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

    • Пристрелите меня
      Ответить
    • бля, научите его пользоваться циклами что ли...
      Ответить
      • циклы пригодятся в php, а для вывода html лучше так, нагляднее
        Ответить
    • А где же закрывающий код </option>
      Ответить
    • Цыклами пользуются только трусы :)
      Ответить
    • Как поможет цикл для значений 2,5,7,14,30,365,100?
      Ответить
    • и что здесь ужасного? типичная задача, типичный "иф" для неё.
      Кроме отсутствия тэга закрывающего - не вижу ничего говнокодистого, ув. Говнокодеры!
      Ответить
      • 2, 5, 7, 14, 30, 365, 1000 можно было в массив запихать и по нему пробегать.
        Ответить
        • +1
          Ответить
        • было бы быстрее или это эстетическая поправка?
          Ответить
          • Эстетическая поправка, чтобы не копипастить.
            Что если таких строк понадобится много?
            Ответить
            • но ведь не понадобилось? зачем изобретать велосипед? достаточно использования условия. конечно более эстетично в массив всунуть, а потом искать в массиве, но для нескольких значений можно и накопипастить.
              Такое ощущение что Вы тут обосрете всё что не кинешь. Логика проста: раз здесь - то говнокод.
              Ответить
              • Отнюдь, за себя скажу, некоторые примеры я не считаю говнокодом и пишу в комментах почему.
                ===
                Ну а тут ты прав, если есть увереность, что код не нужно будет менять.
                Ответить
              • А вдруг прийдется добавлять option'ы?
                Ответить
        • Вариант, но так нагляднее.
          Ответить

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