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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2339)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   +110

   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

   private void Save(string ThreadID, string Board)
           {
               string pathL;
               if (cbGIF.Checked)
               {
                   pathL = String.Format(path, Board, ThreadID, "-gif");
               }
               else
               {
                   pathL = String.Format(path, Board, ThreadID, "");
               }
               string htmlPath = String.Format(threadPath, Board, ThreadID);
   
               WebClient wcli = new GZipWebClient();
               string cThread = wcli.DownloadString(htmlPath);
               
               var rx = new Regex(cbGIF.Checked ? regExGif : regEx);
               var ms = rx.Matches(cThread);
               imgSaved = 0;
               imgCount = ms.Count;
               saveProgress.Minimum = 0;
               saveProgress.Maximum = imgCount;
               saveProgress.Value = 0;
               if (!Directory.Exists(pathL))
               {
                   Directory.CreateDirectory(pathL);
               }
               try
               {
                   foreach (Match m in ms)
                   {
                       WebClient ccl = new WebClient();
                       ccl.DownloadFileCompleted += new AsyncCompletedEventHandler(ccl_DownloadFileCompleted);
                       string[] v = m.Value.Split('"');
                       string sd = v[1].Split('/').Last();
                       string a = url + v[1];
                       string b = pathL + sd;
                       if (File.Exists(b))
                       {
                           imgSaved++;
                           saveProgress.Value = imgSaved;
                           lblSaveProgress.Text = imgSaved.ToString() + "/" + imgCount.ToString();
                           if (imgSaved == imgCount)
                           {
                               btnSave.Enabled = true;
                               lblSaveProgress.Text = "FUKKEN SAVED!";
                           }
                       }
                       else ccl.DownloadFileAsync(new Uri(a),b);
                   }
               }
               catch (WebException e)
               {
                   MessageBox.Show(e.Message + e.StackTrace);
               }          
           }

   В пределах одного метода бросаемся из крайности в крайность в именовании переменных.

   A1mighty, 26 Июля 2013

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

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

   +120

   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

   if (IsSetPrinter)
                           {
                               if (Counter == 0)
                               {
                                   printingIsOkay = PrintDocument(letterIDsString
                                  , row.DocumentTemplateVersionID
                                  , row.DocumentTemplate
                                  , (row.IsHeader_DocumentTemplateVersionIDNull() ? (int?)null : row.Header_DocumentTemplateVersionID)
                                  , (row.IsFooter_DocumentTemplateVersionIDNull() ? (int?)null : row.Footer_DocumentTemplateVersionID)
                                  , true
                                  , Counter
                                  , Counter == documentTemplateVersions.Count - 1);
                               }
                               else
                               {
                                   printingIsOkay = PrintDocument(letterIDsString
                                 , row.DocumentTemplateVersionID
                                 , row.DocumentTemplate
                                 , (row.IsHeader_DocumentTemplateVersionIDNull() ? (int?)null : row.Header_DocumentTemplateVersionID)
                                 , (row.IsFooter_DocumentTemplateVersionIDNull() ? (int?)null : row.Footer_DocumentTemplateVersionID)
                                 , false
                                 , Counter
                                 , Counter == documentTemplateVersions.Count - 1);
                               }
   
                           }
                           else
                           {
                               printingIsOkay = PrintDocument(letterIDsString
                                  , row.DocumentTemplateVersionID
                                  , row.DocumentTemplate
                                  , (row.IsHeader_DocumentTemplateVersionIDNull() ? (int?)null : row.Header_DocumentTemplateVersionID)
                                  , (row.IsFooter_DocumentTemplateVersionIDNull() ? (int?)null : row.Footer_DocumentTemplateVersionID)
                                  , false
                                  , Counter
                                  , Counter == documentTemplateVersions.Count - 1);
                           }

   Я бы выложил весь класс. Он весь такой. Но, думаю, вы поняли кто писал?

   kore_sar, 23 Июля 2013

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

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

   +115

   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

   private string GenerateUserName(ISession session)
   {
     string userName = "";
     while (true)
     {
       if (IsBrand) userName = FName.Replace(" ", "");
       else userName = string.Format("{0}.{1}", FName, LName);
   
       if (String.IsNullOrEmpty(FName) && String.IsNullOrEmpty(LName))
       {
         userName = UserID.ToString();
       }
       else
       {
         var i = 0;
         while (UserBeanHelper.GetUserByUserName(session, userName) != null)
         {
           i++;
           userName = string.Format("{0}.{1}-{2}", FName, LName, i);
         }
       }
   
       UserTransferBean userByUserName = UserBeanHelper.GetUserByUserName(session, userName);
       if (userByUserName != null)
          ;
       else
       {
         break;
       }
     }
     return userName;
   }

   DarkThinker, 19 Июля 2013

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

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

   +124

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

   class Comparer : IComparer<int>
   {
       Random random = new Random();
   
       public int Compare(int x, int y)
       {
           return 1 - random.Next() % 3;
       }
   }

   Ccik, 18 Июля 2013

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

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

   +125

   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

   var colorInfoList = new List<ColorInfo>()
                   {
                       new ColorInfo("#000000", "	black", "	Черный"),
                       new ColorInfo("#000080","	navy","	Темно-синий"),
                       new ColorInfo("#00008B","	darkblue","	Темно-голубой"),
                       new ColorInfo("#0000CD","	mediumblue","	Умеренно-голубой"),
                       new ColorInfo("#0000FF","	blue	","Голубой"),
                       new ColorInfo("#006400","	darkgreen","	Темно-зеленый"),
                       new ColorInfo("#008000","	green","	Зеленый"),
                       new ColorInfo("#008080","	teal","	Чайный"),
                       new ColorInfo("#008B8B","	darkcyan","	Темный циан"),
                       new ColorInfo("#00BFFF","	deepskyblue","	Темный небесно-синий"),
                       new ColorInfo("#00CED1","darkturquoise	","Темно-бирюзовый"),
                       new ColorInfo("#00FA9A","	mediumspringgreen","	Умеренный синевато-серый"),
                       new ColorInfo("#00FF00","	lime","	Известковый"),
                       new ColorInfo("#00FFFF","	aqua","	Синий"),
                       new ColorInfo("#2E8B57","	seagreen","	Морской волны"),
                       new ColorInfo("#32CD32","	limegreen","	Зеленовато-известковый"),
                       new ColorInfo("#4B0082","	indigo","	Индиго"),
                       new ColorInfo("#696969","	dimgray","	Тускло-серый"),
                       new ColorInfo("#7CFC00","	lawngreen","	Зеленой травы"),    
                       new ColorInfo("#7FFFD4","	aquamarine","	Аквамарин"),
                       new ColorInfo("#800000","	maroon","	Оранжево-розовый"),
                       new ColorInfo("#808080"," 	gray","	Серый"),
                       new ColorInfo("#8B0000","	darkred","	Темно-красный"),
                       new ColorInfo("#9ACD32","	yellowgreen","	Желто-зеленый"),
                       new ColorInfo("#A52A2A","	brown","	Коричневый")
                   };
   
   
           public ColorInfo ColorCheck(List<ColorInfo> list, string colorHex)
           {
               for (int i = 0; i < list.Count-1; i++)
               {
                   if (String.Compare(list[i].HexInfo, colorHex, StringComparison.Ordinal) <= 0 && String.Compare(list[i + 1].HexInfo, colorHex, StringComparison.Ordinal)>=0)
                       return list[i+1];
               }
               return new ColorInfo();
           }

   Определение названия цвета

   iofjuupasli, 17 Июля 2013

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

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

   +121

   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

   /// Разрезать строку на части длиной не более maxRecordLen
     private static void splitText(ref List<string> splitTextArray,  string text) {
       splitTextArray = new List<string>();
       for (; ; ) {
         try {
           string str = text.Substring(0, maxRecordLen);
           splitTextArray.Add(str);
           try {
             text = text.Substring(maxRecordLen);
           }
           catch (Exception) {
             break;
           }
         }
         catch (Exception) {
           splitTextArray.Add(text);
           break;
         }
       }
     }

   no comment.

   diimdeep , 17 Июля 2013

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

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

   +129

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

   public class DataLayer
   {
   ...
   public List<Employee> GetEmployees() { ... }
   public List<Department> GetDepartments() {...}
   public List<Roles> GetRoles() {...}
   }

   Обратил внимание, что некоторые коллеги любят использовать в качестве возвращаемых типов не обычные массивы, а обязательно List<T>.
   Долго гадал, что-ж такая за практика интерсная, на стеке прям несколько вопросов подрял.
   Затем, обратил внимание, что все они используют в качестве основного инструмента паттерн MVC.
   Проштудировав самые известные книжки по MVC, таки нашёл виновника:
   http://www.ozon.ru/context/detail/id/19064535/ - Программирование на основе Microsoft ASP.NET MVC (Дино Эспозито)
   Везде где только можно, всё просто обделано LIst'ами. Даже данные передаваемые во View...

   TauSigma, 15 Июля 2013

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

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

   +135

   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

   switch (curWpn)
   {
       case WeaponType.Doublegun: DrawNumber(doublegunAmmo, new Vector2(graphics.PreferredBackBufferWidth - 150, graphics.PreferredBackBufferHeight - 54), 1.0f, doublegunAmmo > 0 ? Color.White : Color.Red); spriteBatch.Draw(doublegunI, new Microsoft.Xna.Framework.Rectangle(graphics.PreferredBackBufferWidth - 64, graphics.PreferredBackBufferHeight - 64, 64, 64), Color.White); break;
       case WeaponType.Grenade: DrawNumber(grenadeCnt, new Vector2(graphics.PreferredBackBufferWidth - 150, graphics.PreferredBackBufferHeight - 54), 1.0f, grenadeCnt > 0 ? Color.White : Color.Red); spriteBatch.Draw(grenadeI, new Microsoft.Xna.Framework.Rectangle(graphics.PreferredBackBufferWidth - 64, graphics.PreferredBackBufferHeight - 64, 64, 64), grenadeTimeout > 0.0f ? Color.Red : Color.White); break;
       case WeaponType.Knife: spriteBatch.Draw(knifeI, new Microsoft.Xna.Framework.Rectangle(graphics.PreferredBackBufferWidth - 64, graphics.PreferredBackBufferHeight - 64, 64, 64), Color.White); break;
       case WeaponType.Launcher: DrawNumber(rocketCnt, new Vector2(graphics.PreferredBackBufferWidth - 150, graphics.PreferredBackBufferHeight - 54), 1.0f, rocketCnt > 0 ? Color.White : Color.Red); spriteBatch.Draw(launcherI, new Microsoft.Xna.Framework.Rectangle(graphics.PreferredBackBufferWidth - 64, graphics.PreferredBackBufferHeight - 64, 64, 64), launchTimeout > 0.0f ? Color.Red : Color.White); break;
       case WeaponType.Revolver: spriteBatch.Draw(revolverI, new Microsoft.Xna.Framework.Rectangle(graphics.PreferredBackBufferWidth - 64, graphics.PreferredBackBufferHeight - 64, 64, 64), Color.White); break;
       case WeaponType.Cannon: DrawNumber(cannonAmmo, new Vector2(graphics.PreferredBackBufferWidth - 150, graphics.PreferredBackBufferHeight - 54), 1.0f, cannonAmmo > 0 ? Color.White : Color.Red); spriteBatch.Draw(cannonI, new Microsoft.Xna.Framework.Rectangle(graphics.PreferredBackBufferWidth - 64, graphics.PreferredBackBufferHeight - 64, 64, 64), cannonTimeout > 0.0f ? Color.Red : Color.White); break;
       case WeaponType.Laser: DrawNumber(laserAmmoCnt, new Vector2(graphics.PreferredBackBufferWidth - 150, graphics.PreferredBackBufferHeight - 54), 1.0f, laserAmmoCnt > 0 ? Color.White : Color.Red); break;
       case WeaponType.Minigun: DrawNumber(minigunAmmoCnt, new Vector2(graphics.PreferredBackBufferWidth - 150, graphics.PreferredBackBufferHeight - 54), 1.0f, minigunAmmoCnt > 0 ? Color.White : Color.Red); break;
   }
   //в другом месте
   switch (curWpnIndex)
   {
       case 0: curWpn = WeaponType.Knife; break;
       case 1: curWpn = WeaponType.Revolver; break;
       case 2: curWpn = WeaponType.Doublegun; break;
       case 3: curWpn = WeaponType.Laser; break;
       case 4: curWpn = WeaponType.Minigun; break;
       case 5: curWpn = WeaponType.Rifle; break;
       case 6: curWpn = WeaponType.Grenade; break;
       case 7: curWpn = WeaponType.Launcher; break;
       case 8: curWpn = WeaponType.Cannon; break;
   }

   Это неописуемо. Нашел у себя...
   Видимо, придется снова идти мыться.

   RaZeR, 15 Июля 2013

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

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

   +128

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

   public void BuildInsertClause(OleDbCommand cmd, ObjectState objState)
           {
                 StringBuilder builder = new StringBuilder();
                 ..........
                 cmd.CommandText = builder.ToString() + "(" + columns.ToString() + ") VALUES (" +
                 values.ToString() + ")";
           }

   http://solidcoding.blogspot.ru/2008/01/linq-to-excel-provider-25.html
   Еще много смешного, для затравки:
   

   object val = reader[col.GetSelectColumn()];
   
        if (val is DBNull)
   
            {
   
                 val = null;
   
            }

   neeedle, 15 Июля 2013

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

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

    +135

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

    Byte[] content = setting.RecipientType == ReportRecipientType.A?
                            Processor.Process(Encoding.ASCII.GetBytes(report.Value),"A",String.Empty) :
                            setting.RecipientType == ReportRecipientType.B || setting.RecipientType == ReportRecipientType.BX?
                            Processor.Process (Encoding.ASCII.GetBytes(report.Value), "B", String.Empty) :
                             setting.RecipientType == ReportRecipientType.C?
                             Processor.Process (Encoding.UTF8.GetBytes(report.Value), "C", String.Empty) :
                             setting.RecipientType == ReportRecipientType.D?
                             Processor.Process (Encoding.UTF8.GetBytes(report.Value), "D", String.Empty) :
                            Processor.Process (Encoding.UTF8.GetBytes(report.Value), filename, DateTime.Now, setting.data);

    Имена переменных и методов обфусцированы, но суть не в них.
    Какой сумрачный гений родил такую управляющую структуру?

    startrack, 11 Июля 2013

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

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

• 193
• …
• 123
• 122
• 121
• 120
• 119
• 118
• 117
• 116
• 115
• …
• 1