1. Java / Говнокод #12503

    +70

    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, 30 Января 2013

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

    • Нормально.
      Печально что нету синхронизации.
      Ответить
      • И RuntimeException вместо более подходящего под ситуацию IllegalArgumentException.
        Ответить
        • Да похер. Давным-давно я тоже пытался типы подбирать.
          Типично распиздяйский код, мне даже чем-то близок, если б не кривая архитектура и отсутствие атомарности на статике.
          Ответить
          • Может doc описан как volatile?
            Ответить
            • Хз. Часто volatile недостаточно
              Из того что опубликовано технически всё правильно:
              >"doc".equals(name)
              Строить догадки что там и как - нет желания.
              Ответить
    • А где-то выше:
      while( true ) {
        try  {
            setManager(name, manager);
            break;
        }
        catch( RuntimeException e )  {  }
      }
      Ответить
    • - вы доктор?
      - нет, я менеджер по телу.
      Ответить

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