Вариации двух условий

[Nordic Warrior]

Всем доброго времени суток.

Возник такой вопрос, как лучше обрабатывать четыре вариации двух условий?

Так:

if(условие 1 верно && условие 2 верно)
{
// some code..
// some identical code ONE..
}
else if(условие 1 НЕ верно && условие 2 верно)
{
// some code..
// some identical code ONE..
}
else if(условие 1 верно && условие 2 НЕ верно)
{
// some code..
// some identical code TWO..
}
else if(условие 1 НЕ верно && условие 2 НЕ верно)
{
// some code..
// some identical code TWO..
}

Или так:

if(условие 1 верно)
{
    if(условие 2 верно)
    {
        //some code..
    }
    else if(условие 2 НЕ верно)
    {
        //some code..
    }
    //some identical code ONE..
}
else if(условие 1 НЕ верно)
{
    if(условие 2 верно)
    {
        //some code..
    }
    else if(условие 2 НЕ верно)
    {
        //some code..
    }
    //some identical code TWO..
}

 

[the_hunter]

Лучше всего будет избегать такого рода код. Как один из вариантов - сделать флаги (bitsFlags |= myFlag) и далее чекать битсумму через switch.
//some identical code - в отдельную функцию.

 

[Nordic Warrior]

the_hunter, я сомневаюсь, что в моем случае можно реализовать это флагами, по крайней мере не вижу способа. Поскольку во-первых, все эти 4 условия предопределяются в теле этой же функции, в которой будет условие, а во-вторых, если условие 2 у меня является булевым, то первое условие определяется значениями числа, и без дополнительного условия не получится задать флаг. Здесь пример определения этого условия. (строка 153). Я готовлю обновление, в котором к нему добавится еще и второе условие..

 

[Garey]

зачем во втором примере else if ? если условия одинаковые можно просто if else.

 

[Ayk]

чекать битсумму через switch

Как?
Особенность switch в том что кейсы должны быть константами.
Для проверки битсуммы надо будет отдельно создавать константы всех возможных значений битсуммы.

И разве компилятор не смотрит на switch как на тот же if/else if?
Оба (if и switch) на выходе будут опкодами JNZ, JZER и т.д.

Nordic Warrior,

if(условие 1 верно)
{
    if(условие 2 верно)
    {
        //some code..
    }
    else
    {
        //some code..
    }
    //some identical code ONE..
}
else // условие 1 НЕ верно
{
    if(условие 2 верно)
    {
        //some code..
    }
    else
    {
        //some code..
    }
    //some identical code TWO..
}

Преимущество if-a в том, что он может сравнивать НЕ константы.

if( OnlinePlayers > get_pcvar_num(RequiredPlayers) )
  // code

 

[Nordic Warrior]

Garey, ну да, логично).

Ayk, оба сделать if/else не получится, поскольку

Здесь пример определения этого условия. (строка 153).

Там может быть третий вариант, который мне не нужен, поэтому придется одно условие делать if/else if.

 

[Ayk]

Nordic Warrior, у тебя там проверка abs > cvar_diff. Это значит, что sign никогда не вернет 0, так что там всего 2 варианта.

 

[Nordic Warrior]

Ayk, точно, я об этом не подумал.

Всем спасибо, кто откликнулся, можно закрывать. Сделал по совету Ayk.