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

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

+66

1
2
3
4
5
6
7
8

function sanitizeString($var)
  {
    trim($var);
    strip_tags($var);
    htmlspecialchars($var);
    stripslashes($var);
    mysql_real_escape_string($var);
  }

Защита от SQL инъекции - экранирование, http://www.askdev.ru/q/10604

Anon39AsAjYN, 14 Июня 2012

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

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

+65

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
54
55
56
57
58
59

<?php

$str = file_get_contents('source.txt');
$out = '';

$len = strlen($str);
$word = '';

$bad_words = array(
    'Аноним',
    'Птн',
    'Сен',
    'Ответ',
    'Пропущено',
    'ответов',
    'изображениями',
    'Нажмите',
    'jpg',
    'png',
    'gif',
    'Кб',
    'x',
);

$str=str_replace('Нажмите "ответ", чтобы увидеть тред целиком.','', $str);
$str=str_replace('Показана уменьшенная копия, оригинал по клику.','', $str);
$str=str_replace('Пропущено ','', $str);
$str=str_replace(' ответов, из них ','', $str);
$str=str_replace(' с изображениями.','', $str);
$str=str_replace('Аноним','', $str);
$str=str_replace('Ответы','', $str);
$str=str_replace('Ответ','', $str);
$str=str_replace('Птн','', $str);
$str=str_replace('Сен','', $str);
$str=str_replace('jpg','', $str);
$str=str_replace('png','', $str);
$str=str_replace('gif','', $str);
$str=str_replace('Кб','', $str);
$str=str_replace('x','', $str);



for ($i = 0; $i < $len; $i++) {
    $c = $str[$i];
    if (($c >= 'a') && ($c <= 'z') ||
            ($c >= 'A') && ($c <= 'Z') ||
            ($c >= 'а') && ($c <= 'я') ||
            ($c >= 'А') && ($c <= 'Я')
    ) {
        $word.=$c;
    } else {
        if ($word != '') {
            $out.=strtolower($word) . "\r\n";
        }
        $word = '';
    }
}
file_put_contents('dictonary.txt', $out);
?>

Из чьей-то пхп-вайпалки для известной имиджборды.
Генерирует словарь слов из текста, взятого с помощью Ctrl+C с любой из страниц борды.

7ion, 14 Июня 2012

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

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

+59

1
2
3
4

<table>
	<tr><th colspan="2">Если вы еще не зарегистрированы:</th></tr>
	<tr><td colspan="2"><a href="/registration.php">Перейдите к регистрации на сайте</a></td></tr>
</table>

Глубокий смысл в наличии colspan="2" постичь никак не получается...

domaster, 14 Июня 2012

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

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)

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