Jawa / Говнокод #12503 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
public static final void setManager(String name, MessageManager manager) {
        if ("doc".equals(name)) {
            doc = manager;
        } else {
            throw new RuntimeException("name is not 'doc' : " + name);
        }
    }

Просто эпично! Даже добавить нечего

Запостил: tuba.linux tuba.linux, (Updated )

Комментарии (7) RSS

    • ava bormand # Ссылка на оригинал 0 ⇈

      И RuntimeException вместо более подходящего под ситуацию IllegalArgumentException.
      Ответить

      • ava 3.14159265 # Ссылка на оригинал 0 ⇈

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

          • ava 3.14159265 # Ссылка на оригинал 0 ⇈

            Хз. Часто volatile недостаточно
            Из того что опубликовано технически всё правильно:
            >"doc".equals(name)
            Строить догадки что там и как - нет желания.
            Ответить

  • ava absolut # Ссылка на оригинал 0

    А где-то выше:
    while( true ) {
  try  {
      setManager(name, manager);
      break;
  }
  catch( RuntimeException e )  {  }
}
    Ответить

Добавить комментарий

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓