Бессметрная классика снова и снова.

[hedin]

  public boolean isRoot() {
      return (isMaster() && !isSlave()) || (!isMaster() && !isSlave());
  }

Comments

[feldgendler]

На первом курсе надо давать задачки на упрощение выражений вроде этого. С реальными примерами из code_wtf.

[trailmax]

Да ладно, надо на последнем такие снова давать - чтобы не забывали.

Я такие решал на первом курсе пачками, лет восемь назад. Сейчас пришлось загуглить как называются правила и законы по которым это все упращается, хотя без названий правил их правильно применил при беглом осмотре кода.

[trailmax]

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

[hedin]

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

[feldgendler]

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

[tophelm]

Автогенерённый код?

[hedin]

Ни в коем случае. Практически domain.

[ewg]

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

[trailmax]

Что за компилятор такой?

[ewg]

Какой-то из ранних Borland C++
Потом подобные фишки выкидывал AVR GNU C.

Каждый раз дело доходило до родимчика, когда программа отказывалась выполнять некоторые действия в живом виде, но прекрасно это делала под отладчиком.

[trailmax]

OO вот это крутой компилятор!

[heilkitty_ru]