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

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
private static final Color COLOR_GREEN = Color.GREEN;
  private static final Color COLOR_GREEN_DARK = JBoard.COLOR_GREEN;
  private static final Color COLOR_RED = Color.RED;
  private static final Color COLOR_RED_DARK = JBoard.COLOR_RED;
  private static final Color COLR_GRAY = Color.GRAY;

упс, это я случайно )))) если вы это видите, то я это уже рефакторю ))))

Запостил: Lure Of Chaos Lure Of Chaos, (Updated )

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

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

    А как так получается, что однородные константы формируются из разных мест ?
    Кто из Color., кто из JBoard.
    Ответить

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

        Понятно, что определено, иначе бы не собралось. Вопрос в другом, разве сразу не видно, что нелогично так делать ?
        Ответить

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

          Вот мне почему-то кажется, что автор кода решил сделать парочку констант (к которым он обращается в коде), но временно не стал заморачиваться с их значением...
          Ответить

          • ava Lure Of Chaos # Ссылка на оригинал 0 ⇈

            на самом деле вначале это были константы хардкод в самом месте, где используются, потом я их вынес в константы класса, при этом не сразу придумал, как нормально назвать - поэтому назвал по значению. Именно в этом виде код попал сюда )
            дальше я избавился от констант, теперь цвета берутся из синглтона "настроек" = )
            Ответить

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

Семь раз отмерь — один отрежь, guest!

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

    ↑
    8
    ↓