По колено в коде — Говнокод.ру

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

+110

1
2
3
4
5
6
7

protected static string CreateSubIndexName(string prefix, int index) {
            return String.Format(CultureInfo.InvariantCulture, "{0}[{1}]", prefix, index);
        }

        protected static string CreateSubIndexName(string prefix, string index) {
            return String.Format(CultureInfo.InvariantCulture, "{0}[{1}]", prefix, index);
        }

Из исходников ASP.NET MVC 2. Класс DefaultModelBinder.
Рядышком друг за другом два метода с совершенно одинаковым содержимым.
Наш девиз: дублирование кода, умещающегося в одну строчку, за дублирование не считается.

Pauchok-Anaynckiy, 24 Ноября 2010

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

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

+118

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

//-----------------------------------------------------------------------------------------------------------
// Создаю ПОСВИДЧЕННЯ ПРО ВИДРЯДЖЕННЯ
// Создаю тело документа
FunForServices.writeToLog(2, ps, "create PPV");

rs1 = stat1.executeQuery("Select D.DEPAR_Name As Dep, rtrim(L.LDAP_Name) as FIO, P.PEOP_Posit As Pos From IPS..PEOP P, IPS..LDAP L, IPS..DEPAR D Where L.LDA_LDAP_Login=D.LDAP_Login and L.LDAP_Login=P.LDAP_Login and L.LDAP_Login='"+rs.getString("Login")+"'");
rs1.next();
rez1="<?xml version=''1.0'' ?><u1><st><![CDATA[";
rez1+="<table width=\"100%\" border=0><tr><td width=\"33%\" align=right><table width=\"33%\"><tr>";
rez1+="<td style=\"FONT-WEIGHT: bold; FONT-SIZE: x-small; FONT-FAMILY: Arial; TEXT-ALIGN: center\">";
rez1+="ЗАТВЕРДЖЕНО</td></tr><tr><td style=\"FONT-WEIGHT: bold; FONT-SIZE: x-small; FONT-FAMILY: Arial; TEXT-ALIGN: center\">";
rez1+="наказом Державної податкової</td></tr><tr><td style=\"FONT-WEIGHT: bold; FONT-SIZE: x-small; FONT-FAMILY: Arial; TEXT-ALIGN: center\">";
rez1+="адмiнiстрацiї України вiд</td></tr><tr><td style=\"FONT-WEIGHT: bold; FONT-SIZE: x-small; FONT-FAMILY: Arial; TEXT-ALIGN: center\">";
rez1+="28.07.97 г. N 260</td></tr></table></td></tr><tr><td><br><br></td></tr><tr><td align=middle><I><B><FONT face=Arial size=4>ПОСВIДЧЕННЯ ПРО ВIДРЯДЖЕННЯ N</FONT></B></I>";
rez1+="<td>                     </td>";
//////////////////////////////////////
//	строк 30 такого
//////////////////////////////////////
rez1+="<TR><TD> </TD></TR><TR><TD><TABLE cellSpacing=0 cellPadding=0 width=\"100%\"><TR><TD align=left><STRONG><EM><FONT face=Arial>КЕРIВНИК</FONT></EM></STRONG></TD><TD align=middle id=pod>Пiдпис</TD>";
rez1+="<TD align=right onclick=StampUtv(\""+sign+"\")><B>"+utvfio+"</B></TD></TR></TABLE></TD></TR><TR><TD> </TD></TR><TR><TD><STRONG><FONT face=Arial>М.П.</FONT></STRONG></TD></TR></TABLE></TD></TR>";
rez1+="</table>";
rez1+="]]></tp></u1>";

rs1 = stat1.executeQuery("Declare @Rez int exec GetCardNum "+FunForServices.Year+", '"+rs.getString("Login")+"', 'A', @Rez out Select @Rez");
rs1.next();
annNumb1 = rs1.getInt(1);

rs1 = stat1.executeQuery("Declare @Rez int exec GetCardNum "+FunForServices.Year+", '"+rs.getString("Login")+"' , 'C', @Rez out Select @Rez");
rs1.next();
ndoc1=rs1.getInt(1);
FunForServices.CreateBody(ndoc1, 0, "DocB", rez1, stat1);
FunForServices.writeToLog(2, ps, ndoc1+" for "+rs.getString("Login"));

rs1=stat1.executeQuery("AddDocCard "+rs.getInt("Act")+", '"+utv+"', "+ndoc+", "+rs.getInt("Grup")+", 1, 4,'"+regndoc.substring(0,regndoc.lastIndexOf("-"))+"', 'ПОСВIДЧЕННЯ ПРО ВIДРЯДЖЕННЯ', null,'"+annot+"', 'Нормальный', 4, null, null, null, null, null, null, '"+pk+"', ';"+utv+"', null, ';"+rs.getString("Login")+"', '"+pkdat+"', @IIdDocCard="+ndoc1+", @IIdAttending = "+annNumb1);
stat1.executeUpdate("Constatation "+ndoc1+", '"+utv+"'");
FunForServices.writeToLog(6, ps, "");

лайно з Украïни...

3.14159265, 24 Ноября 2010

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

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

+78

1
2
3

double price=199990.10;
BigDecimal realPrice=new BigDecimal(price).round(
		new MathContext((int)Math.round(Math.log10(price))+3)));

Округление до копеек.
Реальный финансовый проект.

gavnokoder, 24 Ноября 2010

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

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

+156

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

function uname($v)
{
	return $v;
}

function name($v)
{
	return $v;
}

function url($v)
{
	return $v;
}
		
function icq($v)
{
	return $v;
}
	
function from($v)
{
	return $v;
}
	
function intrest($v)
{
	return $v;
}

foreach( $s_data AS $k=>$v )
{
	$v =  $k($v);
	$parser -> set_var($this -> tmpl_id,$k,$v);
}

Клинический случай. Убрал «$v = $k($v);» и, как и следовало ожидать, ничего не изменилось.

Oonitaz, 24 Ноября 2010

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

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

+174

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

// used to set or reset batch mode
void SetIsBatch(bool bSet)
{
    if(bSet)
    {
        if(m_Chart.GetIsBatched())
            return;
        else
        {
            m_Chart.SetIsBatched(TRUE);
        }
    }
    else
    {
       if(m_Chart.GetIsBatched())
       {
           m_Chart.SetIsBatched(FALSE);
       }
       else
           return;
    }
}

Shumway, 24 Ноября 2010

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

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

−93

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

Private Sub Label10_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Label10.Click

        RadioButton1.Checked = False
        RadioButton2.Checked = False
        RadioButton3.Checked = False
        RadioButton4.Checked = False
        RadioButton5.Checked = False
        RadioButton6.Checked = False
        RadioButton7.Checked = False
        RadioButton8.Checked = False
        RadioButton9.Checked = False
        RadioButton10.Checked = False

        RadioButton11.Checked = False
        RadioButton12.Checked = False
        RadioButton13.Checked = False
        RadioButton14.Checked = False
        RadioButton15.Checked = False
        RadioButton16.Checked = False
        RadioButton17.Checked = False
        RadioButton18.Checked = False
        RadioButton19.Checked = False
        RadioButton20.Checked = False

        RadioButton21.Checked = False
        RadioButton22.Checked = False
        RadioButton23.Checked = False
        RadioButton24.Checked = False
        RadioButton25.Checked = False
        RadioButton26.Checked = False
        RadioButton27.Checked = False
        RadioButton28.Checked = False
        RadioButton29.Checked = False
        RadioButton30.Checked = False

        RadioButton31.Checked = False

        RadioButton33.Checked = False
        RadioButton34.Checked = False
        RadioButton35.Checked = False
        RadioButton36.Checked = False
        RadioButton37.Checked = False
        RadioButton38.Checked = False
        RadioButton39.Checked = False
        RadioButton40.Checked = False
        RadioButton41.Checked = False
        RadioButton44.Checked = False

        Button1.Enabled = True
        Panel5.Visible = True
        Panel7.Visible = False
        Panel3.Visible = False
        Panel4.Visible = False
        Panel6.Visible = False
        Panel8.Visible = False
        Panel10.Visible = False
        Panel2.Visible = False
        Panel14.Visible = False
        Panel16.Visible = False
        Panel1.Visible = False


    End Sub

А вот так кодят ценные специалисты, работающие за неплохие деньги в столичных ВУЗах .

non_def, 23 Ноября 2010

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

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

+86

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

boolean IsWeekStartsMonday() {
if (strDay.equalsTo("Russia") || strDay.equalsTo("Ukraina") || ....)
    return true;
else
    return false;
}

void SomeFunction(){
....
if (IsWeekStartsMonday())
    startDay = java.util.Calendar.getFirstDayOfWeek();
else
    startDay = java.util.Calendar.SUNDAY;
}

из исходников андроидовской апликухи, отличился гражданин Индии :)

AndyFox, 23 Ноября 2010

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

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

+84

1
2
3
4
5
6
7

try {
                if (field[i][j + 1] == 0) {
                    field[i][j + 1] = 2;
                    fifo.add(new Cell(i, j + 1));
                }
            } catch (ArrayIndexOutOfBoundsException ignored) {
            }

Плевать на то что будет стучаться к несуществующему элементу массива, заигнорим и все!

dexatot, 23 Ноября 2010

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

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

+145

1
2
3
4
5
6
7
8

if (users.ToList().Count == 0)
{
          return true;
}
else
{
          return false;
}

Mikita, 23 Ноября 2010

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

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

+155

1
2
3
4
5
6
7
8
9

void XmlElementHandler::startElement(
                    const XMLCh* const uri,
                    const XMLCh* const localname,
                    const XMLCh* const qname,
                    const XERCES_CPP_NAMESPACE_QUALIFIER Attributes& attrs
                    )
{
  static XmlContext *context = XmlContext::getInstance();
  static XmlParser *parser = XmlParser::getInstance();

оптимизируем обращения к синглтонам. потому что одной невидимой глобальной переменной явно не достаточно. вот такое щасце в 30+ методах на реализации интерфейса к Xerces-C.

Dummy00001, 23 Ноября 2010

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

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