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

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

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

1. PHP (5701)
2. C++ (2737)
3. Куча (2340)
4. JavaScript (2028)
5. C# (1928)
6. Java (1502)
7. Си (1102)
8. Pascal (647)
9. Python (582)
10. 1C (535)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (30)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

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

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

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

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

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

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

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

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

   +13.4

   1. 1

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

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

   guest, 26 Декабря 2008

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

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

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

    +12

    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

    [Test]
    public void UserHasNoPermissionToDeleteTransactionWithEditPermission()
    {
      var user = CreatetestUser("test")
        .WithPermissions()
        .Allow("Transaction/Edit").Do();
      TestCurrentUser = user;
    
      var trn = CreateTestTransaction(120, typeof(CreditCardPayment));
    
      try
      {    
        // Who cares about delete, we never do it.
        // I need this test passed just NOW.
    
        // trn.AsIObject().Delete();
        // Assert.Fail("Exception expected");
      }
      catch (PermissionException ex)
      {
        Assert.IsTrue(true, "Cool we got an exception");
      }
      catch (Exception generic)
      {
        Assert.Fail("PermissionException expected.");
      }
    }

    Ничего не тестирующий тест

    guest, 24 Декабря 2008

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

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

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