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

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

+173

1
2
3
4
5
6
7
8

/*
      if($_REQUEST["act_id"]==8){
      а нет. маньяки так не пишут...
*/
define("Э", $_REQUEST["act_id"]);
if( 8===Э ){ 
	...
}

оператор члена :)

basename, 17 Января 2012

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

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

+161

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

<?php
//...
while (($data = fgetcsv($handle, 1000, $this->CSV_SEPARATOR, $this->CSV_ENCLOSURE)) !== FALSE) {
				$num = count($data);
				$item = array();

				for ($c=0; $c < count($data); $c++) {
					$item[] = $data[$c];
				}

//некие действия с $item. $data больше нигде используется

?>

Итого, $num не используется в цикле, да и сам по себе цикл - бесполезен, можно было сделать просто $item = $data; Не говоря уже о том, что можно просто использовать $data.

Код из одного модуля для opencart.

mr.The, 16 Января 2012

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

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

+149

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

<?php
function check_brackets($s)
{
    $brackets = array('(' => ')' , '[' => ']', '{' => '}');
    $stack = array();
    
    for($i = 0; $i < strlen($s); $i++)
    {
        if(in_array($s[$i], array_keys($brackets)))
        {
            array_push($stack, $brackets[$s[$i]]);
        }
        elseif(in_array($s[$i], array_values($brackets)))
        {
            if(empty($stack) || array_pop($stack) != $s[$i])
            {
                return false;
            }
        }
    }
    if(empty($stack)) return true;
}

if($_POST["bracket_string"] && $_POST["bracket_string"] <= 30)
{
    if(check_brackets($_POST["bracket_string"]))
    {
        $message = "Check passed";
    }
    else $message = "Check failed";
}

?>

Финальная версия.

varg242, 16 Января 2012

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

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

+158

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

//Рекурсивная функция цепной проверки категории
	function category(){
		static $category;
		static $level=0;
		$myrow = mysql_query("SELECT id,pot_category FROM categroy WHERE sysname='".$_GET['do'][$level]."'");
		if(mysql_num_rows($myrow)){
		$result = mysql_fetch_assoc($myrow);
		if($result['no_viev']==0){
		if($result['sysname']==end($_GET['do'])){
			return 'category';
			}else{
				$level++;
				$this->category();
				}
				}else{return 'no_viev';}
		}else{
			//Если категории не найдено то показываем ошибку 404
			return FALSE;}
		
		}

Бессмысленная рекурсия

lans8097, 16 Января 2012

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

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

+150

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

function check($string_to_search) {
$stack = array();
foreach (preg_split("//", $string_to_search) as $char) {
switch ($char) {
case '{':
case '(':
case '[':
array_push($stack, $char);
break;
case '}':
if (array_pop($stack) != "{") return false;
break;
case ')':
if (array_pop($stack) != "(") return false;
break;
case ']':
if (array_pop($stack) != "[") return false;
break;
}
}
return true;
}

by kolen

varg242, 16 Января 2012

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

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

+146

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

<?php
function check($s)
{
    $brackets = array(')' => '(', ']' => '[', '}' => '{');
    $stack = array();
    $stack_size = 0;
    for($i = 0; $i < strlen($s); $i++)
    {
	if (in_array($s[$i], array_values($brackets)))
        {
    	    $stack[$stack_size++] = $s[$i];
        }
        else if (in_array($s[$i], array_keys($brackets)))
        {
    	    $last = $stack_size ? $stack[$stack_size-1] : '';
            if ($last != $brackets[$s[$i]])
            {
        	return false;
    	    }
            else
            {
    		unset($stack[--$stack_size]);
    	    }
    	}
    }
    return count($stack) == 0;
}

function check_brackets($s)
{
    if(check($s))
    {
        return true;
    }
    else return false;
}

if($_POST["bracket_string"])
{
    if(check_brackets($_POST["bracket_string"]))
    {
        $message = "Check passed";
    }
    else $message = "Check failed";
}

?>

Пацаны, есть задание. Срочно заговнокодить код, но чтобы точно работало.

varg242, 16 Января 2012

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

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

+155

1
2
3
4
5
6

if ($num > ($i-4)) {
	$delta = $i - $num;	
	if ($delta == 1) $num = $num - 3;
	if ($delta == 2) $num = $num - 2;
	if ($delta == 3) $num = $num - 1;
}

где i это кол-во элементов, num - текущий элемент, delta - разница между количеством и текущим
Задача: уникализировать алгоритм. Мне не нравится вот это
if ($delta == 1) $num = $num - 3;
if ($delta == 2) $num = $num - 2;
if ($delta == 3) $num = $num - 1;
но как сделать это одной строкой пока придумать не могу...

Single, 16 Января 2012

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

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

+147

1
2
3
4
5
6

if($row['status_id'] != 6){
    if($row['assist_type'] == 0){
        $row['assist_type'] == 3;
    }
    ...
}

:-|

temka, 13 Января 2012

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

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

+167

01
02
03
04
05
06
07
08
09
10

$zfunctv=0;
if ($zfunctv==1 || $zfunctv==2 || $zfunctv==3 || $zfunctv==4) require_once('function1.php');
if ($zfunctv==2 || $zfunctv==3) require_once('function2.php');
if ($zfunctv==3 || $zfunctv==4) require_once('function3.php');
if ($zfunctv==''){
	require_once('function1.php');//Для всех окон
	require_once('function2.php');//Для полноразмерных окон
	require_once('function3.php');//Для оборудования
	require_once('admfunct.php');
}

Какого хрена? Как это работает?! Код коммерческой системы UserSide v2.48

Neka, 13 Января 2012

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

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

+171

1
2
3
4
5

$date_day = date("d");
$date_month = date("m");
$date_year = date("Y");
$date_time = date("H:i");
$date_cont = " " . $date_day . "-" . $date_month . "-" . $date_year . " " . $date_time;

Вот так мы получаем дату строкой :)
Впоследствии используется только $date_cont.

andymitrich, 12 Января 2012

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

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

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

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

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

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

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

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

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

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

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

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