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

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

В номинации:

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

За время:

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

+108

1
2
3
4

userPerms = (Permissions)Session["permissions"];
if (userPerms.AllUserRights["rod_view"] == null && userPerms.AllUserRights["rod_view"] == null) {
    throw new Exception("no rights");
}

Я по два раза по два раза не повторяю не повторяю.

sergey_sh, 10 Мая 2012

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

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

+108

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

string result = Encoding.GetEncoding(CharSet).GetString(resultData);
            //       сам паттерн    \[\[\[\"([^"]*)


            String commonString = "";
            commonString += @"\";
            commonString += @"[";
            commonString += @"\";
            commonString += @"[";
            commonString += @"\";
            commonString += @"[";
            commonString += @"\";
            commonString += "\"";
            commonString += @"(";
            commonString += "[";
            commonString += "^";
            commonString += "\"";
            commonString += @"]";
            commonString += @"*";
            commonString += @")";
            String pattern = commonString;

            Regex reg = new Regex(pattern);
            var mathces = reg.Match(result);
            String stringResult = mathces.ToString().Replace("[[[\"", "");

Сборка паттерна регулярки, скажите как можно такое покороче собрать

AndrKn, 11 Апреля 2012

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

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

+108

01
02
03
04
05
06
07
08
09
10
11

public static string RandomWord(int Number)
{    
	Random RandomIndex = new Random();
	string[] ArrayLetter = {"q", "w", "e", "r", "t", "y", "u", "i", "o", "p", "a", "s", "d", "f", "g", "h", "j", "k", "l", "z", "x", "c", "v", "b", "n", "m"};
	string OutputValue = null;
	for (int i = 0; i < Number; i++)
	{
	OutputValue += ArrayLetter[RandomIndex.Next(0, 25)];
	}
	return OutputValue;
}

Метод выводит случайное слово блин.

KusokGovna, 17 Марта 2012

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

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

+108

1

if (rmes<>'')and(rmes<>#13#10)and(rmes<>#13#10#13#10)and(rmes<>#13#10#13#10#13#10) then

Эталонный условный оператор прямо из Парижской Дельфийской палатки мер и весов.

bugmenot, 18 Августа 2011

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

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

+108

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

using(FileStream fs = new FileStream("имя файла", FileMode.CreateNew))
{
  using(StreamWriter sw = new StreamWriter(fs))
  {
    sw.Write("Lloyd ");
    sw.Write("is ");
    sw.Write("cool ");
    sw.Write("guy. ");
    sw.Write(":)");
    sw.Flush();
  }
}

Взято здесь http://www.rsdn.ru/forum/dotnet/394039.flat.aspx
Я не очень часто пишу на C#, но насколько я знаю, использование конструкции using предполагает, автоматическое очищение буфферов в конце блока кода.

g00d, 27 Апреля 2011

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

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

+108

1
2
3
4
5
6
7

private void Form_KeyDown(object sender, KeyEventArgs e)
        {
            if (e.KeyCode == Keys.Escape)
            {
                btnCancel_Click(sender, EventArgs.Empty);
            }
        }

Форма - модальный диалог, Cancel кнопка обычная - никаких причин просто выставить у нее DialogResult в Cancel не было. Но ведь так нагляднее!

eval_2009, 05 Января 2011

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

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

+108

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

if (m_Level != value)
                {
                    if (m_Level != null)
                    {
                        Logger.Unsubscribe(m_Level.Value, Logger_LogEvent);
                    }
                    m_Level = value;
                    if (m_Level != null)
                    {
                        Logger.Subscribe(m_Level.Value, Logger_LogEvent);
                    }
                }

Бывает и так. . .

walash, 03 Декабря 2010

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

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

+108

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

TCalc = Class
  Public
    Order : 1..2;
    BMouseDown : Boolean;
    TSS : TPoint;
    mas : array[1..8,1..8] of Byte;
    Constructor Create;
    Procedure MouseDown (Stroka, Stolb : Byte; Var Bool : Boolean);
    Procedure FiguresCount (Var White, Black : Byte);
    Procedure Messages (i : Byte);
    Procedure Proverka (T : TPoint; Stroka, Stolb : Byte;
                        Var Bool, Double : Boolean; Var Dbl : TPoint);
    Procedure ReloadMas (Str, Stolb : Byte; Point : TPoint; Double : Boolean; DblP : TPoint);
    Procedure BoardHelp (Stroka, Stolb : Byte);
    Procedure NoChangeOrder (Stroka, Stolb : Byte; Var NoChangeOrder : Boolean);
    Procedure Proverka_GoTo (Str, Stolb : Byte; Var masBool : ArrBool;
                             Var masCoord : ArrCoord);
    Procedure Scanner (Var kol : Byte; Var ScanMas : ArrCoord);
    Procedure PlaySound (Wave : TLMDWaveComp; ind : Byte);
    Procedure FigureTrue (Str, Stolb : Byte);
    Procedure Damka_Go (Fig : TPoint; Str, Stolb : Byte;
                        Var Bool, Double : Boolean; Var Dbl : TPoint);
    Procedure IsDamka_Fight (X, Y : Byte; Var Bool : Boolean; Var Coord : TPoint);                    
end;

Объявление класса, отвечающего за логику в игре шашки.
Объектный подход :)

ClassVar, 16 Ноября 2010

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

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

+108

1
2
3
4
5
6
7
8

...
while (true)
{
     if(i > 5 || SourceNumber == Math.Floor(SourceNumber)) break;
     ...
     i++;
}
...

Мартин, 15 Ноября 2010

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

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

+108

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

private void InitializeUser()
{
	textBoxFirstname.Text = _user.FirstName;
	textBoxLastname.Text = _user.LastName;
	textBoxPassword.Text = _user.Password;
	textBoxUsername.Text = _user.UserName;
	textBoxMail.Text = _user.Mail;
	comboBoxRoles.Text = _user.UserRole.Id>0 ? _user.UserRole.RoleName : "";
	
	comboBoxRoles.Enabled = true;

	if(User.CurrentUser.isLoanOfficer || User.CurrentUser.isVisitor || User.CurrentUser.isCashier)
	{
		comboBoxRoles.Enabled = false;
	}
	else if(User.CurrentUser.isAdmin)
	{
		if(_user.isAdmin || _user.isSuperAdmin) comboBoxRoles.Enabled = false;
	}
	else if(User.CurrentUser.isSuperAdmin)
	{
		if(_user.isSuperAdmin) comboBoxRoles.Enabled = false;
	}
}

Код взят из реального open source проекта. Реализация многоуровневой проверки прав текущего пользователя :).

_Ru55_, 22 Октября 2010

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

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