Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

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

+158

1
2
3
4
5

/*	Адаптация под оперу	*/
if ( stristr($_SERVER['HTTP_USER_AGENT'], 'Opera') )
	echo "<input type=hidden value='file_ok' name='action'><br><input type=\"image\" src=\"".$url."images/buttons/button_file_ok.png\">";
else 
	echo "<br><input type=\"image\" src=\"".$url."images/buttons/button_file_ok.png\" value=\"file_ok\" name=\"action\">";

особый говнокод для Оперы
http://my.opera.com/russian/forums/topic.dml?id=918821

xxx, 28 Февраля 2011

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

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

+158

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
42
43
44
45

<?
//include('../root/start.php');
include('start.php');




$hotels_menu_id = GetSettingsParam('hotels_menu_id');

if ($image_id = $_GET['image_id']) {
    $row = mysql_fetch_assoc(mysql_query(
                    'select co.id co_id, co.title co_title, ' .
                    'ci.id ci_id, ci.title ci_title, ' .
                    'h.id h_id, h.title h_title ' .
                    'from ' . _mysql_tbl_prefix . 'countries co ' .
                    'left join ' . _mysql_tbl_prefix . 'cities ci on co.id = ci.country_id ' .
                    'left join ' . _mysql_tbl_prefix . 'hotels h on ci.id = h.city_id ' .
                    'left join ' . _mysql_tbl_prefix . 'hotel_images hi on hi.hotel_id = h.id ' .
                    'where hi.image_id = ' . $image_id));
    echo mysql_error();
# $src = '../../hotel_images/'.(int)($image_id/2000).'/'.$image_id.'.jpg';
    $src = './hotel_images/' . (int) ($image_id / 2000) . '/' . $image_id . '.jpg';
}
?>
<html>
    <head>
        <meta http-equiv="Content-Type" content="text/html; charset=windows-1251" />
        <title>Фото отеля "<?= $row['h_title'] ?>" (<?= $row['ci_title'] . ', ' . $row['co_title'] ?>) - Туроператор</title>
    </head>
    <link rel="stylesheet" href="../hotel_image.css">
    <body>
        <table height="100%" width="100%">
            <tr>
                <td valign="middle" align="center">
                    <h1 class="hotel-image-title"><?=
'Фото отеля "<a href="../index.php?menu_id=' . $hotels_menu_id . '&hotel_id=' . $row['h_id'] . '">' . $row['h_title'] . '</a>" ' .
        '(<a href="../index.php?menu_id=' . $hotels_menu_id . '&city_id=' . $row['ci_id'] . '">' . $row['ci_title'] . '</a>, ' .
        '<a href="../index.php?menu_id=' . $hotels_menu_id . '&country_id=' . $row['co_id'] . '">' . $row['co_title'] . '</a>)'
?></h1>
                    <a href="#" onClick="window.close()"><img src="<?= $src ?>" class="hotel-image" alt="Фото отеля "<?= $row['h_title'] ?>" (<?= $row['ci_title'] . ', ' . $row['co_title'] ?>)"></a>
                </td>
            </tr>
        </table>
    </body>
</html>

Попросили так сазать исправить))
я был ошеломлен "магическим числом" 2000

rainerg, 22 Февраля 2011

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

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

+158

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
42
43
44
45
46
47

if ($_GET[action] == "avatar") {


	if ($info = $_GET[info]) {

			if ($info[avatar]) {
				echo $info[avatar];
			} else {
					header("Content-type: image/png");

					$im = imagecreatetruecolor(80, 80);
					$white = imagecolorallocate($im, 255, 255, 255);
					$grey = imagecolorallocate($im, 128, 128, 128);
					$black = imagecolorallocate($im, 0, 0, 0);
					imagefilledrectangle($im, 0, 0, 80, 80, $white);
					$font = "fonts/avatar.ttf";

					$text = "HET";
					imagettftext($im, 20, 0, 15, 40, $grey, $font, $text);
					imagettftext($im, 20, 0, 14, 39, $black, $font, $text);
					$text = "ABATAPA";
					imagettftext($im, 14, 0, 5, 55, $grey, $font, $text);
					imagettftext($im, 14, 0, 4, 54, $black, $font, $text);
					imagepng($im);
					imagedestroy($im);
			}
	} else {
					header("Content-type: image/png");

					$im = imagecreatetruecolor(80, 80);
					$white = imagecolorallocate($im, 255, 255, 255);
					$grey = imagecolorallocate($im, 128, 128, 128);
					$black = imagecolorallocate($im, 0, 0, 0);
					imagefilledrectangle($im, 0, 0, 80, 80, $white);
					$font = "fonts/avatar.ttf";

					$text = "HET";
					imagettftext($im, 20, 0, 15, 40, $grey, $font, $text);
					imagettftext($im, 20, 0, 14, 39, $black, $font, $text);
					$text = "ABATAPA";
					imagettftext($im, 14, 0, 5, 55, $grey, $font, $text);
					imagettftext($im, 14, 0, 4, 54, $black, $font, $text);
					imagepng($im);
					imagedestroy($im);	
	}

}

Вот такое говнецо встретилось))

Sulik78, 19 Февраля 2011

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

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

+158

1
2
3
4
5
6
7
8
9

public function search() {
    $ret = array();
    foreach($this->dir() as $file)
        if($file->folder())
            $ret = array_merge($ret,$file->search()->asArray());
        else
            $ret[] = $file;
    return new mod_file_list($ret);
}

Красота, в результате работы функции мы получаем объект, состоящий из массива объектов класса file (этот метод в этом классе)... "чудо-поиска" (хотя это вовсе и не поиск, а банальный список файлов всех дочерних папок), особенно будет забавно, если в списке будет несколько тысяч файлов...
В этом классе в принципе много мест, которые наровят насоздавать кучу экземпляров этого же класса, вместо замены одной приватной переменной...

GhOsTMZ, 15 Февраля 2011

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

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

+158

1
2
3
4
5
6
7
8
9

function changeType(what) {
                if (what == "forward") {
                        if (document.forms[0].elements['forward_list'].disabled == true) {
                                document.forms[0].elements['forward_list'].disabled = false;
                        } else {
                                document.forms[0].elements['forward_list'].disabled = true;
                        }
                }
        }

Есть замечательная ispCP и форма для настройки форвардинга е-мэйлов. Тут описывается поведение чек-бокса для включения формы.

Mr_Alone, 09 Февраля 2011

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

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

+158

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

do
        {
                do
                        {
                                if ($myrow_cats["category_id"] == $myrow_oldtovar_1["id"])
                                        {
                                                printf("<tr><td class='newcat_author'><input name='old_id_cat[]' type='checkbox' value='%s' checked >
                                                %s</td></tr>",
                                                $myrow_oldtovar_1["id"],$myrow_oldtovar_1["rubrikatitle"]);
                                        }
                                else
                                        {
                                                printf("<tr><td class='newcat_author'><input name='old_id_cat[]' type='checkbox' value='%s' >%s</td></tr>",
                                                $myrow_oldtovar_1["id"],$myrow_oldtovar_1["rubrikatitle"]);
                                        }       
                        }
                while($myrow_oldtovar_1 = mysql_fetch_array($result_oldtovar_1));
        }
while ( $myrow_cats = mysql_fetch_array($result_cats) );

qbasic, 08 Февраля 2011

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

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

+158

1
2
3
4

$arTime = localtime();
//формируем ядро номера заказа, которое будем подцеплять к разным префиксам

$orderNumberCore = str_pad($arTime[4]+1, 2, "0", STR_PAD_LEFT).str_pad($arTime[3], 2, "0", STR_PAD_LEFT).str_pad($arTime[2], 2, "0", STR_PAD_LEFT).str_pad($arTime[1], 2, "0", STR_PAD_LEFT).str_pad($arTime[0], 2, "0", STR_PAD_LEFT);

Формируется номер заказав интернет-магазине из даты/времени. Но при этом не учитывается год.

$orderNumberCore -- это дата в формате dmhis.

kost, 07 Февраля 2011

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

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