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

• karamba 15 лет ago # 0

  Вот почему надо включать checkstyle на уровне репозитория.

  Ответить

  • Art 15 лет ago # 0 ⇈

    А ведь хорошая мысль...

    Ответить

    • karamba 15 лет ago # 0 ⇈

      У нас настроен при комите: http://checkstyle.sourceforge.net/
      Там много что есть, SimplifyBooleanExpression, SimplifyBooleanReturn. Одно плохо, если его не включили сразу и проект уже большой, то будут проблемы и проще не включать вообще. Иначе будут жестокие diffы, но все новые проекты только с ним и никак без него.

      Ответить

  • roman-kashitsyn 15 лет ago # 0 ⇈

    Да мне кажется, что та же Idea будет ворнинги на таких косяках давать.

    Ответить

    • Art 15 лет ago # 0 ⇈

      Аха, вот только этот код в NetBeans писался...

      Ответить

    • karamba 15 лет ago # 0 ⇈

      Нельзя полагаться только на IDE, централизованная проверка не помешает. Мне вот Eclipse не выдает( Да и всегда найдется человек, который в спешке в 18:59 "зафиксит" баг отредактировав файл в NotePad, зальет, сломает билд, и успеет свалить домой. Ведь IDE дольше открывать)
      
      К тому же Idea не показывает такие вещи, как equals без hashcode, Utils классы без private конструктора и прочие стандартные косяки, которые иногда пролетают через ревью.

      Ответить

      • roman-kashitsyn 15 лет ago # 0 ⇈

        > Нельзя полагаться только на IDE
        Тут я с вами полностью согласен. Сам довольно долго принципиально писал на Java в emacs.

        Ответить

• roman-kashitsyn 15 лет ago # 0

  Не самое страшное гавно, но некрасиво, да. NullPointer словить можно.

  Ответить

• kibberpunk 15 лет ago # 0

  isEmpty() - это для трусов)))

  Ответить

  • bugmenot 15 лет ago # 0 ⇈

    /* пусты ли трусы? */

    Ответить