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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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# / Говнокод #10332

   +96

   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

   private void textBox10_TextChanged(object sender, EventArgs e)
           {
           }
           private void button3_Click(object sender, EventArgs e)
           {
               textfactorname1.Text = "";
               textfactorname2.Text = "";
               textfactorname3.Text = "";
               textfactorname4.Text = "";
               textfactorname5.Text = "";
               textfactorname6.Text = "";
               textfactorname7.Text = "";
               textfactorname8.Text = "";
               textfactorname9.Text = "";
               ...
              
               checkBox9.Checked = false;
               checkBox10.Checked = false;
           }
           private void button2_Click_1(object sender, EventArgs e)
           {
               int n;
               try
               {
                   n = Convert.ToInt32(textItera.Text);
               }
               catch
               {
                   n = 1;
               }
               string results;
               double[] NPVarray = new double[n];
               double[] NPVsquares = new double[n];
               double NPVmax,NPVmin,NPVmid,disp,standdev,CV,zeropoint,pxy,xxx,yyy;
               int lessthanzero, morethanxxx, lessthanyyy;
               try
               {
                   for (int i = 0; i < n; i++)
                   {
                       string factorname1;
                      ...
                       string factorname10;
                       double fixedvalue1;
                      ...
                       double fixedvalue10;
                       double midvalue1;
                       ...
                       double midvalue10;
                       double minvalue1;
                       ...
                       double minvalue10;
                       double maxvalue1;
                       ...
                       double maxvalue10;
                       string varname1;
                      ...
                       string varname10;
                       if (textfactorname1.Text == "")
                       {
                           factorname1 = "What a fuck?";
                       }
                       else
                       {
                           factorname1 = textfactorname1.Text;
                       }
                       if (textfactorname2.Text == "")
                       {
                           factorname2 = "What a fuck?";
                       }

   Вот так вот. Лимит в 100 строк не позаоляет показать все прелести.

   tarabo, 20 Мая 2012

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

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

   +92

   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

   Console.Write("Введите координату Xm: ");
               double Xm = Convert.ToDouble(Console.ReadLine());
               Console.Write("Введите координату Ym: ");
               double Ym = Convert.ToDouble(Console.ReadLine());
   
               Console.Write("Введите координату Xa: ");
               double Xa = Convert.ToDouble(Console.ReadLine());
               Console.Write("Введите координату Ya: ");
               double Ya = Convert.ToDouble(Console.ReadLine());
   
               Console.Write("Введите координату Xb: ");
               double Xb = Convert.ToDouble(Console.ReadLine());
               Console.Write("Введите координату Yb: ");
               double Yb = Convert.ToDouble(Console.ReadLine());
   
               Console.Write("Введите координату Xc: ");
               double Xc = Convert.ToDouble(Console.ReadLine());
               Console.Write("Введите координату Yc: ");
               double Yc = Convert.ToDouble(Console.ReadLine());
   
               if (
                   ((((Xb - Xa) * Ym + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xm <= 0) && ((Xb - Xa) * Yc + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xc <= 0))
                   || (((Xb - Xa) * Ym + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xm >= 0) && ((Xb - Xa) * Yc + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xc >= 0)))
   
                   && ((((Xc - Xa) * Ym + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xm <= 0) && ((Xc - Xa) * Yb + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xb <= 0))
                   || (((Xc - Xa) * Ym + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xm >= 0) && ((Xc - Xa) * Yb + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xb >= 0)))
                   
                   && ((((Xc - Xb) * Ym + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xm <= 0) && ((Xc - Xb) * Ya + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xa <= 0))
                   || (((Xc - Xb) * Ym + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xm >= 0) && ((Xc - Xb) * Ya + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xa >= 0)))
                   ) Console.WriteLine("Точка ({0}, {1}) входит в треугольник", Xm, Ym);
               else Console.WriteLine("Точка ({0}, {1}) не входит в треугольник", Xm, Ym);
               Console.ReadKey();

   Проверка входит ли точка в треугольник.

   maksim_ovcharik, 19 Мая 2012

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

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

   +98

   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

   public class XmlParserClass
       {
           private XmlDocument xmlDoc;
           public static List<Attribute> listOfAttributes;
           public static Dictionary<int, AttributeValue> listOfAttributeValues;
           public static Dictionary<int, File> listOfFiles;        
           public static List<Product> listOfProducts;
           public List<FileCategory> listOfFileCategories;
           public List<Distributor> listOfDistributors;
           public static bool cryptedFiles = false;
   
           private string xmlfileLocation; 
           private string XmlfileLocation  
           {
               get 
               { 
                   if (xmlfileLocation == null)
                       xmlfileLocation = System.IO.Path.GetDirectoryName(Program.GetValue("XMLFileLocation", ""));
                   return xmlfileLocation;
               }
           }
   	       
   	public XmlParserClass(XmlDocument xmlDocument)
           {
               xmlDoc = xmlDocument;
               listOfAttributes = new List<Attribute>();
               listOfAttributeValues = new Dictionary<int, AttributeValue>();
               listOfProducts = new List<Product>();
               listOfFileCategories = new List<FileCategory>();
               listOfFiles = new Dictionary<int, File>();
               listOfDistributors = new List<Distributor>();
   
   			.......
   			
                  //Читаем а с XML ....
                   foreach (XmlNode xmlValue in attr.SelectSingleNode("Values").ChildNodes)
                   {
                      ................
                   }
   
                   listOfAttributes.Add(a);
   
   			
   			.........
   			//Читаем fc с XML....
   			listOfFileCategories.Add(fc);
   			
   			.........
   		}
   	}

   Создали экземпляр чтобы прочитать данные с XML. Куда бы нам их положить? Конечно в статические поля!

   dreamer, 18 Мая 2012

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

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

   +96

   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

   if (IsPostBack)
               {
                   string PostBackerID = Request.Form.Get("__EVENTTARGET");
                   string PostBackerArg = Request.Form.Get("__EVENTARGUMENT");
   
                   if (PostBackerID == "ctl00$cphMainContent$pagerID")
                   {
                       CurrentReviewPage = int.Parse(PostBackerArg);
                       FillReviews();
                   }
                   else if (PostBackerID == "ctl00$cphMainContent$Like")
                   {
                       FillLike();
                   }
                   else if (PostBackerID == "ctl00$cphMainContent$review")
                   {
                       AddReview(PostBackerArg);
                   }
                   else if (PostBackerID == "ctl00$cphMainContent$Follow")
                   {
                       FillFollow();
                   }
                   else if (PostBackerID == "ctl00$cphMainContent$WebResultpagerID")
                   {
                       CurrentWebResultPage = int.Parse(PostBackerArg);
                       FillWebResults();
                   }
                   else if (PostBackerID == "ctl00$cphMainContent$WebResultCluster" || PostBackerID == "ctl00$cphMainContent$HiddenButton")
                   {
                       CurrentWebResultPage = 0;
                       int SelectedClusterID = int.Parse(PostBackerArg);
                       if (AllClusters != null && AllClusters.ContainsKey(SelectedClusterID))
                       {
                           SelectedClusterName = AllClusters[SelectedClusterID];
                           FillWebResults();
                       }
                   }
               }

   компактная логика всей страницы...

   mangyst, 17 Мая 2012

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

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

   +97

   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

   enum TheComponentsOfLanguage 
   {
   	Glottis, 					//— голосовая щель;
   	FerreinsCord, 				//— голосовая складка;
   	VestibularFold, 				//— складка преддверия;
   	EpiglotticFold, 				//— черпалонадгортанная складка;
   	BaseOfTongue, 				//— корень языка;
   	PalatineTonsils, 			//— небные миндалины;
   	BlindHoleLanguage, 			//— слепое отверстие языка;
   	BorderFurrow,				//— пограничная бороздка;
   	LeafBuds,				//— листовидные сосочки;
   	PapillaeSurroundedTheShaf,		//— сосочки, окруженные валом;
   	MushroomShapedPapillae,		//— грибовидные сосочки;
   	BodyLanguage,				//— корпус языка;
   	FiliformPapillae,			//— нитевидные сосочки;
   	DorsumOfTongue,			//— спинка языка;
   	TheTipOfTheTongue			//— верхушка языка;
   }

   KusokGovna, 17 Мая 2012

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

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

   +98

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

   var prevQuery = new { TimePassed = 1, ID = 1 };
   prevQuery = null;
   foreach (var currentQuery in db.Queries.Select(q => new { TimePassed = q.TimePassed, ID = q.ID }))
   {
   	if (prevQuery != null)
   	{
   		//действия с prevQuery и currentQuery
   	}
   	prevQuery = currentQuery;
   }

   HIMen, 17 Мая 2012

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

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

   +96

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

   OdbcCommand dist_cmd = ifx_conn.CreateCommand();
                   dist_cmd.CommandText = string.Format(@"select  -1 as districtid, '-=ВСІ=-' as districtname from systables where  tabname = 'systables'                               union all 
                                           select id as districtid , cpz_name as districtname from  post_cpz where directorate_id = {0} and id != 0 ORDER BY 2", (String.IsNullOrEmpty(DDL_obl.SelectedValue))?"-1":DDL_obl.SelectedValue);
                   reader = dist_cmd.ExecuteReader();

   в процессе рефакторинга, убирая NHibernate код был заменен на это

   bercerker, 17 Мая 2012

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

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

   +86

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

   protected void ListViewAlerts_ItemCommand(object sender, ListViewCommandEventArgs e)
           {
               int commandArgument = e.Item.DataItemIndex;
               var listViewDataItem = (HiddenField)ListViewAlertRegistry.Items[commandArgument].Controls[3];
               int id = Convert.ToInt32(listViewDataItem.Value);
   .....
           }

   Из рабочего проекта. Новый способ узнать id item'а listview. Видимо свойство DataKeyNames кодеру незнакомо - id берет из hiddenfield, причем выбирает этот hiddenfield не по его id, а по позиции контрола по счету в строке. Добавляем еще один элемент в listview, и соответственно метод валится.

   _vinka, 16 Мая 2012

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

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

   +91

   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

   private void changeTabItem(object parameter)
           {
               if (SelectedTabIndex == SelectedTabIndexOld) return;
               if (SelectedTabIndex != 0)
               {
                   if ([надо сохранить])
                   {
                       if ([попытка сохранить удалась]) SelectedTabIndexOld = SelectedTabIndex;
                       else SelectedTabIndex = 0;
                   }
                   else SelectedTabIndexOld = SelectedTabIndex;
   
                   if (SelectedTabIndex == 1)
                       [обновить древо]
   
               }
               else if (SelectedTabIndex != 1)
               {
                   if ([надо сохранить])
                   {
                       if ([попытка сохранить удалась]) SelectedTabIndexOld = SelectedTabIndex;
                       else SelectedTabIndex = 1;
                   }
                   else SelectedTabIndexOld = SelectedTabIndex;
                   if (SelectedTabIndex == 0)
                       [обновить древо]
               }
           }

   Раскопки продолжаются. Теперь Вашему вниманию предлагается "женское" переключение вкладок, работающее "от противного". В данном случае две вкладки, а теперь представим что надо добавить ещё пару. =)

   DukeGonzo, 16 Мая 2012

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

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

    +93

    1. 1

    if (indSpec == false || indZone == false || (FRof != null && FRof == 2)) result = 1;

    Нашли в проекте вот такое "женское" условие. Видимо, префиксом "ind" автор намекала на своё происхождение.

    DukeGonzo, 15 Мая 2012

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

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

• 193
• …
• 98
• 97
• 96
• 95
• 94
• 93
• 92
• 91
• 90
• …
• 1