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

    +77

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    int processRequest(Request rq) {
    try {
        ... blablabla ...
        if ("".equals(object.getPrefix())) {
            throw new IllegalArgumentException(bundle.getMessage(MessageKeys.ILLEGAL_PREFIX));
        }
        ... blablabla ...
    } catch (IllegalArgumentException ex) {
        if (ex.getMessage().equals(bundle.getMessage(MessageKeys.ILLEGAL_PREFIX))) {
            return ErrorCodes.ILLEGAL_PREFIX;
        }
        return ErrorCodes.UNKNOWN_ERROR;
    }
    ... blabla ...
    return ErrorCodes.OK;
}

    Интересный подход

    Запостил: debugger, 27 Марта 2015

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

    • ava wvxvw 28.03.2015 15:41 # 0

      И тем не менее, можно представить ситуацию, когда где-нибудь в глубине blahblahblah кто-то выкидывает исключение с точно таким же сообщением, и чтобы не обрабатывать одну и ту же ситуацию дважды, человек решил таким вот образом поступить.
      Ответить

      • ava guest 28.03.2015 16:43 # 0

        Сообщения для людей, а не для программы.
        Ответить

      • ava debugger 30.03.2015 14:13 # 0

        Даже если допустить эту ситуацию, то проблемма в том что из бандла достаются локализованные строки и именно они участвуют при определении самой ошибки. Да, скорее всего они уникально будут описывать ошибку - ну потому что будет глупо локализовывать разные ошибки одинаковыми строками (хотя и такое допустимо). Проблема тут именно в этом дебильном и не нужном сравнении (equals) двух строк.
        Ответить

    • ava someone 29.03.2015 17:44 # 0 

      if ("".equals(object.getPrefix())) {


      if (object.getPrefix().isEmpty()) не?
      Ответить

      • ava wvxvw 29.03.2015 22:43 # 0

        Неужели в Яве можно создать разные пустые строки? Интересно зачем.
        Ответить

        • ava 3.14159265 30.03.2015 13:57 # 0

          ABA
          Ответить

          • ava wvxvw 30.03.2015 17:43 # 0

            Так не спасет же, ну а вдруг ту же самую пустую строку присвоят (public static final String EMPTY_STRING = "";)
            Ответить

            • ava 3.14159265 30.03.2015 17:50 # 0

              >а вдруг ту же самую пустую строку присвоят
              Тогда ошибка в геноме погромиста.
              Кто знает что делает напишет в нужном месте new String(s) и не будет так писать в других местах.
              Ответить

      • ava guest 29.05.2015 19:34 # 0

        java.lang.String#isEmpty только с Java 1.6. Может код старше.
        Ответить

    • ava chtulhu 29.03.2015 19:40 # 0

      а если object == null?
      Ответить

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