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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2362)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1116)
8. Pascal (648)
9. Python (584)
10. 1C (538)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. PHP / Говнокод #10300

   +79

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

   if ($type['my_event']) {
       $select->where('1=1)))');
   } else {
       $select->where('1=1))');				
   }

   Индусы..

   anycolor, 17 Мая 2012

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

2. PHP / Говнокод #10299

   +62

   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

   $str = CHEREZ_GOPU_NOCACHE ? false : $mem->get($cachename);
    if ($str === false)
    {
   
     /*
       
   мно гомно гомно го кода 
    */
      $mem->set('totalonline', $totalonline, false, MEMCACHE_CACHETIME * 3);
      if (!CHEREZ_GOPU_NOCACHE)
          $mem->set($cachename, $str, false, MEMCACHE_CACHETIME);
   
   }

   и еще 100500 строк в данном файле...

   Sulik78, 17 Мая 2012

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

3. C# / Говнокод #10298

   +96

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

   OdbcCommand dist_cmd = ifx_conn.CreateCommand();
                   dist_cmd.CommandText = string.Format(@"select  -1 as districtid, '-=ВСІ=-' as districtname from systables where  tabname = 'systables'                               union all 
                                           select id as districtid , cpz_name as districtname from  post_cpz where directorate_id = {0} and id != 0 ORDER BY 2", (String.IsNullOrEmpty(DDL_obl.SelectedValue))?"-1":DDL_obl.SelectedValue);
                   reader = dist_cmd.ExecuteReader();

   в процессе рефакторинга, убирая NHibernate код был заменен на это

   bercerker, 17 Мая 2012

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

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

   −102

   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
   81. 81
   82. 82
   83. 83
   84. 84
   85. 85
   86. 86
   87. 87

   Процедура СортировкаСписка(Список) 
   	
   	Перем лСредний;  
   	
   	лЛевый= СоздатьОбъект("СписокЗначений");
   	лПравый = СоздатьОбъект("СписокЗначений");
   	
   	Если Список.РазмерСписка()>1 Тогда  
   		
   		лРазмерСписка = Список.РазмерСписка();
   		лПозицияСреднего = Цел(лРазмерСписка/2);
   		лСреднее = Список.ПолучитьЗначение(лПозицияСреднего);
   		
   		Для й = 1 По лРазмерСписка Цикл
   			
   			Если й<>лПозицияСреднего Тогда
   				
   				лПроверяемоеЗначение =  Список.ПолучитьЗначение(й);
   				
   				Если лПроверяемоеЗначение<лСреднее Тогда
   					
   					лЛевый.ДобавитьЗначение(лПроверяемоеЗначение);
   					
   				ИначеЕсли лПроверяемоеЗначение>лСреднее Тогда
   					
   					лПравый.ДобавитьЗначение(лПроверяемоеЗначение);
   					
   				КонецЕсли;
   				
   			КонецЕсли;
   			
   		КонецЦикла;
   		
   		СортировкаСписка(лЛевый);
   		СортировкаСписка(лПравый);
   		
   		Список.УдалитьВсе();
   		
   		Для й = 1 По лЛевый.РазмерСписка() Цикл
   			Список.ДобавитьЗначение(лЛевый.ПолучитьЗначение(й));
   		КонецЦикла; 
   		
   		Список.ДобавитьЗначение(лСреднее); 
   		
   		Для й = 1 По лПравый.РазмерСписка() Цикл
   			Список.ДобавитьЗначение(лПравый.ПолучитьЗначение(й));
   		КонецЦикла;
   		
   	КонецЕсли;
   	
   	
   КонецПроцедуры
   
   Процедура Выполнить()
   	
   	ОтсортированныйСписок = СоздатьОбъект("СписокЗначений");
   	
   	СписокЗначений = СоздатьОбъект("СписокЗначений");   
   	
   	СписокЗначений.ДобавитьЗначение(2);
   	СписокЗначений.ДобавитьЗначение(7);
   	СписокЗначений.ДобавитьЗначение(15);
   	СписокЗначений.ДобавитьЗначение(16);
   	СписокЗначений.ДобавитьЗначение(20);
   	СписокЗначений.ДобавитьЗначение(21);
   	СписокЗначений.ДобавитьЗначение(3);
   	СписокЗначений.ДобавитьЗначение(5);
   	СписокЗначений.ДобавитьЗначение(4);
   	СписокЗначений.ДобавитьЗначение(14);
   	СписокЗначений.ДобавитьЗначение(12);
   	СписокЗначений.ДобавитьЗначение(13);
   	СписокЗначений.ДобавитьЗначение(11);
   	СписокЗначений.ДобавитьЗначение(19);
   	СписокЗначений.ДобавитьЗначение(1);
   	СписокЗначений.ДобавитьЗначение(8);
   	СписокЗначений.ДобавитьЗначение(6);
   	СписокЗначений.ДобавитьЗначение(17);
   	СписокЗначений.ДобавитьЗначение(9);
   	СписокЗначений.ДобавитьЗначение(10);
   	СписокЗначений.ДобавитьЗначение(18);
   	
   	СортировкаСписка(СписокЗначений);
   	
   	ВыбранноеЗначение = 0;
   	СписокЗначений.ВыбратьЗначение(ВыбранноеЗначение);
   	
   КонецПроцедуры

   informatikmarazmatik, 16 Мая 2012

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

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

   +61

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

   class admin_payment {
       // ...
   }
   
   $m = 'admin_payment';
   $$m = new $m();
   $m->get();

   отборно.

   shmaltorhbooks, 16 Мая 2012

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

6. 1C / Говнокод #10295

   −119

   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

   Функция ЧекМайл(Майл) 
       
   	Перем Чек;
   	
   	Если Найти(Майл,"@mail.ru")<>0 Тогда
   		
   		Чек = 0; 
   		
   	Иначе	
   		
   		Чек = 1;
   		
   	КонецЕсли;
   	
   	Возврат Чек;
   	
   КонецФункции
   
   ...
   	  Если ЧекМайл("[email protected]") = 0 Тогда
   	  	 Предупреждение("Ой! Что-то пошло не так!");
   	  	 Предупреждение("Не беспокойтесь, это наша ошибка. Мы уже знаем о ней и примем меры к ее исправлению!");
   	  	 СтатусВозврата(0);
   	  	 Возврат;
   	  КонецЕсли;

   могли и предупредить ...

   informatikmarazmatik, 16 Мая 2012

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

7. Си / Говнокод #10294

   +138

   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

   typedef int32_t wl_fixed_t;
   
   //было:
   static inline double wl_fixed_to_double(wl_fixed_t f)
   {
     return (double) f / 256.0;
   }
   
   //стало:
   static inline double wl_fixed_to_double(wl_fixed_t f)
   {
     union {
       double d;
       int64_t i;
     } u;
   
     u.i = ((1023LL + 44LL) << 52) + (1LL << 51) + f;
   
     return u.d - (3LL << 43);
   }

   Заоптимизировали...

   rat4, 16 Мая 2012

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

8. JavaScript / Говнокод #10293

   +176

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

   function getAnchor(link){
   	var url=( (!link) ? window.location : link).toString();
   	var idx = url.indexOf("#");
   	return (idx != -1 ? url.substring(idx+1) : '');	
   }
   function setAnchor(anchor){
   	var url=(window.location).toString();
   	var idx = url.indexOf("#");
   	return ( window.location= 
   		(idx != -1 ? url.substring(0,idx+1) : url+'#')+anchor
   	);
   }

   3.14159265, 16 Мая 2012

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

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

   −115

   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

   Тз = СоздатьОбъект("ТаблицаЗначений");
   	Тз.НоваяКолонка("Число");
   	Для й=1 По 100 Цикл
   		Тз.НоваяСтрока();
   		Тз.Число = й;
   	КонецЦикла;
   	КоличествоСтрокТз = Тз.КоличествоСтрок();
   	Для й=-КоличествоСтрокТз По -1 Цикл
   		ЗначениеИзТз = Тз.ПолучитьЗначение(-й,"Число");
   		ПроверяемоеЗначение = ЗначениеИзТз/2-Цел(ЗначениеИзТз/2);
   		Если ПроверяемоеЗначение <> 0 Тогда
   			Тз.УдалитьСтроку(-й);
   		КонецЕсли;
   	КонецЦикла;
   	Тз.ВыбратьСтроку();

   Четные числа до 100

   informatikmarazmatik, 16 Мая 2012

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

10. C# / Говнокод #10291

    +86

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

    protected void ListViewAlerts_ItemCommand(object sender, ListViewCommandEventArgs e)
            {
                int commandArgument = e.Item.DataItemIndex;
                var listViewDataItem = (HiddenField)ListViewAlertRegistry.Items[commandArgument].Controls[3];
                int id = Convert.ToInt32(listViewDataItem.Value);
    .....
            }

    Из рабочего проекта. Новый способ узнать id item'а listview. Видимо свойство DataKeyNames кодеру незнакомо - id берет из hiddenfield, причем выбирает этот hiddenfield не по его id, а по позиции контрола по счету в строке. Добавляем еще один элемент в listview, и соответственно метод валится.

    _vinka, 16 Мая 2012

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

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

• 2105
• …
• 983
• 982
• 981
• 980
• 979
• 978
• 977
• 976
• 975
• …
• 1