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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5709)
2. C++ (2740)
3. Куча (2390)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (38)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

2. JavaScript / Говнокод #10256

   +160

   1. 1
   2. 2
   3. 3
   4. 4

   <button type='button' id='button-download'>download zipfile</button>
   $("#button-download").live("click", function() {
       $.get("http://localhost/admin/zip/002140.zip"); // doesn't work?
   })

   Ссылка на скачивание файла, современный вариант.

   mozg, 13 Мая 2012

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

3. PHP / Говнокод #10199

   +83

   1. 1

   $v = (((bool) $v) ? true : false);

   somnambulism, 05 Мая 2012

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

4. Objective C / Говнокод #10117

   −282

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26

   NSArray *beautifulValues = [[NSArray alloc] initWithObjects:
                                   [NSNumber numberWithFloat:0.00001], 
                                   [NSNumber numberWithFloat:0.00005], 
                                   [NSNumber numberWithFloat:0.0001],
                                   [NSNumber numberWithFloat:0.005],
                                   [NSNumber numberWithFloat:0.001],
                                   [NSNumber numberWithFloat:0.005],
                                   [NSNumber numberWithFloat:0.01],
                                   [NSNumber numberWithFloat:0.05], 
                                   [NSNumber numberWithFloat:0.1],
                                   [NSNumber numberWithFloat:0.5],
                                   [NSNumber numberWithFloat:1],
                                   [NSNumber numberWithFloat:5],
                                   [NSNumber numberWithFloat:10],
                                   [NSNumber numberWithFloat:50],
                                   [NSNumber numberWithFloat:100],
                                   [NSNumber numberWithFloat:500],
                                   [NSNumber numberWithFloat:1000],
                                   [NSNumber numberWithFloat:5000],
                                   [NSNumber numberWithFloat:10000],
                                   [NSNumber numberWithFloat:50000],
                                   [NSNumber numberWithFloat:100000],
                                   [NSNumber numberWithFloat:500000],
                                   [NSNumber numberWithFloat:1000000],
                                   [NSNumber numberWithFloat:5000000],
                                   [NSNumber numberWithFloat:10000000], nil];

   Нашел в одном из проектов.

   itruf, 30 Апреля 2012

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

5. JavaScript / Говнокод #10078

   +155

   1. 1
   2. 2
   3. 3
   4. 4

   <style type="text/css">
   #1{display:<script type="text/javascript">if(NextStrInfo(1) == 1){document.write("block");}else{document.write("none");}</script>;}
   #2{display:<script type="text/javascript">if(NextStrInfo(2) == 1){document.write("block");}else{document.write("none");}</script>;}
   </style>

   Замечено на форуме.

   Nikitian, 29 Апреля 2012

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

6. C# / Говнокод #9995

   +958

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19

   public int RomeToArab (string str)
   {
   	int[] arabian = new int[13] { 1000, 900, 500, 400, 100, 90, 50, 40, 10, 9, 5, 4, 1 };
   	strint[] rome = new string[13] { "M", "CM", "D", "CD", "C", "XC", "L", "XL", "X", "IX", "V", "IV", "I" };
   	int n = 0;
   	int i = 0;
   	do
   	{
   		if (str.StartsWith(rome[i]))
   		{
   			n = n + arabian[i];
   			str = str.Substring(rome[i].Length, str.Length - rome[i].Length);
   		}
   		if (!str.StartsWith(rome[i])
   			i++;
   	}
   	while (i < 13);
   	return n;
   }

   aleksi, 20 Апреля 2012

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

7. C# / Говнокод #9978

   +110

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28
   29. 29
   30. 30
   31. 31
   32. 32
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62
   63. 63
   64. 64
   65. 65
   66. 66
   67. 67
   68. 68
   69. 69
   70. 70
   71. 71
   72. 72
   73. 73
   74. 74
   75. 75
   76. 76
   77. 77
   78. 78
   79. 79
   80. 80

   this.pictureBox14.BeginInit();
         this.pictureBox15.BeginInit();
         this.pictureBox15.BeginInit();
         this.pictureBox21.BeginInit();
         this.pictureBox21.BeginInit();
         this.pictureBox22.BeginInit();
         this.pictureBox22.BeginInit();
         this.pictureBox23.BeginInit();
         this.pictureBox23.BeginInit();
         this.pictureBox24.BeginInit();
         this.pictureBox24.BeginInit();
         this.pictureBox16.BeginInit();
         this.pictureBox16.BeginInit();
         this.pictureBox17.BeginInit();
         this.pictureBox17.BeginInit();
         this.pictureBox18.BeginInit();
         this.pictureBox18.BeginInit();
         this.pictureBox19.BeginInit();
         this.pictureBox19.BeginInit();
         this.pictureBox20.BeginInit();
         this.pictureBox20.BeginInit();
         this.pictureBox9.BeginInit();
         this.pictureBox9.BeginInit();
         this.pictureBox6.BeginInit();
         this.pictureBox6.BeginInit();
         this.pictureBox5.BeginInit();
         this.pictureBox5.BeginInit();
         this.pictureBox7.BeginInit();
         this.pictureBox7.BeginInit();
         this.pictureBox10.BeginInit();
         this.pictureBox10.BeginInit();
         this.panel1.SuspendLayout();
         this.panel1.SuspendLayout();
         this.SuspendLayout();
         this.SuspendLayout();
         this.panel3.Location = new Point(140, 36);
         this.panel3.Location = new Point(140, 36);
         this.panel3.Name = "panel3";
         this.panel3.Name = "panel3";
         this.panel3.Size = new Size(740, 436);
         this.panel3.Size = new Size(740, 436);
         this.panel3.TabIndex = 4;
         this.panel3.TabIndex = 4;
         this.panel3.MouseDown += new MouseEventHandler(this.panel3_MouseDown);
         this.panel3.MouseDown += new MouseEventHandler(this.panel3_MouseDown);
         this.panel3.MouseUp += new MouseEventHandler(this.panel3_MouseUp);
         this.panel3.MouseUp += new MouseEventHandler(this.panel3_MouseUp);
         this.pictureBox2.BackColor = Color.Gray;
         this.pictureBox2.BackColor = Color.Gray;
         this.pictureBox2.Location = new Point(-11, 0);
         this.pictureBox2.Location = new Point(-11, 0);
         this.pictureBox2.Name = "pictureBox2";
         this.pictureBox2.Name = "pictureBox2";
         this.pictureBox2.Size = new Size(938, 36);
         this.pictureBox2.Size = new Size(938, 36);
         this.pictureBox2.TabIndex = 1;
         this.pictureBox2.TabIndex = 1;
         this.pictureBox2.TabStop = false;
         this.pictureBox2.TabStop = false;
         this.pictureBox3.BackColor = Color.Gray;
         this.pictureBox3.BackColor = Color.Gray;
         this.pictureBox3.Location = new Point(880, 12);
         this.pictureBox3.Location = new Point(880, 12);
         this.pictureBox3.Name = "pictureBox3";
         this.pictureBox3.Name = "pictureBox3";
         this.pictureBox3.Size = new Size(36, 466);
         this.pictureBox3.Size = new Size(36, 466);
         this.pictureBox3.TabIndex = 0;
         this.pictureBox3.TabIndex = 0;
         this.pictureBox3.TabStop = false;
         this.pictureBox3.TabStop = false;
         this.pictureBox1.BackColor = Color.Gray;
         this.pictureBox1.BackColor = Color.Gray;
         this.pictureBox1.Location = new Point(-4, 472);
         this.pictureBox1.Location = new Point(-4, 472);
         this.pictureBox1.Name = "pictureBox1";
         this.pictureBox1.Name = "pictureBox1";
         this.pictureBox1.Size = new Size(938, 36);
         this.pictureBox1.Size = new Size(938, 36);
         this.pictureBox1.TabIndex = 0;

   Полная версия: http://pastebin.com/KwDzLR4s

   P4R4, 18 Апреля 2012

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

8. Куча / Говнокод #9933

   +132

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22

   <asp:Label></asp:Label>
   <br>
   <asp:TextBox></asp:TextBox>
   <br>
   <br>
   <br>
   <br></br>
   <asp:Label></asp:Label>
   <br>
   <asp:TextBox></asp:TextBox>
   <br>
   <br>
   <br>
   <br></br>
   <asp:Button />
   <br></br>
   </br>
   </br>
   </br>
   </br>
   </br>
   </br>

   новый сотрудник "наваял" страницу. кто его надоумил использовать закрывающий тег </br>, да еще и по такому странному принципу - неизвестно...

   _vinka, 12 Апреля 2012

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

9. Perl / Говнокод #9736

   −126

   1. 1

   if (!defined $login || (defined $login && length($login)<3 && length($login>16))) {

   PSIAlt, 22 Марта 2012

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

10. C++ / Говнокод #9726

    +995

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78

    Для Каждого ТекСтрокаТовары Из ЭтотОбъект.Товары Цикл
    		
          //Товары в резерве на складах
          Если ЗначениеЗаполнено(ЭтотОбъект.ЗаказПокупателя) Тогда
    
            ЗапросРезерв = Новый Запрос;
            ЗапросРезерв.УстановитьПараметр("Период", КонецДня(ЭтотОбъект.Дата));
            ЗапросРезерв.УстановитьПараметр("Склад", ЭтотОбъект.Склад);
            ЗапросРезерв.УстановитьПараметр("Номенклатура", ТекСтрокаТовары.Номенклатура);
    	ЗапросРезерв.УстановитьПараметр("Цвет", ТекСтрокаТовары.Цвет);
    	ЗапросРезерв.УстановитьПараметр("Регистратор", ЭтотОбъект.ЗаказПокупателя.Ссылка);
    	ЗапросРезерв.Текст = "ВЫБРАТЬ
    	|	ТоварыВРезервеНаСкладах.Номенклатура,
    	|	isnull(ТоварыВРезервеНаСкладах.Количество, 0) КАК Количество,
    	|	isnull(ТоварыВРезервеНаСкладах.Размер42, 0) КАК Размер42,
    	|	isnull(ТоварыВРезервеНаСкладах.Размер44, 0) КАК Размер44,
    	|	isnull(ТоварыВРезервеНаСкладах.Размер46, 0) КАК Размер46,
    	|	isnull(ТоварыВРезервеНаСкладах.Размер48, 0) КАК Размер48,
    	|	isnull(ТоварыВРезервеНаСкладах.Размер50, 0) КАК Размер50,
    	|	isnull(ТоварыВРезервеНаСкладах.Размер52, 0) КАК Размер52,
    	|	isnull(ТоварыВРезервеНаСкладах.Размер54, 0) КАК Размер54,
    	|	isnull(ТоварыВРезервеНаСкладах.Размер56, 0) КАК Размер56,
    	|	isnull(ТоварыВРезервеНаСкладах.Размер58, 0) КАК Размер58,
    	|	isnull(ТоварыВРезервеНаСкладах.Размер60, 0) КАК Размер60
    	|ИЗ
    	|	РегистрНакопления.ТоварыВРезервеНаСкладах КАК ТоварыВРезервеНаСкладах
    	|ГДЕ
    	|	ТоварыВРезервеНаСкладах.ВидДвижения = ЗНАЧЕНИЕ(ВидДвиженияНакопления.Приход)
    	|	ТоварыВРезервеНаСкладах.Период <= &Период
    	|	И ТоварыВРезервеНаСкладах.Регистратор = &Регистратор
    	|	И ТоварыВРезервеНаСкладах.Склад = &Склад
    	|	И ТоварыВРезервеНаСкладах.Номенклатура = &Номенклатура
    	|	И ТоварыВРезервеНаСкладах.Цвет = &Цвет";
    	ТЗРезерв = ЗапросРезерв.Выполнить().Выгрузить();
    			
    	Если ЗапросРезерв.Выполнить().Пустой() = Истина Тогда
    	Иначе
    		ЗапросСнятие = Новый Запрос;
    		ЗапросСнятие.УстановитьПараметр("Период", КонецДня(ЭтотОбъект.Дата));
    		ЗапросСнятие.УстановитьПараметр("Склад", ЭтотОбъект.Склад);
    		ЗапросСнятие.УстановитьПараметр("Номенклатура", ТекСтрокаТовары.Номенклатура);
    		ЗапросСнятие.УстановитьПараметр("Цвет", ТекСтрокаТовары.Цвет);
    		ЗапросСнятие.УстановитьПараметр("ДокументРезерва", ЭтотОбъект.ЗаказПокупателя.Ссылка);
    		ЗапросСнятие.Текст = "ВЫБРАТЬ
    		|	ТоварыВРезервеНаСкладах.Номенклатура,
    		|	isnull(ТоварыВРезервеНаСкладах.Количество, 0) КАК Количество,
    		|	isnull(ТоварыВРезервеНаСкладах.Размер42, 0) КАК Размер42,
    				.....
    		|	isnull(ТоварыВРезервеНаСкладах.Размер60, 0) КАК Размер60			
    		|ИЗ
    		|	РегистрНакопления.ТоварыВРезервеНаСкладах КАК ТоварыВРезервеНаСкладах
    		|ГДЕ
    		|	ТоварыВРезервеНаСкладах.ВидДвижения = ЗНАЧЕНИЕ(ВидДвиженияНакопления.Расход)
    		|	И ТоварыВРезервеНаСкладах.ДокументРезерва = &ДокументРезерва
    		|	И ТоварыВРезервеНаСкладах.Период = &Период
    		|	И ТоварыВРезервеНаСкладах.Склад = &Склад
    		|	И ТоварыВРезервеНаСкладах.Номенклатура = &Номенклатура
    		|	И ТоварыВРезервеНаСкладах.Цвет = &Цвет";
    		ТЗСнятие = ЗапросСнятие.Выполнить().Выгрузить();
    				
    		//Товары в резерве на складах
    		Движение = Движения.ТоварыВРезервеНаСкладах.Добавить();
    		Движение.ВидДвижения = ВидДвиженияНакопления.Расход;
    		Движение.Период = Дата;
    		Движение.Склад = ЭтотОбъект.Склад; 
    		Движение.Номенклатура = ТекСтрокаТовары.Номенклатура;
    		Движение.ДокументРезерва = ЭтотОбъект.ЗаказПокупателя.Ссылка;
    		Движение.Цвет = ТекСтрокаТовары.Цвет;
    		//42
    		Если ТЗРезерв.Итог("Размер42") - ТЗСнятие.Итог("Размер42") > 0 Тогда
    			Если ТЗРезерв.Итог("Размер42") - ТЗСнятие.Итог("Размер42") > ТекСтрокаТовары.Количество Тогда
    				Движение.Размер42 = ТекСтрокаТовары.Размер42;
    			Иначе
    				Движение.Размер42 = ТЗРезерв.Итог("Размер42") - ТЗСнятие.Итог("Размер42");
    			КонецЕсли;
    		КонецЕсли;      
    		...
                    //И так до 60го размера

    Нашел у себя эту простыню. Чтобы вычислить остатки по резерву - делается запрос по движения со знаком плюс, потом запрос с движениями по минус.
    Далее все это дело выгружается в две таблицы значений и высчитывается разность колонок. Вместо простого запроса по регистру остатков) Ну и много чего интересного)

    ErrorEd88, 21 Марта 2012

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

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

    +108

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 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)

• ← влево
• вправо →

• 1975
• …
• 1234
• 1233
• 1232
• 1231
• 1230
• 1229
• 1228
• 1227
• 1226
• …
• 1