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

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

+172

1
2
3

<td class="td_caption" style="width: 40%"><label for="pohuy">Запомнить</label></td>

		<td class="td_value"><input type="checkbox" name="pohuy" checked="checked" id="pohuy"/></td>

"Я ХАЧУ ГАЛОЧКУ ЗАПОМНИТЬ ПОД ПАРОЛЕМ БИЛЯТЬ", - как скажешь начальник.

*параметр "pohuy" как вы поняли нигде и никак не обрабатывается.

jokz, 07 Декабря 2011

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

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

+163

1
2
3

if ($linksCount == 0) $linksCount = -1; // for no error
	$percent = round(($linksOkIndex/$linksCount)*100, 0);
	if ($linksCount == -1) $linksCount = 0; // for no error

Я так избегаю деления на ноль -)

increazon, 06 Декабря 2011

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

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

+167

1
2
3
4
5
6
7
8

if (!$xhr)
{
	//...
} 
else
{
	echo $goodUpload ? 'ok' : 'ne ok';							
}

Ответ формы на AJAX-запрос

f0rm4t, 06 Декабря 2011

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

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

+157

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
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88

function &_static($name = null, $value = null, $unsetVal = false) {
  static $data = array();
  $null = null;
  if (is_array($name)) {
    $data = $name;
    return $null;
  }
  $state = 0;
  $args = array($name, $value, $unsetVal);
  while (count($args)) {
    $arg = array_shift($args);
    switch ($state) {
      case 0:
        if ($arg === null) {
          $state = 8;
        }
        elseif ($arg !== null) {
          $state = 1;
        }
        else {
          throw new \InvalidArgumentException();
        }
        break;

      case 1:
        if ($arg === null) {
          $state = 5;
        }
        elseif ($arg !== null) {
          $state = 2;
        }
        else {
          throw new \InvalidArgumentException();
        }
        break;

      case 2:
        if ($arg === true) {
          $data[$name] = $value; // $name, $value, true - overwrite saved value and return reference on it.
          return $data[$name];
        }
        elseif ($arg === false) {
          if (!array_key_exists($name, $data)) {
            $data[$name] = $value;
          }
          return $data[$name]; // $name, $value, false - save value if not exist, return reference on stored value.
        }
        else {
          throw new \InvalidArgumentException();
        }

      case 5:
        if ($arg === false) {
          if (!array_key_exists($name, $data)) {
            $data[$name] = $value; // $name, null, false, - initialize var if not exist, return reference on stored value.
          }
          return $data[$name];
        }
        elseif ($arg === true) {
          $data[$name] = $value; // $name, $value, true - overwrite saved value and return reference on it.
          return $data[$name]; // $name, null, true - overwrite saved value and return reference on it.
        }
        else {
          throw new \InvalidArgumentException();
        }
        break;

      case 8:
        if ($arg === null) {
          continue;
        }
        if ($arg === true) {
          $data = array(); // null, null, true - unset all
          break 2;
        }
        elseif ($arg === false) {
          return $data; // null, null, false - return all
        }
        else {
          throw new \InvalidArgumentException();
        }

      default:
        throw new \RuntimeException('Unknown state.');
    }
  }
  return $null;
}

Как сделать глобальные переменные своими руками?
Как сделать ветвление когда под рукой не оказалось if, зато есть while и switch?

sectus, 05 Декабря 2011

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

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

+177

1
2
3
4
5
6
7

"
<td><span>\".setDiskont2(\$data[price]*{$as_Quantity}, $current_site[Skidka], \$data[Discount]).\"</span></td>
<th></th>
</tr>
  ")." 
  ".as_if($totalsum+=as_query("SELECT price FROM Data".$as_ObjectID." WHERE Data_ID=$as_ItemID","\$data[price]")*$as_Quantity*$as_Discount,"")."
")."

Оцените.
Этот кусок кода хранится в поле таблицы бд, в котором хранится кусок шаблона формы корзины интернет-магазина.
Функция на шестой строке выводит общую сумму заказа при eval.
У меня ушло более двух часов, чтобы розыскать логику внутри семикратной (! я не шучу) вложенности include_once, чтобы в итоге понять, что запрос который я искал хранится в самой бд, причем не просто храниться, а хранится внутри бд, внутри php который хранится внутри html... Сам Доминик Кобб позавидовал бы степени вложенности моих кошмаров %)

antongorodezkiy, 04 Декабря 2011

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

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

+159

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

$t2 = db_query("SELECT
      			COUNT(*)
  			FROM
     			order_delivery_offers AS odf,
     			uc_orders AS uo
  			WHERE
     			odf.order_id = uo.order_id AND
     			uo.uid = $uid AND
     			uo.order_status = 'accepted' AND
     			odf.status = 'accepted' AND
     			odf.visible = 1 AND
     			odf.active = 1
  			GROUP BY odf.order_id");
  $count_rows2 = 0;
  while ($r2 = db_fetch_array($t2)){
		$count_rows2++;
  }

Подсчет количества записей в запросе.

beertudam, 04 Декабря 2011

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

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

+160

1
2
3
4
5
6
7
8

const ORDER_FUNC_ARGS   = '$a, $b';
	const ORDER_FUNC_LOGIC  = '$a=strlen($a);$b=strlen($b);return$a==$b?0:($a<$b?1:-1);';
// а дальше в одном из методов
		$this->_userAgents 	= array_keys($browsers);
		usort(
			$this->_userAgents,
			create_function(self::ORDER_FUNC_ARGS, self::ORDER_FUNC_LOGIC)
		);

Для меня это странно выглядит

LiteError, 03 Декабря 2011

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

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

+159

1
2
3
4
5
6

protected $_never_allowed_regex = array(
					"javascript\s*:"			=> '[removed]',
					"expression\s*(\(|&\#40;)"	=> '[removed]', // CSS and IE
					"vbscript\s*:"				=> '[removed]', // IE, surprise!
					"Redirect\s+302"			=> '[removed]'
	);

Это не разу не ковнокод, но строкой с // IE, surprise! не поделится, не смог)

ЗЫ. это CodeIgniter 2.0.3 класс Security, строка 52

Zerstoren, 02 Декабря 2011

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

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

+159

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
60
61
62
63
64
65

private function isBonusAddOK($code,$ffStr)
	{
		switch($code)
		{
			case 1:if(
						($this->isBFFOK($ffStr,40) || $this->isBFFOK($ffStr,38) || $this->isBFFOK($ffStr,37)) && 
						$this->isBFFOK($ffStr,24) && $this->isBFFOK($ffStr,19) &&
						($this->isBFFOK($ffStr,42) || $this->isBFFOK($ffStr,43) || $this->isBFFOK($ffStr,44))
						)return true;return false;
						
			case 6:if(
						($this->isBFFOK($ffStr,40) || $this->isBFFOK($ffStr,38) || $this->isBFFOK($ffStr,37)) && 
						$this->isBFFOK($ffStr,24) && $this->isBFFOK($ffStr,20) &&
						($this->isBFFOK($ffStr,42) || $this->isBFFOK($ffStr,43) || $this->isBFFOK($ffStr,44))
						)return true;return false;
						
			case 11:if(
						$this->isBFFOK($ffStr,24) &&
						($this->isBFFOK($ffStr,16) || $this->isBFFOK($ffStr,358) || $this->isBFFOK($ffStr,360) || $this->isBFFOK($ffStr,361)) && 
						$this->isBFFOK($ffStr,62) &&
						($this->isBFFOK($ffStr,64) || $this->isBFFOK($ffStr,352) || $this->isBFFOK($ffStr,356) || $this->isBFFOK($ffStr,363)) &&
						($this->isBFFOK($ffStr,3) || $this->isBFFOK($ffStr,4) || $this->isBFFOK($ffStr,5))
						)return true;return false;
						
			case 14:if(
						$this->isBFFOK($ffStr,24) &&
						($this->isBFFOK($ffStr,16) || $this->isBFFOK($ffStr,358) || $this->isBFFOK($ffStr,360) || $this->isBFFOK($ffStr,361)) && 
						$this->isBFFOK($ffStr,61) &&
						($this->isBFFOK($ffStr,64) || $this->isBFFOK($ffStr,352) || $this->isBFFOK($ffStr,356) || $this->isBFFOK($ffStr,363)) &&
						($this->isBFFOK($ffStr,3) || $this->isBFFOK($ffStr,4) || $this->isBFFOK($ffStr,5))
						)return true;return false;
			
			case 17:if(
						($this->isBFFOK($ffStr,16) || $this->isBFFOK($ffStr,358) || $this->isBFFOK($ffStr,360) || $this->isBFFOK($ffStr,361)) && 
						($this->isBFFOK($ffStr,64) || $this->isBFFOK($ffStr,352) || $this->isBFFOK($ffStr,356) || $this->isBFFOK($ffStr,363)) &&
						($this->isBFFOK($ffStr,3) || $this->isBFFOK($ffStr,4) || $this->isBFFOK($ffStr,5)) &&
						$this->isBFFOK($ffStr,17)
						)return true;return false;
			
			case 20:if(
						($this->isBFFOK($ffStr,16) || $this->isBFFOK($ffStr,358) || $this->isBFFOK($ffStr,360) || $this->isBFFOK($ffStr,361)) && 
						($this->isBFFOK($ffStr,64) || $this->isBFFOK($ffStr,352) || $this->isBFFOK($ffStr,356) || $this->isBFFOK($ffStr,363)) &&
						($this->isBFFOK($ffStr,3) || $this->isBFFOK($ffStr,4) || $this->isBFFOK($ffStr,5)) &&
						$this->isBFFOK($ffStr,25) && $this->isBFFOK($ffStr,21)
						)return true;return false;
						
			case 23:if(
						($this->isBFFOK($ffStr,16) || $this->isBFFOK($ffStr,358) || $this->isBFFOK($ffStr,360) || $this->isBFFOK($ffStr,361)) && 
						($this->isBFFOK($ffStr,64) || $this->isBFFOK($ffStr,352) || $this->isBFFOK($ffStr,356) || $this->isBFFOK($ffStr,363)) &&
						($this->isBFFOK($ffStr,3) || $this->isBFFOK($ffStr,4) || $this->isBFFOK($ffStr,5)) &&
						$this->isBFFOK($ffStr,21)
						)return true;return false;
						
						
		.... еще куча строк до значения 88
			
			default: return true;
		}
	}
	
	private function isBFFOK($ffStr,$pos)
	{
		$val = substr($ffStr,$pos,1);//echo $pos.':'.$val;var_dump(($val != '0'));echo ' ';
		return ($val != '0');
		/*if($val == 0)return false;

ИфЗебок

kyzi007, 02 Декабря 2011

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

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

+167

1

( $resultat == 1 AND $resultat != '' AND $resultat != 0 ) ? true : false;

Что-бы наверняка.

nethak, 02 Декабря 2011

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

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

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

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

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

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

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

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

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

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

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

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