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

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

В номинации:

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

За время:

Си / Говнокод #2816

+137.6

1
2
3
4
5
6
7
8

for( unsigned int i = 0; i < strlen(str); i++)
{
    if ( str[i] == '-')
    {
        str[i] = '\0';
        break;
    }
}

'strlen' в условии цикла - это вычисление инварианта в теле цикла.
Вообще этот цикл реально можно заменить одним вызовом 'strchr'

benderlog, 18 Марта 2010

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

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

+160.6

1
2
3
4
5
6
7

if(isset($_POST['add_st']) && $_POST['add_st'] != false && $_POST['add_st'] != '')
{ 
	if($res=mysql_query("select max(ordering) from category3 where category2=".$cat)) 
		{
			$ord=mysql_fetch_row($res);
			$order=$ord[0]+1;
		}

$_POST['add_st'] - из формы с <input type='submit' name='add_st' value='Сохранить' />

Я уж не говорю, что $order можно было сделать одним запросом

pasha, 16 Марта 2010

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

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

+100.3

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

function rgbagl(r,g,b,a:gldouble):trgbagl;
var
t:trgbagl;
begin
t.r:=r;
t.g:=g;
t.b:=b;
t.a:=a;
rgbagl:=t;
end;
function trgbaintrgbagl(rgba:trgba):trgbagl;
begin
trgbaintrgbagl.r:=rgba.r/255;
trgbaintrgbagl.g:=rgba.g/255;
trgbaintrgbagl.b:=rgba.b/255;
trgbaintrgbagl.a:=rgba.a/255;
end;
function trgbaglintrgba(rgbagl:trgbagl):trgba;
begin
trgbaglintrgba.r:=trunc(rgbagl.r*255);
trgbaglintrgba.g:=trunc(rgbagl.g*255);
trgbaglintrgba.b:=trunc(rgbagl.b*255);
trgbaglintrgba.a:=trunc(rgbagl.a*255);
end;
function trgbaglelmintrgbaelm(elm:gldouble):byte;
begin
trgbaglelmintrgbaelm:=trunc(elm*255);
end;
function trgbaelmintrgbaglelm(elm:byte):gldouble;
begin
trgbaelmintrgbaglelm:=elm/255;
end;

Вот так вот.

Говногость, 13 Марта 2010

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

Java / Говнокод #2774

+79.1

1
2
3
4
5
6
7
8

if (at == null) {
	try {
	throw new NullPointerException();
	}catch (Exception e) {
	e.printStackTrace();
	throw new NullPointerException();
	}
}

проверка корректности полученного аргумента в функции.
хорошо придумано, что бы не вызывать
throw new IllegalArgumentException();

maxt, 13 Марта 2010

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

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

+165

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

if ($check==0)
{
  $query = "SELECT id, title, news, author, date FROM news ORDER BY id DESC LIMIT 10";
}else{
  $query = "SELECT id, title, news, author, date FROM news ORDER BY id LIMIT 10";	
}
$result = mysql_query($query,$db);
$_count=0;
while($row=mysql_fetch_array($result))
{  
	$_count++;
}

$result = mysql_query($query,$db);
while($row=mysql_fetch_array($result))
{  
...

Считаем кол-во строк в mysql-таблице

qwase, 11 Марта 2010

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

Куча / Говнокод #2740

+60.9

1
2
3

<a href="game.php?page=buildings&cmd=insert">
<input style='color:#0F0; font-weight: bold'; type='submit' value='bla-bla-bla'>
</a>

Человек ВНЕЗАПНО понял, как сделать рамку вокруг ссылки "как у кнопки".

Int, 09 Марта 2010

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

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

+65.7

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24

template <class T>
struct SLinkedListNode
{
public:
	SLinkedListNode* m_pPrev;  ///<  Pointer to previous node.
	SLinkedListNode* m_pNext;  ///<  Pointer to next node.
	int Initialize();
	int Finalize();
};
...
struct SWaiterNode : public SLinkedListNode<SContainerNode>, public SLinkedListNode<SWaiterListNode>
{
...
// container node
struct SContainerNode
{
	// no members needed yet (just for masking SLinkedListNode type)
};

// container
struct SContainer
{
public:
	SLinkedListNode<SContainerNode>* m_pObjects;

сначала делаем тимплейтный класс, параметр которого никому не нужен, а потом делаем заглушку для этого параметра.
нормально.

kubrinsky, 19 Февраля 2010

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

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

+149.1

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="<?php echo $this->language; ?>" lang="<?php echo $this->language; ?>" >
<head>
<?php 
	$menu = & JSite::getMenu();
	$flag = ($menu->getActive() == $menu->getDefault()); 
	$option = JRequest::getVar('option', null); 
?>
<jdoc:include type="head" />
<link rel="stylesheet" href="<?php echo $this->baseurl ?>/templates/system/css/general.css" type="text/css" />
<link rel="stylesheet" href="<?php echo $this->baseurl ?>/templates/system/css/system.css" type="text/css" />
<link rel="stylesheet" href="templates/<?php echo $this->template ?>/css/template.css" type="text/css" />
<!--[if lte IE 7]>
<script language="JavaScript" type="text/javascript" src="?php echo $this->baseurl ?>/templates/<?php echo $this->template ?>/js/DD_belatedPNG.js"></script>
<script language="JavaScript" type="text/javascript">
DD_belatedPNG.fix('.icon a, .phone, .menu a, .header '); 
</script>
<![endif]-->
</head>

<body >
	<div class="wrapper">
		<div class="header">
			<ul class="icon">
				<li class="<?php  if ($flag)  {?>active<?php }?>"><a href="#" class="home"><span></span></a></li>
				<li><a href="#" class="sitemap"><span></span></a></li>
				<li><a href="#" class="search"><span></span></a></li>
			</ul>
			<h3 class="phone">
				Тел. 8 (044) 561 66 22
			</h3>
			<div class="menu-wrap">
				<jdoc:include type="modules" name="menu" style="none" />
			</div>
			<div class="why">
				<h1>Преимущества работы с нами</h1>
				<ul>
					<li>1) Умеренные цены</li>
					<li>2) Использование высококлассных материалов</li>
					<li>3) Кротчайшие сроки выполнение работ</li>
					<li>4) Лучший спектр услуг в городе</li>
					<li>5) Гаранатия до 3х лет</li>
				</ul>
			</div>
		</div>
		<div class="content">
		<?php  if ($flag)  {?>
			<div class="scroller">
				<h3>Наши работы</h3>
				<div class="scroll">
					<div class="scroll-visible">
						<ul>
							<jdoc:include type="modules" name="slider" style="none" />
						</ul>
					</div>
					<a href="#" class="scroll-right"></a>
					<a href="#" class="scroll-left"></a>
				</div>
			</div>
			<div class="text-module">
					<jdoc:include type="modules" name="main-text" style="none" />
			</div>
		<?php }else{?>
			<jdoc:include type="component" />
		<?php }?>
		</div>
		<div class="bottom-modules">
			<div class="bottom-modules-item item-left"><jdoc:include type="modules" name="user1" style="xhtml" /></div>
			<div class="bottom-modules-item item-right"><jdoc:include type="modules" name="user3" style="xhtml" /></div>
			<div class="bottom-modules-item item-center"><jdoc:include type="modules" name="user2" style="xhtml" /></div>
		</div>
		<div class="footer-space"></div>
	</div>
	<div class="footer">
		<span class="copyright">
			© 2009<br />
			Все права защищены
		</span>
		<span class="design"></span>
	</div>
</body>
</html>

Areks, 19 Февраля 2010

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

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

+160.9

1
2
3
4

static function &getdbHandler(){
		global $DB_tree;
		return $DB_tree;
}

Shop-Script поражает....

metalalisa, 18 Февраля 2010

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

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

+162.5

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

$s_name=$_POST['s_name'];
$f_name=$_POST['f_name'];
$m_name=$_POST['m_name'];
$login=$_POST['login'];
$password1=$_POST['password1'];
$password2=$_POST['password2'];
$post_id=$_POST['post'];
$query="SELECT  person.login as login	FROM	person	ORDER BY ID";
$res=Select($query);
$flag=false;
		  While(OCIFetch($res))
		        {
			        $login_base=text($res,'LOGIN');
				if ($login==$login_base)
				{
				$flag=true;
				break;
				}
			}
		If($flag)
			{
			echo "</br><center><h3>Такой логин уже существует...придумайте еще что-нибудь!</center></h3></br>";
			?>
			<center><h4><a href="index.php?menu=reg">Назад</a></h4><center>
			<?
			}
               if (!$flag)
		   {
		     if ($password1==$password2)
		           {
		             If (($login!='') and ($password1!='') and ($s_name!='') and ($f_name!='') and ($m_name!=''))
			   {
                                 .... Ну и дальше  добавение и все такое))

ммм.. альтернативные способы защиты от инъекций в запросах???)))

1_and_0, 12 Февраля 2010

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

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