Anti-pattern: избыточная ветвь else, содержащая предложение guard / обработку ошибок

часто вы видите такой код:

public void bla()
{
  if (conditionTrue)
  {
    // long code here
  } else {
    // otherwise do nothing
    return;
  }
  // some more code
}

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

public void bla()
{
  if (!conditionTrue)
    return;

  // long code here

  // some more code
}

некоторые эксперты завершают это, вложив эти лишние ветви в 3, 4 или 5 уровней, создавая код, который абсолютно трудно читать.

Я знаю об этих местах, как-то касаясь этого тема:

это именованный анти-шаблон и каково его имя, если да?

1 ответов


пахнет Код Стрелой анти-шаблон