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

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

+157

1
2
3
4
5
6

/**
 * Появилась из-за опечатки. См. getCurrentEnterpriseIds()
 */
public function getCurrnetEnterpriseIds($user_id) {
    return $this->getCurrentEnterpriseIds($user_id);
}

рефакторинг? не, не слышал

ghost404, 18 Января 2012

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

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

+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

$data = "";
if (isset($_POST["json"]))
 $data = $_POST["json"];
$send_str = "";

 $jsonObj=json_decode($data);
 if(json_last_error() == JSON_ERROR_NONE)
 {
  if ($jsonObj)
  {
   if (strlen($jsonObj->pass) != 32 || strlen($jsonObj->login)==0 || strlen($jsonObj->userName)==0)
   {
    $send_str = '{"RegistrationActionError":{"type":"2"}}';  
   }
   else
   {
    // проверка на совпадение логина
    if ($db->userExists($jsonObj->login))
     $send_str = '{"RegistrationActionError":{"type":"1"}}';
    else
    {
     $user = $db->createUser($jsonObj->login, $jsonObj->pass, $jsonObj->userName, $jsonObj->mail);
     $send_str = '{"RegistrationActionComplete"}';
    }
   }
  }
  else
   $send_str = '{"RegistrationActionError":{"type":"2"}}';
 }
 else
  $send_str = '{"RegistrationActionError":{"type":"2"}}';

 echo $send_str;

Регистрация пользователя написанная человеком который себя позиционирует как пхпе миддл )

kyzi007, 18 Января 2012

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

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

+150

1
2
3
4

if (!isset($pmas_table[0][3])) $pmas_table[0][3]='';
if (!isset($pmas_table[0][5])) $pmas_table[0][5]='';
if (!isset($pmas_table[0][6])) $pmas_table[0][6]='';
if (!isset($pmas_table[0][7])) $pmas_table[0][7]='';

UserSide v2.48 again. Так как там каждая строчка имеет право быть на этом сайте, выкладываю The best of the best =)

Neka, 18 Января 2012

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

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

+161

1

while (!$db->query("INSERT INTO lots (lot_num) VALUES (".($unicnum=rand(1,99999)).")"));

Проверка и создание уникального рандомного идентификатора

dancebass, 17 Января 2012

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

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

+150

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

public function run() {
  $stime = microtime(TRUE);
  $this->header('Content-Type: text/html');
  $this->setcookie('testcookie', '1');
  $this->registerShutdownFunction(function() {
?></html><?php
  });

?><!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>It works!</title>
</head>
<body>
<h1>It works! Be happy! ;-)</h1>
............
............
............

phpDaemon Example.php line 63

LmSys, 17 Января 2012

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

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

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

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 / Говнокод #9159

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

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

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

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

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

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

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

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

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