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

    +161

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    <?
    for($i=2;$i<31;$i++)
    {
    	if (is_file("upload/files/".$args['id']."_img".$i.".jpg"))
    	{
    		print "<p><a target='_blank' href='/upload/files/".$args['id']."_img".$i.".jpg"."'><img width=100px src='/upload/files/".$args['id']."_img".$i.".jpg"."' border=0></a></p>";
    	}
    }
    ?>

    Вывод фотографий в Aquilon CMS. Причем это написано в (типа)View.

    Запостил: dew2, 01 Февраля 2011

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

    • ну прям на все грабли наступили:
      и слишком длинная строка
      и короткие теги
      и волшебные цифры
      и двойные кавычки
      и жестко заданные границы массива
      и жесткие пути
      и копипаста путей, причем целыми выражениями
      и проверка на файл во вьюве
      и вывод хтмл в пхп коде
      и хтмл кавычки одинарные
      и атрибут width вместо ксс
      и незакавыченные значения
      и указание единиц измерения в атрибуте, хоть этого не должно там быть (в ксс надо)

      ну для просто полного счастья говнокодера не хватает только
      что бы тег <p> был бы не закрыт, как многие броузеры позволяют

      и столько говна в трех строчках не считая скобок

      з.ы. кэп ничего не забыл? = )
      Ответить
      • цикл начинается с 2х, т.к. предполагается, что первый файл всегда существует))) а если нет, то насрать, выведем пустой img (это выше по коду было)
        Ответить
        • вот и мне непонятно, почему такое особое отношение к первому?
          а может и ко второму (неясно, граница там 0 или 1)
          Ответить
          • Часто бывает, что первый файл большой, а остальные маленькие. Наверное, там просто width стоит побольше.
            Ответить
      • КЭП не знает какой доктайп используется, поэтому говорить, что что-то не закавычено нельзя. С другой стороны, если это должен был быть XHTML, то тэг img не закрыт.

        Кстати, какая разница какие кавычки в HTML?
        Ответить
        • ну вы даете. Если это хтмл4.0, то можно и писать, по-вашему, без кавычек и в верхнем регистре? и действительно не видите разницы между двойными кавычками, обратный и апострофами? и красиво писать только есть этого требует доктайп? (кстати, доктайп как раз необязателен)
          вот именно по такой же логике у нас и улицы чистые только там, где президент ездит, и электричество только в тех районах, куда чиновников послали.
          Ответить
          • у нас железнодорожные пути заасфальтированы только там, где президент ездит
            политота
            Ответить
            • вы тоже против html в стиле xhtml?
              Ответить
              • модульная архитектура xhtml недостаточно продумана, а поскольку требуется точное следование схеме, поддержка новых фич из пятой версии откладывается на неопределенный срок. так что XHTML мёртв.
                а для красивого форматирования достаточно прохода htmltidy
                Ответить
                • но это не оправдание для пишущих левой задней ногой под лозунгом "ну и что, что говно? работает же!"
                  Ответить
                  • а разве не пишут говнокод, потому что не знают, как можно лучше?
                    Ответить
                    • скорее, потому, что не хотят знать. Иначе же это школокод - детишки играются, пока не научатся
                      Ответить
        • доктайп? не, не слышали. нету там доктайпа вообще нигде)))вот такой вот Аквилон CMS
          Ответить

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