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

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

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

1. PHP (5700)
2. C++ (2734)
3. Куча (2331)
4. JavaScript (2028)
5. C# (1926)
6. Java (1501)
7. Си (1097)
8. Pascal (647)
9. Python (582)
10. 1C (533)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (29)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. C# / Говнокод #5794

   +139

   1. 1

   Response.Write("<" + '/'.ToString() + "script>");

   Даже не знаю, кому и зачем это понадобилось...

   Ordos, 24 Февраля 2011

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

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

   +106

   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

   public static bool KeyToString(Keys key, bool shitKeyPressed,
                                                                       out char character)
           {
               bool result = false;
               character = ' ';
               CharPair charPair;
   
               if ((Keys.A <= key && key <= Keys.Z) || key == Keys.Space)
               {
                   // Use as is if it is A～Z, or Space key.
                   character = (shitKeyPressed) ? (char)key : Char.ToLower((char)key);
                   result = true;
               }
               else if (keyMap.TryGetValue(key, out charPair))
               {
                   // Otherwise, convert by key map.
                   if (!shitKeyPressed)
                   {
                       character = charPair.NormalChar;
                       result = true;
                   }
                   else if (charPair.ShiftChar.HasValue)
                   {
                       character = charPair.ShiftChar.Value;
                       result = true;
                   }
               }
   
               return result;
           }

   Разбираю код, скачанный с майкросовтовского App Hub. В коде и комментариях полно маленьких грамматический ошибок, но это...
   Код выложен на полном серьёзе и качается тут http://create.msdn.com/en-US/education/catalog/utility/performance_utility
   
   P.S. для тупых спойлер: вместо shift написано shit.

   Skelet0n, 23 Февраля 2011

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

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

   +115

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

   var model = Service.GetPlanGatheringIndicators(periodId).ToList();
   ViewData["PeriodName"] = Service.GetPeriods().Where(p => p.Id == periodId).Select(p => p.TextAlias).FirstOrDefault();
   var leafEls = model.Where(x => x.HasChildren == false);
   var leafEl = (leafEls.Count() > 0) ? leafEls.First() : null;
   ViewData["PlanActivated"] = ((leafEl != null) && (leafEl.PlanActivate == true)
   		                             && (leafEl.PredictActivate == true)
   		                             && (leafEl.FactActivate == true));

   Вот в таком чудесном виде и лежит, ожидая рефактора и форматирования.

   fr0mrus, 22 Февраля 2011

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

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

   +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

   public static class EventExtensions
       {
           public static void Fire(this EventHandler eventHandler, object sender)
           {
               eventHandler.Fire(sender, EventArgs.Empty);
           }
   
           public static void Fire(this EventHandler eventHandler, object sender, EventArgs args)
           {
               eventHandler.FireDelegate(sender, args);
           }
   
           public static void Fire<TEA>(this EventHandler<TEA> eventHandler, object sender, TEA args)
               where TEA : EventArgs
           {
               eventHandler .FireDelegate(sender, args);
           }
   
           public static void FireDelegate(this Delegate eventHandler, params object[] parameters)
           {
               if (eventHandler != null)
               {
                   Delegate[] delegates = eventHandler.GetInvocationList();
                   foreach (var del in delegates)
                   {
                       try
                       {
                           del.Method.Invoke(del.Target, parameters);
                       }
                       catch (Exception e)
                       {
                           // use logger here
                           throw;
                       }
                   }
               }
           }
       }

   Надоело писать инвокаторы для ивентов? Как тебе это?

   walash, 21 Февраля 2011

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

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

   +124

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

   private void image_hide_show(object sender, EventArgs e)
           {
               for(int i = 0; i < Controls.Count; i++)
                   if ((sender as PictureBox).Name + "_img" == Controls[i].Name) Controls[i].Visible = !Controls[i].Visible;
           }

   Мартин, 20 Февраля 2011

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

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

   +146

   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

   using System;
   
   class Popovkin
   {
       static void Main()
       {
           Console.ForegroundColor = ConsoleColor.DarkRed;
           Console.BackgroundColor = ConsoleColor.Gray;
           Console.WriteLine("Привет, баребухи! Я Ромик Будин!");
           Console.WriteLine("Уже " + DateTime.Now);
           Console.WriteLine("Нажмите Ентер!");
           Console.ReadLine();
           Console.ForegroundColor = ConsoleColor.Green;
           Console.BackgroundColor = ConsoleColor.Black;
           Console.Clear();
           for (int e = 1; e < 1200000; e++)
           {
               Console.ForegroundColor = ConsoleColor.Blue;
               Console.BackgroundColor = ConsoleColor.White;
               Console.Write("Hello Popovkin!!!");
               Console.Write("Hello Popovkin!!!");
               Console.Write("Hello Popovkin!!!");
               Console.Write("Hello Popovkin!!!");
               Console.Write("Hello Popovkin!!!");
               Console.Write("Hello Popovkin!!!");
               Console.Write("Hello Popovkin!!!");
               Console.Write("Hello Popovkin!!!");
               Console.Write("Hello Popovkin!!!");
               Console.Write("Hello Popovkin!!!");
               Console.Write("Hello Popovkin!!!");
               Console.Write("Hello Popovkin!!!");
               for (int i = 0; i < 2; i++)
               {
                   Console.ForegroundColor = ConsoleColor.Green;
                   Console.BackgroundColor = ConsoleColor.Black;
                   Console.Write("Hello Popovkin!!!");
                   Console.Write("Hello Popovkin!!!");
                   Console.Write("Hello Popovkin!!!");
                   Console.Write("Hello Popovkin!!!");
                   Console.Write("Hello Popovkin!!!");
                   Console.Write("Hello Popovkin!!!");
                   Console.Write("Hello Popovkin!!!");
                   Console.Write("Hello Popovkin!!!");
                   Console.Write("Hello Popovkin!!!");
                   Console.Write("Hello Popovkin!!!");
                   Console.Write("Hello Popovkin!!!");
                   Console.Write("Hello Popovkin!!!");
                   for (int q = 0; q < 2; q++)
                   {
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.Write("Hello Popovkin!!!");
                       Console.WriteLine("Hello Popovkin!!!");
                       Console.ForegroundColor = ConsoleColor.Blue;
                       Console.BackgroundColor = ConsoleColor.Red;
                   }
               }
           }
           Console.ReadLine();
       }
   }

   111111, 19 Февраля 2011

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

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

   +123

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

   string numberAsString = number.ToString();
   
       foreach(char x in numberAsString)
       {
           string y = x.ToString();
           int z = int.Parse(y);
           Console.WriteLine(z*z);
       }

   qbasic, 17 Февраля 2011

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

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

   +121

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

   [AcceptVerbs(HttpVerbs.Post)]
   public ActionResult SaveLastReport(string gridHtml)
   {
   	var buf = Convert.FromBase64String(gridHtml);
   	var str = System.Text.Encoding.UTF8.GetString(buf, 0, buf.Length);
   	System.IO.File.WriteAllText(Server.MapPath("~/content/lastreport.html"), str);
   	return RedirectToAction("LastReport");
   }

   Вот так иногда закрываются формальные требования по ТЗ...

   St0rm, 16 Февраля 2011

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

9. C# / Говнокод #5697

   +109

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

   try
               {
                   splash.Close(new TimeSpan(0, 0, 0, 3, 0));
               }
               catch (Exception ex) { System.Diagnostics.Debug.WriteIf(false, ex.ToString()); }

   Вот так вот эксепшн обработал. Помню, недавно сказал такую вещь "В нашем игровом движке содержится лучший код, который я когда-либо писал" про свой игровой движок. Казалось бы, удобная имплементация, структура и принципы ООП правильно используются. Но потом я увидел ЭТО (см. выше). Надо больше спать, видимо... %)

   RaZeR, 16 Февраля 2011

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

10. C# / Говнокод #5696

    +115

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

    public bool SameTerritoryAsEditor(string terrName)
    {
       return _personService.GetTerritoryByUserIdentity(Thread.CurrentPrincipal.Identity.Name).Name.Equals(terrName);
    }

    Определяем принадлежность пользователя к определенной территории по её названию.

    St0rm, 16 Февраля 2011

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

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

• 193
• …
• 61
• 60
• 59
• 58
• 57
• 56
• 55
• 54
• 53
• …
• 1