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

    • ava 3.14159265 30.01.2013 17:56 # +1

      Нормально.
      Печально что нету синхронизации.
      Ответить

      • ava bormand 30.01.2013 17:59 # +1

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

        • ava 3.14159265 30.01.2013 18:06 # 0

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

          • ava bormand 30.01.2013 18:07 # 0

            Может doc описан как volatile?
            Ответить

            • ava 3.14159265 30.01.2013 18:09 # 0

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

    • ava absolut 30.01.2013 21:47 # 0

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

    • ava Lure Of Chaos 31.01.2013 00:33 # +1

      - вы доктор?
      - нет, я менеджер по телу.
      Ответить

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