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

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

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

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

1. ActionScript / Говнокод #6240

   −102

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

   for (i=0; i<550; i++)
   {
          for (j<0; j<400; j++)
         {
              .....
         }
   }

   Думал почему же цикл выполняеться только 550 раз, долгое время немог понять почему XD

   maldan, 05 Апреля 2011

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

2. Куча / Говнокод #6239

   +146

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

   <p>10.6.                   Участник может проходить этапы flash игр неограниченное количество раз. Результаты по каждой игре сохраняются на персональной странице пользователя и в базе данных Сайта. В рейтинге отражается наилучший результат, достигнутый при прохождении flash игры.</p>
   <p>10.7.                   Участники, которые не ввели уникальный код со скретч-карты, могут играть в Интернет-игру, но не участвуют в присуждении Приза «Стильный MP3 Плеер»</p>
   <p>10.8.                   Призы «Стильный MP3 Плеер» получают участники, набравшие максимальное количество баллов по итогам 2 flash игр.</p>
   
   <p><strong>11.  </strong><strong>Порядок определения Победителей Интернет-игры:</strong></p>
   <p>11.1.                   Первые 25 участников, набравших наибольшее количество баллов по итогам Интернет-игры, получают Приз «Стильный MP3 плеер»;</p>
   <p>11.2.                   В случае одинакового количества баллов, набранных разными участниками, приз достается участнику, чьи результаты были набраны раньше</p>
   <p>11.3.                   Каждый Участник может получить не более одного Приза «Стильный MP3 плеер».</p>

   рылся в портфолио molinos.ru, зашел на их детище http://www.corona-island.ru/ (2010 года выпуска) и увидел вот это в коде. И так весь сайт! Новое веяние - верстка пробелами!!!!!!

   oldfilin, 05 Апреля 2011

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

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

   +184

   1. 1

   BLOG_BLOG_BLOG_NO_BLOG

   Константа в 1C-Bitrix.
   
   Баден-Баден отдыхает.

   maxru, 05 Апреля 2011

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

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

   +132

   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

   /// <summary>
   /// Конвертирование руского текста в английский.
   /// </summary>
   /// <param name="russianText">Русский текст.</param>
   public static string ConvertToEnglish(string russianText)
   {
   	string englishText = russianText.ToLower();
   
   	englishText = englishText.Replace("КПК", "PDA");
   	englishText = englishText.Replace("ПК", "PC");
   	englishText = englishText.Replace("Ач ", "Ah");
   	englishText = englishText.Replace("ПО", "Software");
   	englishText = englishText.Replace("ОС", "OS");
   	.
   	.
   	.
   	//далее еще около 300 подобных замен
   
   	return englishText;
   }

   Изюминка этого говнокода заключается в первой строчке функции.
   Встретил там же где и http://govnokod.ru/6170

   lexx-pnz, 05 Апреля 2011

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

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

   +160

   1. 1

   std::cout << (valid_flag + prior ? 1 : 0) << " " << valid_flag+1-1 << std::endl;

   Говнокод - загадка. Какой тип у valid_flag?

   seregakabancheg, 05 Апреля 2011

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

6. 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)

7. 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)

8. 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)

9. 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)

10. 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)

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

• 2110
• …
• 618
• 617
• 616
• 615
• 614
• 613
• 612
• 611
• 610
• …
• 1