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

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

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

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

+168

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

define('Min2Sec',60);
define('Hour2Sec',3600);
define('Day2Sec',86400);
define('int','integer');
define('real','float');
define('bool','boolean');
define('str','string');
define('mix','array');
define('intmix','int_mix');
define('realmix','real_mix');
define('boolmix','bool_mix');
define('strmix','str_mix');
define('obj','object');
define('nil','null');

Из ядра одной русской CMS, писал видимо Pascal'ист

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

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

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

+158

1
2
3
4
5
6
7

$sql = "SELECT 
			t.topic_id							
		FROM 
			".Config::Get('db.table.topic')." as t,	
		WHERE 
			1=1					
		LIMIT ?d, ?d";

5-6 строки мешают уснуть. Из того же LiveStreet..

hybroid, 04 Сентября 2010

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

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

+156

1
2
3
4
5
6

if (!$this->sCurrentEvent) {
	/**
	 * Показывает дефолтную страницу
	 */
	//а это какая страница?
}

И всё. Никаких else и прочих вещей. Сижу, курю, пытаюсь восстановить поток мысли авторов.
Из того же LiveStreet.

hybroid, 03 Сентября 2010

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

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

+160

1
2
3

if((string)(int)$url_code === (string)$url_code) { 
              /*.... not relevant content */
       }

mykola, 03 Сентября 2010

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

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

+155

1
2
3
4
5

<?php
$url = $_GET['url'];
$url = str_replace('http://', '', $url);
echo '<a href="http://$url">123</a>';
?>

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

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

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

+167

1
2
3
4
5
6
7

public function for_json()
{
     $res = unserialize(serialize($this->params));
     unset($res['wml']);
     unset($res['xhtml']);
     return $res;
}

Моск сломался, пытаясь понять строчку 3 :)

ko1Rn, 03 Сентября 2010

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

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

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

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

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

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

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

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

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

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

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

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