Как избавиться от множества if?

Question

[jejos]

У меня есть задача, которую я решил, однако, не уверен что моё решение можно назвать хорошим. Суть такова:
есть 3 числовых переменных A, B, C. Каждая из них содержит число, они могут быть как равны между собой в разный момент времени, так и не равны друг другу. Также, в 1-й или более переменных значение может отсутствовать (null). Условия задачи выглядят так: "если A > B > C, то ответ считать по формуле A + B + C", или "если ((A == null) && (C > B), то ответ считать по другой формуле (B / C) * 2" и таких условий несколько штук. Я так и закодировал, и мое решение выглядит как несколько подряд идущих if'ов (10 штук), и внутри каждого if'а расписано по какой формуле считать ответ. Но честно говоря, оно мне не нравится. Мне оно видится как куча if'ов, составляющих громоздкую конструкцию, но я не вижу, как можно сделать это более читаемым и понятным. Не понимаю, как обернуть эти условия в понятные абстракции с названиями, чтобы это перестало быть похожим на кучу математических формул. Как можно сделать лучше?
Пример кода, о котором я говорю (и таких if 10 штук подряд):

if ((A < B) && (C > B)) {
value = (A + B) - 1
}
if ((A == null) && (C < B)) {
value = (C + B) * 2
}

PS: лучшее, что я пока смог придумать, это обернуть все эти вычисления в функцию, и дать поясняющие комментарии.

Comments

[Сергей Соколов]

Приоритет операторов позволяет сократить лишние скобки. Может, это слегка разгрузит код:

if (A < B && C > B) {
  value = A + B - 1
} else if (A == null && C < B) {
  value = (C + B) * 2
}

Answer 1

[Saboteur]

Множество if это нормально.

Для оптимизации, надо смотреть вашу задачу целиком. Возможно не хватает опыта или внимания, чтобы найти закономерности, которые можно сократить. Но возможно что их и нет.
Если условия такие разные, что их нельзя сократить, значит будет десять if и магия тут не поможет.

Можете кинуть все условия, вдруг кто-то что-то найдет.

Answer 2

[Михаил Ушенин]

Если нужно избежать копирования кода с проверкой значений переменных, можно написать функцию с именем типа getCaseId, которая будет получать на вход ваши переменные и возвращать строковый идентификатор (или константу с идентификатором), по которому вам будет понятно, какой из случаев в данный момент имеет место. Потом составьте блок switch и в нём для каждого идентификатора пропишите нужные математические действия.

Это может быть полезно, если у вас в нескольких местах кода есть зависимость от значений этих переменных — можно повторно вызывать функцию getCaseId, если нужно, и использовать эти идентификаторы.

Comments

[jejos]

Именно подобного совета и я ожидал. Спасибо! Есть какое-то название у такого приёма?

[Михаил Ушенин]

Это называется самодокументируемый код. Смысл и назначение кода должны быть в основном понятны из используемых идентификаторов: имён переменных, констант, функций, классов. Вы их пишете для себя.

Если вы назвали переменную value, то будьте готовы к тому, что, возможно, будете заново выяснять для себя через пару месяцев, что это за значение. Если вы её назвали template_name_setting_value, то будет сразу видно, что это значение настройки, в которой хранится имя шаблона.

Примерно это же я применил к вашему набору условий — дал каждому условию осмысленное имя, которое и вы сами легко поймёте, читая свой код через полгода, и ваши коллеги, которые будут с ним работать.

Answer 3

[uvelichitel]

switch case

Comments

[historydev]

или тернарные n == X ? n : X

Answer 4

[asd111]

Чтобы тебя больше не смущало большое количество if then else вот тебе пример кода из типичного лексера. Это не полный кусок и в нем больше 20 if then else и по другому писать особо нет смысла.

void next() {
        ...
        else if (token == '=') {
            // parse '==' and '='
            if (*src == '=') {
                src ++;
                token = Eq;
            } else {
                token = Assign;
            }
            return;
        }
        else if (token == '+') {
            // parse '+' and '++'
            if (*src == '+') {
                src ++;
                token = Inc;
            } else {
                token = Add;
            }
            return;
        }
        else if (token == '-') {
            // parse '-' and '--'
            if (*src == '-') {
                src ++;
                token = Dec;
            } else {
                token = Sub;
            }
            return;
        }
        else if (token == '!') {
            // parse '!='
            if (*src == '=') {
                src++;
                token = Ne;
            }
            return;
        }
        else if (token == '<') {
            // parse '<=', '<<' or '<'
            if (*src == '=') {
                src ++;
                token = Le;
            } else if (*src == '<') {
                src ++;
                token = Shl;
            } else {
                token = Lt;
            }
            return;
        }
        else if (token == '>') {
            // parse '>=', '>>' or '>'
            if (*src == '=') {
                src ++;
                token = Ge;
            } else if (*src == '>') {
                src ++;
                token = Shr;
            } else {
                token = Gt;
            }
            return;
        }
        else if (token == '|') {
            // parse '|' or '||'
            if (*src == '|') {
                src ++;
                token = Lor;
            } else {
                token = Or;
            }
            return;
        }
        else if (token == '&') {
            // parse '&' and '&&'
            if (*src == '&') {
                src ++;
                token = Lan;
            } else {
                token = And;
            }
            return;
        }
        else if (token == '^') {
            token = Xor;
            return;
        }
        else if (token == '%') {
            token = Mod;
            return;
        }
        else if (token == '*') {
            token = Mul;
            return;
        }
        else if (token == '[') {
            token = Brak;
            return;
        }
        else if (token == '?') {
            token = Cond;
            return;
        }
        else if (token == '~' || token == ';' || token == '{' || token == '}' || token == '(' || token == ')' || token == ']' || token == ',' || token == ':') {
            // directly return the character as token;
            return;
        }

        ...
}

Comments

[uRoot]

Насколько сильно страдает производительность, если простыня из if else?

[asd111]

CryNet, Производительность не страдает. Даже на python 700 if отрабатывает меньше чем за секунду.
Если у вас меньше миллиона if на одном уровне то можете не волноваться.

Answer 5

[Valera Dobroman]

Можно сократить с помощью циклов for, while и массивов. Но желательно видеть как выглядит твой код.

Answer 6

[Надим]

Вы можете загнать ваши условия в массив, используя Boolean, а затем красиво обработать все if внутри одного цикла for просто перебрав результирующий массив.

Comments

[Station 72]

дебажить одно удовольствие наверное?

[Надим]

Station 72, если автор вопроса хочет любой ценой сократить кол-во условий if, то кто я такой ему мешать?