Си диез / Говнокод #13157 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
public void LogInDB(Exception ex)
{
      // TODO: реализовать
}

А мы то голову ломаем, чего в старых методах на сервере ошибки не логируются!

Запостил: NeoN NeoN, (Updated )

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

  • ava Lure Of Chaos # Ссылка на оригинал 0

    т.е. залогиниться - это что-то из ряда вон выходящее, сродни делению на ноль?
    Ответить

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

          А что не так с методом? Это походу самая обычная запись экцепшена в базу на центральном сервере, чтобы потом логи с клиентов не выгребать и не плодить лишних коннектов, передающих логи другими способами.
          Ответить

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

            а то, что он должен инлайниться, с вызовом специально настроенно логгера.
            Ответить

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

              Жаба - это технология, в которой ручной копипаст и инлайнинг считаются правилами хорошего поведения.
              (Logger)Container.get(Logger.class.getName()).log("Вот так правильно?");
              Ответить

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

                  Про инъекцию понял, надо осилить как-нибудь Spring.

                  А аннотация @Log как работает? Логирует все исключения, возникшие в помеченном ей методе и отпускает их дальше?
                  Ответить

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

                    > практически те же шаблоны
                    О, нет. Далеко не тоже самое. Но в целом жить можно.
                    Ответить

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

                    Ничего подобного, только внешне похожи - внутри совсем разные вещи. А во всему причиной шарпеевская (ну и жабоидная) типобезопастность.
                    Ответить

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

          Зеленая краска кончилась? 😉

          Здесь TODO стояло в одном месте, и достаточно было реализовать этот метод. А если ловить и писать TODO в обработчиках - то потом их все искать и выпиливать...
          Ответить

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

            >Зеленая краска кончилась? 😉
            летом и так всё зеленое 🙂 экономлю
            Ответить

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

          А в обработчике логировать сообщение об ошибке
          Ответить

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

            Ну вариант, конечно, но почему тогда просто не вывалить Function xxx not implemented в лог прямо из нее самой? Раз уж мы не хотим делать отсутствие этой функции фатальным.
            Ответить

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

Из-за тебя ушел bormand, guest!

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

    ↑
    8
    ↓