Чистота кода VS оптимизация

Цитата Serafim @ 09.04.18, 14:44

Ага, конечно

function doSmthng() {
    switch (true) {
        case smthngBad1:
            throw new Exception("Something bad 1");
        case smthngBad2:
            throw new Exception("Something bad 1");
        case smthngBad3:
            throw new Exception("Something bad 1");
    }
 
    do();
}

И как вы, Serafim, переделаете Астаротский пример, если будет обработка с возвратом?.. (или в Яве такого нет?)

Цитата applegame @ 09.04.18, 15:42

И чем тебе в этом случае поможет "нормальный" паттерн-матчинг?

хороший вопрос) в данном случае, кажется, ничем)

Цитата Славян @ 09.04.18, 16:21

И как вы, Serafim, переделаете Астаротский пример, если будет обработка с возвратом?.. (или в Яве такого нет?)

Имеется ввиду посткондишн? Для этого приватные методы и декораторы, кажется, придумали)

При чем тут приватные методы и декораторы ? О_о

Цитата Астарот @ 09.04.18, 21:49

При чем тут приватные методы и декораторы ? О_о

1) Крупные методы принято делить на более мелкие, в частности твой вариант, в языках с возвращаемым значением в операндах (JS, Python, Ruby), вместо кастуемых (Java, PHP) моет выглядеть следующим образом:

// На псевдокоде, а-ля JS или жаба
class Some {
    public function doSmthng() {
        return checkAbility(a) && checkAbility(b) && checkAbility(c) && do(); // В первой группе языков вернётся значение последнего метода
    }
 
    private function checkAbility(value) {
        return (bool)value || throw new Exception('...');
    }
}

а ещё можно воспользоваться редукцией в группе "вторых языков"

private function checkAbilities(...values): bool {
    return values.reduce((i, result) -> result ? !i : result, true); // Возвращает, либо тру, либо фолс, если один из аргументов values его содержит
}

2) В случае если нужна "обработка с возвратом", что бы это не значило, то можно воспользоваться декоратором, и завраппить ответ. Например, из твоего опыта Астарот, могу привести в пример NotNull аннотацию, которая "враппит" (или контейнеризирует, или декорирует, или вообще это монада ) оригинальное значение. Не она, конечно, а подписчик, не допуская нулевого значения в содержимом. В тоже время Ensure и Verify контракты DbC вполне могут сойти за одну из частных применений декораторов на методах. Выбирай любой способ.

Добавлено 10.04.18, 02:16
P.S. Но это всё имеет смысл, когда подобных операций больше трёх. В примере из трёх булевых достаточно:

function doSmthng() {
    if (smthngBad1 || smthngBad2 || smthngBad3) {
        throw new Exception("Something bad");
    }
 
    do();
}

Сообщение отредактировано: Serafim - 10.04.18, 02:16

Цитата Serafim @ 10.04.18, 01:53

моет выглядеть следующим образом

Какой кошмар Однострочник, в котором еще найди нужную do(), или наоборот нужную проверку, которую ты вынес на сторону, и куда нужно еще посмотреть, что б узнать, что именно оно бросает. Бряки на дебаге тоже ставить одно удовольствие. Б-же, зачем себя так не любить?

Цитата Serafim @ 10.04.18, 01:53

а ещё можно

Убивайтунг! Вот просто и сразу - ну, не читается же глазом эта билеберда, так зачем? Потому что можешь? Наверное, никогда не пойму этого...

Добавлено 10.04.18, 06:26

Цитата Serafim @ 10.04.18, 01:53

В примере из трёх булевых

Ага, а потом гадай - какая же из трех "выстрелила"

Цитата Астарот @ 10.04.18, 06:25

Убивайтунг! Вот просто и сразу - ну, не читается же глазом эта билеберда, так зачем? Потому что можешь? Наверное, никогда не пойму этого...

Сильно похоже на ЫГМ - Ынтырпрайз Гойловного Моска.

Цитата Астарот @ 09.04.18, 15:46

то есть за исключением многословности тот же if получается.

Что интересно, в любом случае получится if-else, только на несколько уровней ниже.

Цитата applegame @ 10.04.18, 06:30

Сильно похоже на ЫГМ - Ынтырпрайз Гойловного Моска.

Не знаю, что это, но мне реально страшно - это ж что он может навертеть в действительно не очевидных местах!

Цитата Астарот @ 10.04.18, 07:02

Не знаю, что это, но мне реально страшно - это ж что он может навертеть в действительно не очевидных местах!

Вот что это такое - FizzBuzz Enterprise Edition is a no-nonsense implementation of FizzBuzz made by serious businessmen for serious business purposes.
Вот где настоящий ужас

Сообщение отредактировано: applegame - 10.04.18, 07:49

Не понимаю, что тебе не нравиться, там даже тесты есть

Цитата Астарот @ 10.04.18, 07:58

Не понимаю, что тебе не нравиться, там даже тесты есть

Что значит "даже"?
Это коммерческий код написанный коммерческим программистом в соответствии с высочайшими энтерпрайз стандартами. Там по определению не может не быть тестов.

switch(true) - это сильно
Больше похоже на обфускацию, а не упрощение кода

Цитата Serafim @ 10.04.18, 01:53

моет выглядеть следующим образом

Это прикольный выкрутас, конечно (и где-то даже "классический"). Вот только если попадёт в руки джуну, не каждый сообразит, как сие работает.

Цитата Serafim @ 10.04.18, 01:53

а ещё можно воспользоваться редукцией в группе "вторых языков"

А вот это особенно.

Добавлено 10.04.18, 10:45

Цитата Астарот @ 10.04.18, 06:25

Ага, а потом гадай - какая же из трех "выстрелила"

function doSmthng() {
    var i;
    if (smthngBad1 && (i=1) || smthngBad2 && (i=2) || smthngBad3 && (i=3)) {
        throw new Exception("Something bad " + i);
    }
 
    doIt();
}

Или...

function doSmthng() {
    var i = 1;
    if (smthngBad1 || ++i && smthngBad2 || ++i && smthngBad3) {
        throw new Exception("Something bad " + i);
    }
 
    doIt();
}

А что это switch(true) делает? Я верно понял, что switch разворачивается в пачку if-ов, в каждом из которых проверяется равенство того, что в switch и в case?

OpenGL, получается так...

if (true == smthngBad1) {...}
if (true == smthngBad2) {...}

Сообщение отредактировано: Jin X - 10.04.18, 11:10

Цитата Jin X @ 10.04.18, 10:29

if (smthngBad1 && (i=1) || smthngBad2 && (i=2) || smthngBad3 && (i=3)) {

И потом нарываешься на хитро закопанные грабли в виде смены ключей оптимизации кода, и выясняется... (памяти {$B+})