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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2324)
4. JavaScript (2027)
5. C# (1924)
6. Java (1501)
7. Си (1095)
8. Pascal (647)
9. Python (582)
10. 1C (531)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (196)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (28)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

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

   +152

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

   try {
       if (something) {
           throw new MyGotoException();
       }
       DoSomethingElse();
   }
   catch (MyGotoException) {
   }
   
   DoGotoBlock();

   Очень хитрый вариант реализации схемы goto.

   guest, 02 Апреля 2009

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

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

   +147

   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

   if (chk.Checked)
               {
                   foreach (BV.Star.BLL.PersonGroups.PersonGroup trainee in trainees)
                   {
                       if (trainee.TraineeID.ToString() == gvwTrainees.DataKeys[gvwTrainees.PageIndex == 0 ? gvr.DataItemIndex : (gvr.DataItemIndex % gvwTrainees.PageSize)].Value.ToString())
                         goto Next;
                   }
   
                   InsertTraineeIntoGroup(Int32.Parse(ddlGroups.SelectedValue),
                       Int32.Parse(gvwTrainees.DataKeys[gvwTrainees.PageIndex == 0 ? gvr.DataItemIndex : (gvr.DataItemIndex % gvwTrainees.PageSize)].Value.ToString()),
                     false, DateTime.MaxValue,
                     "");
               Next: ;
   
               }

   И снова на сцене - goto!

   guest, 02 Апреля 2009

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

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

   +143.1

   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

   if (((HttpContext.Current.Profile as ProfileCommon).Role == "3") || ((HttpContext.Current.Profile as ProfileCommon).Role == "4"))
           {
               if (e.Item.Value == " Add New Trainee ")
                   e.Item.Enabled = false;
               if (e.Item.Value == " Add New Group ")
                   e.Item.Enabled = false;
               if (e.Item.Value == " Audit Log ")
                   e.Item.Enabled = false;
               if (e.Item.Value == " Manage Holidays ")
                   e.Item.Enabled = false;
           }
   
           if ( (( HttpContext.Current.Profile as ProfileCommon).Role != "0") && (( HttpContext.Current.Profile as ProfileCommon).Role != "") )
           {
               if (e.Item.Value == " Manage Users ")
                   e.Item.Enabled = false;
               if (e.Item.Value == " Manage Courses ")
                   e.Item.Enabled = false;
               if (e.Item.Value == " Audit Log ")
                   e.Item.Enabled = false;
           }
           if (((HttpContext.Current.Profile as ProfileCommon).Role == "") ||
               ((HttpContext.Current.Profile as ProfileCommon).Role == "0") ||
               ((HttpContext.Current.Profile as ProfileCommon).Role == "1") ||
               ((HttpContext.Current.Profile as ProfileCommon).Role == "2"))
           {
               if (e.Item.Value == " Holidays ")
                   e.Item.Enabled = false;
   
           }

   Новое место работы. Новые порции говнокода.
   Это классика - no comments...

   guest, 02 Апреля 2009

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

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

   +145

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

   /// <summary>
           /// Child border.
           /// </summary>
           private Border childBorder;

   Я бы никогда без комментария не догадался, зачем же это поле

   guest, 30 Марта 2009

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

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

   +958.8

   1. 1

   idString.Length == Guid.NewGuid().ToString.Length()

   Самый имхо индусский (см. лукмор) код проверить что строка это Guid

   guest, 30 Марта 2009

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

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

   +112.5

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

   public void Log(string pSource, string pMessage, EventLogEntryType pEntryType) {
       try {
           if (!EventLog.SourceExists(pSource)) {
               EventLog.CreateEventSource(pSource, "Application");
           }
   
           EventLog.WriteEntry(pSource, pMessage, pEntryType);
       }
       catch (Exception _ex) {
           Log("", _ex.ToString(), EventLogEntryType.Error);
       }
   }

   Пишем в лог - постоянно...
   Автор находки: http://ayende.com/Blog/archive/2009/03/19/how-not-to-write-a-logger.aspx

   guest, 27 Марта 2009

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

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

   +135.9

   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

   String[] days = new String[7];
   for( int i = 0; i < 7; i++ ) {
     switch(i) {
       default:
       case 0:
         days[i] = "Monday";
         break;
       case 1:
         days[i] = "Tuesday";
         break;
       case 2:
         days[i] = "Wednesday";
         break;
       case 3:
         days[i] = "Thursday";
         break;
       case 4:
         days[i] = "Friday";
         break;
       case 5:
         days[i] = "Saturday";
         break;
       case 6:
         days[i] = "Sunday";
         break;
     }
   }

   Боян, наверное, но все-таки...

   guest, 24 Марта 2009

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

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

   +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

   private string StripDuplicateSpaces(string data)
       {
           bool flag = false;
           do
           {
               if (data.IndexOf("  ") != -1)
               {
                   data = data.Replace("  ", " ");
               }
           }
           while (!flag);
           return data.Trim();
       }
   
       private string TrimNewLines(string s)
       {
           return s;
       }

   за это ещё и денег просят :)

   guest, 19 Марта 2009

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

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

   +150

   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

   [Serializable]
       public class Key
       {
           public Key(long id, string code)
           {
               m_Id = id;
               m_Code = code;
           }
   
           public Key(long id) : this(id, null)
           {
           }
   
           public Key(string code) : this(0, code)
           {
           }
   
           public Key() : this(0, null)
           {
           }
   
           public long getId()
           {
               return (m_Id);
           }
   
           public long m_Id;
   
           public string getCode()
           {
               return (m_Code);
           }
   
           public string m_Code;
   
           public static string ToString(Key k)
           {
               return ((k == null) ? "(null)" : k.ToString());
           }
       }

   guest, 18 Марта 2009

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

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

    +150

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

    public const char ExpMonthSeparator = ':';
    
    
    protected void BindPager()
    {
       var colon = string.Format("{0}", ExpMonthSeparator);
       /* And some lines for refactor */
    }

    Фикся очередную багу наткнулся на такую прелесть

    guest, 18 Марта 2009

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

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

• 193
• …
• 13
• 12
• 11
• 10
• 9
• 8
• 7
• 6
• 5
• …
• 1