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

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

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

1. PHP (5707)
2. C++ (2739)
3. Куча (2375)
4. JavaScript (2032)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (588)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (199)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (31)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   +162

   1. 1
   2. 2

   foreach(split(',', '101,102,150,1351,2135,22153,351,15321,5351,235') as $key => $val)
   	$tmparray[] = $val;

   разбиваем строку в массив )))

   pes, 05 Апреля 2011

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

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

   −864

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

   SELECT COUNT(*) FROM apteka.cache_offers WHERE ORG_ID = '21963' AND `subject_id` IN (35,30052,20262,20263,20001,30488,30776,22099,22044,21982,21938,21823,
   (еще триста чисел),
   26313,26358,26478,23381,27430,26367,26418,23433,23404,23372,23289,23321,23285,23315,23282,23406,23287,
   26356,23429,23263,23349,23399,23356,23291,23346,23351,23250,23198,23194,23319,23238,23296,23259,23167,23268,23130,23052,23119,22430,23146,22315,23069,23083,23
   127,23189,23023,23014,23059,22988,23020,23007,22989,22949,22937,22939,22941,22979,22786,22842,22849,22758,22913,22919,22833,22718,22753,22779,22724,22857,2276
   8,22761,22733,22823,22759,22764,22658,22681,22674,22706,22599,22698,22667,22627,22776,22654,22655,22601,22619,22598,22694,22696,22603,22541,22570,22469,22534,
   22515,22676,22516,22477,22491,22677,22513,22421,22537,22451,22507,22482,22440,22353,22523,22422,22510,22397,22473,22348,22383,22424,22333,22380,22359,22508,22
   486,22456,22330);

   Стандартный запрос. При формировании страницы конструкция встречается многократно (>10 раз)

   deadwire, 05 Апреля 2011

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

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

   +117

   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

   int massaotves = 0;
                       var result = db.Vesi500.SingleOrDefault(x => x.Id_vesi500 == 10);
                       massaotves = Convert.ToInt32(result.Value_vesi500);
                       int step500 = 0;
                       //int ost1 = summ500 % massaotves;
                       //if (ost1 == 0)
                       //    step500 = summ500 / massaotves;
                       //else
                       //    step500 = (summ500 / massaotves) + 1;
   
                       int ost1 = summ500 % 450;
                       if (ost1 == 0)
                           step500 = summ500 / 450;
                       else
                           step500 = (summ500 / 450) + 1;
                       #endregion
   
                       #region Отвесы и суммы для 200 весов
                       int massaotves200 = 0;
                       var result1 = db.Vesi200.SingleOrDefault(x => x.Id_vesi200 == 9);
                       massaotves200 = Convert.ToInt32(result1.Value_vesi200);
                       int step200 = 0;
                       //int ost2 = summ200 % massaotves200;
                       //if (ost2 == 0)
                       //    step200 = summ200 / massaotves200;
                       //else
                       //    step200 = (summ200 / massaotves200) + 1;
                       
                       int ost2 = summ200 % 180;
                       if (ost2 == 0)
                           step200 = summ200 / 180;
                       else
                           step200 = (summ200 / 180) + 1;
                       #endregion                   
   
   
                       int itog = 0;
                       //if (step200 > step500)
                       //    itog = step200;
                       //else
                       //    itog = step500;
   
                       int test = (summ200 + summ500) / massaotves200;
                       if (test >= step500 && test >= step200)
                           itog = test;
                       else if (step200 > step500)
                           itog = step200;
                       else   
                           itog = step500;
   
                       textBox3.Text = Convert.ToString(itog);
                       vsegootvesov = itog;

   Выполняет расчет необходимых показателей, для запуска технологического процесса

   fxlancer, 05 Апреля 2011

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

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

   +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
   26. 26
   27. 27
   28. 28
   29. 29
   30. 30
   31. 31
   32. 32
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37

   string[] otgruzka = new string[11] { "Отгрузка компонент 1 Вес 5дк-500", "Отгрузка компонент 2 Вес 5дк-500", "Отгрузка компонент 3 Вес 5дк-500", "Отгрузка компонент 4 Вес 5дк-500", "Отгрузка компонент 5 Вес 5дк-500", "Отгрузка компонент 6 Вес 5дк-500", "Отгрузка компонент 1 Вес 5дк-200", "Отгрузка компонент 2 Вес 5дк-200", "Отгрузка компонент 3 Вес 5дк-200", "Отгрузка компонент 4 Вес 5дк-200", "Отгрузка компонент 5 Вес 5дк-200", };
                       string[] silos = new string[11] { "Силос 1", "Силос 5", "Силос 6", "Силос 7", "Силос 8", "Силос 31-H", "Силос 9", "Силос 11", "Силос 2", "Силос 3", "Силос 4" };
                                          
                       for (int i = 0; i < dataGridView1.RowCount; i++)
                       {
                           if (Convert.ToString(dataGridView1.Rows[i].Cells[5].Value) == "Силос 1")
                               dataGridView1.Rows[i].Cells[3].Value = modbus_master.GetValue(otgruzka[0]);
                           if (Convert.ToString(dataGridView1.Rows[i].Cells[5].Value) == "Силос 5")
                               dataGridView1.Rows[i].Cells[3].Value = modbus_master.GetValue(otgruzka[1]);
                           if (Convert.ToString(dataGridView1.Rows[i].Cells[5].Value) == "Силос 6")
                               dataGridView1.Rows[i].Cells[3].Value = modbus_master.GetValue(otgruzka[2]);
                           if (Convert.ToString(dataGridView1.Rows[i].Cells[5].Value) == "Силос 7")
                               dataGridView1.Rows[i].Cells[3].Value = modbus_master.GetValue(otgruzka[3]);
                           if (Convert.ToString(dataGridView1.Rows[i].Cells[5].Value) == "Силос 8")
                               dataGridView1.Rows[i].Cells[3].Value = modbus_master.GetValue(otgruzka[4]);
                           if (Convert.ToString(dataGridView1.Rows[i].Cells[5].Value) == "Силос 31-H")
                               dataGridView1.Rows[i].Cells[3].Value = modbus_master.GetValue(otgruzka[5]);
                           if (Convert.ToString(dataGridView1.Rows[i].Cells[5].Value) == "Силос 9")
                               dataGridView1.Rows[i].Cells[3].Value = modbus_master.GetValue(otgruzka[6]);
                           if (Convert.ToString(dataGridView1.Rows[i].Cells[5].Value) == "Силос 11")
                               dataGridView1.Rows[i].Cells[3].Value = modbus_master.GetValue(otgruzka[7]);
                           if (Convert.ToString(dataGridView1.Rows[i].Cells[5].Value) == "Силос 2")
                               dataGridView1.Rows[i].Cells[3].Value = modbus_master.GetValue(otgruzka[8]);
                           if (Convert.ToString(dataGridView1.Rows[i].Cells[5].Value) == "Силос 3")
                               dataGridView1.Rows[i].Cells[3].Value = modbus_master.GetValue(otgruzka[9]);
                           if (Convert.ToString(dataGridView1.Rows[i].Cells[5].Value) == "Силос 4")
                               dataGridView1.Rows[i].Cells[3].Value = modbus_master.GetValue(otgruzka[10]);
                       }                    
                       textBox12.Text = Convert.ToString(modbus_master.GetValue("Текущий отвес 5дк-200") + 1);
   
                       if (modbus_master.GetValue("Текущий отвес 5дк-200") > WorkWithArchive.currentOtves200)
                       {
                           WorkWithArchive.WriteArchiveOtves200(modbus_master.GetValue("Текущий отвес 5дк-200"));
                           WorkWithArchive.currentOtves200 = modbus_master.GetValue("Текущий отвес 5дк-200");
                           WorkWithArchive.WriteArchiveKomponent200(dataGridView1);
                           WorkWithArchive.ot200 = true;
                       }

   Программа управления на производстве, взаимодействует с контроллером, получает кучу данных и такую же кучу отправляет.

   fxlancer, 05 Апреля 2011

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

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

   +162

   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

   public function Showday($id)
   	{
   		//echo $id;exit;
   		if($id==1)
   		{
   		$day ='Monday';
   		}
   		elseif($id==2)
   		{
   		$day ='Tuesday';
   		}
   		elseif($id==3)
   		{
   		$day ='Wednesday';
   		}
   		elseif($id==4)
   		{
   		$day ='Thursday';
   		}
   		elseif($id==5)
   		{
   		$day ='Friday';
   		}
   		
   	    return $day;
   	}	
   	
   	
   	public function Showdayno($id)
   	{
   		//echo $id;exit;
   		if($id=='Monday')
   		{
   		$day ='1';
   		}
   		elseif($id=='Tuesday')
   		{
   		$day ='2';
   		}
   		elseif($id=='Wednesday')
   		{
   		$day ='3';
   		}
   		elseif($id=='Thursday')
   		{
   		$day ='4';
   		}
   		elseif($id=='Friday')
   		{
   		$day ='5';
   		}
   		
   	    return $day;
   	}

   Вот такой код попался в ZF проекте, написанном индусами.

   gerra, 05 Апреля 2011

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

6. Java / Говнокод #6230

   +75

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

   String filler = "";
     int tot = text.length();
     int til = 18 - (tot/100);
     for(int i = 0; i < tot; i++)
     {
       if(text.charAt(i) == '\n')
       {
         text = text.substring(0, i) + "<br/>" + text.substring(i+1);
         tot += 3; i += 3;
         til--;
       }
     }

   Продакшн. Форматирование текста - вставка перевода строки.

   a4060276, 05 Апреля 2011

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

7. C++ / Говнокод #6229

   +177

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

   if(x*x > = 0)
   {
   // какие-то действия
   }
   else
   {
   // какие-то действия
   }

   Код встретил у знакомой студентки :3 Не, ну а в вдруг?

   minuzZ, 05 Апреля 2011

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

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

   +165

   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

   javascript:
   document.write(
   document.getElementsByTagName('body')0].
   getElementsByTagName('div')[0].
   getElementsByTagName('div')[0].
   getElementsByTagName('div')[2].
   getElementsByTagName('div')[1].
   getElementsByTagName('div')[0].
   getElementsByTagName('div')[0].
   getElementsByTagName('table')[0].
   getElementsByTagName('tbody')[0].
   getElementsByTagName('tr')[0].
   getElementsByTagName('td')[0].
   getElementsByTagName('div')[0].
   getElementsByTagName('div')[0].
   getElementsByTagName('div')[0].
   getElementsByTagName('div')[1].
   getElementsByTagName('table')[0].
   getElementsByTagName('tbody')[0].
   getElementsByTagName('tr')[0].
   getElementsByTagName('td')[1].
   getElementsByTagName('div')[0].
   getElementsByTagName('b')[0].
   getElementsByTagName('a')[0].value
   );

   http://javascript.ru/tutorial/dom/search#comment-5934
   
   Вот как писать надо!

   fuckyounoob, 05 Апреля 2011

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

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

   −132

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

   Договор = Неопределено;
   
   ВыборкаДоговоров = Справочники.ДоговорыКонтрагентов.Выбрать();
   
   Пока ВыборкаДоговоров .Следующий() Цикл
         Если ВыборкаДоговоров.Ссылка = Документ.ДоговорКонтрагента Тогда
               Договор = ВыборкаДоговоров.Ссылка;
         КонецЕсли;
   КонецЦикла
   
   ТабДокумент.Параметры.Договор = Договор.Наименеование;

   Попросили стажера сделать печатную форму.

   A-tomic, 05 Апреля 2011

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

10. SQL / Говнокод #6226

    −162

    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
    88. 88
    89. 89
    90. 90
    91. 91
    92. 92
    93. 93
    94. 94

    USE [VOLTAREN]
    GO
    /****** Object:  UserDefinedFunction [dbo].[myReplace]    Script Date: 04/05/2011 12:11:42 ******/
    SET ANSI_NULLS ON
    GO
    SET QUOTED_IDENTIFIER ON
    GO
    ALTER FUNCTION [dbo].[myReplace](@NAME varchar(255))
    RETURNS varchar(255)
    BEGIN
    set @NAME = (
    	UPPER(
    replace(
    	replace(
    		replace(
    			replace(
    				replace(
    					replace(
    						replace(
    							replace(
    								replace(
    									replace(
    										replace(
    											replace(
    												replace(
    													replace(
    														replace(
    															replace(
    																replace(
    																	replace(
    																		replace(
    																			replace(
    																				replace(
    																					replace(
    																						replace(
    																							replace(
    																								replace(
    																									replace(
    																										replace(
    																											replace(
    																												replace(
    																													replace(
    																														replace(
    																															replace(
    																																replace(
    																																	replace(
    																																		replace(
    																																			replace(
    																																				@NAME,'*','')
    																																			,'табл','таб')
    																																		,'~','')
    																																	,'`','')
    																																,'<','')
    																															,'>','')
    																														,'}','')
    																													,'{','')
    																												,']','')
    																											,'[','')
    																										,'=','')
    																									,'_','')
    																								,'#','')
    																							,'$','')
    																						,'?','')
    																					,'^','')
    																				,'&','')
    																			,'@','')
    																		,'''','')
    																	,'"','')	
    																,':','')
    															,'%','')
    														,'-','')
    													,'+','')
    												,')','')
    											,'(','')
    										,'N','')
    									,'№','')
    								,'|','')
    							,'\','')
    						,'/','')
    					,'!','')
    				,';','')
    			,',','')
    		,'.','')
    	,' ','')));
    	
    
    return(
    replace(
    	@NAME,
    'покробол','по')
    )
    	
    
    END;

    функция удаления не нужных символов из строки

    cu3blukekc, 05 Апреля 2011

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

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

• 2108
• …
• 617
• 616
• 615
• 614
• 613
• 612
• 611
• 610
• 609
• …
• 1