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

0

  1. 1
hideVoting = (disabled || getVisitor().isAnonymous()) ? true : false;

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

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

    • Код похоже Javascript, а не Java.
      disabled || getVisitor().isAnonymous() не будет возвращать true или false
      а будет
      disabled или getVisitor().isAnonymous()
      А они могут быть чему угодно, хоть строкой.
      так то.
      Так что ? true : false нужно
      Ответить
      • Чтобы переконвертить выражение в true/false в JavaScript достаточно поставить перед ним два отрицания - !!. Впрочем, это читается хуже тернарного оператора.
        Ответить
        • код нормальный.
          именно так надо писать!!!!

          разделение условия и присвоения
          Ответить
  • Доброгость:
    #2

    Баян, то баян, но я вот иногда специально так пишу, чтобы нагляднее было.
    Ответить
  • [quote=Доброгость]специально так пишу, чтобы нагляднее было[/quote]
    Для так кого нагляднее?
    Ответить
  • Иногда такая хрень появляется после рефакторинга. IDE-шки ещё не настолько умные, чтобы автоматически упрощать такие выражения
    Ответить
    • Идея такие выражения специально подсвечивает и предлагает заоптимизировать - убрав лишний тернарный оператор.
      Ответить
    • Плохо, что размывает логику. Однако более понятно для начинающих. В общем, однозначно нельзя сказать - добро или зло.
      Ответить
      • наоборот не использование такого написания размывает логику
        Ответить
  • Не говнокод, просто лишний оператор. А вообще, явное лучше неявного.
    Ответить

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

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

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


    8