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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2323)
4. JavaScript (2027)
5. C# (1924)
6. Java (1501)
7. Си (1095)
8. Pascal (647)
9. Python (582)
10. 1C (531)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (196)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (28)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. C# / Говнокод #18198

   +142

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12

   // ...
   
   Action updatingLoadedItemsList = null;
   
   foreach (Item loadedItem in loadedItems)
       if (loadedItem.Id == currentItem.Id)
           updatingLoadedItemsList = () => loadedItems.RemoveAt(loadedItems.IndexOf(loadedItem));
   
   if(updatingLoadedItemsList != null)
       updatingLoadedItemsList.Invoke();
   	
   // ...

   pushistayapodmyshka, 19 Мая 2015

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

2. C++ / Говнокод #18197

   +906

   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

   class LoaderLock
   	{
   	public:
   		void lock();
   		void unlock();
   		bool IsLocked() const;
   		LoaderLock() = default;
   
   	private:
   		LoaderLock(LoaderLock&) = delete;
   		LoaderLock(LoaderLock&&) = delete;
   		LoaderLock& operator=(LoaderLock&) = delete;
   		LoaderLock& operator=(LoaderLock&&) = delete;
   		bool _Locked = false;
   		ULONG _cookie = 0;
   		std::unique_ptr<void, std::function<void(void*)>> _ntdll{ LoadLibrary("NTDLL.DLL"), [](void* h){if (h)FreeLibrary((HMODULE)h); } };
   		typedef NTSTATUS(__stdcall*LdrLockLoaderLockFunc)(
   			ULONG Flags,
   			ULONG *State,
   			ULONG *Cookie);
   		LdrLockLoaderLockFunc _LdrLockLoaderLock = !_ntdll ? 0 : (LdrLockLoaderLockFunc)GetProcAddress((HMODULE)_ntdll.get(), "LdrLockLoaderLock");
   		typedef NTSTATUS(__stdcall*LdrUnlockLoaderLockFunc)(
   			ULONG Flags,
   			ULONG Cookie);
   		LdrUnlockLoaderLockFunc _LdrUnlockLoaderLock = !_ntdll ? 0 : (LdrUnlockLoaderLockFunc)GetProcAddress((HMODULE)_ntdll.get(), "LdrUnlockLoaderLock");
   	};

   Уже давно минул 2014 год, а C++11 не перестает радовать нас размазанными по интерфейсу конструкторами и деструкторами.

   LispGovno, 18 Мая 2015

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

3. Си / Говнокод #18196

   +137

   1. 1

   printk(KERN_CRIT "AAA\n");

   Ебу и патчу.

   codemonkey, 18 Мая 2015

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

4. 1C / Говнокод #18195

   −161

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

   //AS 
   Если ЭтотОбъект.Проведен И ((глЗначениеПеременной("глТекущийПользователь").Наименование = "Гуля") или 
   (глЗначениеПеременной("глТекущийПользователь").Наименование = "Марина") или 
   (глЗначениеПеременной("глТекущийПользователь").Наименование = "Ирина")) Тогда 
   ЭтаФорма.ТолькоПросмотр = Истина ; 
   КонецЕсли ; 
   //AS\\

   Не мое, стырил на просторах интернета у пользователя Shipik1982, за что ему спасибо - повеселил)

   portwein, 18 Мая 2015

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

5. PHP / Говнокод #18194

   +163

   1. 1

   $age = floor( (strtotime(date('Ymd')) - strtotime($date)) / 31556926);

   31556926 / 365 / 24 / 60 / 60 ~ 1

   crook, 18 Мая 2015

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

6. JavaScript / Говнокод #18193

   +145

   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

   function send_posit(a){
   					a='<'+a+'>';
   					$.ajax(
   						{url: 'handler.php',
   						data : { posit_this: a},
   						type : "POST", 
   						async: false, 
   						dataType: "html",
   						success: function(data){data_br=data;}
   						})
   				}
   				function reset_all(){$("#Sub_menu1").html('');$("#Sub_menu2").html('');$("#Sub_menu3").html('');$("#Sub_menu4").html('');$("#tovar").html('');}
   				function reset_sub_menu2(){$("#Sub_menu3").html('');$("#Sub_menu4").html('');$("tovar").html('');$("#tovar").html('');}
   				function reset_sub_menu3(){$("#Sub_menu4").html('');$("#tovar").html('');}
   				function reset_tovar(){$("#tovar").html('');}
   				function Katalog(p){
   				switch (p) 
   				{
   				case 0:	send_posit("cat01");$("#menu0").html(data_br);send_posit("cat011");$("#Sub_menu1").html(data_br);send_posit("cat012");$("#Sub_menu2").html(data_br);
   					send_posit("cat013");$("#Sub_menu3").html(data_br);send_posit("cat014");$("#Sub_menu4").html(data_br);break;
   				case 1:	send_posit("cat01");$("#menu0").html(data_br);reset_all();send_posit("cat1");$("#Sub_menu1").html(data_br);break;
   				case 2:send_posit("cat02");$("#menu0").html(data_br);reset_all();send_posit("cat2");$("#Sub_menu1").html(data_br);break;
   				case 3:send_posit("cat03");$("#menu0").html(data_br);reset_all();send_posit("cat3");$("#Sub_menu1").html(data_br);break;
   				case 4:send_posit("cat04");$("#menu0").html(data_br);reset_all();send_posit("cat4");$("#Sub_menu1").html(data_br);break;
   				case 11:send_posit("cat11");reset_sub_menu2();$("#Sub_menu2").html(data_br);break;
   				case 111:send_posit("cat111");reset_sub_menu3();$("#Sub_menu3").html(data_br);break;
   				case 1111:send_posit("tov_kart_1111");$("#tovar").html(data_br);break; 
   				case 1112:send_posit("tov_kart_1112");$("#tovar").html(data_br);break; 
   				case 1113:send_posit("tov_kart_1113");$("#tovar").html(data_br);break; 
   				case 1114:send_posit("tov_kart_1114");$("#tovar").html(data_br);break; 
   				case 112:send_posit("cat112");reset_sub_menu3();$("#Sub_menu3").html(data_br);break;
   				case 113:send_posit("tov_kart_1131");reset_sub_menu2();$("#tovar").html(data_br);break;
   				case 1121:send_posit("tov_kart_1121");$("#tovar").html(data_br);break;
   				case 1122:send_posit("tov_kart_1122");$("#tovar").html(data_br);break;
   				case 1123:send_posit("tov_kart_1123");$("#tovar").html(data_br);break;
   				case 12:send_posit("cat12");reset_sub_menu2();$("#Sub_menu2").html(data_br);break;
   				case 121:send_posit("tov_kart_121");reset_sub_menu2();$("#tovar").html(data_br);break;
   				case 122:send_posit("tov_kart_122");reset_sub_menu2();$("#tovar").html(data_br);break;
   				case 123:send_posit("tov_kart_123");reset_sub_menu2();$("#tovar").html(data_br);break;
   				case 13:send_posit("cat13");reset_sub_menu2();$("#Sub_menu2").html(data_br);break;
   				case 131:send_posit("tov_kart_131");reset_sub_menu2();$("#tovar").html(data_br);break;
   				case 132:send_posit("tov_kart_132");reset_sub_menu2();$("#tovar").html(data_br);break;
   				case 133:send_posit("tov_kart_133");reset_sub_menu2();$("#tovar").html(data_br);break;
   				case 134:send_posit("tov_kart_134");reset_sub_menu2();$("#tovar").html(data_br);break;
   				case 14:send_posit("cat14");reset_sub_menu2();$("#Sub_menu2").html(data_br);break;
   				case 141:send_posit("cat141");reset_sub_menu3();$("#Sub_menu3").html(data_br);break;
   				case 142:send_posit("cat142");reset_sub_menu3();$("#Sub_menu3").html(data_br);break;
   				case 16:send_posit("cat16");reset_sub_menu2();$("#Sub_menu2").html(data_br);break;
   				case 161:send_posit("cat161");reset_sub_menu3();$("#Sub_menu3").html(data_br);break;
   				case 162:send_posit("cat162");reset_sub_menu3();$("#Sub_menu3").html(data_br);break;
   				case 163:send_posit("cat163");reset_sub_menu3();$("#Sub_menu3").html(data_br);break;
   				case 1611:send_posit("tov_kart_1611");$("#tovar").html(data_br);break;
   				case 1612:send_posit("tov_kart_1612");$("#tovar").html(data_br);break;
   				case 1613:send_posit("tov_kart_1613");$("#tovar").html(data_br);break;
   				case 1614:send_posit("tov_kart_1614");$("#tovar").html(data_br);break;
   				case 1615:send_posit("tov_kart_1615");$("#tovar").html(data_br);break;
   				case 1621:send_posit("tov_kart_1621");$("#tovar").html(data_br);break;
   				case 1622:send_posit("tov_kart_1622");$("#tovar").html(data_br);break;
   				case 1623:send_posit("tov_kart_1623");$("#tovar").html(data_br);break;
   				case 1631:send_posit("tov_kart_1631");$("#tovar").html(data_br);break;
   				case 1632:send_posit("tov_kart_1632");$("#tovar").html(data_br);break;
   				case 1633:send_posit("tov_kart_1633");$("#tovar").html(data_br);break;
   				case 1634:send_posit("tov_kart_1634");$("#tovar").html(data_br);break;
   				case 1635:send_posit("tov_kart_1635");$("#tovar").html(data_br);break;
   				case 1636:send_posit("tov_kart_1636");$("#tovar").html(data_br);break;
   				case 1637:send_posit("tov_kart_1637");$("#tovar").html(data_br);break;
   				case 1638:send_posit("tov_kart_1638");$("#tovar").html(data_br);break;
   				case 1639:send_posit("tov_kart_1639");$("#tovar").html(data_br);break;
   				case 15:send_posit("cat15");reset_sub_menu2();$("#Sub_menu2").html(data_br);break;
   				case 151:send_posit("cat151");reset_sub_menu3();$("#Sub_menu3").html(data_br);break;
   				case 152:send_posit("cat152");reset_sub_menu3();$("#Sub_menu3").html(data_br);break;
   				case 1511:send_posit("tov_kart_1511");$("#tovar").html(data_br);break;
   				case 1512:send_posit("tov_kart_1512");$("#tovar").html(data_br);break;
   				case 1513:send_posit("tov_kart_1513");$("#tovar").html(data_br);break;

   о майн гад.
   
   Каталог который можно написать на jquery в 15строк

   onotoliy, 18 Мая 2015

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

7. PHP / Говнокод #18192

   +145

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

   if (count($stores) == 1) {
       foreach ($stores as $store) {
           $storeId = $store->id;
       }
   }

   jbot, 18 Мая 2015

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

8. PHP / Говнокод #18191

   +144

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

   /* Convert IP address to unsigned long int. */
   function ip2ulong($ip) {
       return sprintf("%u", ip2long32($ip));
   }

   https://github.com/pfsense/pfsense/blob/master/etc/inc/util.inc#L414-L417
   
   Продолжаем осторожно идти дальше. Это нормально вообще? Да, я видел всякие смехуёчки с магическим приведением типов в PHP и JS, так что наверняка оно где-то правильно конвертится и все работает несмотря ни на что и вопреки. Может, это даже нормальная практика в мире PHP, но мне страшно.

   superhacker777, 17 Мая 2015

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

9. PHP / Говнокод #18190

   +144

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

   if ($ipaddr == $ip_reverse) {
       return true;
   } else {
       return false;
   }

   https://github.com/pfsense/pfsense/blob/master/etc/inc/util.inc#L625-L629
   
   Давно хотел спросить: это как-нибудь здравым смыслом оправдывается? Часто встречаю похожее, когда чужой код смотрю. Может, это профи делают для… читаемости какой-то, я не знаю. Или я ищу хорошее там, где его нет?

   superhacker777, 17 Мая 2015

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

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

    +144

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

    int spisok::kolvo()
    {
    	return spisok::N;
    }

    KGOH, 17 Мая 2015

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

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

• 2095
• …
• 1548
• 1547
• 1546
• 1545
• 1544
• 1543
• 1542
• 1541
• 1540
• …
• 1