Jawa / Говнокод #205 Ссылка на оригинал

0

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

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

Запостил: guest guest, (Updated )

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

  • ava guest # Ссылка на оригинал 0

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

  • ava guest # Ссылка на оригинал 0

    Nick:
    "yes".equals(distKey) - наиболее правильный вариант, т.к. точно не упаде NullPointerException.

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

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

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

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓