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

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

+53

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

<?php $args = array(  
        'type'                     => 'post'  
        ,'child_of'                 => 0  
        ,'parent'                   => ''  
        ,'orderby'                  => 'name'  
        ,'order'                    => 'ASC'  
        ,'hide_empty'               => 1  
        ,'hierarchical'             => 1  
        ,'exclude'                  => ''  
        ,'include'                  => ''  
        ,'number'                   => 0  
        ,'taxonomy'                 => 'category'  
        ,'pad_counts'               => false );?> 

http://wp-kama.ru/function/get_categories

взял тут: http://wp-kama.ru/function/get_categories

Видимо у него в учебнике так было:

У лукоморья дуб зелёный
;Златая цепь на дубе том
:И днём и ночью кот учёный
Всё ходит по цепи кругом
;Идёт направо - песнь заводит
,Налево - сказку говорит
...

А.С. Пушкин

irvis, 13 Июня 2012

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

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

+55

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

protected function _convertArrayToStr($arr = array()) {

	$str = '';
	$data = Controller::getSomeMethod();


	$valus = array_values($arr);
	$keys = array_keys($arr);
	//Количество незаполненых полей
	$count = 0;
	// Формируем строку с title и введеных значени
	for($i = 0; $i < count($arr); ++$i) {

		if($keys[$i] == $data[$i]['name'] ) {
			
			if($valus[$i] == 'on') {
				$valus[$i] = 'Да';
			}

			$str .= $data[$i+$count]['title'] . ": " . $valus[$i] . " \n";

		}else{

			$count++;
			$str .= $data[$i+$count]['title'] . ": " . $valus[$i] . " \n";

		}
	}
	return $str;

}

Стажер реализовал обход ассоциативного массива и преобразование значений массива в строку.

Iliander, 13 Июня 2012

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

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

+61

1
2
3
4
5
6
7

if (empty($columns)) {
	$columns = 'eleven';
} else {
	// Check the function for a returned variable
        // (от автора) КОГО КОГО МЫ ТАМ ЧЕКАЕМ??
	$columns = $columns;
}

Найдено в достойном шаблоне Skeleton для WordPress
/(. . )

blessed, 13 Июня 2012

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

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

+62

1
2
3
4
5

if(isset($_GET['show_all'])){
		set_query('show_all=','',true);
		$show_all = true;
		$Register->set('show_all', $show_all);
}

Найдено в WebAsyst Shop-Script.

BiggestFox, 13 Июня 2012

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

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

+55

01
02
03
04
05
06
07
08
09
10

$i = 0;
foreach($_POST['union'] as $key=>$value) {
	$i++;
	if ($i == 1) {
		$fid = $key;
	}
	if ($i > 1) {
		[код поскипан]
	}
}

Ну почему не используется if { } else { }?

domaster, 13 Июня 2012

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

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

+69

1

$file[] = $_FILES[''.$key.''];

Подобное "приведение типов (?)" для $key просто убивает. Размазано по всему проекту...

domaster, 13 Июня 2012

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

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

+52

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

<?if (!defined("B_PROLOG_INCLUDED") || B_PROLOG_INCLUDED!==true)die();?>
<?if (!empty($arResult)):?>
<ul id="horizontal-multilevel-menu">
<?
$previousLevel = 0;
foreach($arResult as $arItem):?>
	<?if ($previousLevel && $arItem["DEPTH_LEVEL"] < $previousLevel):?>
		<?=str_repeat("</ul></li>", ($previousLevel - $arItem["DEPTH_LEVEL"]));?>
	<?endif?>
	<?if ($arItem["IS_PARENT"]):?>
		<?if ($arItem["DEPTH_LEVEL"] == 1):?>
			<li><a href="<?=$arItem["LINK"]?>" class="<?if ($arItem["SELECTED"]):?>root-item-selected<?else:?>root-item<?endif?>"><?=$arItem["TEXT"]?></a>
				<ul>
		<?else:?>
			<li<?if ($arItem["SELECTED"]):?> class="item-selected"<?endif?>><a href="<?=$arItem["LINK"]?>" class="parent"><?=$arItem["TEXT"]?></a>
				<ul>
		<?endif?>
	<?else:?>
		<?if ($arItem["PERMISSION"] > "D"):?>
			<?if ($arItem["DEPTH_LEVEL"] == 1):?>
				<li><a href="<?=$arItem["LINK"]?>" class="<?if ($arItem["SELECTED"]):?>root-item-selected<?else:?>root-item<?endif?>"><?=$arItem["TEXT"]?></a></li>
			<?else:?>
				<li<?if ($arItem["SELECTED"]):?> class="item-selected"<?endif?>><a href="<?=$arItem["LINK"]?>"><?=$arItem["TEXT"]?></a></li>
			<?endif?>
		<?else:?>
			<?if ($arItem["DEPTH_LEVEL"] == 1):?>
				<li><a href="" class="<?if ($arItem["SELECTED"]):?>root-item-selected<?else:?>root-item<?endif?>" title="<?=GetMessage("MENU_ITEM_ACCESS_DENIED")?>"><?=$arItem["TEXT"]?></a></li>
			<?else:?>
				<li><a href="" class="denied" title="<?=GetMessage("MENU_ITEM_ACCESS_DENIED")?>"><?=$arItem["TEXT"]?></a></li>
			<?endif?>
		<?endif?>
	<?endif?>
	<?$previousLevel = $arItem["DEPTH_LEVEL"];?>
<?endforeach?>
<?if ($previousLevel > 1)://close last item tags?>
	<?=str_repeat("</ul></li>", ($previousLevel-1) );?>
<?endif?>
</ul>
<div class="menu-clear-left"></div>
<?endif?>

Очередной говнокод от битрикса. "Генерация" выпадающего горизонтального меню.

avtorfox, 12 Июня 2012

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

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

+141

1
2
3
4
5

function toArray($xml) {
                $xml = simplexml_load_string($xml);
                $json = json_encode($xml);
                return json_decode($json,TRUE);                        
        }

Но зачем?!

mr.The, 11 Июня 2012

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

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

+54

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

foreach($Articles as $art){
			$id = $art['id'];
			$count++;	
			if(isset($_GET['str'])){
				$str = (int)$_GET['str']; 
				$mn = $number*($str-1)+1;
				}else{
					$str = 1;
					$_GET['str'] = 1;
				}	
			if($count == $mn){
				$flag = true;
				$k=5;
			}		
			if($flag && $k!=0){
				$k--;
				$title = $art['title'];
				$content = $art['content'];
				$author = $art['author'];
				$date = $art['date'];
				printf("<h1>%s</h1><br /><h3 align='left'>%s</h3>%s  <a href='index.php?id=articles&str=%s&nomer=%s'>Далее</a><hr /><br />",$title,$date, substr($content, 0,800),$nav, $id);
					}
		}

Постраничная навигация

BrdNLSrg, 10 Июня 2012

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

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