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

    +73

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    if (jsonValues.toString().contains("acctContext") == true) {
    try {
        objectRelMethod = impl.provisionDevice(
	jsonValues.getString("acctContext"),
	jsonValues.getString("persoIdentityRequest"));
     }catch(JSONException e){
	objectRelMethod = impl.provisionDevice(null, null);
     }
}

    и туда же... и это называется гордым словом "мидл"...

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

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

    • ava myzone 21.12.2012 19:01 # 0

      поправлюсь: гордым словом "инженер"
      Ответить

    • ava wvxvw 21.12.2012 21:02 # 0

      Почему мне слово provision напоминает о яблоках? Неужели Ява и туда добралась?
      Ответить

    • ava wissenstein 22.12.2012 00:31 # +3

      >if (someCondition == true)

      О, горе!
      Ответить

    • ava someone 22.12.2012 12:59 # −1

      В чём ГК?

      Вижу два: поиск значения методом поиска подстроки в toString и == true. Что ещё?
      Ответить

      • ava wvxvw 22.12.2012 14:59 # +2

        Я так понимаю, что это какой-то JSON парсер, который генерирует явовские объекты для того, чтобы представить данные в JSON. Т.е. наверняка есть какая-то возможность узнать существует ли (ключ? значение? подстрока содержащая "acctContext") в этом объекте не переводя его в строку.
        Но не исключен вариант, когда код для "правильного" поиска этой подстроки был бы гораздо сложнее, чем поиск по сгенерированой строке. Но судя по возмущению автора, скорее всего, это просто ключ в хеш таблице, или что-то такое.
        Ответить

        • ava myzone 22.12.2012 15:24 # 0

          Вы абсолютно правы.
          Есть и метод has, и методы получения опциональных полей, которые возвращают дефолт значение, в случае отсутствия значения в мэпе.
          Да и работать это будет быстрее, ибо создание исключения не так дешево...
          Ну да ладно, после первой строчки это мелочи жизни...
          Ответить

          • ava myzone 22.12.2012 15:35 # 0

            Кстати, я только что заметил, что если первого поля не будет, то второго, в оригинальной логике, тоже нет
            Ответить

    • ava 3.14159265 24.12.2012 19:33 # +2

      "быдл"
      Ответить

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