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

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

+4

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

$query = "
	SELECT 	packet, info FROM packet ORDER BY info";
$result = mysql_query($query);
while ($row = mysql_fetch_array($result)) /* Тут очень много строк */
{
	/*Тут один очень жирный запрос*/
	/* ... */
	$res8 = mysql_query("SELECT date FROM analiz_packet2 WHERE date='$dnow' LIMIT 1");
		
	if (mysql_num_rows($res8) == 0)
	{
		echo "<td align=right>{$r[2]}</td>";
	}
	
	echo "<td align=right>{$r[3]}</td>";
	
	$res2 = mysql_query("SELECT date FROM analiz_packet2 WHERE date='$dnow' LIMIT 1	");
	/* Unknown date */
	if (mysql_num_rows($res2) == 1)
	{
		/*Тут еще несколько SQL запросов*/
	}
	
	/* ... */
}

Два ядра грузятся на 178% по данным top-а

Запостил: Werdn, 26 Июля 2010

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

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

+11

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

switch(count($Args)) {
         case 0:
            $Result = new $ClassName; break;
         case 1:
            $Result = new $ClassName($Args[0]); break;
         case 2:
            $Result = new $ClassName($Args[0], $Args[1]); break;
         case 3:
            $Result = new $ClassName($Args[0], $Args[1], $Args[2]); break;
         case 4:
            $Result = new $ClassName($Args[0], $Args[1], $Args[2], $Args[3]); break;
         case 5:
            $Result = new $ClassName($Args[0], $Args[1], $Args[2], $Args[3], $Args[4]); break;
         case 6:
            $Result = new $ClassName($Args[0], $Args[1], $Args[2], $Args[3], $Args[4], $Args[5]); break;
         case 7:
            $Result = new $ClassName($Args[0], $Args[1], $Args[2], $Args[3], $Args[4], $Args[5], $Args[6]); break;
         case 8:
            $Result = new $ClassName($Args[0], $Args[1], $Args[2], $Args[3], $Args[4], $Args[5], $Args[6], $Args[7]); break;
         default:
            throw new Exception();
      }

Взято из форума Vanilla 2...

И еще в одном файле подобное library/core/class.dispatcher.php (со строки 267).

Запостил: Александр Михалицын, 24 Июля 2010

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

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

+1

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

<?php

class Control {

    const
    ENV_VAR = 'REQUEST_URI',
    DIR = _SCRIPT,
    MAIN = 'default',
    FILENAME_MASK = '%s.ctrl.php',
    CHAR = '.';

    private static function _parametrize(&$var) {
        static $params = null;
        (null === $params) && ($params =& self::$_params);

        ((is_numeric($var))
                && (($params[] = $var) || 1)
                || (self::CHAR === $var[0])
                        && (($params[] = substr($var, 1)) || 1)
                || (false !== ($dot = strpos($var, self::CHAR)))
                        && (($params[substr($var, 0, $dot)] = substr($var, $dot + 1)) || 1))
                && ($var = null);
    }


    public static function call($name, $_PARAMS = array()) {
        static $_TPL = null;
        (null === $_TPL) && ($_TPL =& $GLOBALS['_TPL']);
        static $_DATA = null;
        (null === $_DATA) && ($_DATA =& $GLOBALS['_DATA']);
        static $_MY = -1;
        (-1 === $_MY) && ($_MY =& $GLOBALS['_MY']);

        $filename = self::DIR . '/' . sprintf(self::FILENAME_MASK, basename($name));
        return (file_exists($filename) && include($filename));
    }
}

На каком языке раньше писал автор?? Где тут С++?
Платили за каждый символ??

Запостил: doremidon, 23 Июля 2010

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

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

+16

1
2
3
4
5
6
7

<?php 

$visitor = $_SERVER['REMOTE_ADDR']; 
if (preg_match("/217.66.26.85/",$visitor))  
{header('Location: http://site.ru/page.html');} 

?>

http://forum.searchengines.ru/showpost.php?p=7312251&postcount=1

Запостил: GoodTalkBot, 23 Июля 2010

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

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

+13

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

function endPanel()
	{
		if($this->type=="common" || $this->type=="common_blank")
		{
				?>
								</div>
						</div>
				</div>
				<?php
		}
		elseif($this->type=="common_short")
		{
				?>
								</div>
						</div>
				</div>
				<?php
		}
		elseif($this->type=="main" || $this->type=="main_small")
		{
			?>
					</div>
				</div>
				<div class="bottom">
					<div class="bottom-right"></div>
				</div>
			</div>
			<?php
		}
		elseif($this->type=="light" || $this->type=="light_blank" || $this->type=="light_narrow")
		{
			?>
					</div>
				</div>
				<div class="bottom">
					<div class="bottom-right"></div>
				</div>
			</div>
			<?php
		}
	}

Прислали на работе проект на доработку, мало того предыдущий программер не знает про разделение логики и представления, дак еще и такие штуки на каждом шагу встречаются

Запостил: xoros, 22 Июля 2010

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

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

+1

1
2
3
4
5
6
7
8

<?php

if(true) {
...
}
else $blacklist_numbers_error="неверный тип файла";

?>

разбираю на работе код ><

Запостил: DmitryDick, 22 Июля 2010

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

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

+12

1
2
3
4
5
6
7
8
9

<?php
// Парсируем установочный файл
if(@parse_ini_file("../../phpshop/inc/config.ini",1))
$SysValue=parse_ini_file("../../phpshop/inc/config.ini",1);
 elseif(@parse_ini_file("../../../phpshop/inc/config.ini",1))
    $SysValue=parse_ini_file("../../../phpshop/inc/config.ini",1);
	  elseif(@parse_ini_file("../../../../phpshop/inc/config.ini",1))
	      $SysValue=parse_ini_file("../../../../phpshop/inc/config.ini",1);
	     else $SysValue=@parse_ini_file("../../../../../phpshop/inc/config.ini",1);

PHPShop

Запостил: Under, 22 Июля 2010

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

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

−7

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

$db = new sql_db($server_db, $user_db, $pass_db, $name_db);
$db->sql_query("SELECT * FROM `".$prefix."_global` WHERE `name`='global';");
$row = $db->sql_fetchrow();

define("LANGUAGE_ADMIN","{$row['language_admin']}");
define("MODUL_HOME_ADMIN","{$row['home_modul_adm']}");
define("THEMA_ADMIN","{$row['thema_admin']}");

require_once ("language/".LANGUAGE_ADMIN."/ind.php");

if(IN_ADMIN){
define("IN_ADMIN", TRUE);

	if (!file_exists ("modules/".MODUL_HOME_ADMIN."/index.php"))
	die (UNIT_NO_DEFAULT);
	
	if (isset($_GET ['am'])){ $MODULES = $_GET ['am']; }
	else { $MODULES = MODUL_HOME_ADMIN; }

	if (file_exists ("modules/".$MODULES."/index.php")) { 
		require ("modules/".$MODULES."/index.php");
	}
	else{ echo UNIT_IS_NOT_FOUND; }
	
	if (file_exists (Subject_file_path)) {
		$TEMPLATES_SITE = file_get_contents(Subject_file_path);
	}
	else{
		echo SUBJECTS_NOT;
	}

	require(Connect_funct_temp);

	echo $COD_TEMP;

if(!check_user($UserID)) logout();

} else { require(Authorisation_file_path); }

Пишу CMS и просто интересно мнение... Код короткий, так как многое через define()
И очень интересно Ваше мнение по нему, зачем define(), чтобы человек первый встречный не наизменял там чего, чтото типа защита от ламера, НО от ЛАМЕРА только) Просто программер так всё поймёт...

Кароче я тут в первые, и поэтому я ХЗ чё вам сказать ещё))

Запостил: nethak, 21 Июля 2010

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

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

+8

1

$black = imagecolorallocate($img, 255, 255, 255);

Не совсем говнокод, конечно, но все же.

Запостил: Uchkuma, 20 Июля 2010

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

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

+6

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

// говнецо терпимое
...
//говнецо зачетное:
case 'error':
$error=array();  
if (!empty($_POST))  
{  
$login=mysql_escape_string(stripslashes(trim($_POST['name']))); 
$text=mysql_escape_string(stripslashes(trim($_POST['text']))); 
if (empty($login))  
$error['empty']='Ошибка. Возможные причины: Логин';
if (empty($text))  
$error['empty']='Ошибка. Возможные причины: комментарий';

$select = mysql_query("SELECT `id` FROM `blog`");
while ($row_id = mysql_fetch_array($select))
{
if (!$row_id == $_GET[id_news])
{
echo 'нет такойва id новости'; 
}
}

break;

// дальше говнецо

if (!$row_id == $_GET[id_news]) //гениально, оригинал http://forum.wen.ru/?p=3&f=6&t=365157

Запостил: GoodTalkBot, 20 Июля 2010