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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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. Лучший говнокод

   В номинации:

   • Лучший рейтинг
   • Самый комментируемый

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

2. Куча / Говнокод #8249

   +145

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

   500 Internal Server Error
   
   К сожалению, произошел какой-то сбой.
   
   Бригада скорой помощи уже выехала по этому адресу.

   AnimeGovno-_-, 20 Октября 2011

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

3. JavaScript / Говнокод #8235

   +159

   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

   protected function paperSizeOrientationValidator(p_printpropertiesobj:Object):Boolean
         	{
         		var ret_val:Boolean = (p_printpropertiesobj.printJob.paperWidth == _printPaperTab.paperWidth * PRINT_DPI &&
   		 			p_printpropertiesobj.printJob.paperHeight == _printPaperTab.paperHeight * PRINT_DPI)
   		 			||
   					(p_printpropertiesobj.printJob.paperHeight == _printPaperTab.paperWidth * PRINT_DPI &&
   		 			p_printpropertiesobj.printJob.paperWidth == _printPaperTab.paperHeight * PRINT_DPI)
   		 			|| 
   		 			((p_printpropertiesobj.printJob.paperWidth > _paperPrintMarginLow * _printPaperTab.paperWidth * PRINT_DPI && p_printpropertiesobj.printJob.paperWidth < _paperPrintMarginHigh * _printPaperTab.paperWidth * PRINT_DPI) &&  
   		 			(p_printpropertiesobj.printJob.paperHeight > _paperPrintMarginLow * _printPaperTab.paperHeight * PRINT_DPI && p_printpropertiesobj.printJob.paperHeight < _paperPrintMarginHigh * _printPaperTab.paperHeight * PRINT_DPI))
   		 			||
   		 			((p_printpropertiesobj.printJob.paperHeight > _paperPrintMarginLow * _printPaperTab.paperWidth * PRINT_DPI && p_printpropertiesobj.printJob.paperHeight < _paperPrintMarginHigh * _printPaperTab.paperWidth * PRINT_DPI) &&  
   		 			(p_printpropertiesobj.printJob.paperWidth > _paperPrintMarginLow * _printPaperTab.paperHeight * PRINT_DPI && p_printpropertiesobj.printJob.paperWidth < _paperPrintMarginHigh * _printPaperTab.paperHeight * PRINT_DPI))
         		
         		return ret_val;
         	}

   This is actually ActionScript. An unknown former colleague was trying to "validate paper orientation" before sending the page to the printer...

   bot225, 19 Октября 2011

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

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

   +160

   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

   <?php
   class Html {
   	static private $ShowSiblings; // Управляет отображением похожих категорий
   	static function Initialize($showSiblings) {
   		self::$ShowSiblings = $showSiblings;
   	}
   	static function SimpleLink($target, $text) { // Простая ссылка
   		return '<a href="'.$target.'">'.$text.'</a>';
   	}
   	static function CategorySiblingLink($targetCategory, $text) { // Формирует ссылку на похожую категорию
   		return '<a href="products.php?cat='.$targetCategory.'&page=1">'.$text.'</a>';
   	}
   	static function CategoryLink($targetCategory, $text) { // Формирует ссылку на указанную категорию
   		return '<a href="products.php?cat='.$targetCategory.'&page=1">'.$text.'</a>';
   	}
   }
   ?>

   Один из классов каталога продукции.

   rasufa, 12 Октября 2011

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

5. SQL / Говнокод #7999

   −853

   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

   Create Proc char32add1(  
           @char32 nvarchar(1) =Null, 
   	@rez nvarchar(1) output, 
   	@rez1 int=-1 output 
                      )  
   As 
     Begin 
   if (@char32 is Null) return -1 
   Select @rez1=0 
   if (@char32 = "0") Begin Select  @rez="1" 
    return 0 End 
   if (@char32 = "1")  Begin Select  @rez="2"  
   return 0 End 
   if (@char32 = "2")  Begin Select  @rez="3"  
   return 0 End 
   if (@char32 = "3")  Begin Select  @rez="4"  
   return 0 End 
   if (@char32 = "4")  Begin Select  @rez="5" return 0 End 
   if (@char32 = "5")  Begin Select  @rez="6" return 0 End 
   if (@char32 = "6")  Begin Select  @rez="7" return 0 End 
   if (@char32 = "7")  Begin Select  @rez="8" return 0 End 
   if (@char32 = "8")  Begin Select  @rez="9" return 0 End 
   if (@char32 = "9")  Begin Select  @rez="A" return 0 End 
   if (@char32 = "a" or @char32 = "A" )  Begin Select  @rez="B" return 0 End 
   if (@char32 = "b" or @char32 = "B" )  Begin Select  @rez="C" return 0 End 
   if (@char32 = "c" or @char32 = "C" )  Begin Select  @rez="D" return 0 End 
   if (@char32 = "d" or @char32 = "D" )  Begin Select  @rez="E" return 0 End 
   if (@char32 = "e" or @char32 = "E" )  Begin Select  @rez="F" return 0 End 
   if (@char32 = "f" or @char32 = "F" )  Begin Select  @rez="G" return 0 End 
   if (@char32 = "g" or @char32 = "G" )  Begin Select  @rez="H" return 0 End 
   if (@char32 = "h" or @char32 = "H" )  Begin Select  @rez="I" return 0 End 
   if (@char32 = "i" or @char32 = "I" )  Begin Select  @rez="J" return 0 End 
   if (@char32 = "j" or @char32 = "J" )  Begin Select  @rez="K" return 0 End 
   if (@char32 = "k" or @char32 = "K" )  Begin Select  @rez="L" return 0 End 
   if (@char32 = "l" or @char32 = "L" )  Begin Select  @rez="M" return 0 End 
   if (@char32 = "m" or @char32 = "M" )  Begin Select  @rez="N" return 0 End 
   if (@char32 = "n" or @char32 = "N" )  Begin Select  @rez="O" return 0 End 
   if (@char32 = "o" or @char32 = "O" )  Begin Select  @rez="P" return 0 End 
   if (@char32 = "p" or @char32 = "P" )  Begin Select  @rez="Q" return 0 End 
   if (@char32 = "q" or @char32 = "Q" )  Begin Select  @rez="R" return 0 End 
   if (@char32 = "r" or @char32 = "R" )  Begin Select  @rez="S" return 0 End 
   if (@char32 = "s"or @char32 = "S" )  Begin Select  @rez="T" return 0 End 
   if (@char32 = "t" or @char32 = "T" )  Begin Select  @rez="U" return 0 End 
   if (@char32 = "u" or @char32 = "U" )  Begin Select  @rez="V" return 0 End 
   Select @rez1=1 
   if (@char32 = "v" or @char32 = "V" )  Begin Select  @rez="0" return 1 End 
    
   End

   CPlusPlusGovno, 27 Сентября 2011

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

6. Ruby / Говнокод #7902

   −105

   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

   def show
       @updates = UpdateList.new
       update_id = params[:update_id]
   
       if 'twitter' == @provider_key
         provider_api = ProviderApi::Twitter.new(current_user)
   
         if update_id.present?
           begin
             benchmark(" Twitter API: status") do
               @api_response = provider_api.status(update_id)
             end
             @update = update = SocialUpdate.from_twitter_response(@api_response, true)
             while update && (in_reply_to = update.in_reply_to_update_id)
               benchmark(" Twitter API: status") do
                 @previous_status = provider_api.status(in_reply_to)
               end
   
               if error = @previous_status['error']
                 @updates << SocialUpdate.from_twitter_error(error)
                 break
               else
                 update = SocialUpdate.from_twitter_response(@previous_status, true)
                 @updates << update
               end
             end
           rescue => e
             logger.info("Error in fetching status #{in_reply_to || update_id}: #{e}")
           end
         end
       end
   
       @update.flag_for_user(current_user) if @update
       @updates.flag_for_user(current_user)
     end

   sumskyi, 19 Сентября 2011

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

7. C++ / Говнокод #7895

   +143

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

   void crash()
   {
       (( void(*)() )0)();
   }

   Crash ;]

   petersvp, 19 Сентября 2011

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

8. Objective C / Говнокод #7875

   −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

   if(selectedLanguage==French)
       {
   
           FinalString = [[NSString alloc] initWithFormat:@"http://www.xyz.com/api_com.php?page_id=%d",IDValue];
           url = [[NSURL alloc] initWithString:FinalString];
       }
       else if(selectedLanguage==German)
       {
   
           FinalString = [[NSString alloc] initWithFormat:@"http://www.x.com/api_com.php?page_id=%d",IDValue];
           url = [[NSURL alloc] initWithString:FinalString];
       }
       else if(selectedLanguage==Nepali)
       {
   
           FinalString = [[NSString alloc] initWithFormat:@"http://www.xy.com/api_com.php?page_id=%d",IDValue];
           url = [[NSURL alloc] initWithString:FinalString];
   
       }

   mazder, 16 Сентября 2011

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

9. SQL / Говнокод #7748

   −116

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

   --Если идентификатор аккаунта не передан возвращаем NULL
   IF @AccountID <= 0 OR @AccountID IS NULL RETURN 3;
   
   --Для администраторов всегда возвращаем NULL
   IF @Role = 'admin' OR @Role = 'super_admin' RETURN 3;

   Тело одной из хранимых процедур. Комментарии, видать, протухли.

   dante, 02 Сентября 2011

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

10. PHP / Говнокод #7715

    +161

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

    class OnBeginRequest
    {
    	public static function run( CEvent $event )
    	{
    		Order::model()->onOrderCreate = function( Cevent $event ) {};
    	}
    }

    Cервис на yii framework. Какой сакральный смысл заложен, неизвестно. Скорее всего, какой-то невероятной крутости костыль, типа не срабатывало событие.

    mitallast, 31 Августа 2011

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

11. PHP / Говнокод #7606

    +161

    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

    // mysql_affected_rows порядка 200 записей
    while ($row_ex=mysql_fetch_array($res_ex)){
       $e_arr[trim(strtoupper($row_ex[0]))] = $row_ex[1];
        $exc .= "(".preg_replace("/(\*|\+|\(|\))/","\\\\$0",$row_ex[0])."(\ |\,|\<|\n|\.)+)|";
    }
    
    if ($exc){
        preg_match_all("/(".$exc.")(?!<\/a>)/i",$Tour['info'],$maches);
         $mach=array_unique($maches[0]);
         unset($maches);
         foreach ($mach as $e_name){
           $last_c=substr($e_name,-1);
           if ($last_c=="<" || $last_c=="," || $last_c==".") $e_name = substr($e_name,0,-1);
           if ($e_name[trim(strtoupper($e_name))])
           $Tour['info'] = preg_replace("/$e_name(?!.?<\/a>)/","<a href='".$e_arr[trim(strtoupper($e_name))]."\.html'>".$e_name."<\/a>",$Tour['info']);
         }
    
        preg_match_all("/(".$exc.")(?!<\/a>)/i",$Tour['info1'],$maches);
         $mach=array_unique($maches[0]);
         unset($maches);
         foreach ($mach as $e_name){
           $last_c=substr($e_name,-1);
           if ($last_c=="<" || $last_c=="," || $last_c==".") $e_name = substr($e_name,0,-1);
    $e_arr[trim(strtoupper($e_name))]."\.html'>".$e_name."<\/a>",$Tour['info1']);
         }
    
        preg_match_all("/(".$exc.")(?!<\/a>)/i",$Tour['info2'],$maches);
         $mach=array_unique($maches[0]);
         unset($maches);
         foreach ($mach as $e_name){
           $last_c=substr($e_name,-1);
           if ($last_c=="<" || $last_c=="," || $last_c==".") $e_name = substr($e_name,0,-1);
    $e_arr[trim(strtoupper($e_name))]."\.html'>".$e_name."<\/a>",$Tour['info2']);
         }
    
        preg_match_all("/(".$exc.")(?!<\/a>)/i",$Tour['shedulle'],$maches);
         $mach=array_unique($maches[0]);
         unset($maches);
         foreach ($mach as $e_name){
           $last_c=substr($e_name,-1);
           if ($last_c=="<" || $last_c=="," || $last_c==".") $e_name = substr($e_name,0,-1);
         }
    
      }

    http://govnokod.ru/7602 "Регулярки? Да, слышал!"
    Итого + 8-10 секунд на исполнение.

    govnoacc, 21 Августа 2011

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

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

• 1963
• …
• 43
• 42
• 41
• 40
• 39
• 38
• 37
• 36
• 35
• …
• 1