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

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

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

1. PHP (5712)
2. C++ (2746)
3. Куча (2407)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1122)
8. Pascal (648)
9. Python (592)
10. 1C (541)
11. SQL (433)
12. ActionScript (292)
13. Objective C (202)
14. bash (201)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (46)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. Лучший говнокод

   В номинации:

   • Лучший рейтинг
   • Самый комментируемый

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

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

   +137

   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

   private static string SimplifyPath(string path)
   {
   	var dirs = path.Split("\\".ToCharArray());
   
   	var absolutePath = new List<string>();
   	foreach (var dir in dirs)
   	{
   		if (dir == "..")
   		{
   			absolutePath.RemoveAt(absolutePath.Count - 1);
   			continue;
   		}
   		absolutePath.Add(dir);
   	}
   
   	var result = string.Empty;
   
   	for (int i = 0; i < absolutePath.Count - 1; ++i)
   	{
   		result += absolutePath[i];
   		result += "\\";
   	}
   
   	result += absolutePath[absolutePath.Count - 1];
   
   	return result;
   }

   Path.GetFullPath? Нет, мы не ищем легких путей.

   Cpp, 07 Марта 2013

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

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

   +134

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

   public void Children_OnAdd(object sender, EventArgs e)
   {
      ((MyEventArgs<Document>)e).D.Parent = this;
   }

   taburetka, 03 Марта 2013

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

4. JavaScript / Говнокод #12647

   +164

   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

   var num = 3;
   var x1 = Math.floor(Math.random()*num + 1);
   var x2 = Math.floor(Math.random()*num + 1);
   var x3 = Math.floor(Math.random()*num + 1);
   var x4 = Math.floor(Math.random()*num + 1);
   var x5 = Math.floor(Math.random()*num + 1);
   var x6 = Math.floor(Math.random()*num + 1);
   var x7 = Math.floor(Math.random()*num + 1);
   var x8 = Math.floor(Math.random()*num + 1);
   var x9 = Math.floor(Math.random()*num + 1);
   var x10 = Math.floor(Math.random()*num + 1);
   var x11 = Math.floor(Math.random()*num + 1);
   var x12 = Math.floor(Math.random()*num + 1);
   var x13 = Math.floor(Math.random()*num + 1);
   var x14 = Math.floor(Math.random()*num + 1);
   var x15 = Math.floor(Math.random()*num + 1);
   var x16 = Math.floor(Math.random()*num + 1);
   
   
   var i = 0;
   
   while (true) {
   x1 = Math.floor(Math.random()*num + 1);
   x2 = Math.floor(Math.random()*num + 1);
   x3 = Math.floor(Math.random()*num + 1);
   x4 = Math.floor(Math.random()*num + 1);
   x5 = Math.floor(Math.random()*num + 1);
   x6 = Math.floor(Math.random()*num + 1);
   x7 = Math.floor(Math.random()*num + 1);
   x8 = Math.floor(Math.random()*num + 1);
   x9 = Math.floor(Math.random()*num + 1);
   x10 = Math.floor(Math.random()*num + 1);
   x11 = Math.floor(Math.random()*num + 1);
   x12 = Math.floor(Math.random()*num + 1);
   x13 = Math.floor(Math.random()*num + 1);
   x14 = Math.floor(Math.random()*num + 1);
   x15 = Math.floor(Math.random()*num + 1);
   x16 = Math.floor(Math.random()*num + 1);
   	if (x1 === x2){
   		if (x2 === x3) {
   			if(x4 === x5){
   				if (x5 === x6) {
   					if (x6 === x7){
   						if (x7 === x8){
   							if (x8 === x9){
   								if (x9 === x10){
   									if (x10 === x11){
   										if (x11 === x12){
   											if (x12 === x13){
   												if (x13 === x14){
   													if (x14 === x15){
   														if (x15 === x16){
   															if (x1 === x16){
   																document.write('<h3>ЧИСЛО x1 - ' + x1 + ' и x2 - ' + x2 + " x3 - " + x3 +  " x4 - " + x4 + " x5 - " + x5 + " x6 - " + x6 + " x7 - " + x7 + " x8 - " + x8 + " x9 - " + x9 + " x10 - " + x10 + " x11 - " + x11 + " x12 - " + x12 + " x13 - " + x13 + " x14 - " + x14 + " x15 - " + x15  + " x16 - " + x16 + "<b> i= </b> " + i);
   										break;
   															}
   														
   														}
   													}
   												}
   											}
   										}
   									
   									}
   								}
   							}
   						}
   					}
   				}
   			}
   		}
   
   	}else{
   		i++;
   		document.write('<b>x1:  </b>' + x1 + '   <b>  x2:  </b> ' + x2 + " x3 - " + x3 +  " x4 - " + x4 + " x5 - " + x5 + " x6 - " + x6 + " x7 - " + x7 + " x8 - " + x8 + " x9 - " + x9 + " x10 - " + x10 + " x11 - " + x11 + " x12 - " + x12 + " x13 - " + x13 + " x14 - " + x14 + " x15 - " + x15  + " x16 - " + x16 + "  - " + i + "<br>");}
   }

   Генерируем 16 чисел в диапазоне 1-3, условие такое, что они должны быть равными, пока они не равны выводим каждое значение числа, и вот что каждый раз в цикле (генерируем новые случайные числа, в этом же диапазоне)

   kuler, 25 Февраля 2013

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

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

   +145

   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

   if (!$to && !$from)
           {
               $msgInstance->setStatus(1);
               $msgInstance->setDeletedTo(1);
               $msgInstance->setDeletedFrom(1);
               $incomingL = -1;
               $outcomingL = -1;
           }
           elseif (!$to){
               $msgInstance->setStatus(1);
               $msgInstance->setDeletedTo(1);
               $incomingL = -1;
               $outcomingL = 0;
           }
           elseif (!$from)
           {
               $msgInstance->setDeletedFrom(1);
               $incomingL = 0;
               $outcomingL = -1;
           }

   Оцените xD ;)

   mishanon, 19 Февраля 2013

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

6. Куча / Говнокод #12584

   +116

   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

   <style type="text/css">
   	header
   	{
   		display: block;
   	}
   
   	header
   	{
   		clear: both;
   		height: 160px;
   		width: 100%;
   		background: url(/images/imges.jpg) no-repeat;
   		-moz-background-size: 100%;
   		-webkit-background-size: 100%;
   		-o-background-size: 100%;
   		background-size: 100% 100%;
   		background-position: center center;
   
   	}
   	.block1
   	{
   		height: 160px;
   		width: 20%;
   	}
   	.block2
   	{
   		height: 160px;
   		width: 15%;
   	}
   	.block3
   	{
   		height: 160px;
   		width: 30%;
   	}
   	.block4
   	{
   		height: 160px;
   		width: 15%;
   	}
   	.block5
   	{
   		height: 160px;
   		width: 20%;
   	}
   	table
   	{
   		width: 100%;
   		border-collapse: collapse;
   		table-layout: fixed;
   	}
   
   </style>
   
   
   <header>
   	<table>
   		<tr>
   			<td class="block1" onClick="document.location='/' "  onmouseover ="this.style.cursor='pointer' " title= "Главная">
   			</td>
   			<td class="block2">
   			</td>
   			<td class="block3">
   			</td>
   			<td class="block4">
   			</td>
   			<td class="block5" onClick="document.location='/script/enter.php' " onmouseover ="this.style.cursor='pointer' " title= "Вход">
   			</td>
   		</tr>
   	</table>
   </header>

   Найдено на просторах интернета: обычная шапка сайта, ничего лишнего.
   Код был нечитаемым - форматировал я. CSS, касаемо использованных тегов, вся.

   superfood, 13 Февраля 2013

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

7. Java / Говнокод #12567

   +61

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

   public void insert()
   {
       super.insert();
       if (this.ItemId == "")
           this = this;
   }

   нарочно не придумаешь

   bodeaux, 11 Февраля 2013

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

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

   +140

   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

   string num = number.ToString();
   num = num.Replace(',', '.');
   string num2 = number2.ToString();
   num2 = num2.Replace(',', '.');
   int displace = 0;
   
   int i = 0;
   for (i = 0; i <= num.Length - 1; i++)
   {
       if (num.Substring(i, 1) != ".")
       {
           int curNum = Convert.ToInt16(num.Substring(i, 1));
           Microsoft.Xna.Framework.Rectangle source = new Microsoft.Xna.Framework.Rectangle(curNum * 46, 0, 46, 64);
           sb.Draw(numbers, new Microsoft.Xna.Framework.Rectangle((int)position.X + (i * 24), (int)position.Y - 5, (int)(46.0f * scale), (int)(64.0f * scale)), source, color);
       }
       else
       {
           Microsoft.Xna.Framework.Rectangle source = new Microsoft.Xna.Framework.Rectangle(10 * 46, 0, 46, 64);
           sb.Draw(numbers, new Microsoft.Xna.Framework.Rectangle((int)position.X + (i * 24), (int)position.Y - 5, (int)(46.0f * scale), (int)(64.0f * scale)), source, color);
       }
       displace += (int)(46.0f * scale);
   }
   Microsoft.Xna.Framework.Rectangle src = new Microsoft.Xna.Framework.Rectangle(11 * 46, 0, 46, 64);
   sb.Draw(numbers, new Microsoft.Xna.Framework.Rectangle((int)position.X + displace, (int)position.Y - 5, (int)(46.0f * scale), (int)(64.0f * scale)), src, color);
   displace += (int)(46.0f * scale);
   for (i = 0; i <= num2.Length - 1; i++)
   {
       if (num2.Substring(i, 1) != ".")
       {
           int curNum = Convert.ToInt16(num2.Substring(i, 1));
           Microsoft.Xna.Framework.Rectangle source = new Microsoft.Xna.Framework.Rectangle(curNum * 46, 0, 46, 64);
           sb.Draw(numbers, new Microsoft.Xna.Framework.Rectangle((int)position.X + (i * 24) + displace, (int)position.Y - 5, (int)(46.0f * scale), (int)(64.0f * scale)), source, color);
       }
       else
       {
           Microsoft.Xna.Framework.Rectangle source = new Microsoft.Xna.Framework.Rectangle(10 * 46, 0, 46, 64);
           sb.Draw(numbers, new Microsoft.Xna.Framework.Rectangle((int)position.X + (i * 24) + displace, (int)position.Y - 5, (int)(46.0f * scale), (int)(64.0f * scale)), source, color);
       }
   }

   Nuff said.
   Как я тогда давным давно поленился импортировать неймспейс XNA - чёрт его знает.

   RaZeR, 09 Февраля 2013

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

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

   +70

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

   public static final void setManager(String name, MessageManager manager) {
           if ("doc".equals(name)) {
               doc = manager;
           } else {
               throw new RuntimeException("name is not 'doc' : " + name);
           }
       }

   Просто эпично! Даже добавить нечего

   tuba.linux, 30 Января 2013

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

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

    +153

    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

    // ================================================ SET UP DISPLAY OF FIRST SLIDE (CHECK FOR ANCHOR LINK) ============================================================
    
    var anchorLink = self.document.location.hash;
    
    switch (anchorLink)
    {
    	
    	// Intro
    	case "#panel-00-00": var currentSlide = 1; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','none'); $('#slides_holder').css('left','0px'); animateSlide(currentSlide); break;
    	
    	// Table of Contents
    	case "#panel-00-01": var currentSlide = 2; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','none'); $('#slides_holder').css('left','-776px'); animateSlide(currentSlide); break;
    	
    	// Chapter 1
    	case "#panel-01-00": var currentSlide = 3; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','block'); highlightNavItems(3,3); $('#slides_holder').css('left','-1552px'); $('.slide'+currentSlide+'_object').css('display','block'); animateSlide(currentSlide); break;
    	case "#panel-01-01": var currentSlide = 4; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','block'); highlightNavItems(3,4); $('#slides_holder').css('left','-2328px'); $('.slide'+currentSlide+'_object').css('display','block'); animateSlide(currentSlide); break;
    	case "#panel-01-02": var currentSlide = 5; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','block'); highlightNavItems(3,5); $('#slides_holder').css('left','-3104px'); $('.slide'+currentSlide+'_object').css('display','block'); animateSlide(currentSlide); break;
    	case "#panel-01-03": var currentSlide = 6; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','block'); highlightNavItems(3,6); $('#slides_holder').css('left','-3880px'); $('.slide'+currentSlide+'_object').css('display','block'); animateSlide(currentSlide); break;
    	case "#panel-01-04": var currentSlide = 7; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','block'); highlightNavItems(3,7); $('#slides_holder').css('left','-4656px'); $('.slide'+currentSlide+'_object').css('display','block'); animateSlide(currentSlide); break;
    	case "#panel-01-05": var currentSlide = 8; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','block'); highlightNavItems(3,8); $('#slides_holder').css('left','-5432px'); $('.slide'+currentSlide+'_object').css('display','block'); animateSlide(currentSlide); break;
    	case "#panel-01-06": var currentSlide = 9; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','block'); highlightNavItems(3,9); $('#slides_holder').css('left','-6208px'); $('.slide'+currentSlide+'_object').css('display','block'); animateSlide(currentSlide); break;
    	case "#panel-01-07": var currentSlide = 10; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','block'); highlightNavItems(3,10); $('#slides_holder').css('left','-6984px'); $('.slide'+currentSlide+'_object').css('display','block'); animateSlide(currentSlide); break;
    
    	/* ... Ещё таких же 4 чаптера по 7-10 слайдов ... */
    
    	// Quiz
    	case "#panel-06-00": var currentSlide = 51; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','block'); highlightNavItems(51,51); $('#slides_holder').css('left','-38800px'); $('.slide'+currentSlide+'_object').css('display','block'); animateSlide(currentSlide); break;
    	case "#panel-06-01": var currentSlide = 52; $('.slide'+(currentSlide)+'_object').css('display','none'); $('#module_menu').css('display','block'); highlightNavItems(51,51); $('#slides_holder').css('left','-38800px'); $('.slide'+currentSlide+'_object').css('display','block'); animateSlide(currentSlide); break;
    	
    	// Intro (user has accessed "main" page)
    	default: var currentSlide = 1;  $('#module_menu').css('display','none'); break;
    }

    Вот такой прислали слайдер на переделку, главный код опущен, но стиль такой же.

    SerDIDG, 22 Января 2013

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

11. PHP / Говнокод #12448

    +47

    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

    function executeNewsRightRelated(sfWebRequest $request)
      {
        $siteId = $request->getParameter('site')->id;
        $tNewsList = $this->object->News;
    
        $newsList = array();
        foreach ($tNewsList as $news) {
    
          $ids = array();
          foreach ($news->Sites as $idModel) {
            $ids[] = $idModel->id;
          }
    
          if (in_array($siteId, $ids)) {
            $newsList[] = $news;
          }
        }
    
        $this->newsList = $newsList;
      }

    Skiv, 18 Января 2013

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

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

• 1978
• …
• 693
• 692
• 691
• 690
• 689
• 688
• 687
• 686
• 685
• …
• 1