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

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

В номинации:

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

За время:

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

+147.7

1

$this->view->isAdmin = ($auth->getIdentity()->role_id == 1) ? true : false;

Называется: "заработался".
Написал и только потом дошло.

oldfornit, 16 Сентября 2009

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

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

+157.4

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

....
$str = str_replace(',', ' ', $this->searchString);
$str = str_replace(';', ' ', $str);
$str = str_replace('.', ' ', $str);
$str = str_replace(':', ' ', $str);
$str = str_replace('+', ' ', $str);
$str = str_replace('-', ' ', $str);
$str = str_replace('"', ' ', $str);
$str = str_replace("'", ' ', $str);
		
$str = trim(preg_replace('!\s{2,}!', ' ', $str));
.....

вот что я наклал как-то

paranoid, 11 Сентября 2009

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

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

+166.8

1

unset($keys[count($keys)-1][count($keys[count($keys)-1])-1]);

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

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

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)

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