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

    +81

    1. 1
    2. 2
    3. 3
    if (request.equals("") || request == null) {
    Response.status(Response.Status.NO_CONTENT).build();
}

    ну сколько можно...

    Запостил: myzone, 21 Декабря 2012

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

    • ava guest 21.12.2012 18:42 # −2

      Второе условие никогда не выполнится.
      Ответить

    • ava Мистер Хэнки 23.12.2012 20:46 # +4

      Стиля Йоды чую необходимость я.
      Ответить

      • ava bormand 23.12.2012 20:58 # +4 

        if ("".equals(request) || null == request) { }
        Ответить

      • ava myzone 23.12.2012 22:57 # +1

        как вариант - конечно, но зачем сравнивать с пустой строкой, есть же isEmpty?
        Ответить

        • ava wissenstein 24.12.2012 02:55 # +2

          Чтобы isEmpty отработало без ошибок в случае null, надо поменять местами условия:
          if ((request == null) || request.isEmpty()) { }

          Хотя, если сделать import static org.apache.commons.lang.StringUtils.isEm pty, то получится совсем красиво:
          if (isEmpty(request)) { }
          Ответить

          • ava wissenstein 24.12.2012 03:02 # +1

            И несмотря на все наши усилия, этот участок кода останется лишней группой команд. Ведь результат вот этого:
            Response.status(Response.Status.NO_CONTENT).build();
            ничему не присваивается. Просто будет создан объект, который через некоторое время будет всего лишь уничтожен сборщиком мусора.
            Ответить

            • ava myzone 24.12.2012 12:45 # 0

              блин, я даже не заметил, что-то мне отдохнуть пора...

              есть шанс, что там был return, который я криво скопировал, но это не отменяет первого предложения ;)
              Ответить

          • ava myzone 24.12.2012 12:35 # 0

            то, что порядок следует изменить - тоже очевидно.

            про апаче комнс не знал, надо бы посмотреть что там есть, спасибо за наводку.
            Ответить

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