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

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

+165

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

if ($row = mysql_fetch_array($result))
{
	$obegin = $row[0];
	$vbegin = $row[1];
	$lbegin = $row[2];
	$oclose = $row[3];
	$vclose = $row[4];
	$opodk = $row[5];
	$vpodk = $row[6];
	$lpodk = $row[7];
	$oend = $row[8];
	$vend = $row[9];
	$lend = $row[10];
	$ozatr = $row[11];
	$vzatr = $row[12];
	$lzatr = $row[13];
	$ooplat = $row[14];
	$voplat = $row[15];
	$loplat = $row[16];
	$opay = $row[17];
	$vpay = $row[18];
	$lpay = $row[19];
	$otraf = $row[20];
	$vtraf = $row[21];
	$ltraf = $row[22];
}

Даже не смешно

Werdn, 30 Июля 2010

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

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

+159

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

void shiftOrder(int* _order, int _count, int _column)
{
    int i=_count-1;
    while ( _order[i]!=_column )
        i--;

    _order[_count]=_order[i];
    memcpy( _order+i, _order+i+1, (_count-i)*sizeof(int) );

    return;
}

void returnOrder(int* _order, int _count, int _column)
{
    int i=0;
    int j=0;
    int order[MAX_COLS];

    for ( i=0; i< _count; i++)
    {
        if ( m_ColInfo[i].visible )
        {
            order[j]=i;
            j++;
        }
    }

    i=j;
    while ( j<_count )
    {
        if ( _order[i-1]!=_column )
        {
            order[j]=_order[i-1];
            j++;
        }
        i++;
    }
    memcpy(_order, order, MAX_COLS*sizeof(int));

    return;
}

я знаю, это всё торфяники!1

ilardm, 30 Июля 2010

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

Pascal / Говнокод #3831

+94

1
2
3
4
5
6
7
8

if MessageDlg('Удалить проводку №'+srcTransact.DataSet.FieldByName('Trans_ID').AsString+' и ее спецификацию?',
    mtConfirmation, [mbYes, mbNo], 0) = mrYes then
  begin
    CMD.SQL.Add('EXEC dbo.asp_DelTransact '+srcTransact.DataSet.FieldByName('Trans_ID').AsString );
    CMD.Execute;
    _refresh(srcTransact.DataSet);
  end;
end;

Из реального проекта (учётная система).
При определённых условиях можно удалить не только то что пользователь собирался удалить :).
Да и вообще, сам код - говно.

Fes, 30 Июля 2010

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

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

+102

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

private static string GetRootedCurrentConfigurationFile(string configurationFile)
{
if (string.IsNullOrEmpty(configurationFile))
{
throw new ArgumentException(Resources.ExceptionStringNullOrEmpty, "configurationFile");
}
if (!File.Exists(configurationFile))
{
throw new FileNotFoundException(string.Format(CultureInfo.CurrentCulture, Resources.ExceptionConfigurationLoadFileNotFound, new object[] { configurationFile }));
}
if (!Path.IsPathRooted(configurationFile))
{
return Path.Combine(AppDomain.CurrentDomain.BaseDirectory, configurationFile);
}
return configurationFile;
}

Собснно Говнокод by Microsoft (Ent Lib 5.0.414.0)

TrueLauncher, 30 Июля 2010

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

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

+115

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

public static string GetTable(int type, string title, string[][] rows)
{
    bool flag2;
    int num;
    bool isEmpty = false;
    string str = "";
    switch (type)
    {
        case 1:
            str = str + "\r\n                            <div class='clear'> </div>\r\n\t\t\r\n\t\t                        <div class='down'>\r\n                                \r\n\t\t\t\t\t                <div class='ProperyDetailHeading'>\r\n\t\t\t\t\t\t                <span class='color2'>" + title + "</span>\r\n\t\t\t\t\t                </div>\r\n\t\t\t\t\t                <div class='PropertyDetailContent'>\r\n\t\t\t\t\t\t                <div  class='PropertyDetailInformation content_inner'>\r\n\t\t\t\t\t\t\t                <table summary='Test Table'>\r\n\t\t\t\t\t\t\t\t                <tbody>";
            if ((rows != null) && (rows.Length > 0))
            {
                flag2 = true;
                for (num = 0; num < rows.Length; num++)
                {
                    if ((rows[num] != null) && (rows[num].Length > 0))
                    {
                        str = str + GetTableRow(ref flag2, rows[num][0], rows[num][1], ref isEmpty);
                    }
                }
            }
            break;
        case 2:
......

ASP.NET, пакистанские кодеры. Вообще весь код с сайта можна поместить на сайте отдельным разделом. В проекте вообще не используются контролы типа DataGrid/FormView.. даже DataSource. У них и тэмплэйты свои :(.. Грустно так все...(

Coffeeholic, 30 Июля 2010

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

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

+164

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

function isGood($race,$class) {
         switch ($race) {
                case 1:
                     if ($class == 1 || $class == 2 || $class == 4 || $class == 5 || $class == 6 || $class == 8 || $class == 9) { return true; }
                     break;
                case 2:
                     if ($class == 1 || $class == 3 || $class == 4 || $class == 6 || $class == 7 || $class == 8 || $class == 9) { return true; }
                     break;
                case 3:
                     if ($class == 1 || $class == 2 || $class == 3 || $class == 4 || $class == 5 || $class == 6) { return true; }
                     break;
                case 4:
                     if ($class == 1 || $class == 3 || $class == 4 || $class == 5 || $class == 6 || $class == 11) { return true; }
                     break;
                case 5:
                     if ($class == 1 || $class == 4 || $class == 5 || $class == 6 || $class == 8 || $class == 9) { return true; }
                     break;
                case 6:
                     if ($class == 1 || $class == 3 || $class == 6 || $class == 7 || $class == 11) { return true; }
                     break;
                case 7:
                     if ($class == 1 || $class == 4 || $class == 6 || $class == 8 || $class == 9) { return true; }
                     break;
                case 8:
                     if ($class == 1 || $class == 3 || $class == 4 || $class == 5 || $class == 6 || $class == 7 || $class == 8) { return true; }
                     break;
                case 10:
                     if ($class == 2 || $class == 3 || $class == 4 || $class == 5 || $class == 6 || $class == 8 || $class == 9) { return true; }
                     break;
                case 11:
                     if ($class == 1 || $class == 2 || $class == 3 || $class == 5 || $class == 6 || $class == 7 || $class == 8) { return true; }
                     break;
         }
         return false;

}

Oops, 30 Июля 2010

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

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

+156

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

// в данном случае count($xls_data) не подойдет, потому что
// он меньше изза того что индексы не всегда по +1 идут
end($xls_data);
list($xls_dataCount,$unused) = each($xls_data);

// тут некоторый код, тоже пахнет

for($i = 3; $i <= $xls_dataCount; $i++){
	$row = $xls_data[$i];
	if(count($row) == 1 && $estnames[$row[1]])$ename = $row[1];
	else{
		foreach($years as $yindex => $year)
			$data[$ename][$row[1]][$year] = $row[$yindex];
	}
}

Сижу и ковыряюсь в говнице, по уши увяз((((

asics, 30 Июля 2010

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

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

+164

1
2
3
4
5
6

try {
    ...
} catch ( Exception $O_o ) {
    error_log( $O_o->getMessage() );
    ...
}

Совсем неожиданный эксепшин O_o

eee_qqq, 30 Июля 2010

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

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

+74

1

for (Integer i = 0; i<_pwiList.size(); i++)

Вот к чему приводит бездумное использование классов-обёрток "для унификации". Конечно, оптимизатор может здесь сообразить, но не факт. А если данный фрагмент трактовать буквально, то i++ приведёт к unboxing, увеличению и последующему boxing.

konsoletyper, 30 Июля 2010

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

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

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