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

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

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

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

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

   +120

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

   string tmpS;
   //идут всякие монипуляции с этой строкой
   //. . .
   tmpS.Remove(0, tmpS.Length);

   Реальный код из коммерческого продукта.

   inser, 04 Марта 2011

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

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

   +126

   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

   private const string constDefFeedLimitValue = "5";
   private string feedLimit = constDefFeedLimitValue;
   private int feedLmt;
   
   protected override void RenderContents(System.Web.UI.HtmlTextWriter writer)
           {
               try
               {
                   feedLmt = Convert.ToInt32(feedLimit);
               }
               catch (Exception)
               {
                   feedLmt = 0;
               }
               ...
           }

   Автор из Киева, имеет статус MVP.

   Alexander, 03 Марта 2011

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

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

   +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

   Microsoft.Win32.RegistryKey Fregistry =
                   Microsoft.Win32.Registry.LocalMachine.OpenSubKey("SOFTWARE")
                   .OpenSubKey("Microsoft").OpenSubKey("Windows").OpenSubKey("CurrentVersion")
                   .OpenSubKey("Installer").OpenSubKey("UserData")
                   .OpenSubKey("S-1-5-18").OpenSubKey("Products");
               string []Names = Fregistry.GetSubKeyNames() ;
               string uninstall = "";
               string ApplicationName = "Adobe Reader 7.0.8";
               for (int i = 0; i < Names.Length; i++)
               {
                   Microsoft.Win32.RegistryKey FTemp = Fregistry.OpenSubKey(Names[i]).OpenSubKey("InstallProperties");
                   if (FTemp.GetValue("DisplayName").ToString() == ApplicationName)
                   {
                       object obj = FTemp.GetValue("UninstallString");
                       if (obj == null)
                           uninstall = "";
                       else
                           uninstall = obj.ToString();
                       i = Names.Length; 
                   }
               }
    
               System.Console.WriteLine(uninstall);
               System.Diagnostics.Process FProcess = new System.Diagnostics.Process();
               string temp = "/x{" + uninstall.Split("/".ToCharArray())[1].Split("I{".ToCharArray())[2];
               FProcess.StartInfo.FileName = uninstall.Split("/".ToCharArray())[0];
               FProcess.StartInfo.Arguments = temp;
               FProcess.StartInfo.UseShellExecute = false;
               FProcess.Start();
               System.Console.Read();

   qbasic, 03 Марта 2011

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

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

   +131

   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

   static public int IIF(bool condition, int a, int b)
           {
               int x = 0;
               if (condition)
               {
                   x = a;
               }
               else
               {
                   x = b;
               }
               return x;
           }
   
           static public bool IIF(bool condition, bool a, bool b)
           {
               bool x = false;
               if (condition)
               {
                   x = a;
               }
               else
               {
                   x = b;
               }
               return x;
           }
   
           static public Single IIF(bool condition, Single a, Single b)
           {
               float x = 0;
               if (condition)
               {
                   x = a;
               }
               else
               {
                   x = b;
               }
               return x;
           }
   
           static public Double IIF(bool condition, double a, double b)
           {
               double x = 0;
               if (condition)
               {
                   x = a;
               }
               else
               {
                   x = b;
               }
               return x;
           }
   
           static public decimal IIF(bool condition, decimal a, decimal b)
           {
               decimal x = 0;
               if (condition)
               {
                   x = a;
               }
               else
               {
                   x = b;
               }
               return x;
           }

   inser, 03 Марта 2011

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

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

   +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

   case enter:
   						{
   							TreeNodeBackColorChange();
   							if (Connection.Login == "" || Connection.Login == null)
   							{
   								new fmlogin().ShowDialog();
   								try
   								{
   									if (Connection.Login != "")
   										foreach (TreeNode item in tvMenuList.Nodes)
   										{
   											if (item.Name == lk)
   											{
   												item.NodeFont = new Font("arial", 10, FontStyle.Bold);
   												item.Text += " (" + Connection.Login + ")";
   											}
   										}
   								}
   								catch { }
   							}
   							else MessageBox.Show("Вы уже авторизованы!", "Вход в личный кабинет", MessageBoxButtons.OK, MessageBoxIcon.Warning);
   							break;
   						}
   					case leave:
   						{
   							TreeNodeBackColorChange();
   							if (Connection.Login != "" && Connection.Login != null)
   							{
   								if (MessageBox.Show("Вы уверены, что хотите выйти?", "Выход", MessageBoxButtons.OKCancel, MessageBoxIcon.Question) == DialogResult.OK)
   								{
   									Connection.Login = "";
   									Connection.Pass = "";
   									try
   									{
   										foreach (TreeNode item in tvMenuList.Nodes)
   										{
   											foreach (TreeNode item2 in item.Nodes)
   											{
   												foreach (TreeNode item3 in item2.Nodes)
   												{
   													if (item3.Name == lk)
   													{
   														item.NodeFont = tvMenuList.Font;
   														item3.Text = "Личный кабинет";
   													}
   												}
   												if (item2.Name == lk)
   												{
   													item.NodeFont = tvMenuList.Font;
   													item2.Text = "Личный кабинет";
   												}
   											}
   											if (item.Name == lk)
   											{
   												item.NodeFont = tvMenuList.Font;
   												item.Text = "Личный кабинет";
   											}
   										}
   									}
   									catch { }
   									MessageBox.Show("Выход произведен успешно!", "Выход", MessageBoxButtons.OK, MessageBoxIcon.Information);
   								}
   							}
   							else MessageBox.Show("Вы не авторизованы!", "Выход из личного кабинета", MessageBoxButtons.OK, MessageBoxIcon.Warning);
   							break;
   						}

   извиняюсь) форматирование сбивается когда из студии вставляю)
   PS жалко что нельзя вставить больше 100 строк. А тут такие красивые функции есть, которые теряют всю свою зрелишность при их урезании

   slavenin, 02 Марта 2011

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

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

   +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

   if (rbNewPackage.Checked)
                   {
                       var ls = from ent in Program.DB.UserInfos
                                where
                                (from rp in Program.DB.ReceivedPackages
                                 join r in Program.DB.RecordBooks on rp.id equals r.id_package
                                 where rp.id_user == ent.id &&
                                 r.date == (from recb in Program.DB.RecordBooks
                                            join recp in Program.DB.ReceivedPackages on recb.id_package equals recp.id
                                            where recp.id_user == ent.id
                                            select recb.date).Max() && r.id_status == 1
                                 select rp).Count() != 0
                                select ent;
                       grdEntrantList.DataSource = Program.DB.UserInfos.Where(t => ls.ToList().Contains(t));
                   }
                   else if (rbCancelled.Checked)
                   {
                       var ls = from ent in Program.DB.UserInfos
                                where
                                (from rp in Program.DB.ReceivedPackages
                                 join r in Program.DB.RecordBooks on rp.id equals r.id_package
                                 where rp.id_user == ent.id &&
                                 r.date == (from recb in Program.DB.RecordBooks
                                            join recp in Program.DB.ReceivedPackages on recb.id_package equals recp.id
                                            where recp.id_user == ent.id
                                            select recb.date).Max() && r.id_status == 5
                                 select rp).Count() != 0
                                select ent;
                       grdEntrantList.DataSource = Program.DB.UserInfos.Where(t => ls.ToList().Contains(t));
                   }
                   else if (rbConditional.Checked)
                   {
                       var ls = from ent in Program.DB.UserInfos
                                where
                                (from rp in Program.DB.ReceivedPackages
                                 join r in Program.DB.RecordBooks on rp.id equals r.id_package
                                 where rp.id_user == ent.id &&
                                 r.date == (from recb in Program.DB.RecordBooks
                                            join recp in Program.DB.ReceivedPackages on recb.id_package equals recp.id
                                            where recp.id == rp.id
                                            select recb.date).Max() && r.id_status == 3
                                 select rp).Count() != 0
                                select ent;
                       grdEntrantList.DataSource = Program.DB.UserInfos.Where(t => ls.ToList().Contains(t));
                   }
                   else if (rbOfficial.Checked)
                   {
                       var ls = from ent in Program.DB.UserInfos
                                where
                                (from rp in Program.DB.ReceivedPackages
                                 join r in Program.DB.RecordBooks on rp.id equals r.id_package
                                 where rp.id_user == ent.id &&
                                 r.date == (from recb in Program.DB.RecordBooks
                                            join recp in Program.DB.ReceivedPackages on recb.id_package equals recp.id
                                            where recp.id == rp.id
                                            select recb.date).Max() && r.id_status == 4
                                 select rp).Count() != 0
                                select ent;
                       grdEntrantList.DataSource = Program.DB.UserInfos.Where(t => ls.ToList().Contains(t));
                   }

   говногод моего бывшего коллеги по работе))

   slavenin, 02 Марта 2011

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

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

   +118

   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

   foreach (var aiett in _selectApplItemEntranceTrial)
               {
               lbl2:
                   ReceptionReason_EducationLayout rrel_temp = null;
                   EntranceTrialItem_RecResEducLayout eti_rrel = null;
                   bool flag = false;
                   foreach (var rrel in _recReasonEducLay)
                       foreach (var ap in aiett.ListAppItem)
                           foreach (var rr in rrel.ListRecReas_EducLayout)
                               if (ap.ApplItem.Id_ReceptionReason_EducationLayout == rr.Id)
                               {
                                   rrel_temp = rr;
                                   eti_rrel = rrel;
                                   flag = true;
                                   goto lbl;
                               }
               lbl: ;
                   if (flag)
                   {
                       _recReasonEducLay.FirstOrDefault(x => x == eti_rrel).ListRecReas_EducLayout.Remove(rrel_temp);
                       if (_recReasonEducLay.FirstOrDefault(x => x == eti_rrel).ListRecReas_EducLayout.Count == 0)
                           _recReasonEducLay.Remove(eti_rrel);
                       goto lbl2;
                   }
               }

   Мой личный говнокод! Работающий правильно))))

   slavenin, 02 Марта 2011

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

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

   +124

   1. 1

   while (!Pr.HasExited) System.Threading.Thread.Sleep(250);

   Ожидание завершения запущенного процесса

   Мартин, 02 Марта 2011

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

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

   +115

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

   RepaymentEvent e = DatabaseHelper.GetString("event_type", pReader) == "RBLE"
   	                ? new BadLoanRepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
   	                : (DatabaseHelper.GetString("event_type", pReader) == "RRLE"
   	                       ? new RescheduledLoanRepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
   	                       :(DatabaseHelper.GetString("event_type", pReader).StartsWith("P") 
                                   ? new PendingRepaymentEvent (DatabaseHelper.GetString("event_type", pReader)) {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
                                   : new RepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}));

   Вот такую "элегантную" строчку нашел сегодня коллега в коде нашего проекта :)
   Создаем событие пойди разбери какое :)

   _Ru55_, 02 Марта 2011

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

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

    +112

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

    DirectoryInfo dir = new DirectoryInfo(@"C:\Users\origLocation\");
    DirectoryInfo destDir = new DirectoryInfo(@"C:\Users\origLocation\destLocation\");
    FileInfo[] newVerFiles = dir.GetFiles("*.TXT");
    foreach (FileInfo newVerFile in newVerFiles)
    {
        Regex regex = new Regex("\\d+-new-ver.TXT", RegexOptions.IgnoreCase);
        if (regex.IsMatch(newVerFile.Name))
        {
            newVerFile.MoveTo(destDir.FullName);
        }
     }

    rusco_developer, 02 Марта 2011

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

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

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