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

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

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

1. PHP (5707)
2. C++ (2739)
3. Куча (2376)
4. JavaScript (2032)
5. C# (1929)
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 (199)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (31)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. PHP / Говнокод #12692

   +153

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

   $_SESSION["db_options"] = array ('host'     => trim($this->IP).":".trim($this->Port),
   										'user'     => trim($this->User),
   										'password' => trim($this->Pass),
   										'database' => trim($this->DB)
   										);

   Наследование? Не, не слышал

   nonamez, 05 Марта 2013

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

2. Java / Говнокод #12690

   +114

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

   private List<PalletePath> killDupes(List<PalletePath> pathesNew) {
   	List<PalletePath> noDupes = new ArrayList<PalletePath>();
   	for (PalletePath tp : pathesNew) {
   		if (!noDupes.contains(tp)) {
   			noDupes.add(tp);
   		}
   	}
   	return noDupes;
   }

   Set? Не, не слышали.

   someone, 05 Марта 2013

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

3. 1C / Говнокод #12689

   −122

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

   Если ВыгружатьВзаиморасчеты тогда
       Если ВыгружатьВзаиморасчеты() тогда
           УспешноВзаиморасчетыВыгружено=ВыгрузитьВзаиморасчетыНаСайт(СтруктураПараметровСайта);
       КонецЕсли;
   КонецЕсли;

   Херово, когда имен не хватает =(

   kovyl2404, 05 Марта 2013

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

4. PHP / Говнокод #12688

   +154

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

   $code_date = strtotime($sms_action_date);
       $date = date(
           "Y-m-d H:i:s",
           mktime(
               date('H', $code_date),
               date('i', $code_date) + 30,
               date('s', $code_date),
               date("m", $code_date),
               date("d", $code_date),
               date("Y", $code_date)
           )
   );

   Определяем +30 минут грамотно.

   miraage, 05 Марта 2013

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

5. Java / Говнокод #12686

   +74

   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 boolean userInOneRegistrationNode() throws DfException {
   
           String uname = OrganizationStaffStructureHelper.getCurrentUserName();
   
           int i = 0;
           IDfCollection NodesCol = DQLHelper.getCollection(DQL_GET_REGISTRATOR_DIV, new String[]{uname});
           while (NodesCol.next()) {
               if (!(NodesCol == null)) {
                   String group_name = NodesCol.getString(GROUP_NAME);
                   i = i + 1;
               }
           }
   
           if (i == 1) {
               return true;
           }
           return false;
   }

   Заменяется 2мя строками один - select count(*), вторая - полученный результат Integer.ValueOf(...).

   Landing, 04 Марта 2013

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

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

   +140

   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

   [WebMethod]
           public void runCompareService(string mAuthToken, int documentId, int origVerNum, int revisedVerNum)
           {
               //string result = null;
               DirectoryInfo tmpDirInfo = getTempDirectoryPath(documentId);
               try
               {
                   //authToken = authClient.AuthenticateUser(username, password);
                   authToken = mAuthToken;
                   otAuth.AuthenticationToken = authToken;
   
                   try
                   {
                       string origFilePath = String.Format(tmpFilePath, tmpDirInfo.ToString(), origVerNum);
                       string revisedFilePath = String.Format(tmpFilePath, tmpDirInfo.ToString(), revisedVerNum);
                       string resultFilePath = String.Format(tmpResFilePath, tmpDirInfo.ToString(), origVerNum, revisedVerNum);
   
                       string origContextId = getDocContextId(documentId, origVerNum);
                       string revisedContextId = getDocContextId(documentId, revisedVerNum);
   
                       try
                       {
                           downlodFileByContextId(origContextId, origFilePath);
                           downlodFileByContextId(revisedContextId, revisedFilePath);
                           try
                           {
                               doCompare(origFilePath, revisedFilePath, resultFilePath);
                               try
                               {
                                   DownloadToBrowser(resultFilePath);
                                   
                                   //uploadResultToCS(targetId, resultFilePath);
                               }
                               catch (Exception e)
                               {
                                   throw new Exception(String.Format("Failed to Download To Browser. Error: {0}", e.ToString()));
                               }
                           }
                           catch (Exception e)
                           {
                               throw new Exception(String.Format("Failed to do compare method . Error: {0}", e.ToString()));
                           }
                       }
                       catch (Exception e)
                       {
                           throw new Exception(String.Format("Failed to create and download file. Error: {0}", e.ToString()));
                       }
                   }
                   catch (Exception e)
                   {
                       throw new Exception("Failed to get Context ID for version Exeption details: " + e.ToString());
                   }
   
               }
               catch (Exception e)
               {
                   throw new Exception("Failed to auth user. Exeption details: " + e.ToString());
               }
               finally
               {
                   docManClient.Close();
                   contentServiceClient.Close();
                   if (Directory.Exists(tmpDirInfo.ToString()))
                   {
                       tmpDirInfo.Delete(true);
                   }
               }
               
           }

   Landing, 04 Марта 2013

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

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

   +106

   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

   using System;
   using System.Windows.Forms;
   using System.Collections.Generic;
   using System.Text;
   using System.Xml.Linq;
   
   namespace SplitSpace {
     class Program  {
       static void Main(string[] args) { 
           test();
       }
     
       static void test() {
         if(testfun(0, 0, 3, 1020, 0.05*180/Utils.PI, 1,  40, 5.80020838542985, 5.80020838542985, 0.0125032560359095)) return;
         if(testfun(0, 0, 3, 1020, 0.05*180/Utils.PI, 2,  5, 1.75, 9.85005209635745, 0.00156290700448869)) return;
         if(testfun(0, 0, 3, 1020, 0.05*180/Utils.PI, 3,  1.5, 0.0375078144536192, 0.0375078144536192, 0.000468872101346607)) return;
         if(testfun(0, 0, 3, 219, 0.00, 1,  15.000, 4.900000000, 4.900000000, 0.000000000)) return;
         if(testfun(0, 0, 3, 219, 0.50, 1,  15.000, 4.900000415, 4.900000415, 0.000142791)) return;
         if(testfun(0, 0, 3, 219, 5.00, 1,  15.000, 4.900415674, 4.900415674, 0.014290277)) return;
         if(testfun(0, 0, 3, 219, 5.10, 1,  15.000, 4.900441130, 4.900441130, 0.014868082)) return;
         if(testfun(0, 0, 3, 219, 5.90, 1,  15.000, 4.900683170, 4.900683170, 0.019903983)) return;
         // Спустя 8369 строчек
         // ...
         // ...
         // ...
         if(testfun(2, 1, 4, 1420, 55.00, 2,  7.000, 5.512975201, 5.512975201, 1.316853521)) return;
         if(testfun(2, 1, 4, 1420, 60.00, 2,  7.000, 5.817691454, 5.817691454, 1.558845727)) return;
         if(testfun(2, 1, 4, 1420, 66.00, 2,  7.000, 6.589343466, 6.589343466, 1.708953962)) return;
         if(testfun(2, 1, 4, 1420, 75.00, 2,  7.000, 7.385125168, 7.385125168, 2.367207113)) return;
         if(testfun(2, 1, 4, 1420, 85.10, 2,  7.000, 8.438873737, 8.438873737, 3.248627191)) return;
         if(testfun(2, 1, 4, 1420, 89.00, 2,  7.000, 8.848026526, 8.848026526, 3.610990132)) return;
         if(testfun(2, 1, 4, 1420, 89.99, 2,  7.000, 8.932590260, 8.932590260, 3.699354242)) return;
         if(testfun(2, 1, 4, 1420, 90.00, 2,  7.000, 8.932590181, 8.932590181, 3.700000000)) return;
   
         MessageBox.Show("OK!"); 
       }
   
       static bool testfun(int dir, int coolType, int hotType, double dia, double angG, int angType,  double Rad, double Tan1, double Tan2, double Bis) {
         // тут всякие тесты
         // ...
         // ...
         // ...
         return false;
       }
   
     } 
   }

   diimdeep , 04 Марта 2013

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

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

   +106

   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
   70. 70
   71. 71
   72. 72
   73. 73

   using System;
   using System.Collections.Generic;
   using System.ComponentModel;
   using System.Data;
   using System.Drawing;
   using System.Text;
   namespace data_base
   { public class DatA
   { public struct DATA
   { public struct Yslovie { public string yslovie; public bool ActivE;}
   public Yslovie yslovie1;
   public Yslovie yslovie2;
   public Yslovie yslovie3;
   public string deistvie;};
   public DATA.Yslovie yslovie1 = new DATA.Yslovie();
   public DATA.Yslovie yslovie2 = new DATA.Yslovie();
   public DATA.Yslovie yslovie3 = new DATA.Yslovie();
   public DATA[] BD = new DATA[5];
   public void инициализация_БД()
   { BD[0].yslovie1.yslovie = "поставщик проверенный"; BD[0].yslovie1.ActivE = false;
   BD[0].yslovie2.yslovie = "товар качественный"; BD[0].yslovie2.ActivE = false;
   BD[0].yslovie3.yslovie = "цена приемлемая"; BD[0].yslovie3.ActivE = false;
   BD[0].deistvie = "Поставщик подходит";
   BD[1].yslovie1.yslovie = "поставщик давно на рынке"; BD[1].yslovie1.ActivE = false;
   BD[1].yslovie2.yslovie = "жалоб на поставщика не было"; BD[1].yslovie2.ActivE = false;
   BD[1].yslovie3.yslovie = null; BD[1].yslovie3.ActivE = false;
   BD[1].deistvie = "поставщик проверенный";
   BD[2].yslovie1.yslovie = "есть рекомендации от других заказчиков"; BD[2].yslovie1.ActivE = false;
   BD[2].yslovie2.yslovie = "поставка производится в срок"; BD[2].yslovie2.ActivE = false;
   BD[2].yslovie3.yslovie = "товар не получает дефектов в ходе транспатрировки"; BD[2].yslovie3.ActivE = false;
   BD[2].deistvie = "жалоб на поставщика не было";
   BD[3].yslovie1.yslovie = "товар сертифицирован"; BD[3].yslovie1.ActivE = false;
   BD[3].yslovie2.yslovie = "товар прощел государственные экспертизы"; BD[3].yslovie2.ActivE = false;
   BD[3].yslovie3.yslovie = null; BD[3].yslovie3.ActivE = false;
   BD[3].deistvie = "товар качественный";
   BD[4].yslovie1.yslovie = "цена чуть ниже чем у других поставщиков"; BD[4].yslovie1.ActivE = false;
   BD[4].yslovie2.yslovie = null; BD[4].yslovie2.ActivE = false;
   BD[4].yslovie3.yslovie = null; BD[4].yslovie3.ActivE = false;
   BD[4].deistvie = "цена приемлемая";
   }
   public void Констотация_факта(int i)
   { switch (i)
   { case 1: if (BD[0].yslovie1.ActivE == true) BD[0].yslovie1.ActivE = false;
   else BD[0].yslovie1.ActivE = true; break;
   case 2: if (BD[0].yslovie2.ActivE == true) BD[0].yslovie2.ActivE = false;
   else BD[0].yslovie2.ActivE = true; break;
   case 3: if (BD[0].yslovie3.ActivE == true) BD[0].yslovie3.ActivE = false;
   else BD[0].yslovie3.ActivE = true; break;
   case 4: if (BD[1].yslovie1.ActivE == true) BD[1].yslovie1.ActivE = false;
   else BD[1].yslovie1.ActivE = true; break;
   case 5: if (BD[1].yslovie2.ActivE == true) BD[1].yslovie2.ActivE = false;
   else BD[1].yslovie2.ActivE = true; break;
   case 6: if (BD[1].yslovie3.ActivE == true) BD[1].yslovie3.ActivE = false;
   else BD[1].yslovie3.ActivE = true; break;
   case 7: if (BD[2].yslovie1.ActivE == true) BD[2].yslovie1.ActivE = false;
   else BD[2].yslovie1.ActivE = true; break;
   case 8: if (BD[2].yslovie2.ActivE == true) BD[2].yslovie2.ActivE = false;
   else BD[2].yslovie2.ActivE = true; break;
   case 9: if (BD[2].yslovie3.ActivE == true) BD[2].yslovie3.ActivE = false;
   else BD[2].yslovie3.ActivE = true; break;
   case 10: if (BD[3].yslovie1.ActivE == true) BD[3].yslovie1.ActivE = false;
   else BD[3].yslovie1.ActivE = true; break;
   case 11: if (BD[3].yslovie2.ActivE == true) BD[3].yslovie2.ActivE = false;
   else BD[3].yslovie2.ActivE = true; break;
   case 12: if (BD[3].yslovie3.ActivE == true) BD[3].yslovie3.ActivE = false;
   else BD[3].yslovie3.ActivE = true; break;
   case 13: if (BD[4].yslovie1.ActivE == true) BD[4].yslovie1.ActivE = false;
   else BD[4].yslovie1.ActivE = true; break;
   case 14: if (BD[4].yslovie2.ActivE == true) BD[4].yslovie2.ActivE = false;
   else BD[4].yslovie2.ActivE = true; break;
   case 15: if (BD[4].yslovie3.ActivE == true) BD[4].yslovie3.ActivE = false;
   else BD[4].yslovie3.ActivE = true; break;}
   }}}

   оставлю тут

   Psilon, 04 Марта 2013

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

9. Ruby / Говнокод #12681

   −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

   def register_valid_login
   		login = params[:login]
   		return register_error("Слишком длинный ник!") if (login.length > 16) 
   		return register_error("Слишком короткий ник!") if (login.length < 3)
   		return register_error("Недопустимые символы в нике!") if !(/^[a-zA-Z\-_]+$/=~login)
   		return register_error("Можно использовать либо '-', либо '_' в нике!") if (login.count("-") > 0 && login.count("_") > 0)
   		return register_error("Ник должен состоять из трёх частей максимум!") if (login.count("-") > 2 || login.count("_") > 2)
   		login = [login] if (login.count("-") == 0 && login.count("_") == 0)
   		login = login.split("-") if login.count("-") > 0
   		login = login.split("_") if login.count("_") > 0
   		return register_error("Начало ника не может быть меньше трёх символов!") if (login[0].length < 1)
   		case login.count
   			when 2
   				return register_error("Если у вас ник вида Xxx_Yyy, то Yyy может состоять минимум из 3х букв") if (login[1].length < 3)
   			when 3
   				return register_error("Если у вас ник вида AAA_B_CCC, то B должна быть больше одной буквы") if (login[1].length < 1)
   		end
   		for i in 0..login.count-1
   			return register_error("Заглавная буква в середине " + (i+1).to_s + "-ой части ника") if !(/^[a-zA-Z]?[a-z]+$/=~login[i])
   		end
   		return true
   	end

   Процедура проверки ника в ситеме регистрации одного игрового проекта.

   DropWorld, 03 Марта 2013

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

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

    +134

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

    public void Children_OnAdd(object sender, EventArgs e)
    {
       ((MyEventArgs<Document>)e).D.Parent = this;
    }

    taburetka, 03 Марта 2013

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

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

• 2108
• …
• 1143
• 1142
• 1141
• 1140
• 1139
• 1138
• 1137
• 1136
• 1135
• …
• 1