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

    +159

    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
    if ($_POST["type"] == "2") {
	echo '<form name=fm method=post>'.
	'<select class="f" name="per" onChange="mclick(\'/history.php?type=2&per=\'+document.fm.per.value)">'.
	'<option value="2"'; if ($per == "2") { print "selected"; }; echo '>'.$lng['history'][14].''.
	'<option value="5"'; if ($per == "5") { print "selected"; }; echo '>'.$lng['history'][15].''.
	'<option value="7"'; if ($per == "7") { print "selected"; }; echo '>'.$lng['history'][16].''.
	'<option value="14"'; if ($per == "14") { print "selected"; }; echo '>'.$lng['history'][17].''.
	'<option value="30"'; if ($per == "30") { print "selected"; }; echo '>'.$lng['history'][18].''.
	'<option value="365"'; if ($per == "365") { print "selected"; }; echo '>'.$lng['history'][19].''.
	'<option value="1000"'; if ($per == "1000") { print "selected"; }; echo '>'.$lng['history'][20].''.
	'</select>'.
	'</form>';
}

    убейте меня

    Запостил: ratik, 10 Августа 2009

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

    • ava guest 10.08.2009 21:32 # 0

      Пристрелите меня
      Ответить

    • ava guest 11.08.2009 07:53 # +1

      бля, научите его пользоваться циклами что ли...
      Ответить

      • ava guest 11.08.2009 15:56 # 0

        циклы пригодятся в php, а для вывода html лучше так, нагляднее
        Ответить

    • ava guest 11.08.2009 08:15 # +1

      А где же закрывающий код </option>
      Ответить

    • ava guest 11.08.2009 08:21 # +1

      Цыклами пользуются только трусы :)
      Ответить

    • ava guest 11.08.2009 10:47 # −1

      Как поможет цикл для значений 2,5,7,14,30,365,100?
      Ответить

    • ava guest 11.08.2009 10:55 # −2

      и что здесь ужасного? типичная задача, типичный "иф" для неё.
      Кроме отсутствия тэга закрывающего - не вижу ничего говнокодистого, ув. Говнокодеры!
      Ответить

      • ava guest 11.08.2009 11:02 # +2

        2, 5, 7, 14, 30, 365, 1000 можно было в массив запихать и по нему пробегать.
        Ответить

        • ava guest 11.08.2009 12:20 # 0

          +1
          Ответить

        • ava guest 11.08.2009 13:24 # 0

          было бы быстрее или это эстетическая поправка?
          Ответить

          • ava guest 11.08.2009 15:05 # 0

            Эстетическая поправка, чтобы не копипастить.
            Что если таких строк понадобится много?
            Ответить

            • ava guest 11.08.2009 16:29 # 0

              но ведь не понадобилось? зачем изобретать велосипед? достаточно использования условия. конечно более эстетично в массив всунуть, а потом искать в массиве, но для нескольких значений можно и накопипастить.
              Такое ощущение что Вы тут обосрете всё что не кинешь. Логика проста: раз здесь - то говнокод.
              Ответить

              • ava Oleg_quadro 11.08.2009 18:26 # 0

                Отнюдь, за себя скажу, некоторые примеры я не считаю говнокодом и пишу в комментах почему.
                ===
                Ну а тут ты прав, если есть увереность, что код не нужно будет менять.
                Ответить

              • ava guest 11.08.2009 22:55 # 0

                А вдруг прийдется добавлять option'ы?
                Ответить

        • ava guest 21.08.2009 13:45 # 0

          Вариант, но так нагляднее.
          Ответить

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