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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2389)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
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 (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (37)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. PHP / Говнокод #19457

   +2

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

   $host = Doctrine_Core::getTable("TransfertConfij")->find(1)->emailserver;
   $emailto = $form->getValue('addresstestemail');
   $port = Doctrine_Core::getTable("TransfertConfij")->find(1)->port;
   $encriptyon = Doctrine_Core::getTable("TransfertConfij")->find(1)->encriptyon;
   $login = Doctrine_Core::getTable("TransfertConfij")->find(1)->loginemailserver;
   $pass = Doctrine_Core::getTable("TransfertConfij")->find(1)->passemailserver;
   $content = $form->getValue('contenttestemail');
   
   // Далее код отправки письма.

   5 запросов? Какая разница? работает ведь!

   rkgrep, 14 Февраля 2016

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

3. 1C / Говнокод #19392

   −99

   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

   Если Сальдо = Перечисления.ДаНет.Да И Кредит = Перечисления.ДаНет.Да Тогда
               
   	СуммаКОплате   =  Строка.Сальдо +  Строка.Начислено +  Строка.Разовые - Строка.Оплата - Льгота;
   	Строка.Коплате = СуммаКОплате;
               
   ИначеЕсли Сальдо = Перечисления.ДаНет.Нет И Кредит = Перечисления.ДаНет.Да Тогда
               
   	СуммаКОплате   = Строка.Начислено + Строка.Разовые - Строка.Оплата - Льгота - ?(Строка.Сальдо < 0, Строка.Сальдо * -1, 0);
   	Строка.Коплате = СуммаКОплате;
               
   ИначеЕсли Сальдо = Перечисления.ДаНет.Да И Кредит = Перечисления.ДаНет.Нет Тогда
               
   	СуммаКОплате = ?(Строка.Сальдо > 0, Строка.Сальдо , 0) +  Строка.Начислено + Строка.Разовые - Строка.Оплата  - Льгота;
   	Строка.Коплате = СуммаКОплате;
               
   ИначеЕсли Сальдо = Перечисления.ДаНет.Нет И Кредит = Перечисления.ДаНет.Нет Тогда
               
   	СуммаКОплате = Строка.Начислено + Строка.Разовые - Льгота;
   	Строка.Коплате = СуммаКОплате;
               
   КонецЕсли;

   Проняло до слез. Ребята не знают простой математики, зато им известна условная тернарная операция. Привел самый эпичный отрывок, но там впору публиковать всю конфигурацию.

   ivan_0110, 03 Февраля 2016

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

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

   +2

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

   public ActionResult Exception()
   {
     throw new Exception("You better not to execute this!");
   }

   нашел говнокод, оставленный прошлым архитектором)

   govnokoder1488, 28 Января 2016

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

5. C# / Говнокод #19354

   +2

   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

   switch (button.Tag.ToString()) {
                   case "Ezp":
                   {
                       if (nowLayer.OpenedFiles.Count == 0)
                       {
                           nowLayer.OpenedFiles = new List<ExemplarFile> { new ExemplarFile("") };
                       }
                       if (nowLayer.OpenedFiles[0].EdinZemleps == null)
                       {
                           nowLayer.OpenedFiles[0].EdinZemleps = new List<EdinZemlep>();
                       }
                       var strName = ":ЕЗ" + (nowLayer.OpenedFiles[0].EdinZemleps.Count + 1);
                       nowLayer.OpenedFiles[0].EdinZemleps.Add(new EdinZemlep(strName) { Status = StatusEnum.Образуемый });
                       _ezpListTreeView.Add(new EzpTreeView { NameEzp = strName, ContourList = new List<ContourTreeView>(), ParcelList = new List<PartZuTreeView>(), IsEzpVisible = true, IsSelectedEzp = false } );
                       _dictionaryEzpTreeViewFile.Add(_ezpListTreeView[_ezpListTreeView.Count - 1], nowLayer.OpenedFiles[0]);
                       _dictionaryEzpTreeViewEdinZemlep.Add(_ezpListTreeView[_ezpListTreeView.Count - 1], nowLayer.OpenedFiles[0].EdinZemleps[nowLayer.OpenedFiles[0].EdinZemleps.Count - 1]);
                       break;
                   }
                   case "ContourEzp":
                   {
                       indexEz = TreeViewStackPanelMain.Children.IndexOf(((button.Parent as Grid).Parent as StackPanel).Parent as Grid) - _partZuListTreeView.Count;
                       var strName = (nowLayer.OpenedFiles[0].EdinZemleps[indexEz].CadNum != "" ? nowLayer.OpenedFiles[0].EdinZemleps[indexEz].CadNum : nowLayer.OpenedFiles[0].EdinZemleps[indexEz].LastName) + "(" + (nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Contours.Count + 1) + ")";
                       nowLayer.OpenedFiles[0].EdinZemleps[indexEz].AddContour(new Contour(strName) { Status = StatusEnum.Образуемый });
                       if (_ezpListTreeView[indexEz].ContourList == null)
                       {
                           _ezpListTreeView[indexEz].ContourList = new List<ContourTreeView>();
                       }
                       strName = nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Contours.Last().Name;
                       _ezpListTreeView[indexEz].ContourList.Add(new ContourTreeView { NameContour = strName, IsContourVisible = true, IsSelectedContour = false } );
                       for (var i = 0; i < _ezpListTreeView[indexEz].ContourList.Count; i++)
                       {
                           _ezpListTreeView[indexEz].ContourList[i].NameContour = nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Contours[i].Name;
                       }
                       break;
                   }
                   case "ZuEzp":
                   {
                       indexEz = TreeViewStackPanelMain.Children.IndexOf(((button.Parent as Grid).Parent as StackPanel).Parent as Grid) - _partZuListTreeView.Count;
                       var strName = ":ЗУ" + (nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels.Count + 1);
                       if (_ezpListTreeView[indexEz].ParcelList == null)
                       {
                           _ezpListTreeView[indexEz].ParcelList = new List<PartZuTreeView>();
                       }
                       nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels.Add(new Parcel(strName) { Status = StatusEnum.Образуемый });
                       _ezpListTreeView[indexEz].ParcelList.Add(new PartZuTreeView { NamePartZu = strName, ContourList = new List<ContourTreeView>(), PartList = new List<PartTreeView>(), IsPartZuVisible = true, IsSelectedPartZu = false });
                       _dictionaryPartZuTreeViewFile.Add(_ezpListTreeView[indexEz].ParcelList[_ezpListTreeView[indexEz].ParcelList.Count - 1], nowLayer.OpenedFiles[0]);
                       _dictionaryPartZuTreeViewParcel.Add(_ezpListTreeView[indexEz].ParcelList[_ezpListTreeView[indexEz].ParcelList.Count - 1], nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels.Count - 1]);
                       break;
                   }              
                   case "PartZuEzp":
                   {
                       indexEz = TreeViewStackPanelMain.Children.IndexOf(((((((button.Parent as Grid).Parent as StackPanel).Parent as Grid).Parent as StackPanel).Parent as Grid).Parent as StackPanel).Parent as Grid) - _partZuListTreeView.Count;
                       indexZu = ((((((button.Parent as Grid).Parent as StackPanel).Parent as Grid).Parent as StackPanel).Parent as Grid).Parent as StackPanel).Children.IndexOf((((((button.Parent as Grid).Parent as StackPanel).Parent as Grid).Parent as StackPanel).Parent as Grid));
                       var sbPrclName = nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[indexZu].LastName + "/чзу" + (nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[indexZu].Subparcels.Count + 1);
                       nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[indexZu].Subparcels.Add(new Subparcel(sbPrclName) { Status = StatusEnum.Образуемый });
                       if (_ezpListTreeView[indexEz].ParcelList[indexZu].PartList == null)
                       {
                           _ezpListTreeView[indexEz].ParcelList[indexZu].PartList = new List<PartTreeView>();
                       }
                       _ezpListTreeView[indexEz].ParcelList[indexZu].PartList.Add(new PartTreeView { NamePart = sbPrclName, ContourList = new List<ContourTreeView>(), IsPartVisible = true, IsSelectedPart = _ezpListTreeView[indexEz].ParcelList[indexZu].IsSelectedPartZu });
                       _dictionaryPartTreeViewSubparcel.Add(_ezpListTreeView[indexEz].ParcelList[indexZu].PartList[_ezpListTreeView[indexEz].ParcelList[indexZu].PartList.Count - 1], nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[indexZu].Subparcels[nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[indexZu].Subparcels.Count - 1]);
                       break;
                   }

   и это продолжается, продолжается и продолжается....

   deathguard771, 27 Января 2016

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

6. C# / Говнокод #19306

   +1

   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
   95. 95
   96. 96
   97. 97
   98. 98
   99. 99

   namespace SmalltalkPHP
   {
   	class Model
   	{
   		internal static void CreateClass(string className)
   		{
   			ClassObject = new Class(className);
   		}
   
   		public static Class ClassObject { get; set; }
   		public class Message
   		{
   			public class Arguments
   			{
   				private SortedList arguments = new SortedList();
   				public void Add(String key, String name)
   				{
   					this.arguments.Add(key, name);
   				}
   				public String AsPhp()
   				{
   					String[] sb = new String[arguments.Count];
   					int i = 0;
   					foreach (DictionaryEntry arg in arguments)
   					{
   						if ((String)(arg.Value) != "") sb[i] = "$" + arg.Value;
   						i++;
   					}
   					return String.Join(", ", sb);
   				}
   				public String MakeFunctionName()
   				{
   					String[] sb = new String[arguments.Count];
   					int i = 0;
   					foreach (DictionaryEntry arg in arguments)
   					{
   						if ((String)(arg.Key) != "") sb[i] = (String)arg.Key;
   						i++;
   					}
   					return String.Join("_", sb);
   				}
   			}
   			public class Generic
   			{
   				public virtual string AsPhp()
   				{
   					return "nya";
   				}
   			}
   			public class Unary : Generic
   			{
   				public Unary(String name, Boolean isStatic = false)
   				{
   					this.Name = name;
   					this.Arguments = new Arguments();
   					this.IsStatic = isStatic;
   				}
   				public string Name { get; set; }
   				public Arguments Arguments { get; set; }
   				public Boolean IsStatic { get; set; }
   				public string PhpHeader
   				{
   					get
   					{
   						return String.Format("public {0}function {1}()", IsStatic ? "static " : "", Name);
   					}
   				}
   
   				public override string AsPhp()
   				{
   					return PhpHeader;
   				}
   			}
   			public class Keyword : Generic
   			{
   				public Keyword(Boolean isStatic = false)
   				{
   					this.Arguments = new Arguments();
   					this.IsStatic = isStatic;
   				}
   				public string Name { get { return Arguments.MakeFunctionName(); } }
   				public Arguments Arguments { get; set; }
   				public Boolean IsStatic { get; set; }
   				public string PhpHeader
   				{
   					get
   					{
   						return String.Format("public {0}function {1}({2})", IsStatic ? "static " : "", Name, Arguments.AsPhp());
   					}
   				}
   
   				public override string AsPhp()
   				{
   					return PhpHeader;
   				}
   			}
   		}
   	}
   }

   Писал конвертер Smalltalk -> PHP
   Где-то еще валяется Smalltalk -> Erlang

   kgm-rj, 14 Января 2016

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

7. C# / Говнокод #19304

   −2

   1. 1
   2. 2
   3. 3

   var indexes = T.Select((item, index) => new { Item = item, Index = index })
                   .Where(nitem => n.Item == 0).Select(nindex => n.Index).ToArray();
   Console.WriteLine("Первый: {0}, последний: {1}", indexes.First(), indexes.Last());

   Не слишком много item index? index item?

   Govno_Coder, 14 Января 2016

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

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

   +8

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

   if(user.pass = pass_to_hash(password)){
          return done(null, user);
   }else{
          return done('incorrect password');
   }

   Факир был пьян, и запер дверь, забыв ее закрыть)

   Darth, 15 Декабря 2015

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

9. PHP / Говнокод #19176

   +1

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

   var cloths = {
     <?php foreach ($clothList as $cloth): ?>
       "<?=$cloth->id()?>": {
         "id": <?=$cloth->id()?>,
         "name": "<?=$cloth->name()?>",
         "public_name": "<?=str_replace('"', '\"', $cloth->public_name())?>"
       },
     <?php endforeach; ?>
     "dummy": {}
   };

   kgm-rj, 14 Декабря 2015

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

10. PHP / Говнокод #19171

    +2

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

    if (!$subscription) {
        return $this->redirect(['action' => 'cancel_success']);
        throw new BadRequestException();
    }

    epadqwbd, 11 Декабря 2015

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

11. JavaScript / Говнокод #19164

    0

    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

    $(window).load(function () {
    
        $(".sidebar-list").mCustomScrollbar({
          // scrollButtons:{enable:true,scrollType:"stepped"},
          keyboard: {
            scrollType: "stepped"
          },
          mouseWheel: {
            scrollAmount: 188
          }, // theme:"rounded-dark",
          scrollbarPosition: "inside",
          autoExpandScrollbar: false,
          snapAmount: 188,
          snapOffset: 65
        });
        if ("onhashchange" in window && location.hash) {
          match = location.hash.match(/^[?#!]*(.*)$/)[1];
          scrollToPage(match);
        };
    
        if (!isMob.any() && isRoot) {
          // return false;
          var navM = $('.menu a');
          var section = $.map($(".section"), function (e) {
            var $e = $(e);
            var pos = $e.position();
            return {
              top: pos.top - 75,
              bottom: pos.top - 75 + $e.height(),
              hash: '#!' + $e.attr('id'),
              indx: $e.index()
            };
          });
          //Checking scroll 
          var top = null;
          var changed = false;
          var currentHash = null;
    
          $(window).scroll(function () {
            var newTop = $(document).scrollTop();
            changed = newTop != top;
            if (changed) {
              top = newTop;
              // step();
            }
          });
          function step() {
            if (!changed) {
              return setTimeout(step, 200);
            }
            var count = section.length;
            var p;
            while (p = section[--count]) {
              if (p.top >= top || p.bottom <= top) {
                continue;
              }
              if (currentHash == p.hash) {
                break;
              }
              var scrollTop = $(document).scrollTop();
              window.location.hash = currentHash = p.hash;
              $(document).scrollTop(scrollTop);
            }
            setTimeout(step, 200);
          }
          setTimeout(step, 200);
        }
      });

    Сей гениальный код должен управлять прокруткой и главным меню на сайте.

    moveax3, 10 Декабря 2015

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

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

• 1974
• …
• 309
• 308
• 307
• 306
• 305
• 304
• 303
• 302
• 301
• …
• 1