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

    • ava bormand 22.06.2012 10:24 # 0

      А что не так (ну кроме chekbox)?
      Ответить

      • ava Vasiliy 22.06.2012 10:33 # +1

        без свича.
        Ответить

        • ava bormand 22.06.2012 10:37 # +1

          Ну если там по 100500 строк - то свич был бы не понятнее. Хотя тогда оптимально, конечно, свич + по отдельной функции на каждый блок.
          Ответить

          • ava domaster 22.06.2012 10:53 # +3

            Вообще-то хотелось указать на информативность комментариев. Вот нафига так писать? Лучше уж совсем не писать, чем так.
            Ответить

            • ava TarasB 22.06.2012 10:54 # +2

              Лишняя строчка
              Ответить

            • ava bormand 22.06.2012 11:09 # +4

              Ну цветная строчка, по которой хорошо видно начало следующего блока ;)

              Кстати респект автору, что он хоть пустые строки между ними расставил.
              Ответить

      • ava domaster 22.06.2012 10:57 # +3

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

        // checkbox
        if ($type=="checkbox") {
        [skip]
        }
        Ответить

        • ava bormand 22.06.2012 11:10 # +2

          А чем кстати отличаются эти 2 компонента?
          Ответить

          • ava Steve_Brown 22.06.2012 11:32 # +6

            chekbox почему-то не работает.
            Ответить

          • ava domaster 22.06.2012 11:56 # +2

            Это не элементы формы! :) Это просто проверка на содержимое :) Встречаются еще значения "select1", "select11", "select11-1" :)
            Ответить

            • ava bormand 22.06.2012 12:02 # +5

              > Встречаются еще значения "select1", "select11", "select11-1"
              Какие говорящие названия...
              Ответить

            • ava bormand 22.06.2012 12:07 # 0

              > Это просто проверка на содержимое :)
              Где-то еще есть ассоциативный массив, в котором каждому полю сопоставлен его $type?
              Ответить

              • ava Vasiliy 22.06.2012 12:17 # 0

                почему тогда type не логичней было использовать id
                Ответить

              • ava domaster 22.06.2012 12:21 # +2

                Вообще-то, есть. Выше. И там как раз проставляются эти типы. А вот результат - разный. Например, по "select11-1" формируется <select> для формы, а по "select11" генерится html. Но комментарии в коде просто не поддаются никаким комментариям! :)
                Ответить

                • ava bormand 22.06.2012 12:28 # 0

                  > Это просто проверка на содержимое
                  > формируется
                  > генерится
                  Путаемся в показаниях ? ;)

                  Т.е. все-таки это нечто типа генератора неких стандартных элементов на форме по их типу?
                  Ответить

                  • ava domaster 22.06.2012 12:45 # 0

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

    • ava guest 23.06.2012 10:15 # −5

      показать все, что скрытоКакой багор )))
      Ответить

    • ava guest 26.06.2012 17:46 # +1

      тогда надо было хотя бы elseif использовать...
      Ответить

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