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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2386)
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# / Говнокод #382

   −15.6

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

   graphics.DrawString(i.ToString(),
       ScaleFont,
       brush,
       LeftField - _arrowHeight - ScaleFont.Size * (i.ToString().Length / 1.2F),
       (Height - BottomField) - i * ScaleY * _gridStep - ScaleFont.Height / 2);

   Вычисление размера строки, вместо Graphics.MeasureString

   guest, 13 Января 2009

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

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

   −14

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

   private static bool EndsWithSaurus(String s)
   {
       if ((s.Length > 5) && 
           (s.Substring(s.Length - 6).ToLower() == "saurus"))
       {
           return true;
       }
       else
       {
           return false;
       }
   }

   MSDN

   guest, 11 Января 2009

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

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

   +25

   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

   private void OnRdbDatesCheckedChanged(object sender, EventArgs e)
   {
               if (rdbMonth.Checked)
                   MakeDateViewVisible("month");
               else if (rdbQuarter.Checked)
                   MakeDateViewVisible("quarter");
               else if (rdbYear.Checked)
                   MakeDateViewVisible("year");
               else
                   MakeDateViewVisible("individual");
   }
   
   private void MakeDateViewVisible(string dateView)
   {
               switch (dateView)
               {
                   case "year":
                       YearView.Visible = true;
                       QuarterView.Visible = false;
                       MonthView.Visible = false;
                       IndividualDatesView.Visible = false;
                       break;
                   case "quarter":
                       YearView.Visible = false;
                       QuarterView.Visible = true;
                       MonthView.Visible = false;
                       IndividualDatesView.Visible = false;
                       break;
                   case "month":
                       YearView.Visible = false;
                       QuarterView.Visible = false;
                       MonthView.Visible = true;
                       IndividualDatesView.Visible = false;
                       break;
                   case "individual":
                       YearView.Visible = false;
                       QuarterView.Visible = false;
                       MonthView.Visible = false;
                       IndividualDatesView.Visible = true;
                       break;
               }
   }

   Зато красиво методы вызываются!

   guest, 07 Января 2009

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

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

   +1.1

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

   while (true) {
       try {
           break;
       }
       catch {
   
       }
   }

   интересно сработает?

   guest, 30 Декабря 2008

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

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

   +849.2

   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

   switch (driver.Status)
         {
           case ClientStatus.Unknown:
             return m_driverStatusNames[ClientStatus.Unknown];
           case ClientStatus.Free:
             return m_driverStatusNames[ClientStatus.Free];
           case ClientStatus.Busy:
             return m_driverStatusNames[ClientStatus.Busy];
           case ClientStatus.InWay:
             return m_driverStatusNames[ClientStatus.InWay];
           case ClientStatus.Work:
             return m_driverStatusNames[ClientStatus.Work];
           case ClientStatus.Break:
             return m_driverStatusNames[ClientStatus.Break];
           case ClientStatus.Alarm:
             return m_driverStatusNames[ClientStatus.Alarm];
         }

   :)

   guest, 30 Декабря 2008

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

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

   +1021.6

   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

   public static void brute(long diap)
   {
   n = 68491; //задаем некое число
   NewMessage("w8. Bruteforce working..."); //вызываем событие
   for (int i = 0; i < diap; i++) //мутим цикл
   {
   d = i;
   string brute = Convert.ToString(bug.Equals(n, d)); //сравниваем и возвращаем результат в строку
   if (brute == "True")
   {
   BruteComplete(d.ToString()); //вызываем событие при удачном переборе
   break;
   }
   }
   if (ok == false)
   {
   NewMessage("Brete Failed:(");
   }
   }

   Еще один:
   Данный говнокод взят из журнала "ХАКЕР". Вот ссылка, http://www.xakep.ru/magazine/xs/071/076/1.asp
   Тут программа должна подобрать число n. Как я понял тип n тут или не указан или переменная глобальная. Но нахрена тогда её объявлять глобальной если она применяется только в этом методе. Далее, d = i это вообще пиздец, да конечно же, если n это объект, а не int то все правильно. Но мне кажется автор полный мудак после того что создал строку brute, блять, конвертировал bool d string и сравнивает строку вида True с true. пиздец, пиздец....уебки...руки поотрывать...
   хакеры пля...=\

   guest, 28 Декабря 2008

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

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

   +887.2

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

   if (current_switch == true)
   {
   //вырезано
   }
   else if (current_switch == false)
   {
   //вырезано
   }

   Данный говнокод взят из журнала "ХАКЕР". Вот ссылка, http://www.xakep.ru/magazine/xs/071/076/1.asp
   
   хакеры пля...=\

   guest, 28 Декабря 2008

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

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

   −7

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

   int CompareTo (A a1, A a2)
   {
      if (a1.Year>a2.Year) return 1;
      if (a1.Year<a2.Year) return -1;
       return 0;
   }

   реализация интерфейса IComparable, чем не понравилось стандартное норминрование: return a1.Year - a2.Year; так и не понял

   guest, 28 Декабря 2008

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

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

   +25

   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

   private string RemoveNonAlphaChars(string txt)
   {
   string returnText = txt;
   
   returnText = returnText.Replace("~", "");
   returnText = returnText.Replace("`", "");
   returnText = returnText.Replace("!", "");
   returnText = returnText.Replace("@", "");
   returnText = returnText.Replace("#", "");
   returnText = returnText.Replace("$", "");
   returnText = returnText.Replace("%", "");
   returnText = returnText.Replace("^", "");
   returnText = returnText.Replace("&", "");
   returnText = returnText.Replace("*", "");
   returnText = returnText.Replace("(", "");
   returnText = returnText.Replace(")", "");
   returnText = returnText.Replace("-", "");
   returnText = returnText.Replace("_", "");
   returnText = returnText.Replace("+", "");
   returnText = returnText.Replace("=", "");
   returnText = returnText.Replace("{", "");
   returnText = returnText.Replace("}", "");
   returnText = returnText.Replace("[", "");
   returnText = returnText.Replace("]", "");
   returnText = returnText.Replace("|", "");
   returnText = returnText.Replace("\\", "");
   returnText = returnText.Replace("\"", "");
   returnText = returnText.Replace("'", "");
   returnText = returnText.Replace(";", "");
   returnText = returnText.Replace("<", "");
   returnText = returnText.Replace(",", "");
   returnText = returnText.Replace(">", "");
   returnText = returnText.Replace(".", "");
   returnText = returnText.Replace("/", "");
   returnText = returnText.Replace("?", "");
   returnText = returnText.Replace(" ", "");
   
   return returnText.ToUpper();
   }

   Убирает разделители. ;)

   guest, 26 Декабря 2008

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

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

    +13.4

    1. 1

    prNewProduct.Description = Convert.ToString(tbDescription.Text);

    А мы её ещё стрингее сделаем!

    guest, 26 Декабря 2008

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

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

• 193
• …
• 8
• 7
• 6
• 5
• 4
• 3
• 2
• 1