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

• guest 14 лет ago # 0

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

  Ответить

  • myzone 14 лет ago # 0 ⇈

    спасибо, капитан

    Ответить

• Мистер Хэнки 14 лет ago # 0

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

  Ответить

  • bormand 14 лет ago # 0 ⇈

    if ("".equals(request) || null == request) { }

    Ответить

  • myzone 14 лет ago # 0 ⇈

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

    Ответить

    • wissenstein 14 лет ago # 0 ⇈

      Чтобы isEmpty отработало без ошибок в случае null, надо поменять местами условия:
      

      if ((request == null) || request.isEmpty()) { }

      
      Хотя, если сделать import static org.apache.commons.lang.StringUtils.isEm pty, то получится совсем красиво:
      

      if (isEmpty(request)) { }

      Ответить

      • wissenstein 14 лет ago # 0 ⇈

        И несмотря на все наши усилия, этот участок кода останется лишней группой команд. Ведь результат вот этого:
        

        Response.status(Response.Status.NO_CONTENT).build();

        ничему не присваивается. Просто будет создан объект, который через некоторое время будет всего лишь уничтожен сборщиком мусора.

        Ответить

        • myzone 14 лет ago # 0 ⇈

          блин, я даже не заметил, что-то мне отдохнуть пора...
          
          есть шанс, что там был return, который я криво скопировал, но это не отменяет первого предложения 😉

          Ответить

      • myzone 14 лет ago # 0 ⇈

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

        Ответить