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

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

В номинации:

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

За время:

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

+168.6

01
02
03
04
05
06
07
08
09
10

<?php
function turn($x)
     {
     $y = abs($x);
     if ($y == $x)    
         return "-$y";    
     else        
         return "$y";    
     }
?>

Функция, возвращающая противоположное значение числа.

Отсюда - http://php.net/manual/en/function.abs.php#94768

Slasyz, 23 Декабря 2009

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

SQL / Говнокод #2305

−125.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

create procedure [dbo].[pbsp_GetClientUsers]

	(
        @ClientId int,
        @fname varchar(100),
        @lname varchar(100),
        @email varchar(150)
	)

AS
	
DECLARE @SQL varchar(1000)

Set @SQL = 'Select TOP 500 *, tblRoles.title AS Role from tblUser INNER JOIN
        tbl_mtm_UserRoles ON tblUser.UserId = tbl_mtm_UserRoles.UserId INNER JOIN
        tblRoles ON tbl_mtm_UserRoles.RoleId = tblRoles.Id where tblUser.ClientId = ' + STR(@ClientId) + ' ' 

if LEN(@fname) > 0
    Set @SQL = @SQL + ' AND tblUser.fName like ''' + @fname + '%'' '
    
if LEN(@lname) > 0
    Set @SQL = @SQL + ' AND tblUser.lName like ''' + @lname + '%'' '
    
if LEN(@email) > 0
    Set @SQL = @SQL + ' AND tblUser.Email like ''' + @email + '%'' '        	
    
Set @SQL = @SQL + ' Order by tblUser.lName, tblUser.Fname'        	
exec (@SQL)

По долгу работы приходится местами переписывать унаследованный код. Я держалась неделю, но после этого шедевра все таки зарегилась на сайте и решила поделиться перлом. Интересно, что бы делали, если бы параметров еще штук пускай даже двадцать добавить?

Крендель, 21 Декабря 2009

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

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

+79.1

1
2
3
4
5
6
7

if ((su != null) & (!su.getId().equals(id))) {
                flag = false;
 } else if ((su != null) & (su.getId().equals(id))) {
                flag = true;//true- not exist
 } else {
                flag = true;
 }

не пишите код по ночам... а то такое выходит..))

maki, 24 Ноября 2009

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

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

+51.8

1
2
3
4
5
6

if(p) {
     if (p->m_String[0] == 0) {
         strcpy("foo", p->m_String);
      }
      DrawText(hdc, p->m_String, strlen(p->m_String), &rc, DT_SINGLELINE | DT_CENTER | DT_VCENTER);
}

и такие веселые куски попадаются в официальном примере написания плагинов на сайте Mozilla
пруф линк - http://mxr.mozilla.org/seamonkey/source/modules/plugin/samples/npruntime/plugin.cpp - строка 750

Coach, 13 Ноября 2009

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

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

+146.3

1
2
3
4
5
6
7

<?if (count($arResult["ERRORS"])):?>
	<?=ShowError(implode("<br />", $arResult["ERRORS"]))?>
<?endif?>

<?if (strlen($arResult["MESSAGE"]) > 0):?>
	<?=ShowNote($arResult["MESSAGE"])?>
<?endif?>

уже глаза разрывает от этого классного форматирования

y6uTbIu_CMEXOM, 10 Ноября 2009

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

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

+163.7

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

print '<SELECT name="date_rozd_year">
<option>1933</option><option>1934</option><option>1935</option><option>1936</option>
<option>1937</option><option>1938</option><option>1939</option><option>1940</option>
<option>1941</option><option>1942</option><option>1943</option><option>1944</option>
<option>1945</option><option>1946</option><option>1947</option><option>1948</option>
<option>1949</option><option>1950</option><option>1951</option><option>1952</option>
<option>1953</option><option>1954</option><option>1955</option><option>1956</option>
<option>1957</option><option>1958</option><option>1959</option><option>1960</option>
<option>1961</option><option>1962</option><option>1963</option><option>1964</option>
<option>1965</option><option>1966</option><option>1967</option><option>1968</option>
<option>1969</option><option>1970</option><option>1971</option><option>1972</option>
<option>1973</option><option>1974</option><option>1975</option><option>1976</option>
<option>1977</option><option>1978</option><option>1979</option><option>1980</option>
<option>1981</option><option>1982</option><option>1983</option><option>1984</option>
<option>1985</option><option>1986</option><option>1987</option><option>1988</option>
<option>1989</option><option>1990</option><option>1991</option><option>1992</option>
<option>1993</option><option>1994</option><option>1995</option><option>1996</option>
<option>1997</option><option>1998</option><option>1999</option><option>2000</option>
<option>2001</option><option>2002</option><option>2003</option><option>2004</option>
<option>2005</option><option>2006</option><option>2007</option><option>2008</option>
<option>2009</option><option>2010</option>
</SELECT>";

вот такой код выбора года рождения :)

marten.spb, 02 Ноября 2009

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

Perl / Говнокод #1854

−91.4

01
02
03
04
05
06
07
08
09
10

sub inet_aton{
  my $n_ip;
  $sth1 = $dbh->prepare("SELECT inet_aton(\"$_[0]\") as inet_aton");
  $sth1->execute();
  while (my $ips = $sth1->fetchrow_hashref()) {
    $n_ip = "$ips->{'inet_aton'}";
  };
  $sth1->finish();
  return $n_ip;
};

вот так в одном коммерческом продукте получают ip-адрес в виде int'а

Mihard, 17 Сентября 2009

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

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

+57.9

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

bool mod_lesser ( int elem1, int elem2 ) {
if (elem1 < 0)
elem1 = - elem1;
if (elem2 < 0)
elem2 = - elem2;
return elem1 < elem2;
}

template <class T>
void printVector(const vector<T> &vec);

int _tmain(int argc, _TCHAR* argv[])
{
ostream_iterator <int> output (cout, " ");
vector <int> v1;
vector <int> v2;
vector <int> v3;
int x=0, i=0;
vector<int>::iterator endLocation;

do
{
cout << "Vvedite " << i << "element mnozhestva A: ";
cin >> x;
if (x!=-99999)
{
v1.push_back(x);
i++;
}
else break;
}
while (x!=99999);
i=0; x=0;

do
{
cout << "Vvedite " << i << "element mnozhestva B: ";
cin >> x;
if (x!=-99999)
{
v2.push_back(x);
i++;
}
else break;
}
while (x!=99999);
i=0; x=0;

do
{
cout << "Vvedite " << i << "element mnozhestva C: ";
cin >> x;
if (x!=-99999)
{
v3.push_back(x);
i++;
}
else break;
}
while (x!=99999);

cout << "A = { ";
printVector (v1);
cout << "}";

cout << '\n';

cout << "B = { ";
printVector (v2);
cout << "}";

cout << '\n';

cout << "C = { ";
printVector (v3);

//Выполнение операции A U B
cout << "}";
vector <int> results (v1.size()+v2.size());
cout << '\n';
sort (v1.begin(), v1.end(), mod_lesser);
sort (v2.begin(), v2.end(), mod_lesser);
sort (v3.begin(), v3.end(), mod_lesser);
merge (v1.begin(), v1.end(), v2.begin(), v2.end(), results.begin());
endLocation=unique (results.begin(), results.end());
cout << "A U B = {";
copy (results.begin(), endLocation, output);
cout << "}";
cout << '\n';

sort (results.begin(), results.end(), mod_lesser);

unique (v3.begin(), v3.end());
cout << "(A U B)\\C = {";
set_difference (results.begin(), results.end(), v3.begin(), v3.end(), output);
cout << "}";
cout << '\n';
system ("pause");
return 0;
}

Вот такие крутые программисты сидят в контакте. Особенно радуют три повторяющихся куска кода для ввода вектора и переменная i, единственное предназначение которой - зачем-то плюсоваться в цикле.

andrey.goryachev, 13 Сентября 2009

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

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

+129.9

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

public static void ClearUsers()
        {
            var toremove = new List<string>();
            foreach (var user in SignedUsers.Values)
            {
                if (!user.IsActive)
                    toremove.Add(user.UserName);
            }
            foreach (var usr in toremove)
            {
                SignedUsers.Remove(usr);
            }

        }

Удаление юзера

DavidM, 04 Сентября 2009

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

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

+32.9

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

#include <stdlib.h>

...

class CAbstractControl
{
	...
};

class CWindow
{
	...	
};

class CButton: public CAbstractControl
{
	...
};

class CLabel: public CAbstractControl
{
	...
};

...

bool operator+(CWindow& win, CAbstractControl& Control)
{
	return win.AddControl(Control);
};

...

void main()
{
	CWindow& w=new CWindow(10,10,300,300,"Mail Messanger v11.0");
	CAbstractControl& b1=new CButton(270,280,300,300,"OK");
	...
	CAbstractControl& l1=new CLabel(10,10,"Mail:");
	...
	if(!(w+b1))abort();
	if(!(w+b2))abort();
	...
};

Программа делает аборт();
:D

Говногость, 19 Августа 2009

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

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