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

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

+143

1
2
3
4
5
6

$a = "1";
$b = "$a";

if($a == $b) {
echo '$a = $b';
}

Восемь часов за компом.. Посмотрел на код еще раз :D

bafoed, 07 Сентября 2010

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

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

+143

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

// Так мы генерируем страницу:
ob_start();

renderPage($price, $order);

$renderPage = ob_get_clean();

// Функция renderPage выглядит так:
function renderPage(&$setup, &$order){
?>
<div id="setupMenu">
<!-- дальше простой html с подстановкой php-переменных -->
<?php } ?>

Вот почему у функции нельзя было сделать простой return, а надо было закрывать скобки php и затем при вызове использовать ob_*, чтобы получить html.

netrain, 07 Сентября 2010

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

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

+166

1

$need	= explode(',', 'name,phone');

Оригинальный способ создания массивов

netrain, 07 Сентября 2010

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

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

+162

01
02
03
04
05
06
07
08
09
10
11

if(!function_exists('getImg')){
    function getImg($isVip)
    {
    $type = false;
    if (isset($_GET['f']) && $_GET['f']) $type = 1;
    if (isset($_GET['bl']) && $_GET['bl']) $type = 16;
    if (isset($_GET['lw']) && $_GET['lw']) $type = 'author';
    if (isset($_GET['sym']) && $_GET['sym'] && $isVip) $type = 2;
    return $type;
    }
}

Функция getImg: хм... название как бэ намекает нам, что должно на выходе получиться что-нибудь типа картинки или её адреса, НО НЕТ! нам возвращается какой-то тип...
Что это за тип (ну да, да, php бестиповой язык, но нужно же как-то хотя бы намёки на тип делать) - одному Богу известно, НО! эта переменная изначально логическая, потом ВНЕЗАПНО превпращается в число, ну и под конец может быть и строкой! (ну а почему бы и нет)
Причём конечно же всем очевидна последовательность: 1, 16, (опять-таки ВНЕЗАПНО) 'author', ну и по уже "понятной" логике в конце 2

Да, ну и разумеется вокруг оооочень много комментариев поясняющие этот кусок кода...
А вы на индусов ругаетесь... Так-то!

Zemtall, 06 Сентября 2010

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

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

+165

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

class microTimer {
	function start() {
		global $starttime;
		$mtime = microtime();
		$mtime = explode( ' ', $mtime );
		$mtime = $mtime[1] + $mtime[0];
		$starttime = $mtime;
	}
	function stop() {
		global $starttime;
		$mtime = microtime();
		$mtime = explode( ' ', $mtime );
		$mtime = $mtime[1] + $mtime[0];
		$endtime = $mtime;
		$totaltime = round( ($endtime - $starttime), 5 );
		return $totaltime;
	}
}

DLE 9.0

Morgan, 06 Сентября 2010

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

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

+171

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

define('ERROR', 1);
define('WARNING', 2);
define('PARSE', 4);
define('NOTICE', 8);
define('CORE_ERROR', 16);
define('CORE_WARNING', 32);
define('COMPILE_ERROR', 64);
define('COMPILE_WARNING', 128);
define('USER_ERROR', 256);
define('USER_WARNING', 512);
define('USER_NOTICE', 1024);
// ---------------------------
$errortype = array(
1=>'Ошибка',
2=>'Предупреждение!',
4=>'Ошибка разборщика',
8=>'Замечание',
16=>'Ошибка ядра',
32=>'Предупреждение ядра!',
64=>'Ошибка компиляции',
128=>'Предупреждение компиляции!',
256=>'Пользовательская Ошибка',
512=>'Пользовательскаое Предупреждение!',
1024=>'Пользовательскаое Замечание',
2048=>'Небольшое замечание',
8192=>'Устаревший код');

Мартин, 06 Сентября 2010

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

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

+160

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

<?php
require_once 'mainfunc.php';
$template->msg('SET TITLE', $conf['wiki']['title']);
$rev = intval(@$_GET['rev']);
if(@$_GET['ae']){session_destroy();unset($admin);}
?>
<h1><?= $conf['wiki']['title']; ?></h1>
<p><a href="create.php">[Создать статью]</a> <?php
if($rev) echo '<a href="index.php?rev=0">[Перевернуть]</a>';
else echo '<a href="index.php?rev=1">[Перевернуть]</a>';
?></p>
<?php
$tpr = array();
echo '<p>';
$fls = array();
foreach(scandir('files') as $k) if($k{0}!='.') $fls[]=$k;
natsort($fls);
if($rev) $fls=array_reverse($fls);
foreach($fls as $k){
  if($k{0}=='-') continue;
  $f=file('files/'.$k);
  if($f[0]{0}=='#') $f[0]{0}='';  $tpr[] = '<a href="view.php?id='.$k.'">'.$f[0].'</a><br>';
}
$pnav = new PageNavigator($tpr, 'index.php?page=$p&rev='.$rev);
$pnav->page();
echo '</p>';
echo '<p class="nav">';
$pnav->panel();
echo '</p>';
if(@$admin) echo '<p>Вы админ <a href="index.php?ae=1">[X]</a></p>';
$template->act(); //Запускаем вывод
?>

Инновационный вики-движок для вап-интернета, в действии можно посмотреть здесь: http://wiki.wup.ru/
От одного index.php становится худо.
Плюс работает это еще с помощью какого-то фреймворка, при этом весь html размазан практически по всему проекту, да и все это еще на файлах.
Куча дыр, фильтрующие регулярки не пашут, даже банального антифлуда нет - можно жать f5 и радоваться.
Полностью сорцы можно скачать тут: http://wiki.wup.ru/wikibackups/

7ion, 06 Сентября 2010

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

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

+158

01
02
03
04
05
06
07
08
09
10

if ($_REQUEST['name']){
	$nameProduct = $_REQUEST['name'];
	$products = $db->products($_REQUEST['name']);
} else {
	$products = 'error';	
}

if ($products == 'error'){
	$http->redirect('index.php');
}

Из частного проекта.
Во-первых почему-бы не использовать $nameProduct второй раз вместо $_REQUEST['name']?
Во-вторых вместо $products = 'error'; можно сразу было сделать редирект, т.к. все последующие использования $products не имеют смысла, если редирект выполнен.

mikhailu, 06 Сентября 2010

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

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

+164

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

if($result){
				list($Title_block, $Sections, $Description,$Moderators) = mysql_fetch_row($result)
				//$row = mysql_fetch_assoc($result)
				echo "<form>\n";
					echo "\t<table border='4' cellspacing='1' cellpadding='1' width='1000' height='60'>\n";
						echo "\t\t<tr>\n";
							echo "\t\t\t<td colspan='5' bgcolor='gray' align='leftmargin'>$Title_block</td>\n";
						echo "\t\t</tr>\n";
				do{
					echo "\t\t<tr>\n";
						echo "\t\t\t<td bgcolor='#A0A0A0'></td>\n";
						echo "\t\t\t<td bgcolor='#A0A0A0' width='500'>Форум</td>\n";
						echo "\t\t\t<td bgcolor='#A0A0A0'>Тем</td>\n";
						echo "\t\t\t<td bgcolor='#A0A0A0'>Сообщений</td>\n";
						echo "\t\t\t<td bgcolor='#A0A0A0'>Последних сообщений</td>\n";
					echo "\t\t</tr>\n";
					echo "\t\t<tr>\n";
						echo "\t\t\t<td bgcolor='##D0D0D0'><img src='mus.png'></td>\n";
						echo "\t\t\t<td bgcolor='#D0D0D0' width='500' class='row4'>\n";
							echo "\t\t\t\t<b><a href='Click' style='color:black;'>$Sections</a></b>\n";
							echo "\t\t\t\t<br />$Description<br />Модераторы: $Moderators</td>\n";
						echo "\t\t\t<td bgcolor='#D0D0D0'></td>\n";
						echo "\t\t\t<td bgcolor='#D0D0D0'></td>\n";
						echo "\t\t\t<td bgcolor='#D0D0D0'></td>\n";
					echo "\t\t</tr>\n";
				}while(list($Title_block, $Sections, $Description,$Moderators) = mysql_fetch_row($result));
					echo "\t</table>\n";
				echo "</form>\n";

Часть одного самописного форума

uadeveloper, 06 Сентября 2010

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

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

+144

1

HTML разметка вот этого форума (http://science-noir.kzforum.info/forum.htm) доставляет настоящее эстетическое наслаждение :-D

ageron, 05 Сентября 2010

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

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

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

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

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

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

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

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

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

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

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

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