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

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

В номинации:

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

За время:

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

+154.3

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

There is a little problem with iconv in such using:

$mytext = iconv('windows-1251', 'utf-8', $mytext);
echo $mytext;

This code isn't work correctly. Solution is:

$mytext_utf = iconv('windows-1251', 'utf-8', $mytext);
echo $mytext_utf;
//or just
echo iconv('windows-1251', 'utf-8', $mytext);

ссылка: http://www.php.net/manual/en/function.iconv.php#83511

смотрел ман, наткнулся на вот такой комментарий
Если я еще не ослеп, то чувак одну один и тот же код три раза переписал
и он якобы у него заработал :D

getme, 03 Сентября 2009

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

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

+50.1

1
2
3
4
5
6
7
8
9

...
double total, free;
GetCensoredServer()->GetCensoredMemory(total, free);
if (free<0.) // <-- чудесная проверка
{
	MessageBox(AfxGetMainWnd()->m_hWnd, censored("Please check censored for enough memory!")), _T(""), MB_OK|MB_ICONWARNING);
	return false;
}
...

Найдено в рабочем коде. Некоторые идентификаторы изменены.

Xentrax, 01 Сентября 2009

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

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

−866.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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67

CREATE TABLE [Battles] (
	[name] [varchar] (20) NOT NULL ,
	[date] [datetime] NOT NULL 
) ON [PRIMARY]
GO
CREATE TABLE [Classes] (
	[class] [varchar] (50) NOT NULL ,
	[type] [varchar] (2) NOT NULL ,
	[country] [varchar] (20) NOT NULL ,
	[numGuns] [tinyint] NULL ,
	[bore] [real] NULL ,
	[displacement] [int] NULL 
) ON [PRIMARY]
GO
CREATE TABLE [dbo].[Ships] (
	[name] [varchar] (50) NOT NULL ,
	[class] [varchar] (50) NOT NULL ,
	[launched] [smallint] NULL 
) ON [PRIMARY]
GO
CREATE TABLE [dbo].[Outcomes] (
	[ship] [varchar] (50) NOT NULL ,
	[battle] [varchar] (20) NOT NULL ,
	[result] [varchar] (10) NOT NULL 
) ON [PRIMARY]
GO
ALTER TABLE [dbo].[Battles] ADD 
	CONSTRAINT [PK_Battles] PRIMARY KEY  CLUSTERED 
	(
		[name]
	)  ON [PRIMARY] 
GO
ALTER TABLE [dbo].[Classes] ADD 
	CONSTRAINT [PK_Classes] PRIMARY KEY  CLUSTERED 
	(
		[class]
	)  ON [PRIMARY] 
GO
ALTER TABLE [dbo].[Ships] ADD 
	CONSTRAINT [PK_Ships] PRIMARY KEY  CLUSTERED 
	(
		[name]
	)  ON [PRIMARY] 
GO
ALTER TABLE [dbo].[Outcomes] ADD 
	CONSTRAINT [PK_Outcomes] PRIMARY KEY  CLUSTERED 
	(
		[ship],
		[battle]
	)  ON [PRIMARY] 
GO
ALTER TABLE [dbo].[Ships] ADD 
	CONSTRAINT [FK_Ships_Classes] FOREIGN KEY 
	(
		[class]
	) REFERENCES [dbo].[Classes] (
		[class]
	) NOT FOR REPLICATION 
GO
ALTER TABLE [dbo].[Outcomes] ADD 
	CONSTRAINT [FK_Outcomes_Battles] FOREIGN KEY 
	(
		[battle]
	) REFERENCES [dbo].[Battles] (
		[name]
	)
GO

Это с sql-ex.ru , база "корабли"
Может, упражнения там интересные,
но сама база истинный говнокод. Почему?
Строчка
"В отношение Outcomes могут входить корабли, отсутствующие в отношении Ships."
делает её говнокодом.

lexx, 27 Августа 2009

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

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

+80.3

1

// here must be english equivalent for simple russian 'nahuya?'

Bobby, 27 Августа 2009

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

JavaScript / Говнокод #1692

+161.9

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

function checkAllRemarks()
{ // ЧТО ЭТО!?!?!?!?17СЕМНАДЦАТЬ!!!!1111
   for (i = 0; i < document.getElementsByTagName("input").length; i++)
   {
        if (document.getElementsByTagName("input")[i].name.split("|").length>=2)
        {
            if (document.getElementsByTagName("input")[i].name.split("|")[0]=="CBR")
            {
                if (document.getElementsByTagName("input")[i].name.split("|")[1].length==36)
                {
                    if (document.getElementsByName("allRemarks")[0].checked == true)
                        document.getElementsByTagName("input")[i].checked = true;
                    else
                        document.getElementsByTagName("input")[i].checked = false;            
                }
            }
        }
    } 
}

Много кривого кода я видел на работе, но глядя на эту функцию я не мог не поразиться.

iley, 26 Августа 2009

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

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

+70.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

import javax.microedition.lcdui.*;
import java.lang.*;
import java.io.*;

public class LNRES
{
	public InputStream is=null;
	public String lines[]=null;
	
	LNRES(String name,int size) 
	{
		int x=0;
		is=getClass().getResourceAsStream(name);
		byte mb[]=new byte[size];
		try { x=is.read(mb); } catch (Exception ex) { }
		char mc[]=new char[x];
		for (int i=0; i<x; i++) mc[i]=(char)mb[i];
		
		int n=0,a=0,l=0;
		String mas2[]=null;
		for (int i=0; i<mc.length; i++) {
			if ((mb[i]==13)||(i==mc.length-1)) {
				mas2=new String[n+1];
				for (int j=0; j<n; j++) mas2[j]=new String(lines[j]);
				mas2[n]=new String(mc,a,l+((i==mc.length-1)?1:0)); n++; lines=new String[n];
				for (int k=0; k<n; k++) lines[k]=new String(mas2[k]);
				a=i+2; l=0; i++;
			} else l++;
		}
		lines=new String[n];
		for (int i=0; i<n; i++) lines[i]=new String(mas2[i]);		
	}	
}

Писалось под j2me. Класс считывает весь файл и превращает его в массив строк.

k06a, 26 Августа 2009

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

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

+156

1
2
3
4
5
6
7
8
9

class DbSimple_Generic_Database extends DbSimple_Generic_LastError
{
...
    /**
     * Virtual protected methods
     */
    function ____________PROTECTED() {} // for phpEclipse outline
...
}

ой какая красотень у нас будет в аутлайне теперь...

Smoke, 25 Августа 2009

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

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

+129.7

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

[StructLayout(LayoutKind.Explicit, Size=264)]
public struct STRRET
{
     [FieldOffset(0)]
     public UInt32 uType;    // One of the STRRET_* values

     [FieldOffset(4)]
     public IntPtr pOleStr;    // must be freed by caller of GetDisplayNameOf

     [FieldOffset(4)]
     public IntPtr pStr;        // NOT USED

     [FieldOffset(4)]
     public UInt32 uOffset;    // Offset into SHITEMID

     [FieldOffset(4)]
     public IntPtr cStr;        // Buffer to fill in (ANSI)
}

Обертка над структурой STRRET. Доставляет тут http://www.pinvoke.net/default.aspx/Structures/STRRET.html

dethtroll, 20 Августа 2009

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

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

+165.1

1
2
3
4
5

public function setSomething($var) {
    if ($var === true or $var === false){
        $this->something = $var;
    }
}

Отечественный type hinting для bool.

striker, 20 Августа 2009

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

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