1. C# / Говнокод #13157

    +124

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

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

    Запостил: NeoN, 11 Июня 2013

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

    • ava Lure Of Chaos 12.06.2013 06:43 # 0

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

      • ava bormand 12.06.2013 07:02 # +1

        > залогиниться
        Залогировать...
        Ответить

      • ava crastinus 12.06.2013 12:36 # +2

        Меня больше интересует почему ТС TODO не просматривал.
        Ответить

        • ava Lure Of Chaos 12.06.2013 14:24 # −2

          потому что само наличие метода - абсурд.
          Ответить

          • ava bormand 12.06.2013 15:35 # +1

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

            • ava Lure Of Chaos 12.06.2013 15:52 # 0

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

              • ava bormand 12.06.2013 18:14 # 0

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

                • ava Lure Of Chaos 12.06.2013 18:18 # +1 

                  @Inject
private Logger logger;

void doSmth() {
  this.logger.log("упс");
}
                  или
                  @Log
void doSmth() {
}
                  Ответить

                  • ava bormand 12.06.2013 18:20 # 0

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

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

                    • ava Lure Of Chaos 12.06.2013 18:58 # 0

                      аннотация @javax.inject.Inject не специфична для Spring, а есть в любом IoC контейнере

                      да, @Log именно так и работает, декорирует любой метод на логирование исключений
                      Ответить

                    • ava Lure Of Chaos 12.06.2013 19:04 # +1

                      з.ы. пользуясь случаем, прорекламирую свой любимый фреймворк
                      http://tapestry.apache.org
                      , и даже дам ссылку на проект-пример, который можно использовать как quickstart 
                      https://dl.dropboxusercontent.com/u/73797209/heartsong.zip
                      Ответить

                      • ava bormand 12.06.2013 20:30 # +2

                        Блин, почему я pom всегда читаю как porn?
                        Ответить

                        • ava anonimb84a2f6fd141 12.06.2013 20:44 # −2

                          Зрение? Шревты?
                          Ответить

                        • ava Lure Of Chaos 12.06.2013 20:46 # +3

                          > почему я pom всегда читаю как porn?
                          потому что тоже дофига грязного траха со стонами и матом по своей камасутре
                          Ответить

                          • ava roman-kashitsyn 12.06.2013 21:01 # +1

                            Это вы, батенька, кресты давно не собирали. Наверняка про это кто-то уже снял порно.
                            Ответить

                            • ava Lure Of Chaos 12.06.2013 21:44 # 0

                              > Наверняка про это кто-то уже снял порно.
                              правило №34 еще никто не отменял
                              Ответить

                • ava anonimb84a2f6fd141 12.06.2013 18:45 # −4

                  Жаба - это С++ со сборкой мусора и без шаблонов.
                  Ответить

                  • ava Мистер Хэнки 12.06.2013 19:36 # −1

                    generic'и есть. практически те же шаблоны
                    Ответить

                    • ava roman-kashitsyn 12.06.2013 19:40 # +1

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

                    • ava kegdan 14.06.2013 00:18 # 0

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

    • ava absolut 12.06.2013 15:32 # +3

      Надо не туду писать, а NotImplemented кидать.
      Ответить

      • ava bormand 12.06.2013 15:32 # +3

        Так оно же падать будет.
        Ответить

        • ava absolut 12.06.2013 15:40 # 0

          так надо ловить :) и в обработчике писать уже TODO:
          Ответить

          • ava bormand 12.06.2013 15:42 # +1

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

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

            • ava absolut 12.06.2013 15:52 # +4

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

          • ava myaut 13.06.2013 08:07 # 0

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

            • ava bormand 13.06.2013 08:21 # 0

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

    • ava fasol8 12.06.2013 17:46 # 0

      Это вин!
      Ответить

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