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

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

В номинации:

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

За время:

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

+99.7

1
2
3
4
5

//Use the Name field
string[] nameParts = customer.Name.Split(' ');

string firstName = nameParts[0];
string lastName = customer.Name.Replace(nameParts[0], "").TrimStart(' ');

Как вам такое разбиение полного имени на части?!

pmalyavkin, 20 Октября 2009

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

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

+170.2

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

kadr1=function(){
document.getElementById('banner').style.marginLeft='1'
window.setTimeout('kadr2()',100)
};

kadr2=function(){
document.getElementById('banner').style.marginLeft='2'
window.setTimeout('kadr3()',100)
};

kadr3=function(){
document.getElementById('banner').style.marginLeft='3'
window.setTimeout('kadr4()',100)
};

и так 50 кадров анимации)) я улез под стол

Infamous, 20 Октября 2009

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

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

+136.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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40

protected void WriteLoginBlock()
    {
        string s = "";
        if (SiteSecurity.IsAuthenticated)
        {
            s = @"<div style=""margin-top: 3px;""><a href=""/login/"" style=""text-decoration: underline; color:#000; background:none;"">" + SiteSecurity.CurrentUser.Name + "</a></div>";
        }
        else
        {
            s = @"
			
			<form action=""/login/"" method=""POST"" id=""gLoginForm"">
		<div class=""rel_i"">
			    <input class=""txtinp"" type=""text"" value=""логин"" id=""gLogin"" name=""cllgn"" />
			    <input class=""txtinp"" type=""text"" name=""clpwd"" id=""gPwd"" value=""пароль"" onfocus=""if (this.value==this.defaultValue) {this.value='';this.type='password'}"" onblur=""if(this.value=='') {this.value=this.defaultValue;this.type='text'}"" />
				
				
				
				
				<!--<label for=""clpwd"" class=""pwd_label"" id=""gPwdLabel"">пароль</label>-->
			    <a href=""/login/"" id=""blogin"">&#160;</a>
				</div>
		    </form>
			
		    <script type=""text/javascript"">
			    function isChanged(obj) { return $(obj)[0]._changed; }
			    function isValue(obj, value) { return $.trim($(obj).val()).toLowerCase() == value.toLowerCase(); }
			    $(""#gLogin"")
			    .keydown ( function() { this._changed = true; } )
			    .focus( function() { if (!isChanged(this) && isValue(this, 'логин')) $(this).val(''); } )
			    .blur( function() { if(isValue(this, '')) { $(this).val('логин'); $(this)[0]._changed=false; } } );

                $(""#blogin"").click(function(){
			    if(!isChanged(""#gLogin"") || isValue(""#gLogin"", '') || isValue(""#gPwd"",'')) return false;
			    $('#gLoginForm').submit();
			    return false; });
		    </script>";
        }
        Response.Write(s);
    }

bniwredyc, 15 Октября 2009

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

VisualBasic / Говнокод #1972

−124.6

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

''' <summary>
''' Представляет обект RSS-новость
''' </summary>
''' <remarks></remarks>
Public Class NewsItems
...
    Public Property AgencyTitle() As String
        Get
            Return Me._agencyTitle
        End Get
        Set(ByVal value As String)
            Me._agencyTitle = value
        End Set
    End Property
...
    ''' <summary>
    ''' Вставка региональной новости в БД
    ''' </summary>
    ''' <param name="Lang">язык</param>
    ''' <param name="DateFormat">формат даты в рассылке</param>
    ''' <param name="region">ID региона</param>
    ''' <remarks></remarks>
    Public Sub AddNewRegion(ByVal Lang As String, ByVal DateFormat As String, ByVal region As Integer)
        If AgencyTitle.StartsWith("зБЪЕФБ.Ru") Then
            Exit Sub
        End If
...
    End Sub
...
End Class

заставили разбираца с проектом на ВБ...
и вот шо я там нашол.
...ну конешно... я бы тоже испугался новостного агенства с таким названием )))

xeonix, 13 Октября 2009

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

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

+154.6

1

$headers .= "From: " . '[email protected]'. " <" . '[email protected]' . ">\n";

Made by real Indians!

getme, 05 Октября 2009

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

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

+89.4

1
2
3
4
5
6
7

<option name="d">
      <select value="collection">Коллекции</select>
      <select value="cash">В наличии</select>
      <select value="documents">Выставки</select>
      <select value="bio">Биография</select>
      <select value="int">Интерьер</select>
  </option>

HTML без комментариев) Я валялся от смеха, когда увидел :D

greevex, 30 Сентября 2009

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

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

+132

1
2
3
4
5

<?php

echo "If you see this then PHP works fine!";

?>

Проверка работоспособности интерпретатора PHP.

XPSystemServices, 29 Сентября 2009

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

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

+144.3

1
2

if(preg_match("/^((https?\:\/\/)?([a-z0-9]+\-*[a-z0-9]+)+\.)+[a-z0-9]{2,7}(\:[0-9]{1,4})?$/i", $_GET['url'])) print 'тута пля не ерор !';
else{print 'неверный url';}

no comments :D

greevex, 29 Сентября 2009

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

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

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

/// Из aspx файла

//<asp:Repeater ID="Repeater1" runat="server">
//    <HeaderTemplate>
//        <table width="100%" cellspacing="5">
//    </HeaderTemplate>
//    <ItemTemplate>
//        <tr>
//            <td>
//                ...
//            </td>
//            <asp:Label runat="server" Visible='<%# DataBinder.Eval(Container.DataItem, "visibleCol2") %>'>
//                <td>
//                    ...
//                </td>
//            </asp:Label>
//            <asp:Label ID="LCol3" runat="server" Visible='<%# DataBinder.Eval(Container.DataItem, "visibleCol3") %>'>
//                <td>
//                    ...
//                </td>
//            </asp:Label>
//        </tr>
//    </ItemTemplate>
//    <FooterTemplate>
//        </table>
//    </FooterTemplate>
//</asp:Repeater>

// Из cs файла

DataTable dtres = // получаем товары
if (dtres != null && dtres.Rows.Count > 0)
{
    DataTable dt = new DataTable();
	/*...*/
    
    for (int i=0;i<dtres.Rows.Count;)
    {
        DataRow dr=dtres.Rows[i];
        /*...*/

        int col2Index = i + 1;
        if (col2Index < dtres.Rows.Count)
        {
            /*...*/
            i++;
            int col3Index = i + 1;
            if (col3Index < dtres.Rows.Count)
            {
                /*...*/
                i++;
            }
            else { /*...*/ }
        }
        else { /*...*/ }

        dt.Rows.Add(newRow);
        i++;
    }

    /*...*/
}

Человеку нужно было сделать товары в сетке 3x6.
От того что он сделал у меня пропал дар речи. (Чтобы тут очень много кода не бы большую (не нужную для понимания) часть заменил на "...")
Вобщем, в двух словах, он поместил вторую и третью ячейки таблицы в серверные контролы (причем в Label), а в коде берёт по три товара и засовывает в один RepeaterItem, ну а если количество товаров не делится нацело на три, то в последний RepeaterItem засовываются пустые данные, а Label делается невидимым.
Как-то так.
И самое главное, я не знаю, как человеку объяснить, что так нехорошо делать, т.к он главный программист...

Ordos, 27 Сентября 2009

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

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