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

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

В номинации:

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

За время:

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

+75.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
33
34
35
36
37
38
39

public static ru.project.subpackage.PersonDTO convertOshPersonDtoToPersonDto(OshPersonDTO person){
        ru.project.subpackage.PersonDTO dto = new ru.project.subpackage.PersonDTO();
        dto.setPersonid(person.getPersonid().longValue());
        dto.setNamelast(person.getNamelast());
        dto.setNamefirst            (person.getNamefirst            ());
        dto.setNamesec              (person.getNamesec              ());
        dto.setInitials             (person.getInitials             ());
        dto.setSex                  (convertSkVocValue(person.getSex()));
        dto.setReason               (person.getReason               ());
        dto.setWorkphone            (person.getWorkphone            ());
        dto.setWorkphonedigit       (person.getWorkphonedigit       ());
        dto.setLocalphone           (person.getLocalphone           ());
        dto.setLocalphonedigit      (person.getLocalphonedigit      ());
        dto.setHomephone            (person.getHomephone            ());
        dto.setHomephonedigit       (person.getHomephonedigit       ());
        dto.setMobilephone          (person.getMobilephone          ());
        dto.setMobilephonedigit     (person.getMobilephonedigit     ());
        dto.setFax                  (person.getFax                  ());
        dto.setFaxdigit             (person.getFaxdigit             ());
        dto.setPager                (person.getPager                ());
        dto.setEmail                (person.getEmail                ());
        dto.setWeb                  (person.getWeb                  ());
        dto.setNamelastdative       (person.getNamelastdative       ());
        dto.setNamefirstdative      (person.getNamefirstdative      ());
        dto.setNamesecdative        (person.getNamesecdative        ());
        dto.setNamelastaccusative   (person.getNamelastaccusative   ());
        dto.setNamefirstaccusative  (person.getNamefirstaccusative  ());
        dto.setNamesecaccusative    (person.getNamesecaccusative    ());
        dto.setNamelastgenitive     (person.getNamelastgenitive     ());
        dto.setNamefirstgenitive    (person.getNamefirstgenitive    ());
        dto.setNamesecgenitive      (person.getNamesecgenitive      ());
        dto.setNamelastinstrumental (person.getNamelastinstrumental ());
        dto.setNamefirstinstrumental(person.getNamefirstinstrumental());
        dto.setNamesecinstrumental  (person.getNamesecinstrumental  ());
        dto.setNamelastprepositional(person.getNamelastprepositional());
        dto.setNamefirstprepositional(person.getNamefirstprepositional());
        dto.setNamesecprepositional (person.getNamesecprepositional ());
        return dto;
    }

И так далее еще несколько сотен строк. А главное переупаковка из одних объектов в другие и обратно бессмысленна, т.к. можно использовать исходные (они доступны в приложении)

johnsoft, 07 Сентября 2009

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

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

+127

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

/// <summary>
    ///  Retrieve currency rates from an external site to be sure they are up to date.
    /// In this case just checking the one currency (Australian Dollar) so no need to dynamically parse the site.
    /// </summary>
    /// <returns>currency rates or msg indicating an error</returns>
    private String getCurrencyRates()
    {
        string strURL = @"http://www.x-rates.com/d/JPY/table.html";
        HttpWebRequest txtRequest = (HttpWebRequest)WebRequest.Create(strURL);

        txtRequest.Method = "GET";
        txtRequest.ContentType = "application/x-www-form-urlencoded";
        string response;
        using (StreamReader streamReader = new StreamReader(txtRequest.GetResponse().GetResponseStream()))
        {
            response = streamReader.ReadToEnd();
            if (response.IndexOf("Australian Dollar") > 0)
            {
                //parse the returned page for the two values of the currency rate based on the existing design
                int ind_jpy = (response.IndexOf("/d/AUD/JPY/graph120.html") + 39);
                int ind_aud = (response.IndexOf("/d/JPY/AUD/graph120.html") + 39);
                String jpy_aud = response.Substring(ind_jpy, (response.IndexOf("</a>",ind_jpy) - ind_jpy) );
                String aud_jpy = response.Substring(ind_aud, (response.IndexOf("</a>", ind_aud) - ind_aud));
                Session["curr_rate"] = "set";
                Session["JPY"] = jpy_aud;
                Session["AUD"] = aud_jpy;
                return aud_jpy + " / " + jpy_aud;
            }
            //else present msg to user that unable to obtain currency rates
        }
        return "";
    }

Еще один кандидат

OlgaWolga, 07 Сентября 2009

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

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)

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