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

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

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

1. PHP (5709)
2. C++ (2740)
3. Куча (2389)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (37)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. Лучший говнокод

   В номинации:

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

   За время:

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

2. JavaScript / Говнокод #18996

   −3

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

   var api = angular.module('GithubApi', []);
   api.controller('GithubStatus', function ($scope, $http) {
       $http.get('/api.json')
           .success(function (response) { $scope.api = response; });
   });

   Пейсатели ж.скрипт фреймоврков не перестают радовать. Код несколько упрощен, но суть должна быть очевидна каждому: переменная api будет затерта как только она будет создана строкой типа "[object Object]" потому что в $scope есть поле с таким же значением.

   wvxvw, 09 Ноября 2015

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

3. Си / Говнокод #16375

   +136

   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

   #include studio.h
   
   main()
   {
     uint16 final,num;
     printf(“enetr the unsigned integer 16bit number “);
     scanf(“%d”, &num);
     final= numbitset(num);
     printf(“%d”, final);
   }
   
   unit16 numbitset( unit16 x)
   {
     int i, j,result, total=0;
     uint16 no,modify
     for(i=1;i<=4;i++)
     {
       j=pow(10,i);
       no= (x%(j))>>(i-1)*4;
       if(no==0)
       {
         result=0;
       }
       else if(no==1)
       {
         result=1;
       }
       else if(no==2)
       {
         result=1;
       }
       else if(no==3)
       {
         result=2;
       }
      else
      {
         result = othernum(no/4)+othernum(no%4);	
       }
       total = total+result;
     }
   }
   
   uint16 othernum( uint16 y)
   {
     switch(y)
     {
       case 0:
         return(0);
         break;
     case 1:
         return(1);
        break;
     case 2:
       return(1);
       break;
      case 3:
         return(2);
         break;
      default:
       return;
       break;
     }
   }

   Посчитать количество значащиз битов в 16ти разрядном целом. Реальный тест на собеседовании дал такой вот результат. Угадайте откуда кандидат :)

   multilexa, 18 Июля 2014

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

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

   +152

   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

   function get_price () {
   	
   	$item_price =  get_p('item_price');
   	
   	$price = str_split($item_price);
   	
   	if ( sizeof($price)<=1 and sizeof($price) !== 0 ) {
   		
   		echo '--'; 
   		
   		} 
   	
   	if ( sizeof($price)<=3 and sizeof($price) !== 0 ) {
   		
   		echo get('item_price'); 
   		
   		} 
   			
   	if ( sizeof($price)<=4 and sizeof($price) !== 3  and sizeof($price) !== 2) {
   		
   		echo $price[0].' ';
   		
   		for ($i=1; $i<=sizeof($price);$i++) {
   			
   			echo $price[$i];
   			
   			}
   		
   		}
   		
   	if ( sizeof($price)<=5 and sizeof($price) !== 4 and sizeof($price) !== 3 and sizeof($price) !== 2  ) {
   		
   		echo $price[0].$price[1].' ';
   		
   		for ($i=2; $i<=sizeof($price);$i++) {
   			
   			echo $price[$i];
   			
   			}
   		
   		}	
   
   
     // И еще 2 подобных условия
   		
   
   	}

   Взял заказ на фрилансе с доработкой магазина. Так вот эта функция делает из цены вида 1740 такую 1 740, то есть добавляет пробел. Супер серверный хардкор, js не не слышал ...

   werdan, 21 Октября 2013

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

5. Python / Говнокод #12850

   −99

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

   # 2017991 => 20/17/99
                   # 658581 => 65/85
                   # 6585 => 65
   
                   id = id[:(len(id)%2) - 2]
                   subfolders = ''.join([(i and i % 2 == 0 and '/' or '') + x for  i, x  in enumerate(id)])

   В 4 часа утра написал такой вот щит. Можно по вашему мнению это как-то упросить?

   sbb, 04 Апреля 2013

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

6. Java / Говнокод #12211

   +78

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

   if ((act_type == 15) || (act_type == 24)) {
   } else {
               if (duration != null)
               dts.setDuration(dts.getDuration().add(duration));
   }

   Вся пунктуация сохранена и НИЧЕГО НЕ ВЫРЕЗАНО
   Волшебный if
   Пруф [[ скрыт ]]
   строка 112 - 116

   smpl, 28 Ноября 2012

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

7. Pascal / Говнокод #8744

   +93

   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

   procedure _FinalizeArray(p: Pointer; typeInfo: Pointer; elemCount: Cardinal);
   asm
           CMP     ECX, 0                        { no array -> nop }
           JE      @@zerolength
   
           PUSH    EAX
           PUSH    EBX
           PUSH    ESI
           PUSH    EDI
           MOV     EBX,EAX
           MOV     ESI,EDX
           MOV     EDI,ECX
   
           XOR     EDX,EDX
           MOV     AL,[ESI]
           MOV     DL,[ESI+1]
   
           CMP     AL,tkLString
           JE      @@LString
   
           CMP     AL,tkWString
           JE      @@WString
   
           CMP     AL,tkVariant
           JE      @@Variant
   
           CMP     AL,tkArray
           JE      @@Array
   
           CMP     AL,tkRecord
           JE      @@Record
   
           CMP     AL,tkInterface
           JE      @@Interface
   
           CMP     AL,tkDynArray
           JE      @@DynArray
   
           JMP     @@error
   
   @@LString:
           CMP     ECX,1
           MOV     EAX,EBX
           JG      @@LStringArray
           CALL    _LStrClr
           JMP     @@exit
   @@LStringArray:
           MOV     EDX,ECX
           CALL    _LStrArrayClr
           JMP     @@exit
   
   @@WString:
           CMP     ECX,1
           MOV     EAX,EBX
           JG      @@WStringArray
           CALL    _WStrClr
           JMP     @@exit
   @@WStringArray:
           MOV     EDX,ECX
           CALL    _WStrArrayClr
           JMP     @@exit
   @@Variant:
           MOV     EAX,EBX
           ADD     EBX,16
           CALL    _VarClr
           DEC     EDI
           JG      @@Variant
           JMP     @@exit
   @@Array:
           PUSH    EBP
           MOV     EBP,EDX
   @@ArrayLoop:
           MOV     EDX,[ESI+EBP+2+8]
           MOV     EAX,EBX
           ADD     EBX,[ESI+EBP+2]
           MOV     ECX,[ESI+EBP+2+4]
           MOV     EDX,[EDX]
           CALL    _FinalizeArray
           DEC     EDI
           JG      @@ArrayLoop
           POP     EBP
           JMP     @@exit
   
   @@Record:
           PUSH    EBP
           MOV     EBP,EDX
   @@RecordLoop:
           { inv: EDI = number of array elements to finalize }
   
           MOV     EAX,EBX
           ADD     EBX,[ESI+EBP+2]

   Это так в дельфах автофинализация строк, длиннострок, вариантов, массивов, записей, интерфейсов, динмассивов реализована.
   Вместо того, чтобы напрямую вызвать деструктор, там в рантайме (ПИЗДЕЦ, В РАНТАЙМЕ БЛЯТЬ!!!) проверяется тип поля, требующего финализацию и через три таких жопы наконец-то вызывается деструктор. И это, блять, я ещё создал свой TInterfacedObject, потому что иначе бы деструктор вызывался не через три жопы, а через четыре, и одна из них - системный вызов.
   Да, это при включённой оптимизации всё, если чё.

   TarasB, 06 Декабря 2011

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

8. C++ / Говнокод #8040

   +155

   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

   #include <iostream>
   #include <memory>
   #include <assert.h>
   using namespace std;
    
   template <class T>
   class megaClass
   {
   public:
    
       void hello()
       {
           assert(dynamic_cast<T*>(this)!=NULL);
           static_cast<T*>(this)->hello();
       }
       virtual ~megaClass() {}
   };
    
   class cleft : public megaClass<cleft>
   {
   public:
    
       void hello()
       {
           std::cout << "left::hello()" << std::endl;
       }
   };
    
   class cright : public megaClass<cright>
   {
   public:
    
       void hello()
       {
           std::cout << "right::hello()" << std::endl;
       }
   };
    
    
   int main()
   {
       scoped_ptr<megaClass<cleft> > a1=new cleft;
       a1->hello();
       scoped_ptr<megaClass<cright> > a2=new cright;
       a2->hello();
       
       return 0;
   }

   Пытался продемонстрировать статический полиморфизм TarasB и получилась такая какашка. Кто действительно крут и может сабдж продемонстрировать? Я где-то видел пример, но не помню где...
   
   Ещё продемонстрировал статический полиморфизм через стратегии:

   struct Strategy1
   
   {
   
   static void do(){printf("Lol1");}
   
   };
   
   struct Strategy2
   
   {
   
   static void do(){printf("Lol2");}
   
   };
   
   template<class Strategy>
   
   class MegaClass
   
   {public:
   
   void do()
   
   {
   
     printf("Mega");
   
     Strategy::do();//Класс Strategy можно было и создать для хранения состояния.
   
     printf("/n\");
   
   }
   
   };
   
   //...
   
   
   
   Дальше в разных частях кода создаем:
   
   MegaClass<Strategy1> o;
   
   o.do();
   
   //...
   
   MegaClass<Strategy2> o;
   
   o.do();

   "Один" класс ведёт себя по разному. Понятно, что это не совсем полиморфизм. Но очень часто именно в таком контексте используют динамический полиморфизм, хотя такого статического здесь достаточно выше крыши.
   Плюсы этого подхода :
   1)Создаётся объект в стеке, значит быстро, а не в куче. Хотя можно и не в стеке.
   2)Используется шаблон, значит компиль будет инлайнить.
   
   Минус:
   1)Если понадобится резкой перейти от статического полиморфизма к динамическому - придётся переписывать на виртуальные функции или на истинный статический полиморфизм.
   
   Обсуждения здесь:
   

   http://govnokod.ru/8025#comment110773

   
   
   Сразу исключим детсадовский вариант статического функционального полиморфизма c перегрузкой функций:

   Class1 o1;
   
   foo(o1);
   
   Class2 o2;
   
   foo(o2);
   
   
   
   void foo(Class1 o){/*...*/};
   
   void foo(Class2 o){/*...*/};

   
   
   
   Кто-нибудь реально умеет can into нормальный статический полиморфизм?

   CPPGovno, 30 Сентября 2011

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

9. C++ / Говнокод #7978

   +151

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

   void f_ai(intrusive_ptr<serial> i);
   //...
   static serial ai;
   serial* ii=&ai;
   //...
   f_ai(ii);

   Компилируется, а потом грохается. С++ такой Си...

   CKrestKrestGovno, 25 Сентября 2011

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

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

    0

    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

    <?php
    error_reporting(0);
    ini_set('display_errors', '0');
    mb_internal_encoding('UTF-8');
    header('Access-Control-Allow-Origin: *');
    function str_replace_first($from, $to, $subject){
    $from='/'.preg_quote($from, '/i').'/';
    return preg_replace($from, $to, $subject, 1);
    }
    require_once('morphy/src/common.php');
    $opts=array(
    'storage'=>PHPMORPHY_STORAGE_MEM,
    'predict_by_suffix'=>true, 
    'predict_by_db'=>true,
    'graminfo_as_text'=>true
    );
    $dir='morphy/dicts';
    $lang='ru_RU';
    try{
    $morphy=new phpMorphy($dir, $lang, $opts);
    }catch(phpMorphy_Exception $e){
    die('Error occurred while creating phpMorphy instance: '.PHP_EOL.$e);
    }
    $morphy->getDefaultGrammemsProvider()->excludeGroups('С', 'род');
    $wanal=array();
    $wanal['p']=explode(' : ', file_get_contents('wanal_p.txt'));
    $wanal['s']=explode(' : ', file_get_contents('wanal_s.txt'));
    $wanal['g']=explode(' : ', file_get_contents('wanal_g.txt'));
    shuffle($wanal['p']);
    shuffle($wanal['s']);
    shuffle($wanal['g']);
    $current_wanal=$wanal;
    $string=trim($_GET['q']);
    $modified_comment=$string;
    preg_match_all('/(\w+)/u', str_replace('_', '', $string), $matches);
    $words=$matches[1];
    foreach($words as $word){
    if(mb_strtoupper($word)=='НАХУЙ' || mb_strtoupper($word)=='ХУЙ' || mb_strtoupper($word)=='ТУТ'){
    continue;
    }
    $info=$morphy->getGramInfo(mb_strtoupper($word));
    $type='';
    if(trim($info[0][0]['pos'])=='П'){
    $type='p';
    }
    if(trim($info[0][0]['pos'])=='С'){
    $type='s';
    }
    if(trim($info[0][0]['pos'])=='Г' || trim($info[0][0]['pos'])=='ИНФИНИТИВ'){
    $type='g';
    }
    if($type!=''){
    $replacement=trim($current_wanal[$type][array_rand($wanal[$type])]);
    if($replacement!=''){
    $casted=$morphy->castFormByPattern(mb_strtoupper($replacement), mb_strtoupper($word), null, true);
    if(count($casted)>0){
    $replacement=reset($casted);
    if(md5(mb_substr($word, 0, 1))==md5(mb_strtolower(mb_substr($word, 0, 1)))){
    $replacement=mb_strtolower($replacement);
    } elseif(md5(mb_strtoupper($word))==md5($word)){
    $replacement=mb_strtoupper($replacement);
    } elseif(md5(mb_substr($word, 0, 1))==md5(mb_strtoupper(mb_substr($word, 0, 1)))){
    $replacement=mb_convert_case(mb_strtolower($replacement), MB_CASE_TITLE);
    }
    $modified_comment=str_replace_first($word, $replacement, $modified_comment);
    $base=$morphy->getBaseForm(mb_strtoupper($word));
    if($base){
    $wanal[$type][]=mb_strtoupper($base[0]);
    $wanal[$type]=array_filter(array_unique($wanal[$type]));
    }
    }
    }
    }
    }
    file_put_contents('wanal_p.txt', implode(' : ', $wanal['p']));
    file_put_contents('wanal_s.txt', implode(' : ', $wanal['s']));
    file_put_contents('wanal_g.txt', implode(' : ', $wanal['g']));
    echo $modified_comment;
    ?>

    Какой-то такой модификатор чужих комментариев на основе ещё более ранних комментариев.

    g0_1494089148657, 14 Февраля 2018

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

11. Куча / Говнокод #23501

    +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
    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

    [0mSaving to: ���jdk-8u131-linux-x64.tar.gz���
    
      2050K .......... .......... .......... .......... ..........  1%  137K 4m16s
      2100K .......... .......... .......... .......... ..........  1% 1.06M 4m14s
      2150K .......... .......... .......... .......... ..........  1%  613K 4m14s
      2200K .......... .......... .......... .......... ..........  1%  318M 4m9s
      2250K .......... .......... .......... .......... ..........  1%  609K 4m10s
      2300K .......... .......... .......... .......... ..........  1%  612K 4m10s
      2350K .......... .......... .......... .......... ..........  1%  610K 4m11s
      2400K .......... .......... .......... .......... ..........  1%  612K 4m12s
      2450K .......... .......... .......... .......... ..........  1%  612K 4m13s
      2500K .......... .......... .......... .......... ..........  1%  616K 4m13s
      2550K .......... .......... .......... .......... ..........  1%  616K 4m14s
      2600K .......... .......... .......... .......... ..........  1%  614K 4m15s
      2650K .......... .......... .......... .......... ..........  1%  644K 4m15s
      2700K .......... .......... .......... .......... ..........  1% 1.15M 4m13s
      2750K .......... .......... .......... .......... ..........  1% 1.07M 4m11s
      2800K .......... .......... .......... .......... ..........  1%  306K 4m17s
      2850K .......... .......... .......... .......... ..........  1%  181M 4m13s
      2900K .......... .......... .......... .......... ..........  1%  307K 4m18s
      2950K .......... .......... .......... .......... ..........  1%  608K 4m19s
      3000K .......... .......... .......... .......... ..........  1%  421K 4m21s
      3050K .......... .......... .......... .......... ..........  1%  400K 4m24s
      3100K .......... .......... .......... .......... ..........  1%  605K 4m25s
      3150K .......... .......... .......... .......... ..........  1%  613K 4m25s
      3200K .......... .......... .......... .......... ..........  1%  611K 4m25s
      3250K .......... .......... .......... .......... ..........  1%  303K 4m30s
      3300K .......... .......... .......... .......... ..........  1% 1.34M 4m28s
      3350K .......... .......... .......... .......... ..........  1%  306K 4m32s
      3400K .......... .......... .......... .......... ..........  1%  394K 4m35s
      3450K .......... .......... .......... .......... ..........  1%  416K 4m37s
      3500K .......... .......... .......... .......... ..........  1%  401K 4m39s
      3550K .......... .......... .......... .......... ..........  1%  413K 4m41s
      3600K .......... .......... .......... .......... ..........  2%  421K 4m43s
      3650K .......... .......... .......... .......... ..........  2%  611K 4m43s
      3700K .......... .......... .......... .......... ..........  2%  610K 4m43s
      3750K .......... .......... .......... .......... ..........  2%  574K 4m43s
      3800K .......... .......... .......... .......... ..........  2%  610K 4m43s
      3850K .......... .......... .......... .......... ..........  2%  610K 4m43s
      3900K .......... .......... .......... .......... ..........  2%  611K 4m43s
      3950K .......... .......... .......... .......... ..........  2%  651K 4m43s
      4000K .......... .......... .......... .......... ..........  2%  613K 4m43s
      4050K .......... .......... .......... .......... ..........  2%  612K 4m43s
      4100K .......... .......... .......... .......... ..........  2% 1.08M 4m42s

    Дженкинс прислал телеграму: девопс потрудился наславу.
    (это только 0.5% от всего сообщения)

    wvxvw, 08 Ноября 2017

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

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

• 1975
• …
• 1882
• 1881
• 1880
• 1879
• 1878
• 1877
• 1876
• 1875
• 1874
• …
• 1