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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2373)
4. JavaScript (2031)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (588)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Haskell (30)
22. Swift (27)
23. Kotlin (14)

1. C++ / Говнокод #9624

   +156

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

   if (coms->Strings[1]=="install") {
             if (GetFile("localhost",WinDir+comms->Strings[3]) {
               execCommand=true;
             }else{
               execCommand=false;
             }

   вот такое вот чудо нашел в одном проЭкте предыдущего програмера, который до меня работал)
   вместо localhost передавалось имя файла на сервере

   stasuss, 07 Марта 2012

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

2. C++ / Говнокод #9623

   +997

   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
   74. 74
   75. 75
   76. 76
   77. 77
   78. 78
   79. 79
   80. 80
   81. 81
   82. 82
   83. 83
   84. 84
   85. 85
   86. 86
   87. 87
   88. 88
   89. 89
   90. 90
   91. 91

   char* CFastSmtp::_formatHeader()
   {
   // check for at least one recipient
       if(Recipients.size() <= 0 )    {
           printf("Please add a message recipient!\r\n");
           return NULL;
       }
       int s=0;
       char *msgHeader = new char[16385];
       //char to[1024];
       for (unsigned int i=s=0;i<Recipients.size();i++) {        
           s+=strlen(Recipients.at(i)->GetEmail())+1;
       } if (s==0) s=1; char *to = new char[s];
       //char cc[1024];
       for (i=s=0;i<CCRecipients.size();i++) {        
           s+=strlen(CCRecipients.at(i)->GetEmail())+1;
       } if (s==0) s=1; char *cc = new char[s];
       //char bcc[1024];
       for (i=s=0;i<BCCRecipients.size();i++) {        
           s+=strlen(BCCRecipients.at(i)->GetEmail())+1;
       } if (s==0) s=1; char *bcc = new char[s];
   
       TCHAR szDate[500];
       TCHAR sztTime[500];
   
   // create the recipient string, cc string, and bcc string
       to[0] = '\0';        
       for (i=0;i<Recipients.size();i++) {        
           i > 0 ? strcat(to,","):strcat(to,"");
           strcat(to,Recipients.at(i)->GetEmail());
       }
   
       cc[0] = '\0';    
       for (i=0;i<CCRecipients.size();i++) {
           i > 0 ? strcat(cc,","):strcat(cc,"");
           strcat(cc,CCRecipients.at(i)->GetEmail());
       }
   
       bcc[0] = '\0';    
       for (i=0;i<BCCRecipients.size();i++) {
           i > 0 ? strcat(bcc,","):strcat(bcc,"");
           strcat(bcc,BCCRecipients.at(i)->GetEmail());
       }
   // get the current date and time
       SYSTEMTIME st={0};
       ::GetSystemTime(&st);
       ::GetDateFormat(LOCALE_SYSTEM_DEFAULT,0,&st,"ddd',
                       ' dd MMM yyyy",szDate,sizeof(szDate));
       ::GetTimeFormat(LOCALE_SYSTEM_DEFAULT,TIME_FORCE24HOURFORMAT,&st,
                       "HH':'mm':'ss tt",sztTime,sizeof(sztTime));
   // here it is...the main data of the message
       wsprintf(msgHeader,"DATE: %s %s\r\n", szDate, sztTime);    
       if (m_pcFromName != NULL) {
           strcat(msgHeader,"FROM: ");
           strcat(msgHeader, m_pcFromName);
           strcat(msgHeader, "\r\n");
       }
       strcat(msgHeader,"To: ");
       strcat(msgHeader, to);
       strcat(msgHeader, "\r\n");
       strcat(msgHeader,"Cc: ");
       strcat(msgHeader, cc);
       strcat(msgHeader, "\r\n");
       if (m_pcSubject != NULL) {
           strcat(msgHeader, "Subject: ");
           strcat(msgHeader, m_pcSubject);
           strcat(msgHeader, "\r\n");
       }
       if (m_pcXMailer != NULL) {
           strcat(msgHeader,"X-Mailer: ");
           strcat(msgHeader, m_pcXMailer);
           strcat(msgHeader, "\r\n");
       }
   // start optional fields
       if (m_pcReplyTo != NULL) {
           strcat(msgHeader, "Reply-To: ");
           strcat(msgHeader, m_pcReplyTo);
           strcat(msgHeader, "\r\n");
       }
   // start MIME versions
       strcat(msgHeader, 
              "MIME-Version: 1.0\r\nContent-type: text/plain; charset=US-ASCII\r\n");
   // send header finish command
       strcat(msgHeader, "\r\n");    
   // clean up
       delete to;
       delete cc;
       delete bcc;
   // done    
       return msgHeader;    
   }

   аццкий говнокодер
   http://www.codeproject.com/Articles/623/CFastSmtp-Fast-and-easy-SMTP-class

   63F45EF45RB65R6VR, 07 Марта 2012

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

3. PHP / Говнокод #9622

   +153

   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

   function fnCheckNetwork($intNetworkId)
   {
       if (intval($intNetworkId) == 0)
       {
           return false;
       }
   
       if ($intNetworkId != $this->fnGetNetworkId())
       {
           return false;
       }
   
       return true;
   }

   Замечу, что функция использовалась как приватная, а $intNetworkId никогда не мог быть 0.
   
   Вобщем оригинальный способ проверить на равентсво два числа ))

   step307, 06 Марта 2012

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

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

   +130

   1. 1

   public void GetLocalUsers(out List<string> members) { ... }

   Первый раз вижу что-то подобное.

   fr0mrus, 06 Марта 2012

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

5. JavaScript / Говнокод #9620

   +197

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

   this.setlocale('LC_ALL',0);return text.search(this.php_js.locales[this.php_js.localeCategories.LC_CTYPE].LC_CTYPE.an)!==-1;},ctype_alpha:function(text){if(typeof text!=='string'){return false;}
   this.setlocale('LC_ALL',0);return text.search(this.php_js.locales[this.php_js.localeCategories.LC_CTYPE].LC_CTYPE.al)!==-1;},ctype_cntrl:function(text){if(typeof text!=='string'){return false;}
   this.setlocale('LC_ALL',0);return text.search(this.php_js.locales[this.php_js.localeCategories.LC_CTYPE].LC_CTYPE.ct)!==-1;},ctype_digit:function(text){if(typeof text!=='string'){return false;}
   this.setlocale('LC_ALL',0);return text.search(this.php_js.locales[this.php_js.localeCategories.LC_CTYPE].LC_CTYPE.dg)!==-1;},ctype_graph:function(text){if(typeof text!=='string'){return false;}
   this.setlocale('LC_ALL',0);return text.search(this.php_js.locales[this.php_js.localeCategories.LC_CTYPE].LC_CTYPE.gr)!==-1;},ctype_lower:function(text){if(typeof text!=='string'){return false;}
   this.setlocale('LC_ALL',0);return text.search(this.php_js.locales[this.php_js.localeCategories.LC_CTYPE].LC_CTYPE.lw)!==-1;},ctype_print:function(text){if(typeof text!=='string'){return false;}
   this.setlocale('LC_ALL',0);return text.search(this.php_js.locales[this.php_js.localeCategories.LC_CTYPE].LC_CTYPE.pr)!==-1;},ctype_punct:function(text){if(typeof text!=='string'){return false;}
   this.setlocale('LC_ALL',0);return text.search(this.php_js.locales[this.php_js.localeCategories.LC_CTYPE].LC_CTYPE.pu)!==-1;},ctype_space:function(text){if(typeof text!=='string'){return false;}
   this.setlocale('LC_ALL',0);return text.search(this.php_js.locales[this.php_js.localeCategories.LC_CTYPE].LC_CTYPE.sp)!==-1;},ctype_upper:function(text){if(typeof text!=='string'){return false;}
   this.setlocale('LC_ALL',0);return text.search(this.php_js.locales[this.php_js.localeCategories.LC_CTYPE].LC_CTYPE.up)!==-1;},ctype_xdigit:function(text){if(typeof text!=='string'){return false;}

   При беглом просмотре бросился мне тут в глаза кусочек из http://phpjs.org/downloads/php.common.min.js

   3.14159265, 06 Марта 2012

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

6. Objective C / Говнокод #9618

   −85

   1. 1

   http://pastebin.com/UaTdN6K2

   А выложу ка я это тут.
   Гуано трёхлетней выдержки. Фрагментации не поддаётся. Ныне отправлено на удобрения.

   krypt, 06 Марта 2012

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

7. JavaScript / Говнокод #9617

   +158

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

   function SetCeilingType(Txt) {
   
       if (Txt == "1") {
           document.getElementById("сeilType").value = Txt //"Одноуровневый";
       }
       else {
           document.getElementById("сeilType").value = Txt //"Двухуровневый";
       }
       createLink();
   }

   Сраная школота уходит в буддизм.
   Вот такая сентенция была найдена за flash-кодером, хуже всего что функция глобальна, так как дергается ИЗ ФЛЕША при изменении в нем данных. Творческое прочтение паттерна инверсии зависимости, сцуко!

   Meettya, 06 Марта 2012

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

8. Java / Говнокод #9615

   +78

   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

   Drawable d = getResources().getDrawable(R.drawable.screen_width);
           int width = d.getMinimumWidth();
           switch (width) {
               case 1024:
                   CAMERA_WIDTH = 1024;
                   CAMERA_HEIGHT = 600;
                   break;
               case 800:
                   CAMERA_WIDTH = 800;
                   CAMERA_HEIGHT = 480;
                   break;
               case 480:
                   CAMERA_WIDTH = 480;
                   CAMERA_HEIGHT = 320;
                   break;
               case 320:
                   CAMERA_WIDTH = 320;
                   CAMERA_HEIGHT = 240;
                   break;
               default:
                   CAMERA_WIDTH = 800;
                   CAMERA_HEIGHT = 480;
                   break;
           }

   Вот так автор определял размер экрана в Android...по размеру загруженной картинки в ресурсах
   Про getResources().getConfiguration().screen Layout наверное он не знал...

   TeknoMatik, 05 Марта 2012

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

9. Objective C / Говнокод #9614

   −111

   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
   74. 74
   75. 75
   76. 76
   77. 77
   78. 78
   79. 79
   80. 80
   81. 81
   82. 82
   83. 83
   84. 84
   85. 85
   86. 86
   87. 87
   88. 88
   89. 89
   90. 90
   91. 91
   92. 92
   93. 93
   94. 94
   95. 95
   96. 96
   97. 97
   98. 98

   -(void)handleNotification:(id<INotification>)notification
   {
   	NSLog(@"NOTIFICATION:%@", [notification description]);
   	if ([[notification name] isEqualToString:ShowMetronomeInTab]) 
   	{
   		[self.viewComponent showMetronomeInTab];
   	}
   	else if ([[notification name] isEqualToString:AlertOnScreen]) 
   	{
   		isShowRateAlert = YES;
   	}
   	else if ([[notification name] isEqualToString:NoAlertOnScreen]) 
   	{
   		isShowRateAlert = NO;
   	}
   	else if ([[notification name] isEqualToString: ShowHowToEnter]) 
   	{
   		id body = [notification body];
   		[self.viewComponent showHowToEnter:body];
   	}
   	else if ([[notification name] isEqualToString:ShowResultView]) 
   	{
   		[self.viewComponent showResultView:[notification body] Animated:YES];
   	}
   	else if ([[notification name] isEqualToString:ShowIndicatorView]) 
   	{
           if ([[notification body] isKindOfClass:[NSString class]]) 
           {
               [self.viewComponent showLoadingIndicator:[notification body]];
           }
           else 
           {
               if ([[notification body] isKindOfClass:[NSArray class]]) 
               {
                   [self.viewComponent showLoadingIndicator:[[notification body] objectAtIndex:0] WithSecondString:[[notification body]objectAtIndex:1]];
               }
           }		
   	}
       else if ([[notification name] isEqualToString:ShowBlackIndicatorView]) 
   	{
           [self.viewComponent showBlackLoadingIndicator:[notification body]];	
   	}
   	else if ([[notification name] isEqualToString:RemoveIndicatorView]) 
   	{
   		[self.viewComponent removeLoadingIndicator];
   	}
   	else if ([[notification name] isEqualToString:ShowTimeOutAlert]) 
   	{
   		[self.viewComponent displayAlertOnConnectionTimeOut];
   	}
   	else if ([[notification name] isEqualToString:ShowNetworkUnavailableAlert]) 
   	{
   		[self.viewComponent displayAlertOnNetworkError];
   	}
   	else if ([[notification name] isEqualToString:ShowServiceUnavailableAlert]) 
   	{
   		[self.viewComponent displayAlertOnConnectionError];
   	}
   	else if ([[notification name] isEqualToString:ShowHomeView]) 
   	{
   		[self.viewComponent showHomeView];
   	}
   	else if ([[notification name] isEqualToString:ShowProcessingView]) 
   	{
   		[self.viewComponent showProcessingView:[notification body]];
   	} else if ([[notification name] isEqualToString:ShowResultViewNotAnimated]) 
   	{
   		[self.viewComponent showResultView:[notification body] Animated:NO];
   	} else if ([[notification name] isEqualToString:ShowTimeOutAlertWithParent]) 
   	{
   		[self.viewComponent displayAlertOnConnectionTimeOutWithParent:notification.body];
   	}
   	else if ([[notification name] isEqualToString:ShowNetworkUnavailableAlertWithParent]) 
   	{
   		[self.viewComponent displayAlertOnNetworkErrorWithParent:notification.body];
   	}
   	else if ([[notification name] isEqualToString:ShowServiceUnavailableAlertWithParent]) 
   	{
   		[self.viewComponent displayAlertOnConnectionErrorWithParent:notification.body];
   	} 
       else if ([[notification name] isEqualToString:RotationIndicator]) 
   	{
           [self.viewComponent rotationIndicator];
   	}
   	else if ([[notification name] isEqualToString:StartMusic])
       {
           if (applicationProxy.mediaPlayerWasPlaying)
   		{	
   			applicationProxy.mediaPlayerIsPlaying = YES;
   			[self sendNotification:MediaPlayerChangeState];
   			MPMusicPlayerController *mpPlayer = [MPMusicPlayerController iPodMusicPlayer];
   			[mpPlayer play];
   		}
       }
   	else if ([[notification name] isEqualToString:StopMusic])
       {
           applicationProxy.mediaPlayerIsPlaying  = NO;
   		[self sendNotification:MediaPlayerChangeState];

   И так 50 elseif'ов. И, сцуко, все разные. И в последней трети отформатированы лесенкой.
   Местное видение PureMVC

   krypt, 05 Марта 2012

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

10. Python / Говнокод #9613

    −100

    1. 1
    2. 2
    3. 3

    @type('inline hack', (object,), {'__new__':lambda cls, fnc: wraps(lambda request, *args, **kwargs: setattr(request, 'holder', None) and fnc(request, *args, **kwargs))})
    def create_receiver(request, id):
        pass

    Отсутствие инлайновых декораторов предыдущему фрилансеру не помеха

    nvbn, 05 Марта 2012

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

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

• 2107
• …
• 926
• 925
• 924
• 923
• 922
• 921
• 920
• 919
• 918
• …
• 1