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

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

В номинации:

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

За время:

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

+162

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
48
49
50
51
52
53
54

<?php
$t1 = microtime();
define('CMS_LOAD', TRUE);

require_once('./inc/config.inc.php');
require_once('./inc/main_fnc.inc.php');


cms_MysqlConnect();
unset($cms); // попутно используемые данные скрипта
unset($vars); // $_POST, $_COOKIE, $_GET данные
cms_RegVars();

$cms['page'] = !empty($vars['page']) ? $vars['page'] : 'main';
$cms['md_page'] = !empty($vars['md_page']) ? $vars['md_page'] : '';

$cms['ESTRING'] = !get_magic_quotes_gpc() ? TRUE : FALSE;
$cms['strpos'] = strpos($cms['page'], '_');

$cms['q_page'] = mysql_query("SELECT name, ptext FROM ".CMS_PREFIX."_pages WHERE page='".cms_MysqlEStr($cms['page'])."' LIMIT 1");
$cms['is_page'] = mysql_num_rows($cms['q_page']);


// если в _pages нет такой страницы, перемещаем на главную
if($cms['is_page'] > 0)
{
	list($cms['page_name'], $cms['page_text']) = mysql_fetch_row($cms['q_page']);
}
else
{
	header("Location: /");
}
mysql_free_result($cms['q_page']);

if($cms['strpos'] > 0 && !empty($cms['md_page']))
{
	$cms['q_md_product1'] = mysql_query("SELECT name, descript2, cost FROM ".CMS_PREFIX."_productions WHERE id=".intval($cms['md_page'])." LIMIT 1");
	
	if(mysql_num_rows($cms['q_md_product1']) > 0)
	{
		list($cms['page_name'], $cms['md_prod_descr2'], $cms['md_prod_cost']) = mysql_fetch_row($cms['q_md_product1']);
	}
	else
	{
		header("Location: /");
	}
	
	mysql_free_result($cms['q_md_product1']);
}


//edited by DEMIS [[email protected]]

$cms['page_text_header'] = $cms['page_name'];

Index.php в самописной CMS улыбает. И это только начало +)

be3, 23 Февраля 2011

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

C++ / Говнокод #5710

+162

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

#ifdef WIN32
    string nameOfLibToLoad("C:\opt\lib\libctest.dll");
    lib_handle = LoadLibrary(TEXT(nameOfLibToLoad.c_str()));
    if (!lib_handle) {
        cerr << "Cannot load library: " << TEXT(nameOfDllToLoad.c_str()) << endl;
    }
#else
    string nameOfLibToLoad("/opt/lib/libctest.so");
    lib_handle = dlopen(nameOfLibToLoad.c_str(), RTLD_LAZY);
    if (!lib_handle) {
        cerr << "Cannot load library: " << dlerror() << endl;
    }
#endif

кроссплатформенная загрузка шаренной библиотеки... может я чего то не понимаю в этой жизни, но... совершенно здесь лишний класс строки, который делает совершенно лишний вызов конструктора и добавляет совершенно лишние n-килобайт к размеру сборки...
ссыль: http://www.yolinux.com/TUTORIALS/LibraryArchives-StaticAndDynamic.html

ReL, 18 Февраля 2011

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

C# / Говнокод #5697

+109

1
2
3
4
5

try
            {
                splash.Close(new TimeSpan(0, 0, 0, 3, 0));
            }
            catch (Exception ex) { System.Diagnostics.Debug.WriteIf(false, ex.ToString()); }

Вот так вот эксепшн обработал. Помню, недавно сказал такую вещь "В нашем игровом движке содержится лучший код, который я когда-либо писал" про свой игровой движок. Казалось бы, удобная имплементация, структура и принципы ООП правильно используются. Но потом я увидел ЭТО (см. выше). Надо больше спать, видимо... %)

RaZeR, 16 Февраля 2011

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

C# / Говнокод #5695

+128

1
2
3
4
5

public static Boolean GetAttBool(XmlNode node, string attName)
 {
     long value = GetAttLong(node, attName);
      return value == 1 ? true: false;
 }

uaavg, 16 Февраля 2011

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

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

+162

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

public static function perform($table, $data, $action = 'insert', $parameters = '') {
		reset($data);
		if ($action == 'insert') {
			$query = 'INSERT INTO ' . $table . ' (';
			while (list($columns, ) = each($data)) {
				$query .= $columns . ', ';
			}
			$query = substr($query, 0, -2) . ') values (';
			reset($data);
			while (list(, $value) = each($data)) {
				switch ((string)$value) {
					case 'now()':
						$query .= 'NOW(), ';
						break;
					case 'null':
						$query .= 'NULL, ';
						break;
					default:
						$query .= '\'' . mysql_escape_string($value) . '\', ';
						break;
				}
			}
			$query = substr($query, 0, -2) . ')';
		} elseif ($action == 'update') {
			$query = 'UPDATE ' . $table . ' SET ';
			while (list($columns, $value) = each($data)) {
				switch ((string)$value) {
					case 'now()':
						$query .= $columns . ' = NOW(), ';
						break;
					case 'null':
						$query .= $columns .= ' = NULL, ';
						break;
					default:
						$query .= $columns . ' = \'' . mysql_escape_string($value) . '\', ';
						break;
				}
			}
			$query = substr($query, 0, -2) . ' WHERE ' . $parameters;
		}

		self::request($query);
	}

Вот так у нас вставляют в базу :)

wmmorgun, 15 Февраля 2011

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

Pascal / Говнокод #5665

+103

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
48

glpushmatrix;
if ((commandflat=2)and((commandaxis=2) or(commandaxis=3)))or((commandflat=1)and(commandaxis=1)) then
 glrotatef(faza,rotation[0],rotation[1],rotation[2]);

  glcolor3f(kubcol[Y[2,1,1]].R,kubcol[Y[2,1,1]].G,kubcol[Y[2,1,1]].B);
 glbegin(gl_quads);
  glvertex3f(-1.5,1.5,1.5);
  glvertex3f(-1.5,1.5,0.5);
  glvertex3f(-0.5,1.5,0.5);
  glvertex3f(-0.5,1.5,1.5);
 glend;
 glcolor3f(kubcol[X[1,1,1]].R,kubcol[X[1,1,1]].G,kubcol[X[1,1,1]].B);
 glbegin(gl_quads);
  glvertex3f(-1.5,1.5,1.5);
  glvertex3f(-1.5,1.5,0.5);
  glvertex3f(-1.5,0.5,0.5);
  glvertex3f(-1.5,0.5,1.5);
 glend;
 glcolor3f(kubcol[Z[2,1,1]].R,kubcol[Z[2,1,1]].G,kubcol[Z[2,1,1]].B);
 glbegin(gl_quads);
  glvertex3f(-1.5,1.5,1.5);
  glvertex3f(-0.5,1.5,1.5);
  glvertex3f(-0.5,0.5,1.5);
  glvertex3f(-1.5,0.5,1.5);
 glend;
 glcolor3f(0.5,0.5,0.5);
 glbegin(gl_quads);
  glvertex3f(-1.5,0.5,1.5);
  glvertex3f(-1.5,0.5,0.5);
  glvertex3f(-0.5,0.5,0.5);
  glvertex3f(-0.5,0.5,1.5);
 glend;
 glcolor3f(0.5,0.5,0.5);
 glbegin(gl_quads);
  glvertex3f(-0.5,1.5,1.5);
  glvertex3f(-0.5,1.5,0.5);
  glvertex3f(-0.5,0.5,0.5);
  glvertex3f(-0.5,0.5,1.5);
 glend;
 glcolor3f(0.5,0.5,0.5);
 glbegin(gl_quads);
  glvertex3f(-1.5,1.5,0.5);
  glvertex3f(-0.5,1.5,0.5);
  glvertex3f(-0.5,0.5,0.5);
  glvertex3f(-1.5,0.5,0.5);
 glend;
glpopmatrix;
///////112

Захотел как то кубик рубика накодить, это фрагмент модуля из 1214 строк который выводит 26 кубиков. Цвет лицевых граней берется из массива, остальные - серые.

Dent, 14 Февраля 2011

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

C++ / Говнокод #5623

+175

1
2
3
4
5
6
7

template<class Exception>
bool GenerateException(const Exception* const e)
{
	throw e;
	delete e;
	return true;
}

Продолжение цикла говнокодов про исключения из того же самого проекта.
Проблем от неё почти нет, тк в основном исключения генерятся так:

ExceptionТакойто e;

GenerateException(&e);

Говногость, 10 Февраля 2011

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

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

+161

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

# Добавление пользователя
	if ($_GET['action'] == "add_user" OR $_POST['action'] == "add_user"){
		#Проверяем достаточно ли прав на добавление пользователя
		$pal = 15;
		if (access_level() < $pal) {
			echo "Извените, у вас недостаточно прав для данной операции<br>";
			echo "Ваш уровень ".access_level()." а доступ возможен только с ".$pal."<br>";
		}
		else{
		echo "Уровень доступа - $pal<br>";
		echo "<form action=\"index.php\" method=\"post\">\n";
		echo "<select name=type><option value=1>support<option value=2>installer</select><br>";
		echo "Логин ( первая буква имени + фамилия (en))  <input type=\"text\" name=\"add_username\"><Br> passwd <input type=\"password\" name=\"add_passwd\"><Br>\n access_level <input type=text name=access_level><br>\n";
		echo "Полное имя (Фамилия, Имя, [Отчество]) <input type=\"text\" name=\"add_full_name\"><br>";
		echo "Телефон (только 10 цифр, без +7, 8 и т.п.) <input type=\"text\" name=\"add_phone\"><Br>";
		echo "<input type=\"hidden\" name=\"action\" value=\"add_user\">\n";
		echo "<input type=\"submit\" name=\"submit\" value=\"add\"><br>\n</form>";

		$add_name=mysql_real_escape_string($_POST['add_username']);
		$add_pass=mysql_real_escape_string($_POST['add_passwd']);
		$add_full_name=mysql_real_escape_string($_POST['add_full_name']);
		$add_phone=mysql_real_escape_string($_POST['add_phone']);
		$add_access_level=mysql_real_escape_string($_POST['access_level']);
		$add_pass = md5($add_pass);
		if($_POST['type'] == "1"){
			echo $add_name."-".$add_pass."-".$add_access_level."<br>";
			$sql = 'INSERT INTO `switches`.`creators` (`id`, `name`, `passwd`, `access_level`, `descr`, `creator`, `create_date`) VALUES (\'\', \''.$add_name.'\', \''.$add_pass.'\', \''.$add_access_level.'\', \'\', \''.$_SESSION['user_id'].'\', NOW());';
			mysql_query($sql,$dbutm);
			echo "<br>Пользователь добавлен.<Br>";
		}
		elseif($_POST['type'] == "2"){
			$sql = 'INSERT INTO `switches`.`installers` (`id`, `name`, `full_name`, `passwd`, `access_level`, `descr`, `creator`, `create_date`, `phone`) VALUES (\'\', \''.$add_name.'\', \''.$add_full_name.'\', \''.$add_pass.'\', \''.$add_access_level.'\', \'\', \''.$_SESSION['user_id'].'\', NOW(), \''.$add_phone.'\');';
			mysql_query($sql,$dbutm);
			echo "<br>Монтажник добавлен.<Br>";
		}
	}
}

Система биллинга. Начинаю нервничать...

Gogogo, 08 Февраля 2011

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

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