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

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

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

1. PHP (5711)
2. C++ (2742)
3. Куча (2395)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1120)
8. Pascal (648)
9. Python (591)
10. 1C (540)
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 (44)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   +159

   1. 1
   2. 2

   define('ASC', 'ASC');
   define('DESC', 'DESC');

   *facepalm*

   codename, 24 Февраля 2012

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

2. SQL / Говнокод #9536

   −123

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

   SELECT keywords_ru AS keywords, title_ru AS title, description_ru AS description
   FROM alo_seo
   WHERE
   	( 1 AND city_id="348" AND cat_id=(SELECT id FROM alo_categories WHERE 1 AND path="peryferija" LIMIT 1) ) OR
   	( 1 AND city_id="348" AND cat_id=(SELECT id FROM alo_categories WHERE 1 AND path="komp-yuterna-tehnika" LIMIT 1) ) OR
   	( 1 AND cat_id=(SELECT id FROM alo_categories WHERE 1 AND path="peryferija" LIMIT 1) ) OR
   	( 1 AND cat_id=(SELECT id FROM alo_categories WHERE 1 AND path="komp-yuterna-tehnika" LIMIT 1) )
   LIMIT 1;

   andnix, 24 Февраля 2012

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

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

   +158

   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

   $PROPERTY_VALUE = array(
                   "payment_1_term" => $_POST["payment_1_term"], "payment_1_sum" => $_POST["payment_1_sum"],
                   "payment_2_term" => $_POST["payment_2_term"], "payment_2_sum" => $_POST["payment_2_sum"],
                   "payment_3_term" => $_POST["payment_3_term"], "payment_3_sum" => $_POST["payment_3_sum"],
                   "payment_4_term" => $_POST["payment_4_term"], "payment_4_sum" => $_POST["payment_4_sum"],
                   "payment_5_term" => $_POST["payment_5_term"], "payment_5_sum" => $_POST["payment_5_sum"],
                   "payment_6_term" => $_POST["payment_6_term"], "payment_6_sum" => $_POST["payment_6_sum"],
                   "payment_7_term" => $_POST["payment_7_term"], "payment_7_sum" => $_POST["payment_7_sum"],
                   "payment_8_term" => $_POST["payment_8_term"], "payment_8_sum" => $_POST["payment_8_sum"],
                   "payment_9_term" => $_POST["payment_9_term"], "payment_9_sum" => $_POST["payment_9_sum"],
                   "payment_10_term" => $_POST["payment_10_term"], "payment_10_sum" => $_POST["payment_10_sum"],
                   "payment_11_term" => $_POST["payment_11_term"], "payment_11_sum" => $_POST["payment_11_sum"],
                   "payment_12_term" => $_POST["payment_12_term"], "payment_12_sum" => $_POST["payment_12_sum"],
                   "payment_13_term" => $_POST["payment_13_term"], "payment_13_sum" => $_POST["payment_13_sum"],
                   "payment_14_term" => $_POST["payment_14_term"], "payment_14_sum" => $_POST["payment_14_sum"],
                   "payment_15_term" => $_POST["payment_15_term"], "payment_15_sum" => $_POST["payment_15_sum"],
                   "cost" => $_POST["cost"]
           );

   шлакоблок кода

   SuperChel, 24 Февраля 2012

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

4. PHP / Говнокод #9534

   +159

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

   ...
   </tr>
   </tbody>
   <?  if($color == "#ffffff") 
             $color = "#ffffff"; 
         else 
             $color = "#ffffff";  
   ?>
   </table>

   Гений блин... Правда так и не понятно что хотелось этим сказать...

   vahminator, 24 Февраля 2012

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

5. 1C / Говнокод #9533

   −151

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

   Процедура ВводНаОсновании(ДокОснование) 
       // .....
       ДатаДок=РабочаяДата();
      // .....
   КонецПроцедуры

   Типовая торговля для Украины.
   Зачем то явно указывается рабочая дата. Причём ровно один раз.

   Xbrut, 24 Февраля 2012

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

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

   −146

   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

   Процедура ОбработкаНоменклатуры()
       
   НомерЛистаТипы = 1;
   Попытка ExcelТипы = СоздатьОбъект("Excel.Application");
    ФайлДанныхТипы = ExcelТипы.Workbooks.Open(НазваниеФайла);        
   Исключение
   КонецПопытки;
   //Лист = ФайлДанныхТипы.WorkSheets(НазваниеФайла);
    
   ЗагрузкаТипы = ФайлДанныхТипы.Worksheets(НомерЛистаТипы).UsedRange;    
   //КоличествоСтрокЭкселеТипы = Лист.Rows.Count;    
    
   КоличествоСтрокЭкселеТипы = ФайлДанныхТипы.Worksheets(НомерЛистаТипы).UsedRange.Rows.Count;
   ExcelТипы.Sheets(НомерЛистаТипы).Select();    
   
   Спр2 = СоздатьОбъект("Справочник.Номенклатура");
   СпрВидНом = СоздатьОбъект("Справочник.ВидыНоменклатуры");
   СпрНДС = СоздатьОбъект("Справочник.СтавкиНДС");
   СпрНП = СоздатьОбъект("Справочник.СтавкиНП");
   СпрЕд = СоздатьОбъект("Справочник.ЕдиницыИзмерений");
   
   Для СтрТипы = НачСтр+1 По КонСтр Цикл      
       
   Спр = СоздатьОбъект("Справочник.Номенклатура");
   Если Спр.НайтиПоНаименованию(Строка(СокрЛП(ExcelТипы.Cells(СтрТипы,1).Value))) =  1 Тогда
       Сообщить("Номенклатура найдена");
   Если ВидыЦен = 1 Тогда
       Спр.Себестоимость = Строка(СокрЛП(ExcelТипы.Cells(СтрТипы,4).Value));
   ИначеЕсли ВидыЦен = 2 Тогда
       Спр.Цена = Строка(СокрЛП(ExcelТипы.Cells(СтрТипы,4).Value));
   КонецЕсли;
   Спр.Записать(); 
   Иначе
   Спр.Новый(); 
   Спр.ИспользоватьДату('01.11.2011');
   Если Спр2.НайтиПоКоду("Сй111")=1 Тогда
   Спр.Родитель =Спр2.ТекущийЭлемент(); 
   Конецесли;
   Спр.Наименование = Строка(СокрЛП(ExcelТипы.Cells(СтрТипы,1).Value));  
   Сообщить("Создается номенклатура" + Спр.Наименование);
   Спр.ТипТовара = Перечисление.ТипыТоваров.Собственный; 
   Если СпрВидНом.НайтиПоНаименованию("Товар")=1 Тогда
   Спр.ВидНоменклатуры = СпрВидНом.ТекущийЭлемент();
   КонецЕсли;
   Спр.ПолнНаименование = Спр.Наименование;
   Если ВидыЦен = 1 Тогда
       Спр.Себестоимость = Строка(СокрЛП(ExcelТипы.Cells(СтрТипы,4).Value));
   ИначеЕсли ВидыЦен = 2 Тогда
       Спр.Цена = Строка(СокрЛП(ExcelТипы.Cells(СтрТипы,4).Value));
   КонецЕсли;
   Спр.СтранаПроисхожд = 1;   
   Если СпрНДС.НайтиПоКоду("3")=1 Тогда
   Спр.СтавкаНДС = СпрНДС.ТекущийЭлемент(); 
   КонецЕсли;
   Если СпрНП.НайтиПоКоду("1")=1 тогда
   Спр.СтавкаНП = СпрНП.ТекущийЭлемент(); 
   КонецЕсли;        
   Если СпрЕд.НайтиПоНаименованию("шт")=1 Тогда
   Спр.ЕдиницаИзмерения = СпрЕд.ТекущийЭлемент(); 
   КонецЕсли;
   Спр.Записать();
   КонецЕсли;     
   
   КонецЦикла;
   
   КонецПроцедуры

   В бух 7.7 нужно было загрузить определенный данные из экселя. С 7.7 не работаю - начал сочинять. В начале писал по аналогии с 8.х:
   Спр.СтавкаНДС = СпрНДС.НайтиПоКоду("3") и т.д.
   Потом на инфостарте и мисте подсказали как правильно сделать и посоветовали создать здесь эту запись)

   ErrorEd88, 24 Февраля 2012

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

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

   +155

   1. 1
   2. 2
   3. 3

   $pathToAction = str_replace("\\", "/", $pathToAction);
   while (substr($pathToAction, strlen($pathToAction) - 1, 1) == "/")
       $pathToAction = substr($pathToAction, 0, strlen($pathToAction) - 1);

   nikita2206, 24 Февраля 2012

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

8. Java / Говнокод #9530

   +82

   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

   private static final String QUERY = 
           new StringBuffer("SELECT p.id FROM policysummary p ") 
               .append(" WHERE p.currentRevisionInd     = 1 ") 
               .append("AND p.policyStatusCd          IN ('issued') ") 
               .append("AND p.timedPolicyStatusCd     IN ('inForce') ") 
               .append("AND p.txType                  IN ('policy','renewal','endorsement','reinstatement') ") 
               .append("AND p.expiration <= :requestDate ") 
               .append("AND :requestDate <= (p.expiration + :requestTo) ") 
               .append("AND NOT EXISTS ") 
               .append("  (SELECT prs.id ") 
               .append("  FROM PolicyReportStatus prs ") 
               .append("  WHERE prs.policyId  = p.id ") 
               .append("  AND prs.mvrOrderDt IS NOT NULL ") 
               .append("  )") 
               .toString();

   Недавно выдавили заграничные коллеги

   roman-kashitsyn, 24 Февраля 2012

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

9. Куча / Говнокод #9529

   +120

   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

   Пришёл крешлог нашего приложения от пользователя
   
   Error Message:
   Segmentation fault
   
   Backtrace:
   
   <someapp>:0:in `0x00076d9f ht_handle_signal + 94'
   libsystem_c.dylib:1:in `0x37cb9539 _sigtramp + 48'
   JavaScriptCore:2:in `0x362d7aef WTFReportBacktrace + 146'
   JavaScriptCore:3:in `0x3621d173 _ZN3WTF12_GLOBAL__N_125ARC4RandomNumberGenerator12stirIfNeededEv + 26'
   JavaScriptCore:4:in `0x3621d0b3 _ZN3WTF29cryptographicallyRandomNumberEv + 102'
   JavaScriptCore:5:in `0x36227991 _ZN3WTF12randomNumberEv + 8'
   WebCore:6:in `0x32c3104b _ZN7WebCore17JSDOMGlobalObjectC2ERN3JSC12JSGlobalDataEPNS1_9StructureEN3WTF10PassRefPtrINS_15DOMWrapperWorldEEEPNS1_8JSObjectE + 138'
   <...>
   
   Внимательно смотрим лог, идём по ссылке
   http://trac.webkit.org/browser/releases/Apple/Safari%205.1/JavaScriptCore/wtf/Assertions.h
   и впечатляемся названиями переменных
   Краш, кстати, в генераторе случайных чисел, судя по логу...

   krypt, 24 Февраля 2012

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

10. JavaScript / Говнокод #9528

    +154

    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 trySearch(form)
    {
    	var loc = "/?";
    	
    	var query_len = $.trim(form.query.value).length;
    	
    	if(form.game.value !== "all")
    	{
    		loc += 'game='+form.game.value;
    	}
    	
    	if(!query_len && form.search_by.value !== 'hostname' && form.search_by.value !== 'mode')
    	{
    		$('#display').slideDown();
    		Timer.set(function(){$('#display').slideUp();},5000);
    		
    		form.query.focus();
    		var msg = '';
    		
    		switch(form.search_by.value)
    		{
    			case 'map':
    				msg = 'Введите название карты которую хотите найти, например: <a href="#" onclick="$(\'#query\').val(\'de_dust2,de_inferno\').submit(); return false;">de_dust2,de_inferno</a>';
    				break;
    			case 'player':
    				msg = 'Введите имя игрока которого хотите найти';
    				break;
    			case 'rules':
    				msg = 'Введите настройку которую хотите найти, например: <a href="#" onclick="$(\'#query\').val(\'amxmodx_version 1.8.2-dev\').submit(); return false;">amxmodx_version 1.8.2-dev</a>';
    				break;
    			case 'ip':
    				msg = 'Введите ип-адрес который хотите найти';
    				break;
    		}
    		
    		$('#Msg').html(msg);
    		
    		return false;
    	}
    	if(query_len > 0 || form.search_by.value !== 'hostname')
    		loc += (form.game.value !== "all" ? '&' : '')+"search_by="+form.search_by.value;
    	if(query_len > 0)
    	{
    		loc += "&query="+escape(form.query.value);
    	}
    	
    	location.href = loc;
    	return false;
    }

    Zend, 24 Февраля 2012

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

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

• 2113
• …
• 918
• 917
• 916
• 915
• 914
• 913
• 912
• 911
• 910
• …
• 1