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

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

+147

1
2
3
4

$connect=mysql_connect($server,$login,$password);
if(!@$connect)
{echo "<b><font color='#CC0000'>Нет соединение mysql</font></b>";
exit();}

SunnyMagadan, 23 Августа 2010

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

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

+143

1
2
3
4
5
6

<?php

if (! (false === $cnt))
{
    // ...
}

Сначала от этой конструкции у меня взорвался мозг. Потом, когда я увидел аналогичную, только вместо false пустая строка, я понял, что «просто в пхп нет оператора "!===" ».

user654321, 23 Августа 2010

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

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

+144

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

function constructor(){
		parent::XXXModule();
		if (!class_exists('XXXSU'))
			require_once(join_path(dirname(__FILE__), 'classes', 'class.XXXSU.php'));
			
		if (!class_exists('XXXSUsRouter'))
			require_once(join_path(dirname(__FILE__), 'classes', 'class.XXXSUsRouter.php'));
			
		if (!class_exists('XXXSUFingerprint'))
			require_once(join_path(dirname(__FILE__), 'classes', 'class.XXXSUFingerprint.php'));
			
		if (!class_exists('XXXSUDao'))
			require_once(join_path(dirname(__FILE__), 'classes', 'class.XXXSUDao.php'));
			
		if (!class_exists('XXXSUCategory'))
			require_once(join_path(dirname(__FILE__), 'classes', 'class.XXXSUCategory.php'));
		if (!class_exists('XXXSUsLocale'))
			require_once(join_path(dirname(__FILE__), 'classes', 'class.XXXSUsLocale.php'));
			
		if (!class_exists('XXXSUsHelper'))
			require_once(join_path(dirname(__FILE__), 'classes', 'class.XXXSUsHelper.php'));
			
		if (!class_exists('XXXSUsEmailTemplate'))
			require_once(join_path(dirname(__FILE__), 'classes', 'class.XXXSUsEmailTemplate.php'));
		if (!class_exists('XXXSUsEmailTemplatesDao'))
			require_once(join_path(dirname(__FILE__), 'classes', 'class.XXXSUsEmailTemplatesDao.php'));
	}

конструктор класса - после некоторой маскировки имен(constructor,XXX,SU,join_path).

Писал javист, волею судеб, но против своей, пересевший на ненавистный php

функция join_path - не его, а из некоторого говновелосипеда.

Lure Of Chaos, 23 Августа 2010

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

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

+145

1
2
3
4
5
6
7

<?php
$names = array("Иван","Петр","Семен"); 
if ($names[0]=="Иван"): 
?>
<?php
Привет, Ваня!
endif; ?>

Вот такому коду учат на intuit.ru

Ded_Maksim, 23 Августа 2010

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

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

+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

Часть 1
=====================================
foreach ($_POST['massive'] as $key => $val)
{
@mysql_query("INSERT INTO table VALUES (NULL,'".$val."'");
} 	
======================================
Часть 2
======================================
if (!empty($_POST['massive'])/*&&is_array($_POST['massive']) для любителей пожестче=)*/)
{
//Так мы точно знаем что нам что-то передали(не выскочит нотис).
//Далее надо сформировать правильный запрос. Никогда, НИКОГДА нельзя делать кучу инсертов циклом, т.к. скорее всего это закончиться смертью сервера.
//А сделать как-нить так

foreach ($_POST['massive'] as $key => $val)
{
 $query.="('".mysql_real_escape_string($val)."'),"; //Cделаем заготовку для запроса, попутно параметр экранируется, что бы злобные хакеры не подсунули нам кавычку и прочую гадость.
}
$query= 'INSERT INTO table VALUES'.substr($query, 0, -1).';';//Обрежем последнюю лишнюю запятую, и в итоге получим запрос вида INSERT INTO table VALUES (),(),()..();
if (!mysql_query($query)) //Скажу по секрету, что ставить собаку это моветон.
{
 print "Упс, что-то сломалось";
 print mysql_error();
}else {print "Вот и все"}
==========================================

Вобщем, увидел на одном форуме часть 1.
Будучи в хорошем настроении решил поправить. И Выдал часть 2, прямо вот в форуме. Думал отправить.. Но попахивает же, да?

vitaly, 23 Августа 2010

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

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

+104

1
2
3
4

$a = !empty($_POST['a'])?$_POST['a']:null;
   if (empty($_POST['submit']) || !is_numeric($a)) {
     } else {
        echo "Прошол!";

Vasiliy, 23 Августа 2010

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

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

+171

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

(
	$hours >= 21 && $minutes >= 0	
	or $hours == 0 && $minutes >= 0
	or $hours == 1 && $minutes >= 0
	or $hours == 2 && $minutes >= 0
	or $hours == 3 && $minutes >= 0
	or $hours == 4 && $minutes >= 0
	or $hours == 5 && $minutes >= 0
	or $hours == 6 && $minutes >= 0
	or $hours == 7 && $minutes >= 0
	or $hours == 8 && $minutes >= 0
)

http://webcodes.ru/publ/1-1-0-3524

uWeb, 23 Августа 2010

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

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

+165

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

<?php

$luser=$this->get_authorized_user();
if (!$luser) retuen -101;

// ...

  if (!$lpwd) 
  {
   $db->disconnect();
   return -102;
  }

// ...

 if (!is_array($array)){$db->disconnect();return -103;}
  
 if (0 == count($array)){$db->disconnect();return -104;}

// ...

 if (!is_array($array)){$db->disconnect();return -105;}
  
 if (0 == count($array)){$db->disconnect();return -106;}

орфография и правописание сохранены

вот таких отрицательных "статусов" по всему проекту просто миллион. Только одному автору известно, что они значат.

и работает это все с неправильно написанным "return" в первом блоке

user654321, 23 Августа 2010

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

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

+175

1
2

sleep(10);
header('Location: http://example.com');

Называется "Редирект через 10 секунд".
Собственно автору требовалось это: <meta http-equiv="refresh" content="10; url=index.php">.

Uchkuma, 23 Августа 2010

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

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