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

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

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

1. PHP (5708)
2. C++ (2739)
3. Куча (2379)
4. JavaScript (2032)
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. Java / Говнокод #9944

   +74

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

   for(int i = 0; i < fCount; i++)
   {
          result += Integer.parseInt(String.valueOf((fPart.charAt(i))), 10)*Math.pow(base, fCount-(i+1));	    			
   }

   brainy, 14 Апреля 2012

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

2. C# / Говнокод #9943

   +121

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

   private static string GetShortDisciplineName(string longDsciplineName){
               string[] words = longDsciplineName.Split(' ');
               switch (words.Count()){
                   case 1:{
                       return longDsciplineName;
                       break;
                       }

   pm2t29jf47, 13 Апреля 2012

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

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

   +157

   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

   mysql_connect($MYSQL_HOST,$MYSQL_USER,$MYSQL_PASSWORD) OR DIE("Не могу создать соединение "); 
           mysql_select_db($MYSQL_DB_NAME) or die(mysql_error());  
           $query = "SELECT * FROM `Subdivision` WHERE `Checked`=1";  
           $result = mysql_query($query) or die(mysql_error()); 
           $i=0;
           while($row = mysql_fetch_assoc($result))
             {
               $data[$i]=$row;
               $i++;
             }
           $query = "SELECT * FROM `Subdivision` WHERE `Subdivision_Name`='Спорт'";
           $result = mysql_query($query) or die(mysql_error()); 
           $row = mysql_fetch_assoc($result);
           $data[]=$row;
           $data['22']['Subdivision_Name']='Главная (интернет магазин прикольных футболок с надписью)';
           $data['19']['Subdivision_Name']='Креативные футболки с прикольными рисунками';
           $data['2']['Subdivision_Name']='Классные футболки для офиса';
           $data['3']['Subdivision_Name']='Оригинальные студенческие футболки';
           $data['4']['Subdivision_Name']='Самые лучшие футболки sexy';
           $data['6']['Subdivision_Name']='Правильные футболки для админов';
           $data['32']['Subdivision_Name']='Клевые футболки South Park ';
           $data['35']['Subdivision_Name']='Эксклюзивные футболки на хэллоуин'; 
           $array = array(13,9,11,10,12,14,15,16);
           $sub_array = array(33,19,31,38,0,1,2,3,4,5,6,7,8,20,32,35,36,37);
           $sitemap ="<ul>";
           $sitemap.="<li><a href='".$data['22']['Hidden_URL']."'>".$data['22']['Subdivision_Name']."</a></li>"; 
           $sitemap.="<ul>";
           for($i=0;$i<count($sub_array);$i++)
             {
                $sitemap.="<li><a href='".$data[$sub_array[$i]]['Hidden_URL']."'>".$data[$sub_array[$i]]['Subdivision_Name']."</a></li>"; 
             }
           $sitemap.="</ul>";
           for($i=0;$i<count($array);$i++)
             {
                $sitemap.="<li><a href='".$data[$array[$i]]['Hidden_URL']."'>".$data[$array[$i]]['Subdivision_Name']."</a></li>"; 
             }
           $sitemap.="</ul>";
           
           return $sitemap;

   комментарии излишни

   freeadler, 13 Апреля 2012

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

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

   +148

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

   php_value register_globals 1
   php_flag register_globals on
   php_value magic_quotes 1
   php_flag magic_quotes on
   php_value magic_quotes_gpc 1
   php_flag magic_quotes_gpc on

   n/c

   miraage, 13 Апреля 2012

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

5. Pascal / Говнокод #9940

   +78

   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

   > В какую середину?) он ставиться на свое место. У тебя не верное представление об устройстве данных в памяти. 
   
   procedure TForm1.Button1Click(Sender: TObject);
   type
   parr = ^tarr;
   tarr = array of integer;
   var
   pparr: array of parr;
   begin
   setlength(pparr,1);
   memo1.Lines.Add('first array pointer '+inttostr(integer(@pparr)));
   new(pparr[0]);
   memo1.Lines.Add('second array pointer '+inttostr(integer(pparr[0])));
   memo1.Lines.Add('first element '+inttostr(integer(pparr[0]^)));
   end;
   
   > вывод 
   
   first array pointer 1242664
   second array pointer 10822692
   first element 0

   Человек нашёл единственно верный путь использования двумерных динмассивов. Через указатель на указатель, правда он сам этого не понимает. Авторское форматирование и орфография сохранены.
   
   Источник:
   http://www.gamedev.ru/projects/forum/?id=161043&page=3#m35

   TarasB, 13 Апреля 2012

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

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

   +78

   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

   mLoadingTextureMap.put(LINE +i, createFromResource(mLinesAtlas, this, R.drawable.loading_line1, 0, y));
   y+=drawable.getMinimumHeight();
   i++;
   mLoadingTextureMap.put(LINE +i, createFromResource(mLinesAtlas, this, R.drawable.loading_line2, 0, y));
   y+=drawable.getMinimumHeight();
   i++;
   mLoadingTextureMap.put(LINE +i, createFromResource(mLinesAtlas, this, R.drawable.loading_line3, 0, y));
   y+=drawable.getMinimumHeight();
   i++;
   mLoadingTextureMap.put(LINE +i, createFromResource(mLinesAtlas, this, R.drawable.loading_line4, 0, y));
   y+=drawable.getMinimumHeight();
   i++;
   mLoadingTextureMap.put(LINE +i, createFromResource(mLinesAtlas, this, R.drawable.loading_line5, 0, y));
   y+=drawable.getMinimumHeight();
   i++;
   mLoadingTextureMap.put(LINE +i, createFromResource(mLinesAtlas, this, R.drawable.loading_line6, 0, y));
   y+=drawable.getMinimumHeight();
   i++;
   mLoadingTextureMap.put(LINE +i, createFromResource(mLinesAtlas, this, R.drawable.loading_line7, 0, y));
   y+=drawable.getMinimumHeight();
   i++;
   mLoadingTextureMap.put(LINE +i, createFromResource(mLinesAtlas, this, R.drawable.loading_line8, 0, y));
   y+=drawable.getMinimumHeight();
   i++;
   mLoadingTextureMap.put(LINE +i, createFromResource(mLinesAtlas, this, R.drawable.loading_line9, 0, y));

   вот так один паренек загружает в атлас текстуры....
   нафиг циклы вообще нужны (=

   TeknoMatik, 13 Апреля 2012

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

7. JavaScript / Говнокод #9938

   +157

   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

   $(document).ready(function(){ 
   
           // ...
   
   	var nom1=0;
   	var nom2=0;
   	
   	$("#first").live("hover", function() {
   		$(".blocks1").addClass('activ_block');
   		$(".blocks1 .block1").addClass('activ_block');
   		$(".blocks1 .block2").addClass('activ_block');
   		$(".blocks1 .block2 ul").eq(0).addClass('activ_block');
   		$(".blocks1 .block1 a").eq(0).addClass('active_red');
   		nom1 = 0;
   		
   		$(".blocks").removeClass('activ_block');
   		$(".blocks2").removeClass('activ_block');
   		$(".blocks3").removeClass('activ_block');
   		$(".blocks .blocks ul").removeClass('activ_block');
   		$(".blocks .blocks div").removeClass('activ_block');
   		$(".blocks .block1").removeClass('activ_block');
   		$(".blocks .block2").removeClass('activ_block');
   		$(".blocks .block3").removeClass('activ_block');
   		$(".blocks .block1 a").removeClass('active_reds');
   		$(".blocks .block2 a").removeClass('active_reds');
   	});
   	
   	$(".blocks1 .block1 a").live("click", function() { 
   		nom1 = $(".blocks1 .block1 a").index(this); // id level 2
   		$(".blocks1 .block1 a").removeClass('active_red');
   		$(this).addClass('active_red');
   		$(".blocks1 .block2 ul").removeClass('activ_block');
   		$(".blocks1 .block2").addClass('activ_block');
   		$(".blocks1 .block2 ul").eq((nom1)).addClass('activ_block');	
   	});
   	
   	$(".blocks1 .block2 a").live("hover", function() { 
   		nom2 = $(".blocks1 .block2 ul.activ_block a").index(this); // id level 3
   		
   		$(".blocks1 .block3 div").removeClass('activ_block');
   		$(".blocks1 .block3 div div").removeClass('activ_block');
   		$(".blocks1 .block3").addClass('activ_block');
   		$(".blocks1 .block3 div.z").eq((nom1)).addClass('activ_block');
   		$(".blocks1 .block3 div.activ_block div").eq((nom2)).addClass('activ_block');
   
   		$(".blocks1 .block2 a").removeClass('active_reds');
   		$(this).addClass('active_reds');		
   	});
   	
   	$(".blocks1 .block1").hover(function() { 
   		$(".blocks1 .block3").removeClass('activ_block');
   		$(".blocks1 .block2 a").removeClass('active_reds');
   		$(".blocks1 .block3 div").removeClass('activ_block');
   		$(".blocks1 .block3 div div").removeClass('activ_block');
   	} , 
   	function() {
   		
   	});
   	
   	$(".blocks1 .block3").hover(function() { 
   	} , 
   	function() {
   		
   	});
   	
   	$(".blocks1 .block2").hover(function() {
   		$(".blocks1 .block3").removeClass('activ_block');
   		$(".blocks1 .block3 div").removeClass('activ_block');
   		$(".blocks1 .block2 a").removeClass('active_reds');
   	} , 
   	function() { 
   		if ($(".blocks1 .block3").hasClass('activ_block')) {
   		} else {
   		}
   	});
   
           // ...
   
   });

   Очередной шедевр jQuery-мастеров. Скрипт меню, приведенный код - около трети от общего количества такого же.

   sap1ens, 13 Апреля 2012

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

8. C# / Говнокод #9937

   +125

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

   if (FileAssociation.IsAssociated(".avi") || !FileAssociation.IsAssociated(".avi"))
               {
                   FileAssociation.Associate(".avi", "Flvplayer", ".avi", a, b);
               }

   http://www.cyberforum.ru/csharp-beginners/thread304416.html

   Somov, 12 Апреля 2012

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

9. Java / Говнокод #9936

   +68

   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

   if (seriesOrder == SeriesRenderingOrder.REVERSE) {
                   //render series in reverse order
                   for (int pass = 0; pass < passCount; pass++) {
                       int seriesCount = dataset.getSeriesCount();
                       for (int series = seriesCount - 1; series >= 0; series--) {
                           int firstItem = 0;
                           int lastItem = dataset.getItemCount(series) - 1;
                           if (lastItem == -1) {
                               continue;
                           }
                           if (state.getProcessVisibleItemsOnly()) {
                               int[] itemBounds = RendererUtilities.findLiveItems(
                                       dataset, series, xAxis.getLowerBound(),
                                       xAxis.getUpperBound());
                               firstItem = Math.max(itemBounds[0] - 1, 0);
                               lastItem = Math.min(itemBounds[1] + 1, lastItem);
                           }
                           state.startSeriesPass(dataset, series, firstItem,
                                   lastItem, pass, passCount);
                           for (int item = firstItem; item <= lastItem; item++) {
                               renderer.drawItem(g2, state, dataArea, info,
                                       this, xAxis, yAxis, dataset, series, item,
                                       crosshairState, pass);
                           }
                           state.endSeriesPass(dataset, series, firstItem,
                                   lastItem, pass, passCount);
                       }
                   }
               }
               else {
                   //render series in forward order
                   for (int pass = 0; pass < passCount; pass++) {
                       int seriesCount = dataset.getSeriesCount();
                       for (int series = 0; series < seriesCount; series++) {
                           int firstItem = 0;
                           int lastItem = dataset.getItemCount(series) - 1;
                           if (state.getProcessVisibleItemsOnly()) {
                               int[] itemBounds = RendererUtilities.findLiveItems(
                                       dataset, series, xAxis.getLowerBound(),
                                       xAxis.getUpperBound());
                               firstItem = Math.max(itemBounds[0] - 1, 0);
                               lastItem = Math.min(itemBounds[1] + 1, lastItem);
                           }
                           state.startSeriesPass(dataset, series, firstItem,
                                   lastItem, pass, passCount);
                           for (int item = firstItem; item <= lastItem; item++) {
                               renderer.drawItem(g2, state, dataArea, info,
                                       this, xAxis, yAxis, dataset, series, item,
                                       crosshairState, pass);
                           }
                           state.endSeriesPass(dataset, series, firstItem,
                                   lastItem, pass, passCount);
                       }
                   }
               }

   нашел в jfreechart
   Обе ветки if-else идентичны, только в одной цикл идет от нуля, в другой в обратную сторону.
   Говнокод или нет?

   krige, 12 Апреля 2012

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

10. Python / Говнокод #9935

    −99

    1. 1
    2. 2
    3. 3

    referer = ''
    if 'HTTP_REFERER' in request.META :
        referer = request.META['HTTP_REFERER']

    И подобного кода в проекте тонны...

    Scipetr, 12 Апреля 2012

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

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

• 2109
• …
• 955
• 954
• 953
• 952
• 951
• 950
• 949
• 948
• 947
• …
• 1