Плохо ли размещать if / else внутри другого?

Question

if() {

}else {
    if (IsAlternateRow=='true')
        IsAlternateRow = 'false';
    else
        IsAlternateRow = 'true';
}

Могу ли я поместить оператор if и else внутри другого оператора else?

Answer 1

Примечание: вопрос был помечен как JavaScript после публикации этого ответа (изначально он был о Java, поэтому этот ответ касается Java).

В общем, этоПравильно размещать if и else внутри предложения else.

Однако в вашем коде есть несколько проблем.IsAlternateRow=='true' не является синтаксически допустимым.

• Если это сравнение строк, вы должны использовать двойные кавычки и .equals();
• Если это булево сравнение, вам лучшепросто сделав IsAlternateRow = !IsAlternateRow вместо всего вложенного if.

Answer 2

try

IsAlternateRow = !IsAlternateRow;

(обновлено, чтобы показать, как это будет выглядеть в вашем коде)

var IsAlternateRow = false;

if(/* -- insert equation here -- */) 
{
 // do something
} 
 else 
{
    IsAlternateRow = !IsAlternateRow;
}

Answer 3

Да, размещение if внутри else является вполне приемлемой практикой, но в большинстве случаев использование else if является более понятным и понятным. Э.Г.

if (test) {
   // Do something
} else if (otherTest) {
   // Do something else
} else {
   // Do a third thing
}

Фактически это сокращение для

if (test) {
   // Do something
} else {
     if (otherTest) {
         // Do something else
     } else {
         // Do a third thing
     }
}

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

Ваш пример кода не очень понятен и не будет правильно скомпилирован, более ясный пример кода может помочь нам помочь вам.

Answer 4

return (IsAlternateRow.equals ('true'))? 'false': 'true'

Answer 5

В общем, я бы сказал, что для ответа на подобные вопросы вы должны спросить себя:

Легко ли это прочитать?

Мелко вложенное if/else высказывания не страшны, но как только вы начнете вкладывать средства до тошноты, вам, вероятно, следует провести рефакторинг.

Answer 6

Это хорошо, но есть более простые способы сделать то, что вы делаете:

IsAlternateRow = !IsAlternateRow

Answer 7

Правильный способ сделать это:

if() {

} else if (IsAlternateRow=='true') {
    IsAlternateRow = 'false';
}
 else
{
    IsAlternateRow = 'true';
}

Answer 8

Чтобы ответить на ваш вопрос, да, вы можете бесконечно вкладывать операторы if / else.Код, который вы предоставили, без проблем скомпилируется.Несмотря на то, что вы говорите о Java, я думаю, что если это не псевдокод, он не даст желаемого результата.

Answer 9

Да, вы можете. Если вы хотите вычислять один и тот же объект / переменную несколько раз, вы можете использовать оператор switch, но во многих случаях он находится в шахматном порядке, если оператор будет выполнять работу точно так же.