1. PHP / Говнокод #11277

    +66

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    // text
    if ($type=="text") {
    	[skip]
    }
    
    // order
    if ($type=="order") {
    	[skip]
    }
    
    // select
    if ($type=="select") {
    	[skip]
    }
    
    // chekbox
    if ($type=="chekbox") {
    	[skip]
    }

    Таких проверок штук 20. Коментарии на всех соответствующие.

    Запостил: domaster, 22 Июня 2012

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

    • А что не так (ну кроме chekbox)?
      Ответить
      • без свича.
        Ответить
        • Ну если там по 100500 строк - то свич был бы не понятнее. Хотя тогда оптимально, конечно, свич + по отдельной функции на каждый блок.
          Ответить
          • Вообще-то хотелось указать на информативность комментариев. Вот нафига так писать? Лучше уж совсем не писать, чем так.
            Ответить
            • Лишняя строчка
              Ответить
            • Ну цветная строчка, по которой хорошо видно начало следующего блока ;)

              Кстати респект автору, что он хоть пустые строки между ними расставил.
              Ответить
      • Кстати, checkbox нашелся далеко внизу - и его обрабтка отличается от того, что скипнуто в chekbox. Комментарий написан аналогично:

        // checkbox
        if ($type=="checkbox") {
        [skip]
        }
        Ответить
        • А чем кстати отличаются эти 2 компонента?
          Ответить
          • chekbox почему-то не работает.
            Ответить
          • Это не элементы формы! :) Это просто проверка на содержимое :) Встречаются еще значения "select1", "select11", "select11-1" :)
            Ответить
            • > Встречаются еще значения "select1", "select11", "select11-1"
              Какие говорящие названия...
              Ответить
            • > Это просто проверка на содержимое :)
              Где-то еще есть ассоциативный массив, в котором каждому полю сопоставлен его $type?
              Ответить
              • почему тогда type не логичней было использовать id
                Ответить
              • Вообще-то, есть. Выше. И там как раз проставляются эти типы. А вот результат - разный. Например, по "select11-1" формируется <select> для формы, а по "select11" генерится html. Но комментарии в коде просто не поддаются никаким комментариям! :)
                Ответить
                • > Это просто проверка на содержимое
                  > формируется
                  > генерится
                  Путаемся в показаниях ? ;)

                  Т.е. все-таки это нечто типа генератора неких стандартных элементов на форме по их типу?
                  Ответить
                  • Нет. Всегда возвращается html-код. Только в одном случае это элемент формы ("select11-1" сформирует и отдаст <select></select>), а в другом - просто код: таблица, например. Т.е. по названию или по комментарию никак не поймешь, что получится на выходе. А по "chekbox" (единственному!) возвращается true/false.
                    Ответить
    • показать все, что скрытоКакой багор )))
      Ответить
    • тогда надо было хотя бы elseif использовать...
      Ответить

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