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

    −37.2

    1. 1
    isRegKeyRequired = distKey == null ? false : distKey.equals("yes") ? true : false;

    Уматный ХавнАкоД коллеги из Индии. Ну просто няйсь. Смотрится куда зло*буче чем желаемый вариант (isRegKeyRequired = "yes".equals(distKey));

    Запостил: guest, 16 Декабря 2008

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

    • Проходилмимо:
      Вообще
      >> "yes".equals(distKey))
      говнокод ещё тот. Ибо человек его читает как: а равно ли "yes" тому-то? Правильней писать:
      >> distKey.equals("yes"))
      вот почему null-ы сосут.
      Ответить
    • Nick:
      "yes".equals(distKey) - наиболее правильный вариант, т.к. точно не упаде NullPointerException.

      Это стандартная практика, и я знаю, что во многих конторах е%ут, если ты сравниваешь контанту, ставя ее в аргумент (distKey.equals("yes")).

      Поэтому сравнения типа "".equals(someVal) или "MaZaFaKa".equalsIgnoreCase(someVal) - нормалоьныя распространенная практика.
      Ответить

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