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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2338)
4. JavaScript (2030)
5. C# (1929)
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# / Говнокод #6632

   +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

   IList<Hashtable> records = crit.List<Hashtable>();
   Guid[] personsId = records.Select(item => (Guid)item["PersonID"]).Distinct().ToArray();
   List<EmployeeData> empDatas = new List<EmployeeData>();
   
   foreach(Guid personId in personsId) {
   	IEnumerable<Hashtable> employeeRecords = records.Where(item => (Guid)item["PersonID"] == personId);
   	Hashtable employeeRecord = employeeRecords.FirstOrDefault(item => !(bool)item["IsLoad"] || (DateTime)item["EventDate"] == employeeRecords.Max(unit => (DateTime)unit["EventDate"]));
   
   	Hashtable employeeRecordAddition = new GenericNHibernateDao<BaseDocument>().CreateCriteria()
   		.CreateAlias("Department", "department")
   		.CreateAlias("Employee", "employee")
   		.CreateAlias("WorkDescription.Schedule", "schedule", JoinType.LeftOuterJoin)
   		.CreateAlias("WorkDescription.EmployeeApperance", "employeeApperance", JoinType.LeftOuterJoin)
   		.Add(Restrictions.Eq("EmployeeStamp.TabNo", employeeRecord["TabNo"]))
   		.Add(Restrictions.Eq("IsHalf", false))
   	...
   }

   No comments %)

   Guid, 12 Мая 2011

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

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

   +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

   private string GetConnectionString() 
       { 
         string connString = String.Empty; 
         string location = Assembly.GetExecutingAssembly().Location; 
         int pos = location.LastIndexOf('\\'); 
         location = location.Remove(pos); 
         pos = location.LastIndexOf('\\'); 
         location = location.Remove(pos); 
         pos = location.LastIndexOf('\\'); 
         location = location.Remove(pos); 
         location += @"\server\conf\config.conf"; 
         using (StreamReader sr = File.OpenText(location)) 
         { 
           string s = ""; 
           while ((s = sr.ReadLine()) != null) 
           { 
             connString += s; 
           } 
         } 
         connString += "database = ***; charset = utf8;"; 
         return connString; 
       }

   mahalex, 11 Мая 2011

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

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

   +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

   ----Something.cs
   public partial class Something
   {
     // some implementation.
   }
   ----Something.Bla.cs
   partial class Something
   {
     private class Bla
     {
     }
   }
   ----Something.Foo.cs
   partial class Something
   {
     private class Foo
     {
     }
   }
   ----Something.Bar.cs
   partial class Something
   {
     private class Bar
     {
     }
   }

   Нормально ли использовать partial классы исключительно для хранения private nested классов? К примеру если количество таковых доходит до 5-15?

   walash, 11 Мая 2011

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

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

   +116

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

   mainDays = 0;
   
   for (var d = emplDoc.EventDate.AddMonths(1).AddDays(-1).Date; d <= emplDoc.DateEndWork.Date; d = d.AddMonths(1)) {
         mainDays += 2;
   }

   Guid, 11 Мая 2011

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

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

   +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

   /// <summary>
           /// Return a DateTime version of the given Jabber date.  Example date: 20020504T20:39:42
           /// </summary>
           /// <param name="dt">The pseudo-ISO-8601 formatted date (no milliseconds)</param>
           /// <returns>A (usually UTC) DateTime</returns>
           public static DateTime JabberDate(string dt)
           {
               if ((dt == null) || (dt == ""))
                   return DateTime.MinValue;
               try
               {
                   return new DateTime(int.Parse(dt.Substring(0, 4)),
                                       int.Parse(dt.Substring(4, 2)),
                                       int.Parse(dt.Substring(6, 2)),
                                       int.Parse(dt.Substring(9,2)),
                                       int.Parse(dt.Substring(12,2)),
                                       int.Parse(dt.Substring(15,2)));
               }
               catch
               {
                   return DateTime.MinValue;
               }
           }
           /// <summary>
           /// Get a jabber-formated date for the DateTime.   Example date: 20020504T20:39:42
           /// </summary>
           /// <param name="dt">The (usually UTC) DateTime to format</param>
           /// <returns>The pseudo-ISO-8601 formatted date (no milliseconds)</returns>
           public static string JabberDate(DateTime dt)
           {
               return string.Format("{0:yyyy}{0:MM}{0:dd}T{0:HH}:{0:mm}:{0:ss}", dt);
           }

   Перевод DateTime в строку вида "20020504T20:39:42" и обратно. Из исходников библиотеки Jabber-net.
   TryParseExact и ToString с форматом "yyyyMMddTHH:mm:ss" - это пусть лентяи используют.

   Nagg, 09 Мая 2011

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

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

   +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
   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

   public static Bitmap DrawBarsChart(ChartType t, Size s)
       {
         double[] values = DataValues;
         string[] names = DataNames;
         Bitmap bmp = new Bitmap(s.Width, s.Height);
         if (t != ChartType.bars || names.Length != values.Length || names.Length < 2)
           return bmp;
         else
         {
           Graphics g = Graphics.FromImage(bmp as Image);
           g.Clear(Color.White);
           g.DrawLines(new Pen(Brushes.Black), new Point[] { new Point(20, 20), new Point(20, s.Height - 20), new Point(s.Width - 200, s.Height - 20) });
           int Columnwidth = (s.Width - 240) / values.Length;
           if (Columnwidth > 150) Columnwidth = 150;
           double maxvalue = values.Max();
           int counter = 1;
           int rangefirst;
           int rangesecond;
    
           while (true)
           {
             if (maxvalue / Math.Pow(10, counter) < 10)
             {
               rangefirst = (int)Math.Floor(maxvalue / Math.Pow(10, counter));
               rangesecond = (int)(maxvalue - rangefirst * Math.Pow(10, counter));
               break;
             }
             else
             {
               counter++;
             }
           }
           int rangepix = (s.Height - 60) / (rangefirst + 1);
           for (int i = 0; i < rangefirst + 1; i++)
           {
             g.DrawString((i * Math.Pow(10, counter)).ToString(), new Font(FontFamily.GenericSansSerif, 10, FontStyle.Regular),
               Brushes.Black, new PointF(0, s.Height - 30 - rangepix * i));
             g.DrawLine(new Pen(Brushes.Black), new Point(17, s.Height - 20 - rangepix * i), new Point(20, s.Height - 20 - rangepix * i));
           }
           Colors colors = new Colors(); //класс-контейнер цветов (99 штук)
           int startcolor = new Random(DateTime.Now.Millisecond).Next(99);
           int j = startcolor;
           int startx = 21;
           int ColumnNumber = 1;
           foreach (var value in values)
           {
             int curfirstrange = (int)Math.Floor(value / Math.Pow(10, counter));
             int cursecondrange = (int)(value - curfirstrange * Math.Pow(10, counter));
             int rangesmallpix = (int)(cursecondrange * rangepix / Math.Pow(10, counter));
             g.FillRectangle(new SolidBrush(colors.GetNextColor(j)), startx,
               s.Height - 20 - curfirstrange * rangepix - rangesmallpix, Columnwidth, curfirstrange * rangepix + rangesmallpix);
             g.DrawString(value.ToString(), new Font(FontFamily.GenericSerif, 10, FontStyle.Regular), new SolidBrush(Color.Black), new PointF(startx + Columnwidth / 2 - 10,
               s.Height - 20 - curfirstrange * rangepix - rangesmallpix - 20));
             g.DrawString(ColumnNumber.ToString(), new Font(FontFamily.GenericSerif, 10, FontStyle.Regular), new SolidBrush(Color.Black), new PointF(startx + Columnwidth / 2 - 10,
               s.Height - 10));
             ColumnNumber++;
             j++; if (j > 99) j = 0;
             startx += Columnwidth;
           }
           j = startcolor;
           int TopMargin = 20;
           foreach (string str in names)
           {
             string tmp = str;
             if (str.Length > 15)
               tmp = str.Substring(0, 15);
             g.FillRectangle(new SolidBrush(colors.GetNextColor(j)), s.Width - 200, TopMargin, 10, 10); //pucyeM LIBeTHbIe kBagpaTuku
             g.DrawString(tmp, new Font(FontFamily.GenericSerif, 12, FontStyle.Italic), Brushes.Black, new PointF(s.Width - 180, TopMargin - 6)); //pucyeM Hagnucu
             TopMargin += 20;
             j++; if (j > 99) j = 0;
           }
           return bmp;
         }
       }

   qbasic, 06 Мая 2011

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

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

   +127

   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 void bbbut_Click(object sender, EventArgs e)
           {
               if (this.plugDescr.SelectedText.Length > 0)
               {
                   ToolStripButton button = (ToolStripButton) sender;
                   if (button.Name == "bbB")
                   {
                       this.plugDescr.SelectedText = BB_HTMLwork.WriteBBToString(this.plugDescr.SelectedText, "", "b");
                   }
                   if (button.Name == "bbI")
                   {
                       this.plugDescr.SelectedText = BB_HTMLwork.WriteBBToString(this.plugDescr.SelectedText, "", "i");
                   }
                   if (button.Name == "bbU")
                   {
                       this.plugDescr.SelectedText = BB_HTMLwork.WriteBBToString(this.plugDescr.SelectedText, "", "u");
                   }
                   if (button.Name == "bbS")
                   {
                       this.plugDescr.SelectedText = BB_HTMLwork.WriteBBToString(this.plugDescr.SelectedText, "", "s");
                   }
                   if (button.Name == "bbURL")
                   {
                       this.plugDescr.SelectedText = BB_HTMLwork.WriteBBToString(this.plugDescr.SelectedText, this.bbAddInfoContent.Text, "url");
                   }
               }
           }

   Обработчик кнопок ббкода в утилите для генерации ридми. Весь ее код выполнем в таком духе.
   Утилита: http://fullrest.ru/forum/topic/34114-generator-readme/
   Сорцы: http://depositfiles.com/files/kvi4gsajy

   Говногость, 06 Мая 2011

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

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

   +124

   1. 1

   litFreeMinets.Text = FreeMinutes.Count;

   uassya, 06 Мая 2011

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

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

   +122

   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

   switch (item.Value.ModuleConfiguration.SystemModule) // у свойства SystemModule тип bool, а не bool?
   {
          case true:
          {
               CreateModuleDomain<ISystemModuleProxy>(moduleContainer);
               (moduleContainer.ModuleProxy as ISystemModuleProxy).Init(moduleContainer.ModuleConfiguration, this as ISystemCoreProvider);
                  
                break;
          }
          case false:
          {
               CreateModuleDomain<IBusinessModuleProxy>(moduleContainer);
               (moduleContainer.ModuleProxy as IBusinessModuleProxy).Init(moduleContainer.ModuleConfiguration, this as ICoreProvider);
   
               break;
          }
          default:
               break;
   }

   Фрагмент кода официального Senior Developer. Пример абсолютно надежного кода, который умеет обрабатывать даже будущие состояния булевого типа (default: break;) Будет надежен даже, если Microsoft неожиданно расширит тип состояниями, например MayBeTrue, OfCourseFalse, DontUnderstand и т.п. :)

   anzu, 05 Мая 2011

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

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

    +124

    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

    if (command == "help") ShowHelp();
                else if (command == "") Error(1);
                else
                {
                    string[] pcmd = command.Split(' ');
                    string[] args = { "0,0", "0,0" };
                    string argv = "Black";
                    if (pcmd[0] != "setColor" &&
                        pcmd[0] != "save" &&
                        pcmd[0] != "fill" &&
                        pcmd[0] != "saveCMD" &&
                        pcmd[0] != "close" &&
                        pcmd[0] != "if" &&
                        pcmd[0] != "from" &&
                        pcmd[0] != "setVar" &&
                        pcmd[0] != "getVar" &&
                        pcmd[0] != "setRGB" &&
                        pcmd[0] != "fillFRGB" &&
                        pcmd[0] != "fillRGB" &&
                        pcmd[0] != "setPenSize" &&
                        pcmd[0] != "fillF" &&
                        pcmd[1] != "#?") args = pcmd[1].Split('|');
                    else if (pcmd[0] == "setVar") argv = pcmd[1] + " " + pcmd[3];
                    else if (pcmd[1] == "#?")
                    {
                        Help4command(pcmd[0]);
                        return;
                    }
                    else argv = pcmd[1];
                    switch (pcmd[0])
                    {
                        case "save":
                            try { bmp.Save(argv + "\\" + Name + ".png",ImageFormat.Png); }
                            catch { Error(2); }
                            break;
                        case "line":
                            {
                                string[] xy0 = args[0].Split(',');
                                string[] xy1 = args[1].Split(',');
                                int x0 = 0, y0 = 0, x1 = 0, y1 = 0;
                                if (xy0[0] == "w" || xy0[0] == "W") x0 = W;
                                if (xy0[1] == "h" || xy0[1] == "H") y0 = H;
                                if (xy1[0] == "w" || xy1[0] == "W") x1 = W;
                                if (xy1[1] == "h" || xy1[1] == "H") y1 = H;
                                if (VL.isExistVar(xy0[0])) x0 = int.Parse(VL.Get(xy0[0]));
                                if (VL.isExistVar(xy0[1])) y0 = int.Parse(VL.Get(xy0[1]));
                                if (VL.isExistVar(xy1[0])) x1 = int.Parse(VL.Get(xy1[0]));
                                if (VL.isExistVar(xy1[1])) y1 = int.Parse(VL.Get(xy1[1]));
                                try
                                {
                                    x0 = int.Parse(xy0[0]);
                                    y0 = int.Parse(xy0[1]);
                                    x1 = int.Parse(xy1[0]);
                                    y1 = int.Parse(xy1[1]);
                                }
                                finally { }
                                graph.DrawLine(new Pen(usesCol, penSize), new Point(x0, y0), new Point(x1, y1));
                            }
                            break;
                        case "rect":
                            {
                                if (fiilF == false)
                                {
                                    string[] xy0 = args[0].Split(',');
                                    string[] xy1 = args[1].Split(',');
                                    int x0 = 0, y0 = 0, x1 = 0, y1 = 0;
                                    if (xy0[0] == "w" || xy0[0] == "W") x0 = W;
                                    if (xy0[1] == "h" || xy0[1] == "H") y0 = H;
                                    if (xy1[0] == "w" || xy1[0] == "W") x1 = W;
                                    if (xy1[1] == "h" || xy1[1] == "H") y1 = H;
                                    if (VL.isExistVar(xy0[0])) x0 = int.Parse(VL.Get(xy0[0]));
                                    if (VL.isExistVar(xy0[1])) y0 = int.Parse(VL.Get(xy0[1]));
                                    if (VL.isExistVar(xy1[0])) x1 = int.Parse(VL.Get(xy1[0]));
                                    if (VL.isExistVar(xy1[1])) y1 = int.Parse(VL.Get(xy1[1]));
                                   //еще +100500 строк говнокода

    Только что копался в старых проектах и наткнулся на это. Это была попытка сделать что-то вреде ЯП для рисования, своево рода черепашья графика, но со своими свистелками и сами знаете чем.

    psina-from-ua, 05 Мая 2011

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

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

• 193
• …
• 69
• 68
• 67
• 66
• 65
• 64
• 63
• 62
• 61
• …
• 1