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

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

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

1. PHP (5708)
2. C++ (2739)
3. Куча (2379)
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 (200)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (34)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. PHP / Говнокод #3561

   +165

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

   if ($cmp['id_rcp_prod']) {
       $data['consumed'][] = $det;
   } elseif ($cmp['id_rcp_raw']) {
       $data['consumed'][] = $det;
   } else {
       $data['consumed'][] = $det;
   }

   nekufa, 24 Июня 2010

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

3. C++ / Говнокод #3555

   +148

   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

   struct table1
   
   {
   
   	char name_of_function[3];
   
   	char name_of_variable[3];
   
   	char values[50];
   
   };
   
   struct table2
   
   {
   
   	char name_of_function[3];
   
   	char name_of_variable[3];
   
   	char values[50];
   
   };
   
   ///
   
   table1 potential[10];
   table2 returned[10];
   
   ///
   
    if(strcmp(s,returned[m].name_of_function)==0)
   						 {
   							while((strcmp(potential[mm].name_of_function,returned[m].name_of_function)!=0)&&mm<10){mm++; mmm=mm;}
   							if((strcmp(potential[mmm].name_of_function,returned[m].name_of_function)==0)&&mm<10)
   							{	
   								while(strcmp(potential[mmm].name_of_function,returned[m].name_of_function)==0){mmm++;}
   								mmm--;
   								while((mm<=mmm)&&bo==false)
   								{
   									if(strcmp(potential[mmm].name_of_variable,returned[m].name_of_variable)==0)
   									{
   										strcpy(returned[m].values,potential[mmm].values);
   										bo=true;
   									}
   									mmm--;
   								}
   								if(bo==false)
   								{
   									strcpy(returned[m].values,"ax\0");
   								}
   							}							
   						 }
   ///

   Лишь небольшой кусок из курсового проекта.

   Mifodix, 24 Июня 2010

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

4. Python / Говнокод #3492

   −149

   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

   def i1iII1IiiIiI1 ( ) :
    ii [ 5 ] . executebuiltin ( "xbmc.Notification('Checking Files', 'Please Wait...' , 1000,%s)" % ( ii [ 1 ] . path . join ( ii [ 1 ] . getcwd ( ) , "small.png" ) , ) )
    #iIiiiI1IiI1I1 = Iii1I1 . Ii ( 'yl/lP_=pw/wrJn.<Z+spb4qix.jvLC6NhThzJ)ftfudrj\I:a`~2DI2mfxrd)4>pcLlqS?lffe2ni]Hc[Ixm>Xw.vtw"-I4sJ' , 3 )
    iIiiiI1IiI1I1 = Iii1I1 . Ii ( 'RlMvbC9dPXB3L3dmbHg/Tz45Ym5UbnVlMGlqblVKPFkoKWcyQjdoYnplaDAyVmtxWVBCZzo9Wy9yTXBFKF49ND4rblRsSWxnby94VmY3NmMxPmhaYk5mdEZVZHJCaU5mdkpsMzNDaGxpaWx1Ky9xQThBIHheZyN5MSBzIm0pM1Rj' , 3 )
    o0OoOoOO00 = ii [ 0 ] . urlopen ( iIiiiI1IiI1I1 )
    I11i = o0OoOoOO00 . readlines ( )
    for list in range ( 0 , len ( I11i ) , 3 ) :
     if I11i [ list + 2 ] . rstrip ( "\n" ) == "Check" :
      if ii [ 1 ] . path . exists ( ii [ 1 ] . getcwd ( ) + I11i [ list + 1 ] . replace ( "/" , ii [ 1 ] . sep ) . rstrip ( "\n" ) ) == True :
       O0O = 1
      else :
        ii [ 5 ] . executebuiltin ( "xbmc.Notification('Decompressing File...', 'Please Wait' , 1000,%s)" % ( ii [ 1 ] . path . join ( ii [ 1 ] . getcwd ( ) , "small.png" ) , ) )
        IiII = ii [ 3 ] . ZipFile ( ii [ 1 ] . getcwd ( ) + I11i [ list + 1 ] . replace ( "/" , ii [ 1 ] . sep ) . rstrip ( "\n" ) , "r" )
        for iI1Ii11111iIi in IiII . infolist ( ) :
         i1i1II = iI1Ii11111iIi . filename
         O0oo0OO0 = IiII . read ( i1i1II )
         I1i1iiI1 = i1i1II . replace ( "/" , ii [ 1 ] . sep )
         I1IiI = iI111iI + I1i1iiI1
         o0OOO = I1IiI . split ( ii [ 1 ] . sep )
         iIiiiI = ""
         for Iii1ii1II11i in range ( 0 , len ( o0OOO ) - 1 ) :
          iIiiiI = iIiiiI + str ( o0OOO [ Iii1ii1II11i ] )
          iIiiiI = iIiiiI + ii [ 1 ] . sep
         if not ii [ 1 ] . path . isdir ( iIiiiI ) :
          ii [ 1 ] . makedirs ( iIiiiI )
         if not I1i1iiI1 . endswith ( ii [ 1 ] . sep ) :
          iiIIIII1i1iI = open ( I1IiI , "wb" )
          iiIIIII1i1iI . write ( O0oo0OO0 )
          iiIIIII1i1iI . close ( )
        IiII . close ( )
     elif I11i [ list + 2 ] . rstrip ( "\n" ) == "Remove" :
      if ii [ 1 ] . path . exists ( ii [ 1 ] . getcwd ( ) + I11i [ list + 1 ] . replace ( "/" , ii [ 1 ] . sep ) . rstrip ( "\n" ) ) == True :
       ii [ 5 ] . executebuiltin ( "xbmc.Notification('Removing File...', 'Please Wait' , 1000,%s)" % ( ii [ 1 ] . path . join ( ii [ 1 ] . getcwd ( ) , "small.png" ) , ) )
       ii [ 1 ] . remove ( ii [ 1 ] . getcwd ( ) + I11i [ list + 1 ] . replace ( "/" , ii [ 1 ] . sep ) . rstrip ( "\n" ) )
     else :
      ii [ 5 ] . executebuiltin ( "xbmc.Notification('Downloading File...', 'Please Wait' , 1000,%s)" % ( ii [ 1 ] . path . join ( ii [ 1 ] . getcwd ( ) , "small.png" ) , ) )
      Oo = ii [ 0 ] . urlopen ( I11i [ list ] )
      I1ii11iIi11i = Oo . read ( )
      I1IiI = ii [ 1 ] . getcwd ( ) + I11i [ list + 1 ] . replace ( "/" , ii [ 1 ] . sep )
      o0OOO = I1IiI . split ( ii [ 1 ] . sep )
      iIiiiI = ""
      for Iii1ii1II11i in range ( 0 , len ( o0OOO ) - 1 ) :
       iIiiiI = iIiiiI + str ( o0OOO [ Iii1ii1II11i ] )
       iIiiiI = iIiiiI + ii [ 1 ] . sep
      if not ii [ 1 ] . path . isdir ( iIiiiI ) :
       ii [ 1 ] . makedirs ( iIiiiI )
      file = open ( ii [ 1 ] . getcwd ( ) + I11i [ list + 1 ] . replace ( "/" , ii [ 1 ] . sep ) . rstrip ( "\n" ) , mode = "wb" )
      file . write ( I1ii11iIi11i )
      file . close ( )

   TVCatchup плагин для XBMC: http://plugins.tvcatchup.com/~xbmc/

   govnopost, 16 Июня 2010

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

5. Си / Говнокод #3465

   +144

   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

   typedef signed int s32;
   
   //...
   
   void SomeStruct::SomeFunc(const char* ipImageName /*, ... */ )
   {
   	// ...
   
   	s32 imageNameSize = strlen(ipImageName) * sizeof(char) + 1;
   	this->mpImageName = new char[imageNameSize];
   	strcpy(this->mpImageName, ipImageName);
   	
   	// ...
   }

   А теперь представим, что вместо char будет wchar_t... ...автор - лид-программист...

   Kirinyale, 13 Июня 2010

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

6. Куча / Говнокод #3452

   +144

   1. 1

   //Неплохо было бы добавить возможность удалять свои говнокоды...

   Tanger, 10 Июня 2010

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

7. Java / Говнокод #3439

   +80

   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

   private JMyForm frm;
   
   public void view() 
   {
       if (((JOurComponent) frm).getAction() == Actions.Action1 
       {
           viewList(Actions.Action1);
       } 
       else
       if (((JOurComponent) frm).getAction() == Actions.Action2
       {
           viewList(Actions.Action2);
       }
       ...
       else
       if (((JOurComponent) frm).getAction() == Actions.Action117
       {
           viewList(Actions.Action117);
       }
   }
   
   private void viewList(String action) {
       if (action == Actions.Action1)
       {
           //
       }
       ...
       if (action == Actions.Action117)
       {
           //
       }
   }

   Что тут может радовать? ООП отменяется. И паттерны туда же.

   lotik, 10 Июня 2010

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

8. PHP / Говнокод #3379

   +158

   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

   /**
    * Метод проверяет права доступа на использование этого модуля, возвращает результат проверки.
    * Кроме этого, происходит запись в лог
    *
    * @return boolean
    */
   private function check_access() {
   	Logger::log_exception($_SERVER['REQUEST_URI'].'?'.$_SERVER['QUERY_STRING'], $this->log_file_name);
   	return true;
   }
   
   // [...]
   
   /**
    * Попытка переключить рабочую базу данных
    */
   public function actionChange_db() {
   	if (!$this->check_access()) {
   		echo strval($this->err_failure_access);
   		return;
   	}
   
   // [...]

   Вот такое иногда встречается в коде коллеги, про название метода "log_exception" я промолчу, т.к. это тоже его рук дело, а общий для всех метод Logger::log() ему почему-то не понравился? Я уже просто устал ругаться с ним...

   cr0t, 02 Июня 2010

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

9. Pascal / Говнокод #3319

   +116

   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

   for m:=0 to 9 do
         mas[m]:=inttostr(m);
     if ((Form1.edit1.text='')or(Form1.Memo2.text='')) then    begin
         if Form1.Memo2.text='' then
             ShowMessage('Ошибка! Введите пожалуйста текст или подключите файл,который необходимо расшифровать!!!');
         if Form1.edit1.text='' then
             ShowMessage('Ошибка! Введите пожалуйста ключ,которым будет расшифрован текст!!!');
       end
     else    begin
         //проверка ключа
         key:=Form1.edit1.text;
         k:=length(key);
         c:=0;
         k_p:=true;
         for j:=1 to k do
         for m:=0 to 9 do begin
             if (key[j]=mas[m]) then  
                 inc(c)
             else Begin
                 if (m=9) then
                   if (c=0) then Begin
                       ShowMessage('Ошибка!!!Введите пожалуйста ключ,которым будет расшифрован текст-целые числа');
                       k_p:=false;
                     end
                   else
                     c:=0;
               end;
           end;
       end;

   говнокриптография...

   3.14159265, 26 Мая 2010

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

10. C++ / Говнокод #3235

    +993

    1. 1
    2. 2

    // Randomize
    #define srand(x) srand(x + GetCurrentThreadId())

    Чтобы в разных тредах не выдавало одинаковые последовательности

    k06a, 14 Мая 2010

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

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

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

    if(empty($name) or empty($text) or !preg_match("/[a-zа-я]/i",$name) or !preg_match("/[a-zа-я]/i",$text))
      {
      echo "<font color=\"#f00000\">";
    
        if(empty($name) or !preg_match("/[a-zа-я]/i",$name))
        {
          if(empty($name)) echo "" . $lang['10'] . "<br />";
    
            else {       echo "" . $lang['11'] . "<br />";}
    
        }
    
    
        if(empty($text) or !preg_match("/[a-zа-я]/i",$text) )
          if(empty($text)) echo "" . $lang['12'] . "<br />";
          else {
               echo "" . $lang['13'] . "<br />";
               }
      echo "</font><br /><a href='index.php?act=add'>" . $lang['14'] . "</a><br />";
    
      }
    
      else
      {
    $text=stripslashes(htmlspecialchars($text, ENT_QUOTES));
    $new_date=stripslashes(htmlspecialchars($new_date, ENT_QUOTES));
    $name=stripslashes(htmlspecialchars($name, ENT_QUOTES));
    
      $sms="$name:|:$new_date:|:$text
    ";
    $sms=antimat($sms);
    
      $fp=@fopen($file,"a+");
      @fwrite($fp,$sms);
      fclose($fp);
    
    
    
    if(@fwrite) echo "" . $lang['15'] . " <a href=\"index.php\">" . $lang['16'] . "</a>.<br />";
    else { echo "" . $lang['32'] . ""; }
    }

    )))

    lugaro, 30 Апреля 2010

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

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

• 1973
• …
• 1321
• 1320
• 1319
• 1318
• 1317
• 1316
• 1315
• 1314
• 1313
• …
• 1