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

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

+173

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

<config.php>


define("head", "<!DOCTYPE HTML PUBLIC \"-//IETF//DTD HTML 2.0//EN\">
<HTML><HEAD><TITLE>{$scriptname}</TITLE>

*... пропущено 8 строк *

</STYLE><SCRIPT type=\"text/javascript\"><!--
function confirmB(v)
{
	if (v == 'fulluninstall') var a = confirm('Are you sure you want to грохнуть всё?');
	if (v == 'refresh') var a = confirm('Are you sure you want to очистить out?');
	*... пропущено 6 строк * 
}
//-->
</SCRIPT></HEAD><BODY>");

define ("funchead", "<BODY onload=setTimeout(\"document.forms[0].submit()\",15000)>");

define("log_in", "<FORM name=\"log_in\" action=\"{$_SERVER['SCRIPT_NAME']}\" method=\"post\"><TABLE style=\"width:100%;border:0\"><TR><TD style=\"text-align:center;vertical-align:middle\"><H3>Login</H3>
Username: <INPUT type=\"password\" name=\"log_in\"> <INPUT type=\"submit\" value=\"Log in\">
</TD></TR></TABLE></FORM><BR>\r\n");

define("in_get", "<H3>Качать</H3><FORM name=\"in_get\" action=\"{$_SERVER['SCRIPT_NAME']}\" method=\"post\">
	*... пропущено 12 строк * 
	</FORM>\r\n");

define("in_sort", "<H3>Сортировать</H3><FORM name=\"in_sort\" action=\"{$_SERVER['SCRIPT_NAME']}\" method=\"post\">
	*... пропущено 5 строк * 
	</FORM>\r\n");


Еще штук 20 дефайнов в конце.

Мы не ищем легких путей и шаблонизаторов.

dzugaru, 02 Сентября 2010

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

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

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

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

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

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

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

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

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

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

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

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