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

    +92

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    public Boolean checkPermission(String login) {
    if (login == null && login.trim().length() == 0)
        throw new Exception("Не указан логин пользователя.");
    /* ... */
}

    Чудесное условие. Есть в этом Exception что-то неуловимое - его никто никогда не поймает.

    Запостил: roman-kashitsyn, 14 Июня 2011

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

    • ava Lure Of Chaos 14.06.2011 18:59 # 0

      ошибка?

      и, хм, возвращать Boolean тоже круто.
      Ответить

      • ava roman-kashitsyn 14.06.2011 19:41 # +1

        продакшн код, работает уже 7 лет. и никто не исправил... Видимо, никто не пробовал null'ы подставлять. Юнит тесты рулят.
        Ответить

    • ava Lure Of Chaos 14.06.2011 20:30 # +1

      кстати, как это может быть в продакшене, если кидается Exception, который checked, и требует быть объявленным в списке throws - тут throws Exception я в упор не вижу.
      Ответить

      • ava 3.14159265 14.06.2011 20:54 # 0

        сразу в глаза бросилось.
        Ответить

        • ava Lure Of Chaos 14.06.2011 21:18 # 0

          возможно,фейк или companyname.Exception extends RuntimeException
          Ответить

          • ava 3.14159265 14.06.2011 22:40 # 0

            да походу. любой нормальный IDE ворнинг по идее должен выдавать.
            забавно другое.
            там все-равно-то будет вылазить Exception, тот который nullPointer и unchecked, лол.
            Ответить

            • ava Lure Of Chaos 14.06.2011 22:57 # 0

              ну это в рантайме. А компилятору достаточно и формальное наличие Exception, что бы упереться рогом.
              Ответить

      • ava roman-kashitsyn 15.06.2011 08:28 # +1

        Там метод реально принимает 8 (!) параметров, я умышленно его упростил, случайно отбросив и throws. Кстати, Boolean там возвращается как new Boolean(true)
        Ответить

        • ava Lure Of Chaos 15.06.2011 09:32 # +4

          самое смешное выбросили.
          Ответить

        • ava 3.14159265 15.06.2011 11:49 # 0

          >new Boolean(true)
          если java без автобоксинга (1.4), а вполне возможно код писался под нее, то это совсем не удивительно
          я бы конечно использовал Boolean.TRUE
          Ответить

          • ava guest 15.06.2011 13:47 # 0

            >если java без автобоксинга
            Вот 3.14159265♬расы...
            Ответить

          • ava gegMOPO4 15.06.2011 19:24 # 0

            А Boolean.TRUE для кого? Или хотя бы Boolean.valueOf(true)?
            Ответить

            • ava 3.14159265 15.06.2011 20:24 # +1

              >А Boolean.TRUE для кого?
              brotip:read the spoiler up there

              >Вот 3.14159265♬расы...
              fresh joke. ololo
              Ответить

              • ava gegMOPO4 16.06.2011 10:41 # 0

                С этой поправкой — да, не удивительно. Но java 1.4 тут ни при чём.
                Ответить

        • ava Infinity 15.06.2011 12:17 # +1

          А почему бы не использовать Boolean.TRUE?
          Ответить

    • ava mos 16.06.2011 00:59 # 0

      .
      Ответить

      • ava guest 16.06.2011 03:25 # 0

        А Вы тоже хотели сказать что-то про Boolean.TRUE? :p
        Ответить

        • ava mos 16.06.2011 04:38 # 0

          Нет, я вчера не мог добавить комментарий, просто попробовал :p
          А про Boolean.TRUE или FALSE, я им вообще не пользуюсь
          Ответить

    • ava guest8 08.04.2019 20:56 # −999

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

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