По колено в коде — Говнокод.ру

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

+144

1
2
3

<?php
echo "<html><head><meta http-equiv='Refresh' content='0; url=index.php'></head></html>";
?>

Народ, мне сегодня на собеседовании сказали, что такой редирект - это говнокод. Эт правда?

master_olegi, 26 Ноября 2010

Комментарии (16)

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

+162

01
02
03
04
05
06
07
08
09
10
11
12
13
14

if (isset($items['1994-2008'])) {
	$temp = $items['1994-2008'];
	unset($items['1994-2008']);
	$items['1994-2009'] = $temp;
}

....

$items = $archive+$results;
arsort($items);
$items = array_slice($items, 0, 4);
foreach ($items as $key=>$arr) {
	$content['items'][]=$arr;
}

$archive и $results - два массива :-)

aleksey, 26 Ноября 2010

Комментарии (12)

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

+155

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53

/*фрагмент из компонента*/
$arProps = array(
"NAMES"=>$props["NAMES"]["VALUE"],
"DESCRIPTIONS"=>$props["DESCRIPTIONS"]["VALUE"],
"LINKS"=>$props["LINKS"]["VALUE"],
"IMAGES"=>$props["PICTURES"]["VALUE"],
"URLS" => array(
"0"=>CFile::GetPath($props["PICTURES"]["VALUE"][0]),
"1"=>CFile::GetPath($props["PICTURES"]["VALUE"][1]),
"2"=>CFile::GetPath($props["PICTURES"]["VALUE"][2]),
"3"=>CFile::GetPath($props["PICTURES"]["VALUE"][3]),
"4"=>CFile::GetPath($props["PICTURES"]["VALUE"][4]),
),
);

/*код из шаблона*/
<div class="text-blocks">
<?if($arResult["PROPS"]["DESCRIPTIONS"][0]["TEXT"]):?>
  <div id="banner_text_1" class="text curr" style="left:92px; opacity:0.8; display: block">
<table border="0" cellpadding="0" cellspacing="0"><tr><td>
    <a href="<?=$arResult["PROPS"]["LINKS"][0]?>"><?=$arResult["PROPS"]["DESCRIPTIONS"][0]["TEXT"]?></a>
</td></tr></table>
  </div>
<?endif;?>
<?if($arResult["PROPS"]["DESCRIPTIONS"][1]["TEXT"]):?>
  <div id="banner_text_2" class="text" style="left:156px; opacity:0; display: none">
<table border="0" cellpadding="0" cellspacing="0"><tr><td>
    <a href="<?=$arResult["PROPS"]["LINKS"][1]?>"><?=$arResult["PROPS"]["DESCRIPTIONS"][1]["TEXT"]?></a>
</td></tr></table>
  </div>
<?endif;?>
<?if($arResult["PROPS"]["DESCRIPTIONS"][2]["TEXT"]):?>
  <div id="banner_text_3" class="text" style="left:220px; opacity:0; display: none">
<table border="0" cellpadding="0" cellspacing="0"><tr><td>
    <a href="<?=$arResult["PROPS"]["LINKS"][2]?>"><?=$arResult["PROPS"]["DESCRIPTIONS"][2]["TEXT"]?></a>
</td></tr></table>
  </div>
<?endif;?>
<?if($arResult["PROPS"]["DESCRIPTIONS"][3]["TEXT"]):?>
  <div id="banner_text_4" class="text" style="left:284px; opacity:0; display: none">
<table border="0" cellpadding="0" cellspacing="0"><tr><td>
    <a href="<?=$arResult["PROPS"]["LINKS"][3]?>"><?=$arResult["PROPS"]["DESCRIPTIONS"][3]["TEXT"]?></a>
</td></tr></table>
  </div>
<?endif;?>
<?if($arResult["PROPS"]["DESCRIPTIONS"][4]["TEXT"]):?>
  <div id="banner_text_4" class="text" style="left:284px; opacity:0; display: none">
<table border="0" cellpadding="0" cellspacing="0"><tr><td>
    <a href="<?=$arResult["PROPS"]["LINKS"][4]?>"><?=$arResult["PROPS"]["DESCRIPTIONS"][3]["TEXT"]?></a>
</td></tr></table>
  </div>
<?endif;?>
</div>

Выборка свойств картинок и последующий вывод в шаблоне (выдернуто из самописного компонента Битрикс). Писал это очень давно, вот возникла необходимость править, открыл - посмотрел - случилась истерика!!! Очевидно что такое циклы я тогда еще не подозревал))

akovalenko, 26 Ноября 2010

Комментарии (13)

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

+148

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24

<?
	$optSelect['10'] = ($_REQUEST['sort_count']=='10') ? 'selected' : '';
	$optSelect['20'] = ($_REQUEST['sort_count']=='20') ? 'selected' : '';
	$optSelect['50'] = ($_REQUEST['sort_count']=='50') ? 'selected' : '';
	$optSelect['all'] = ($_REQUEST['sort_count']=='all') ? 'selected' : '';
	
	$remParams = array('sort_count');
        //функция GetCurPageParam в битрикс возвращает путь к текущей странице с добавлением/удалением параметров (автор поста) 
	$optValueURI['10'] = $APPLICATION->GetCurPageParam('sort_count=10', $remParams);
	$optValueURI['20'] = $APPLICATION->GetCurPageParam('sort_count=20', $remParams);
	$optValueURI['50'] = $APPLICATION->GetCurPageParam('sort_count=50', $remParams);
	$optValueURI['all'] = $APPLICATION->GetCurPageParam('sort_count=all', $remParams);
			
	$optValue['10'] = $siteURL . $optValueURI['10'];
	$optValue['20'] = $siteURL . $optValueURI['20'];
	$optValue['50'] = $siteURL . $optValueURI['50'];
	$optValue['all'] = $siteURL . $optValueURI['all'];
?>
<select onChange="location=this.value">
	<option <?=$optSelect['10']?> value="<?=$optValue['10']?>">10</option>
	<option <?=$optSelect['20']?> value="<?=$optValue['20']?>">20</option>
	<option <?=$optSelect['50']?> value="<?=$optValue['50']?>">50</option>
	<option <?=$optSelect['all']?> value="<?=$optValue['all']?>">Все</option>
</select>

Реализация Селекта количества выводимых элементов на странице. Сайт делал топовый партнёр 1c-Bitrix.
Я думаю, что разумнее было бы сделать так:
$arPages = array(10, 20, 50, 'all');
foreach ($arPages as $page) {?>
<option <?if($page == $_REQUEST['sort_count']){?>selected<?}?>>$page</option>
<?}?>
Ну и для сабмита можно какую нибудь нормальную функцию в JS написать.

Cool-Di, 26 Ноября 2010

Комментарии (292)

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

+171

1

echo eval($myrow_res['text']);

Выкакыш.

Uchkuma, 26 Ноября 2010

Комментарии (11)

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

+159

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23

function get_existing_boxes () {
		global $wp_meta_boxes, $more_fields;
		$data = $wp_meta_boxes;
		$boxes = array();
		foreach ((array) $data as $data1) {
			foreach ((array) $data1 as $data2) {
				foreach ((array) $data2 as $data3) {
					foreach ((array) $data3 as $box) {
						 if ($title = $box['title']) {
							 $boxes[$box['id']] = $box;
						 }
					}
				}
			}						
		}
		//if (is_object($more_fields)) {
	//		$mfs = $more_fields->get_objects(array('_plugin_saved', '_plugin'));
	//		foreach ($mfs as $mf_key => $mf) $boxes[$mf_key] = $mf;
	//	}
		//__d($wp_meta_boxes);

		return $boxes;
	}

Из плагина "more-post-types" для WP

Anderson, 25 Ноября 2010

Комментарии (2)

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

+145

1
2
3

function CalculateShippingCost() {
  return 20.00;
}

Из одного "реального" интернет-магазина!

dwinner, 25 Ноября 2010

Комментарии (3)

JavaScript / Говнокод #4753

+158

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17

function checkboxHandler(e) {
	var el = e.target;
	if(el.className == "checked") {
		el.className = "unchecked";
		showThumbs(el, false); 
	}
	else {
		el.className = "checked";
		showThumbs(el, true); 
	}
}
function showThumbs(el, stat) {
	if(stat)
		addThumbs(el.id);
	else
		removeThumbs(el.id);
}

zhegan, 25 Ноября 2010

Комментарии (7)

C# / Говнокод #4752

+145

1
2
3
4

public partial class TaskViewSearchParameter
{
    public int Top5 { get; set; }
}

Часть класса одного из наших проектов

ZlobnyiSerg, 25 Ноября 2010

Комментарии (5)

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