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

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

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

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

2. 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)

3. 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)

4. 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)

5. 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)

6. 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)

7. 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)

8. 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)

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

   +13.4

   1. 1

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

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

   guest, 26 Декабря 2008

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

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

    +25

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

    в aspx:
    <button id="btnNewHint" runat="server" onserverclick=" NewHint"><%=GetLanguageString("AdminHintCreate")%></button>
    
    в cs:
    
    Response.Write("\n<script>\n" + "window.open('AddHint.aspx?CategoryID=" + CategoryID + "','','width=850,height=540,menubar=no,location=no,scrollbars=yes,resizable=yes,status=no,toolbar=no');" + "</script>\n");

    как бы onclick="window.open(...);" написать нельзя

    guest, 26 Декабря 2008

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

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

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