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

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

В номинации:

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

За время:

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

+138

1

http://www.microsoft.com/ru-ru/smb/campaigns/get-modern/default.aspx

Таймер идёт назад.
Да они поехали!

laMer007, 08 Апреля 2014

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

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

+138

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

private function toDBFields($fields){
			$db_fields = array();
			if ($fields){
				foreach($fields as $field){
					switch ($field){

                        case 'p_bits':
                            if (!array_search('b1',$db_fields))	$db_fields[] = 'b1';
                            break;
                        case 'n2':
                            if (!array_search('b1',$db_fields))	$db_fields[] = 'b1';
                            break;
                        case 'n1':
                            if (!array_search('b1',$db_fields))	$db_fields[] = 'b1';
                            break;
						case 'cntdf':
							if (!array_search('cntdf',$db_fields))	$db_fields[] = 'cntdf';
							break;
						case 'first_pckg':
						
							break;
						case 'az':
							if (!array_search('az',$db_fields))		$db_fields[] = 'az';
							break;
						case 'akb_v':
							//if (!array_search('b2',$db_fields))		$db_fields[] = 'b2';
							if (!array_search('b3',$db_fields))		$db_fields[] = 'b3';
							break;
						case 'p1code':
                        case 'ignition_ap1':
							if (!array_search('b4',$db_fields))		$db_fields[] = 'b4';
							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
							break;
						case 'fuel_p2':
                        case 'fuel_p2_debug':
                        case 'p2code':
							if (!array_search('b5',$db_fields))		$db_fields[] = 'b5';
							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
							break;
                        case 'fuel_p3':
                        case 'fuel_p3_debug':
						case 'p3code':
                        case 'p3code_debug':
							if (!array_search('b7',$db_fields))		$db_fields[] = 'b7';
							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
							break;
                        case 'fuel_p3p4_2drt':
                            if (!array_search('b8',$db_fields))		$db_fields[] = 'b8';
							if (!array_search('b7',$db_fields))		$db_fields[] = 'b7';
							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
							break;
                        case 'fuel_p2p3':
                            if (!array_search('b5',$db_fields))		$db_fields[] = 'b5';
							if (!array_search('b7',$db_fields))		$db_fields[] = 'b7';
							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
							break;
                        case 'fuel_p4':
                        case 'p4code_debug':
						case 'p4code':
							if (!array_search('b8',$db_fields))		$db_fields[] = 'b8';
							if (!array_search('b6',$db_fields))		$db_fields[] = 'b6';
							break;
					}
				}
			}

			return $db_fields;

Вот такой кейс который используется в системе для перевода типа переменной к названию поле в таблице ....

Deimm, 13 Февраля 2014

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

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

+138

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

/// generic Singleton<T> (потокобезопасный с использованием generic-класса и с отложенной инициализацией)
 
/// <typeparam name="T">Singleton class</typeparam>
public class Singleton<T> where T : class
{
  /// Защищённый конструктор необходим для того, чтобы предотвратить создание экземпляра класса Singleton. 
  /// Он будет вызван из закрытого конструктора наследственного класса.
  protected Singleton() { }
 
  /// Фабрика используется для отложенной инициализации экземпляра класса
  private sealed class SingletonCreator<S> where S : class
  {
    //Используется Reflection для создания экземпляра класса без публичного конструктора
    private static readonly S instance = (S) typeof(S).GetConstructor(
                BindingFlags.Instance | BindingFlags.NonPublic,
                null,
                new Type[0],
                new ParameterModifier[0]).Invoke(null);
 
    public static S CreatorInstance
    {
      get { return instance; }
    }
  }
 
  public static T Instance
  {
    get { return SingletonCreator<T>.CreatorInstance; }
  }
 
}
 
/// Использование Singleton
public class TestClass : Singleton<TestClass>
{
    /// Вызовет защищенный конструктор класса Singleton
    private TestClass() { }
 
    public string TestProc()
    {
        return "Hello World";
    }
}

Код из Википедии. Создание синглтона на шарпике с излишествами и извращениями

В самом деле, кому нужна инкапсуляция, нам же нужен синглтон

P.S может я чего не понимаю.

kegdan, 26 Декабря 2013

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

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

+138

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

const bool ShowOkIfAllIsCool = true;

book OK()
{
    return ShowOkIfAllIsCool;
}

bool GetFileExists(string filePath)
{
     return File.Exists(filePath);
}

int GetLengthFileName(string filePath)
{
    return filePath.Lenght;
}

int SetMaxValue(int maxValue)
{
    _OPENWINDOWSAFTERINITIALIZATION = maxValue;
}

Программированием занимаюсь много лет, а так и не могу понять, для чего создавать подобные методы в 1 строку и константы заглавными буквами огромной длины? Их люди пишут или это автоматически созданный код какой-то прогой? В чужих программах такое часто можно встретить.

sbs, 25 Декабря 2013

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

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

+138

1
2
3
4
5

Control authorizationPanel= iddbd
      .Controls.Cast<Control>().Where(x => x.Name == "panelMaxim").FirstOrDefault()
      .Controls.Cast<Control>().Where(x => x.Name == "panelInternetB").FirstOrDefault()
      .Controls.Cast<Control>().Where(x => x.Name == "panelPassiveIB").FirstOrDefault()
      .Controls.Cast<Control>().Where(x => x.Name == "panelPhone").FirstOrDefault();

не, ну а как его еще достать? У нас это квест такой на работе - "Найди контрол" называется

taburetka, 18 Декабря 2013

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

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

+138

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

// прога у нас многопоточная, данный код выполняется в потоке. Создание и настройка Idhttp прописана в конструкторе потока.
while not terminated do
    begin
       synchronize(ReadProxy);
        if ThreadExit then break;
        try
           memorystream.Position:=0;
          try
            idHTTP.Get(linknew,memorystream);
            idhttp.disconnect;
          except
            idhttp.Disconnect;
          end;
          finally
            temp:=memorystreamtostring(memorystream); // если этого не сделать, память будет утекать.
            memorystream.clear;
          end;

Я нашел способ предотвратить утечки памяти в Indy, при Get-запросе -
многие грешат на Indy, мол, нестабильная она в потоках, провоцирует утечки.
Оказалось, что дело-то в tmemorystream, я его маму бля ебал!!!

Stertor, 19 Ноября 2013

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

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

+138

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

if ("A" == Key.ToUpper().Substring(startIndex, 1))
        num = 11L;
      else if ("B" == Key.ToUpper().Substring(startIndex, 1))
        num = 12L;
      else if ("C" == Key.ToUpper().Substring(startIndex, 1))
        num = 13L;
      else if ("D" == Key.ToUpper().Substring(startIndex, 1))
        num = 14L;
      else if ("E" == Key.ToUpper().Substring(startIndex, 1))
        num = 15L;
      else if ("F" == Key.ToUpper().Substring(startIndex, 1))
        num = 16L;
      else if ("0" == Key.ToUpper().Substring(startIndex, 1))
        num = 0L;
      else if ("1" == Key.ToUpper().Substring(startIndex, 1))
        num = 1L;
      else if ("2" == Key.ToUpper().Substring(startIndex, 1))
        num = 2L;
      else if ("3" == Key.ToUpper().Substring(startIndex, 1))
        num = 3L;
      else if ("4" == Key.ToUpper().Substring(startIndex, 1))
        num = 4L;
      else if ("5" == Key.ToUpper().Substring(startIndex, 1))
        num = 5L;
      else if ("6" == Key.ToUpper().Substring(startIndex, 1))
        num = 6L;
      else if ("7" == Key.ToUpper().Substring(startIndex, 1))
        num = 7L;
      else if ("8" == Key.ToUpper().Substring(startIndex, 1))
        num = 8L;
      else if ("9" == Key.ToUpper().Substring(startIndex, 1))
      {
        num = 9L;
      }

Программист, писавший ЭТО считал себя очень большим талантом и был даже тех.диром, пока не уволили....))

CraxyFright, 07 Октября 2013

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

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

+138

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
68
69
70
71

persjempre <[email protected]>,Akka Knebekaize <[email protected]>
Значит слушай внимательно, тупица, что я тебе скажу.
Никакой ты не крутой пацан, ты сосунок, который нарывается на ссору с
органами. В предыдущем письме ты четко сдал себя со всеми потрохами,
указав двух свидетелей твоего беспредела. 30 лет работы в прокуратуре
меня кое-чему научили. Я не поленился найти этого паренька,
администратора того форума программистов и пообщаться с ним. Виталий
оказался примилейшим человеком, и показал мне вашу с ним перепалку.

Да, я много гопоты повидал, и ты пожалуй один из немногих уникалов,
который повеселил и меня и наш отдел по защите персональной
информации. С тебя ржали все мои бойцы, сказали что давно такого
детского сада не видели. Это же надо - упорно месить грязь под разными
именами с разных каналов! Вот молодец. Мало того, что засветился как
свеча в темной комнате, мне даже не потребуется делать усиленных
телодвижений для вычисления твоего адреса места проживания, так еще и
упорно плодишь кучу доказательств своей неправоты. Слышал такую
пословицу: "Язык до Киева доведет"? Не? Поясняю - твоя клевета и
оскорбления обязательно будут доведены до криминала, если будешь
продолжать с таким же упорством кричать в публичных местах интернета,
демонстрируя свою храбрость и безнаказанность. Как человек с
многолетним опытом, наблюдавший не один десяток таких дел скажу -
найдет твоя коса на каменюку, обязательно.

Значит слушай теперь, малыш, внимательно. Зовут меня Вениамин
Валентинович, и работаю я прокурором. Прикинь, прокуроры тоже умеют
пользоваться интернетом, и соблюдать нетикет (хотя вряд ли ты о нем
слышал, твои понятия морали судя по твоему языку на нуле). Я
предупредил Виталия, что если он или кто-то из них пострадает от
твоего грязного рта, пусть они мне напишут, предоставив твои слова. Я
постараюсь помочь честным людям наказать обидчика. Я дам ход
"плавному" делу. Поверь мне выписать ордер  мне ничего не стоит, и
любая организация, занимающаяся связью с радостью мне поможет,
предоставив журнал связи абонентов. А учитывая твой многоликий янус,
следы которого остались как оказалось не только на том сайте, пробить
по совпадениям твой адрес прописки легче простого.

Если ты еще не в курсе, или попросту в школе не учился поясняю:
"Преступления в сфере информационных технологий включают как
распространение вредоносных вирусов, взлом паролей, кражу номеров
кредитных карточек и других банковских реквизитов (фишинг), так и
распространение противоправной информации (клеветы, материалов
порнографического характера, материалов, возбуждающих межнациональную
и межрелигиозную вражду и т.п.) через Интернет, коммунальные
объекты[1]."
К твоему вниманию перечитывать до полного просветления слова "Клевета"
и "межнациональная вражда". Сюда же относится и оскорбления по половым
признакам, а они в твоих постах есть. Будь уверен при необходимости
особое внимание суд уделит именно им, как показаниям зачинщика
преступления, т.е. основной первопричины.
Учитывая твои угрозы мы так уж и быть сошлемся на статью 273 УК.  А
так же ст. 130. И все они будут направленны прежде всего на тебя как
на инициатора, и главного организатора беспредела.

Советую тебе сразу искать крутого адвоката, если нет желания забиться
в угол и перестать терроризировать интернет. Я обязательно попрошу еще
своих бойцов проверить тебя на сайте xaker.ru, где вполне можно
прихватить за взломы.

В общем мой тебе добрый совет, не ломай свою жизнь, я уверен что ты
еще молодой, и схлопотать срок, пусть даже условный не желаешь. Даже
административные наказания не пройдут тебе даром, если ты
несовершеннолетний то для твоих родителей.
Кончай баловаться с огнем! Я тебя предупредил - еще раз выскажешься
неадекватно в мою сторону, или Виталий (или любой другой) пожалуется
на тебя, будешь иметь дело с правоохранительными органами, это я тебе
обещаю.


<[email protected]>
Ваш унылый троллинг все больше напоминает попытки запихнуть штепсель от холодильника в USB разъем. Я на такое не ведусь.

Вот какой он, суровый дядя прокурор с Программерсфорума)
Реально же эта писанина принадлежит одной пизде-мордераторше с programmersforum.ru (Ice Valery)

Stertor, 29 Августа 2013

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

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

+138

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

Object.valuesNP = function(obj){
    var result = [];
    for (var i in obj)
        if (obj.hasOwnProperty(i))
            result.push(obj[i]);
    return result;
};

function findCSS(name){
    var css = document.styleSheets;
    var flag = false;
    var result = null;
    Object.valuesNP(css).each(function(content,index){
        if (!flag && content instanceof Object)
            Object.valuesNP(content.cssRules).each(function(content,index){
                if (content instanceof Object && content.selectorText == name && !flag ){
                    flag = true;
                    result = content;
                }
            })
        });
    return result;
}

Надо было поменять параметры некоторого класса(стиль). Ну и за пару минут было накидано вот енто.
Самое смешное, что через неделю это уже не потребовалось.
Вызывать так:
var buttonClass = findCSS('.buttonClass') || console.log('CSS .buttonClass not found');

Dart_Sergius, 16 Августа 2013

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

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