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

    +74

    1. 1
    2. 2
    3. 3
    Logger.getLogger(OriginReaderImpl.class).error("somebody calls reading origins without filtering",
    new RuntimeException());
return Cf.newArrayList();

    У меня был когнитивный диссонанс: в логах стектрейсы, а транзакция успешно завершена.

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

    Запостил: roman-kashitsyn, 26 Марта 2014

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

    • ava KonardinoHuyardino 26.03.2014 16:58 # −8

      показать все, что скрытоvanished
      Ответить

    • ava wvxvw 26.03.2014 18:29 # 0

      Ну а чего, в больших проектах, где случайно две библиотеки пользуются одним логгером ничего лучше обычно и не придумаешь, чем warn / error, потому что если, например, с info распечатать, то в вывод польются тонны неинтереcного говна, типа отчета хайбернейта о создании 100500 таблиц и констрейнтов.
      Ответить

      • ava roman-kashitsyn 26.03.2014 19:40 # 0

        тут в лог печатается стектрейс исключения, которое никто никогда не выбрасывал
        Ответить

        • ava wvxvw 26.03.2014 21:38 # 0

          Почему бы и нет? Я тоже так делаю, чтобы потом можно было найти причину почему какую-то функцию вызвали.
          Ответить

    • ava Lure Of Chaos 26.03.2014 19:52 # 0

      может, это ошибочная транзакция, и она не должна завершаться успешно?
      Ответить

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