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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2341)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   В номинации:

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

   За время:

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

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

   +144

   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

   (function(){
    
   	var ToDoList = {
    
   		targets: {
   			dasha:   [ 'Dasha' , 21 , 'female' ],
   			masha:   [ 'Masha' , 32 , 'female' ],
   			vasya:   [ 'Vasily' , 43 , 'male' ],
   			oksana:  [ 'Oksana' , 13 , 'female' ],
   			volodya: [ 'Vladimir' , 68 , 'male' ],
   			olesya:  [ 'Olesya' , 25 , 'female' ],
   			irina:   [ 'Irina' , 34 , 'female' ],
   			olga:    [ 'Olga' , 17 , 'female' ],
   		},
   		chpokedList: [],
   		chpock: function() {
    
   			for( var i in this.targets ) {
    
   				if( this.testChpokableProps( this.targets[i] ) ) {
   					console.log( this.targets[i][0] )
   						this.chpokedList[i] = this.targets[i][0];
   						console.log( 'chpock was success with ' + this.targets[i][0] );		
   				}			
   			}
   		},
   		testChpokableProps: function( subj ) {
    
   			if ( subj[2] !== 'male') {
    
   				return ( subj[1] >= 18 && subj[1] <= 38 ) ? true : false;
    
   			}
    
   		}
   	};
   	var Test = new function() {
   		this.Go = ToDoList;
   	};
   	Test.Go.chpock();
    
   })();
   
   
   
   	var Test = new function() {
   		this.Go = ToDoList;
   	};
   	Test.Go.chpock();

   Меня прокляли, что сделать?

   Stealth, 08 Августа 2013

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

3. C++ / Говнокод #13398

   +74

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6

   Дорогой участник! Не спеши покидать страницу. Я прошу тебя помочь:
   не поленись, пройдись по всем постам, и поставь каждому участнику по минусу;
   минусуя других участников ты воспитываешь в себе чувство меры и помогаешь обществу снижать ЧСВ. 
   Заранее спасибо, друг мой. Ты будешь вознагражден за это деяние!
   
   Тот, кто покинул страницу, но не выполнил моей просьбы, умрет через 9 дней.

   Дорогой участник! Не спеши покидать страницу. Я прошу тебя помочь:
   не поленись, пройдись по всем постам, и поставь каждому участнику по минусу;
   минусуя других участников ты воспитываешь в себе чувство меры и помогаешь обществу снижать ЧСВ.
   Заранее спасибо, друг мой. Ты будешь вознагражден за это деяние!
   
   Тот, кто покинул страницу, но не выполнил моей просьбы, умрет через 9 дней.

   Stertor, 11 Июля 2013

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

4. C# / Говнокод #12969

   +136

   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

   // Было
   string postCode;
   if (person != null)
   {
     if (HasMedicalRecord(person) && person.Address != null)
     {
       CheckAddress(person.Address);
       if (person.Address.PostCode != null)
         postCode = person.Address.PostCode.ToString();
       else
         postCode = "UNKNOWN";
     }
   }
   // Стало
   string postCode = this.With(x => person)
       .If(x => HasMedicalRecord(x))]
       .With(x => x.Address)
       .Do(x => CheckAddress(x))
       .With(x => x.PostCode)
       .Return(x => x.ToString(), "UNKNOWN");

   "как можно использовать более “монадический” синтаксис в C# для того, чтобы __повысить удобочитаемость__ исходного кода"
   http://www.gotdotnet.ru/blogs/nesteruk/6975/

   Sh1tM4ker, 07 Мая 2013

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

5. C++ / Говнокод #12756

   +15

   1. 1
   2. 2
   3. 3

   String testName;
   //...
   std::swap(testName,  _testName);

   String из thirdparty-библиотеки, а swap везде в нашем коде. По очевидным причинам получаем подение производительности.

   LispGovno, 16 Марта 2013

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

6. C++ / Говнокод #12610

   +80

   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

   /* Попытка обьединить четыре  цикла в один.
   	Первый - Присвоение элементам массива значения.
   	Второй - Поиск минимального значения.
   	Третий - Поиск максимального значения.
   	Четвертый - Вывод содержимого массива.
   */
   #include <iostream>
   #include <cstdlib>
   using namespace std;
   
   int main()
   {
   	int array[100];
   	int flag;
   	int min = 0;
   	int max = 0;
   
   	for (flag=0;flag<100;flag++){					   
   		array[flag]=rand();
   		
   		if ( ! min && ! max){	   //Инициализация переменных, для первой итерации.
   		max = array[0];
   		min = array[0];
   		}
   
   		if (array[flag] >=max) max = array[flag];
   		
   		if (array[flag] <= min) min = array[flag];
   		
   		cout << array[flag] << "\n";
   	}
   	
   	cout << "\nМаксимальное значение: " << max;	
   
   	cout << "\nМинимальное значение: "  << min << '\n';
   	
   	return 0;
   }

   Писал я, пытаясь переделать пример из книжки, в котором для этой же задачи использовалось 4 цикла.
   Просьба обьективно обгадить сей код, дабы мне было над чем подумать.
   
   P.S только учусь.

   shim, 18 Февраля 2013

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

7. Java / Говнокод #12583

   +79

   1. 1
   2. 2
   3. 3

   public class StringToObjectMap extends HashMap<String, Object> {
   
       public StringToObjectMap(Map<? extends String, ? extends Object> map)

   Нет слов выразить мою печаль.

   nafania217518, 13 Февраля 2013

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

8. C++ / Говнокод #12488

   +19

   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

   struct Base { };
   
   struct Class : Base
   {
       int mem{ 0 };  // init non-static data member
   
       Class(int i)
       : Base{}   // init base class
       , mem{i}   // init member
       {
         int j{i};   // init local var
   
         int k = int{0};  // init temporary
   
         f( { 1 } );  // init function arg
   
         int* p = new int{1};  // new init
   
         // int k(int());  // most vexing parse, declares function
         int k{ int{} };   // ok, declares variable
   
         int i[4]{ 1,2,3,4 };   // init array
       }
   
       Class f(int i)
       {
         return { i };   // init return value
       }
   };
   
   Class c{1};   // init global var

   LispGovno, 28 Января 2013

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

9. 1C / Говнокод #12383

   −119

   1. 001
   2. 002
   3. 003
   4. 004
   5. 005
   6. 006
   7. 007
   8. 008
   9. 009
   10. 010
   11. 011
   12. 012
   13. 013
   14. 014
   15. 015
   16. 016
   17. 017
   18. 018
   19. 019
   20. 020
   21. 021
   22. 022
   23. 023
   24. 024
   25. 025
   26. 026
   27. 027
   28. 028
   29. 029
   30. 030
   31. 031
   32. 032
   33. 033
   34. 034
   35. 035
   36. 036
   37. 037
   38. 038
   39. 039
   40. 040
   41. 041
   42. 042
   43. 043
   44. 044
   45. 045
   46. 046
   47. 047
   48. 048
   49. 049
   50. 050
   51. 051
   52. 052
   53. 053
   54. 054
   55. 055
   56. 056
   57. 057
   58. 058
   59. 059
   60. 060
   61. 061
   62. 062
   63. 063
   64. 064
   65. 065
   66. 066
   67. 067
   68. 068
   69. 069
   70. 070
   71. 071
   72. 072
   73. 073
   74. 074
   75. 075
   76. 076
   77. 077
   78. 078
   79. 079
   80. 080
   81. 081
   82. 082
   83. 083
   84. 084
   85. 085
   86. 086
   87. 087
   88. 088
   89. 089
   90. 090
   91. 091
   92. 092
   93. 093
   94. 094
   95. 095
   96. 096
   97. 097
   98. 098
   99. 099
   100. 100

   Функция РазобратьКодыНеисправностей(СтрокаКодов, ТипКода)
   	СпрКодов = СоздатьОбъект("Справочник.КодыГарантийныхНеисправностей");
   	ГруппаТ1 = СпрКодов.НайтиПоКоду("0",0);
   	ГруппаТ2 = СпрКодов.НайтиПоКоду("00",0);
   	ГруппаТ3 = СпрКодов.НайтиПоКоду("000",0);
   	Если СтрокаКодов <> "" Тогда
   		Разд1 = Найти(СтрокаКодов, ";");
   		Разд2 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд1), ";") + Разд1;
   		Разд3 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд2), ";") + Разд2;
   		Разд4 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд3), ";") + Разд3;
   		Разд5 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд4), ";") + Разд4;
   		Разд6 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд5), ";") + Разд5;
   		Разд7 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд6), ";") + Разд6;
   		Разд8 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд7), ";") + Разд7;
   		Разд9 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд8), ";") + Разд8;
   		Разд10 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд9), ";") + Разд9;
   		Разд11 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд10), ";") + Разд10;
   		Если ТипКода = "Т1" Тогда
   			Если Разд1 > 1 Тогда
   				СпрКодов.ИспользоватьРодителя(ГруппаТ1);
   				Если СпрКодов.НайтиПоКоду("0/" + Лев(СтрокаКодов,Разд1-1),2)  = 1 Тогда 
   					ПолучТ1 = СпрКодов.ТекущийЭлемент();
   					Возврат ПолучТ1;
   				КонецЕсли;
   			КонецЕсли;
   		ИначеЕсли ТипКода = "Т2" Тогда
   			Если Разд2-Разд1 > 1 Тогда
   				СпрКодов.ИспользоватьРодителя(ГруппаТ2);
   				Если СпрКодов.НайтиПоКоду("00/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд1), Разд2 - Разд1 - 1), 2) = 1 Тогда
   					ПолучТ2 = СпрКодов.ТекущийЭлемент();
   					Возврат ПолучТ2;
   				КонецЕсли;
   			КонецЕсли;
   		ИначеЕсли ТипКода = "Т3_1" Тогда
   			Если Разд3-Разд2 > 1 Тогда
   				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
   				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд2), Разд3 - Разд2 - 1), 2) = 1 Тогда
   					ПолучТ3_1 = СпрКодов.ТекущийЭлемент();
   					Возврат ПолучТ3_1;
   				КонецЕсли;
   			КонецЕсли;
   		ИначеЕсли ТипКода = "Т3_2" Тогда
   			Если Разд4-Разд3 > 1 Тогда
   				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
   				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд3), Разд4 - Разд3 - 1), 2) = 1 Тогда
   					ПолучТ3_2 = СпрКодов.ТекущийЭлемент();
   					Возврат ПолучТ3_2;
   				КонецЕсли;
   			КонецЕсли;
   		ИначеЕсли ТипКода = "Т3_3" Тогда
   			Если Разд5-Разд4 > 1 Тогда
   				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
   				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд4), Разд5 - Разд4 - 1), 2) = 1 Тогда
   					ПолучТ3_3 = СпрКодов.ТекущийЭлемент();
   					Возврат ПолучТ3_3;
   				КонецЕсли;
   			КонецЕсли;
   		ИначеЕсли ТипКода = "Т3_4" Тогда
   			Если Разд6-Разд5 > 1 Тогда
   				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
   				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд5), Разд6 - Разд5 - 1), 2) = 1 Тогда
   					ПолучТ3_4 = СпрКодов.ТекущийЭлемент();
   					Возврат ПолучТ3_4; 
   				КонецЕсли;
   			КонецЕсли;
   		ИначеЕсли ТипКода = "Т3_5" Тогда
   			Если Разд7-Разд6 > 1 Тогда
   				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
   				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд6), Разд7 - Разд6 - 1), 2) = 1 Тогда
   					ПолучТ3_5 = СпрКодов.ТекущийЭлемент();
   					Возврат ПолучТ3_5;
   				КонецЕсли;
   			КонецЕсли;
   		ИначеЕсли ТипКода = "Т3_6" Тогда
   			Если Разд8-Разд7 > 1 Тогда
   				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
   				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд7), Разд8 - Разд7 - 1), 2) = 1 Тогда
   					ПолучТ3_6 = СпрКодов.ТекущийЭлемент();
   					Возврат ПолучТ3_6; 
   				КонецЕсли;
   			КонецЕсли;
   		ИначеЕсли ТипКода = "Т3_7" Тогда
   			Если Разд9-Разд8 > 1 Тогда
   				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
   				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд8), Разд9 - Разд8 - 1), 2) = 1 Тогда
   					ПолучТ3_7 = СпрКодов.ТекущийЭлемент();
   					Возврат ПолучТ3_7; 
   				КонецЕсли;
   			КонецЕсли;
   		ИначеЕсли ТипКода = "Т3_8" Тогда
   			Если Разд10-Разд9 > 1 Тогда
   				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
   				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд9), Разд10 - Разд9 - 1), 2) = 1 Тогда
   					ПолучТ3_8 = СпрКодов.ТекущийЭлемент();
   					Возврат ПолучТ3_8; 
   				КонецЕсли;
   			КонецЕсли;
   		ИначеЕсли ТипКода = "Т3_9" Тогда
   			Если Разд11-Разд10 > 1 Тогда
   				СпрКодов.ИспользоватьРодителя(ГруппаТ3);

   ещё один высер от автодилера.
   1. начнём сначала, там есть строки:
   ...
   ГруппаТ1 = СпрКодов.НайтиПоКоду("0",0);
   ..., обращаю внимание автора, что функция НайтиПоКоду не возвращает элемент справочника, она лишь говорит есть он или нет, для позиционирования на элементе не хватает строк типа ГруппаТ1 = СпрКодом.ТекущийЭлемент(); и т.д.
   2. дальше. в функцию передаётся параметр СтрокаКодов - это строка с запятыми в качестве разделителей. я конечно понимаю, что в 1С нет встроенной функции для получения массива частей строки типа split в java, но написать свою вообще не сложно, но видимо у автора не хватило мозг..., извините, времени.
   3. теперь смотрим на подобные (их несколько в функции) блоки кода типа:
   СпрКодов.ИспользоватьРодителя(ГруппаТ1);
   Если СпрКодов.НайтиПоКоду("0/" + Лев(СтрокаКодов,Разд1-1),2) = 1 Тогда
   ПолучТ1 = СпрКодов.ТекущийЭлемент();
   Возврат ПолучТ1;
   КонецЕсли;
   строка СпрКодов.ИспользоватьРодителя(ГруппаТ1); не имеет смысла, т.к. ГруппаТ1 содержит значение 1 (см. п.1), а не ссылку на элемент справочника, код работает только благодаря второму параметру - 2, т.к. он заставляет функцию НайтиПоКоду искать по всему справочнику, используя полный код.

   VladC, 02 Января 2013

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

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

    +21

    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

    //Было:
    int foo()//                              ;
    {                                        ;  
        int a                                ;
        float b                              ; //В правой части старые 
        typedef int testing                  ; //полезные комментарии
        std::cout << "Ho-ho-ho"              ;
        a=0                                  ;
        b=1.0f                               ;
        if (a==b)//                          ;
        {                                    ;
            std::cout <<"Yet another caption";
            std::cout <<"Hi-hi-hi"           ;
            bar()                            ;
                                             ;
        }                                    ;
        using namespace std                  ;
        for (int i =0;i<10;i++)//            ;
        {                                    ;
                                             ;
            std::cout <<"Hello, world!"      ;
            std::cout <<"Yet another line"   ;
        }                                    ;
        return 0                             ;
                                             ;
    }                                        ;
    
    //Кто-то что-то решил поменять
    //Стало:
    int foo()//                              ;
    {                                        ;  
        int a                                ;
        float b                              ; //В правой части старые 
        typedef int testing                  ; //полезные комментарии
        std::cout << "Ho-ho-ho"              ;
        a=0                                  ;
        b=1.0f                               ;
        if (a==b)//                          ;
        {                                    ;
            std::cout <<"Yet another caption";
            std::cout <<"Hi-hi-hi"           ;
            bar()                            ;
                                             ;
        }                                    ;
        double c;
        //Слева комментарии требующие
        //проверки. Wtf???Зачем здесь с???    
        using namespace std                  ;
        for (int i =0;i<10;i++)//            ;
        {                                    ;
                                             ;
            std::cout <<"Hello, world!"      ;
            std::cout <<"Yet another line"   ;
        }                                    ;
        return 6666                             ;
                                             ;
    }                                        ;
    
    //Быстро определили по линии из точек c
    //запятой место в котором произошло изменение.
    //Если согласились с ним восстанавливаем линию.

    Бесплатная распределенная говносистема контроля версий. Git и SVN нервно курят в сторонке.

    igumnovf, 20 Ноября 2012

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

11. JavaScript / Говнокод #12002

    +160

    1. 1

    new Array(100).map(function(x) { return Math.random() * 1000; });

    Угадайте результат.

    wvxvw, 25 Октября 2012

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

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

• 1963
• …
• 1666
• 1665
• 1664
• 1663
• 1662
• 1661
• 1660
• 1659
• 1658
• …
• 1