ЙажаСценарий / Говнокод #3662 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
  18. 18
  19. 19
  20. 20
switch (columnType)
{
    case 'first':
    {
        className = '.active-tooltip-first';
        break;
    }

    case 'second':
    {
        className = '.active-tooltip-second';
        break;
    }

    case 'third':
    {
        className = '.active-tooltip-third';
        break;
    }
}

Подобных перлов в данном проекте немало. По возможности вычищаю

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

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

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

        className в верстке могут поменяться и прощайте ваша конструкция с конкатенацией.

        Ясное дело, предложенный вами вариант удобен и красив в ДАННОМ случае, но это не говнокод ни разу.
        Ответить

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

          > className в верстке могут поменяться

          Ну вот поменяется, и перепишу в свитчи/словарь. Нефиг мудрствовать лукаво.
          Ответить

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

          Забавная логика.


          --Зачем у тебя 25 раз скопирована один и тот же блок кода? Давай вынесим его в функцию!
          --Что ты? А вдруг один из них поменяется -- и прощай твой код.

          Рефакторинг -- зло. Копипаст -- рулез
          Ответить

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

        '.active-tooltip-' - это нужно в константу запихнуть, тогда если это когда-то поменяется - проблем не будет. Непридётся по всему проекту лазить автозаменой.
        Ответить

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

      Классы first, second и third не поменяются, так как несут смысл не с точки зрения дизайна, а функциональности (для JQuery). Данный пример до боли напоминает if (a == 1) a = 1; else a = a;
      Ответить

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

        >Классы first, second и third не поменяются

        А если поменяются классы .active-tooltip-second?
        Ответить

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

          А если китайцы начнут третью мировую ядерную войну, сделают Россию провинцией РОСЯСЯ ЧЖОУХУЙ, и все, соответственно, начнут программировать на китайском? Что тогда делать? Ведь переписывать придётся. Посоветуйте, как избежать.
          Ответить

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

            >>>третью мировую ядерную войну
            забей к тому времени как она закончится 90% что ты сдохнешь - так что не парься понапрасну.

            >>>Посоветуйте, как избежать.
            напасть на Китай первыми xD
            Ответить

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

Из-за тебя ушел bormand, guest!

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

    ↑
    8
    ↓