Говнокод #3806 — PHP — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

PHP / Говнокод #3806

+158

<div class="item" style="padding-bottom:60px">
	<a class="no-border" href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y"><img src="/bitrix/templates/producer/i/items/dis-generator.jpg" alt="Дизельные генераторы" title="Дизельные генераторы"/></a>
	<a href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y">Дизельные генераторы</a>
</div>
<div class="item">
	<a class="no-border" href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode('Бензин А-92(95)')?>&set_filter=Y"><img src="/_producer/i/items/oil-generator.jpg" alt="Бензиновые генераторы" title="Бензиновые генераторы"/></a>
	<a href="catalogue/?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode('Бензин А-92(95)')?>&set_filter=Y">Бензиновые генераторы</a>
</div>
<div class="item">
	<a class="no-border" href="catalogue/?SECTION_ID=333"><img src="/bitrix/templates/producer/i/items/generator.jpg" alt="Газовые генераторы" title="Газовые генераторы"/></a>
	<a href="catalogue/?SECTION_ID=333">Газовые генераторы</a>
</div>
<div class="item">
	<a class="no-border" href="catalogue/?SECTION_ID=326"><img src="/bitrix/templates/producer/i/items/container.jpg" alt="Контейнеры" title="Контейнеры"/></a>
	<a href="catalogue/?SECTION_ID=326">Контейнеры</a>
</div>	
<?
// тут куча кода
?>
<div class="item" style="padding-top:0px; margin-top:-15px">
	<? if ($arrFilter_pf[TOPLIVO] != 'Дизельное') { ?><a class="no-border" href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/dis-generator.jpg" alt="Дизельные генераторы" title="Дизельные генераторы"/></div><? if ($arrFilter_pf[TOPLIVO] != 'Дизельное') { ?></a>
	<a href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode("Producer(Турция)")?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Дизельное")?>&set_filter=Y"><? } ?>Дизельные генераторы<? if ($arrFilter_pf[TOPLIVO] != 'Дизельное') { ?></a><? } ?>
</div>
<div class="item" style="padding-top:0px; margin-top:-15px">
	<? if ($arrFilter_pf[TOPLIVO] != 'Бензин А-92(95)') { ?><a class="no-border" href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode('Producer(Турция)')?>&arrFilter_pf[TOPLIVO]=<?=urlencode('Бензин А-92(95)')?>&set_filter=Y"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/oil-generator.jpg" alt="Бензиновые генераторы" title="Бензиновые генераторы"/></div><? if ($arrFilter_pf[TOPLIVO] != 'Бензин А-92(95)') { ?></a>
	<a href="?SECTION_ID=327&arrFilter_pf[PRODUCER]=<?=urlencode("Producer(Турция)")?>&arrFilter_pf[TOPLIVO]=<?=urlencode("Бензин А-92(95)")?>&set_filter=Y"><? } ?>Бензиновые генераторы<? if ($arrFilter_pf[TOPLIVO] != 'Бензин А-92(95)') { ?></a><? } ?>
</div>
<div class="item" style="padding-top:0px; margin-top:-15px">
	<? if ($_GET['SECTION_ID'] != '333') { ?><a class="no-border" href="?SECTION_ID=333"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/generator.jpg" alt="Газовые генераторы" title="Газовые генераторы"/></div><? if ($_GET['SECTION_ID'] != '333') { ?></a>
	<a href="?SECTION_ID=333"><? } ?>Газовые генераторы<? if ($_GET['SECTION_ID'] != '333') { ?></a><? } ?>
</div>
<div class="item" style="padding-top:0px; margin-top:-15px">
	<? if ($_GET['SECTION_ID'] != '326') { ?><a class="no-border" href="?SECTION_ID=326"><? } ?><div class="product_div"><img src="/bitrix/templates/producer/i/items/container.jpg" alt="Контейнеры" title="Контейнеры"/></div><? if ($_GET['SECTION_ID'] != '326') { ?></a>
	<a href="?SECTION_ID=326"><? } ?>Контейнеры<? if ($_GET['SECTION_ID'] != '326') { ?></a><? } ?>
</div>

Это часть шаблона сайта, сделанного на битриксе. Хотя битрикс тут и не нужен, автор и без него справился бы с задачей. Без него было бы даже проще :)))

Название сайта я заменил в коде на "Producer", не хочу уже позорить. Жаль что весь код не влез, там в таком же духе дальше.

Запостил:

alex322, 28 Июля 2010

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

Kornew 28.07.2010 21:06 # +1

думаю, первых 16 строк вполне хватило бы, для описания этого ГК (мб даже и меньше)
Ответить
Uchkuma 28.07.2010 23:21 # −2

Ну это же Битрикс! Надо следовать требованиям Битрикса.
Ответить
- Lure Of Chaos 29.07.2010 10:06 # +1
  
  а при чем тут битрикс?
  Ответить
  - asics 29.07.2010 10:33 # +1
    
    Битрикс тут не причем, все дело в кривых руках! Хотя я далеко не поклонник битрикса.
    Ответить
  - Uchkuma 29.07.2010 11:09 # −5
    
    показать все, что скрытоа где здесь C++?
    Ответить
    - 3.14159265 29.07.2010 12:37 # 0
      
      ААА!!111 СПАЛИЛСЯ!!! )))
      тем гостем был ты?!?!
      Ответить
Wave 29.07.2010 00:50 # −1

И после этого некоторые удивляются, почему же так битрикс не любят.
Ответить
- Lure Of Chaos 29.07.2010 10:07 # −2
  
  использование битрикса уже показатель.
  между прочим, битрикс не бесплатен. спрашивается, накуя за говно платить?
  Ответить
- Анонимус 29.07.2010 14:03 # +1
  
  перефразирую:
  
  "И после этого некоторые удивляются, почему же так PHP не любят."
  Ответить
atarix12 29.07.2010 11:46 # +1

какая разница на чём пишед идеот ? джумла или битрикс..самописаная заточка.
если мозгов нет - то тут не поможет и платное ПО!
Ответить
- bugmenot 29.07.2010 12:28 # 0
  
  да чего вы докопались...
  коммерческое оно потому, что microsoft 1c, иначе просто не умеют работать.
  а платят за него чтобы было кому мозг ети ("поддержка")
  Ответить
  - atarix12 29.07.2010 12:36 # 0
    
    там есть за что платить;)
    Ответить
    - bugmenot 29.07.2010 12:57 # 0
      
      тут можно, конечно, выкачать nulled и изучить, что там наши виликие энтерпрайз архитекты удумали; но это - второстепенное. главное же - в разделении ответственности за website is down, например
      Ответить
      - Uchkuma 29.07.2010 15:35 # 0
        
        На то под Битриксы специальные тарифы на хостингах бывают.
        Ответить
        
        bugmenot 29.07.2010 17:57 # 0
        
        Хм, не знал, теперь гложет любопытство - почему так? (как там распил бабок происходит?)
        Ответить
        
        mrbig66 29.07.2010 18:49 # 0
        
        Птому что не каждый сервак такое говно выдержит.
        Ответить
punishment 01.03.2011 21:59 # −2

Что не так? Автор дурак?
Что заказчик попросил в последний момент - то подопытные и делают.

Ежели хочешь свободно парить в облаках и писать сферические приложения на том, что по душе - сиди дома. Хочешь денег - засунь свои желания в жопу и пиши быстро и работоспособно.
Приведенный код работоспособен? Да.
Выглядит нечитабельно и так, что поддержку отдадут снова его создателю? Да.
Цель достигнута.

Странные у вас вобщем говнокоды, скажу я вам.
Ответить

Говнокод: по колено в коде.

PHP / Говнокод #3806

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

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