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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2372)
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 (29)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

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

   +131

   1. 1

   Сосево из ширинки, ебля с порога.

   KonardStuard, 16 Апреля 2014

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

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

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

   $db = Db::getInstance();
   $result = $db->ExecuteS('SELECT * FROM `'._DB_PREFIX_.'payment_module_settings`;');
   $text = "";
   for($i=2;$i<sizeof($result[0]);$i++)
   {
       if($i == 2)
       {
           $text .= "[general]"."\n";
           $text .= "url = '".$result[0][url]."'"."\n";
       }
       if($i == 3)
       {
           $text .= "[payment]"."\n";
           $text .= "sequritySender = '".$result[0][security_sender]."'"."\n";
       }
       if($i == 4)
       {
           $text .= "transactionChannel = '".$result[0][transaction_Channel]."'"."\n";
       }
       if($i == 5)
       {
           $text .= "transactionMode = '".$result[0][transaction_Mode]."'"."\n";
       }
       if($i == 6)
       {
           $text .= "userLogin = '".$result[0][user_Login]."'"."\n";
       }
       if($i == 7)
       {
           $text .= "userPwd = '".$result[0][user_Pwd]."'"."\n";
       }
       if($i == 8)
       {
           $text .= "paymentType = '".$result[0][payment_Type]."'"."\n";
       }
       if($i == 9)
       {
           $text .= "presentationCurrency = '".$result[0][presentation_Currency]."'"."\n";
       }
       if($i == 10)
       {
           $text .= "[view]"."\n";
           $text .= "src = '".$result[0][src]."'"."\n";
       }
       if($i == 11)
       {
           $text .= "language = '".$result[0][language]."'"."\n";
       }
       if($i == 12)
       {
           $text .= "style = '".$result[0][style]."'"."\n";
       }
      
   }
   
   if($_GET['paymentType'] == 'CC')
   {
     $text .= "brands[] = 'VISA'"."\n";
     $text .= "brands[] = 'MASTER'"."\n";
    
   }
   if($_GET['paymentType'] == 'VA')
   {
     $text .= "brands[] = 'PAYPAL'"."\n";
   }
   
   $text .= "[soap]"."\n";
   $text .= "url = 'http://debugservices.fine-trade.org/PayOnOrderHandling.svc?WSDL'"."\n";
   $text .= "username  = 'extensions'"."\n";
   $text .= "password = 'testExtensions'"."\n";
   $text .= "options[] = false"."\n";
   $text .= "importIfPending = false"."\n";
   
   if (file_exists('LoviitLib/config.ini')) {
       unlink('LoviitLib/config.ini');
   } 
   $myFile = "LoviitLib/testFile.txt";
   $fh = fopen($myFile, 'w') or die("can't open file");
   fwrite($fh, $text);
   fclose($fh);
   rename("LoviitLib/testFile.txt","LoviitLib/config.ini");

   Преамбула: для конфигурации библиотеки передаётся ассоциативный массив в конструктор. Если передаётся null, то для конфигурации используется файл config.ini с дефолтными настройками, который находится в корне библиотеки.
   Этот код исполняется при каждой инициализации библиотеки (при каждом чекауте).
   Ремарка: после запроса к db результатом является ассоциативный массив.
   И да, это код из модуля для prestashop cms.

   aronsky, 15 Апреля 2014

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

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

   +133

   1. 1

   Добрый вечер, цыплята.

   Сосево из ширинки, ебля с порога.

   KonardStuard, 14 Апреля 2014

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

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

   +73

   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

   StartElement startElement = event.asStartElement();
   if(startElement.getName().getLocalPart().equals(REPORT)){
   	report = new Report();
   }else if (startElement.getName().getLocalPart().equals(CODE)){
   	event = eventReader.nextEvent();
   	report.setCode(event.asCharacters().getData());
   	continue;
   }else if(startElement.getName().getLocalPart().equals(SHORT_NAME)){
   	event = eventReader.nextEvent();
   	report.setShortName(event.asCharacters().getData());
   }else if(startElement.getName().getLocalPart().equals(NAME)){
   	event = eventReader.nextEvent();
   	report.setName(event.asCharacters().getData());
   	continue;
   }else if(startElement.getName().getLocalPart().equals(TYPE)){
   	event = eventReader.nextEvent();
   	report.setType(ReportType.valueOf(event.asCharacters().getData()));
   	continue;
   }else if(startElement.getName().getLocalPart().equals(CON_CMN_REPORT)){
   	event = eventReader.nextEvent();
   	String conRepCode = event.asCharacters().getData();
   	report.setConnectedCommonReport(getReportByCode(conRepCode, reports));
   	continue;
   }else if(startElement.getName().getLocalPart().equals(BEFORE)){
   	event = eventReader.nextEvent();
   	report.setAvaliableBefore(Boolean.valueOf(event.asCharacters().getData()));
   	continue;
   }else if(startElement.getName().getLocalPart().equals(QUANTITY)){
   	event = eventReader.nextEvent();
   	report.setQuantity(ReportQuantity.valueOf(event.asCharacters().getData()));
   	continue;
   }else if(startElement.getName().getLocalPart().equals(CREATOR_CLASS_NAME)){
   	event = eventReader.nextEvent();
   	report.setCreatorClassName(event.asCharacters().getData());
   }

   и не лень же было

   evg_ever, 14 Апреля 2014

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

6. C++ / Говнокод #15750

   +12

   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

   int main()
   {
   MyList<MySubList*> *list;
   ...
   int error=makeZeroItemsNull(list)+deleteZeroItems(list);
   delete list;
   return error;
   }
   
   int makeZeroItemsNull(MyList *list)
   {
   //Удаляем указатели в списке и обнуляем
   if(list==0||error)return 1;
   return deleteZeroItems(list); //На всякий случай проверяем остались ли не удаленные элементы
   }
   
   int deleteZeroItems(MyList *list)
   {
   //Удаляем из списка обнуленные элементы
   if(list==0||error)return 2;
   return makeZeroItemsNull(list); //На всякий случай проверяем остались ли не обнуленные элементы
   }

   Учим все функции возвращать int(0) когда нет ошибок.

   IGHOR, 14 Апреля 2014

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

7. PHP / Говнокод #15676

   +142

   1. 1

   Лошади уже здесь.

   Konardillo, 06 Апреля 2014

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

8. SQL / Говнокод #15646

   −123

   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

   select ...
   case 
    -- периоды пересекаются только в одной точке (2)
   when (vw.hEndDate=vw.rStartDate) 
   then vw.hEndDate
   when (vw.rEndDate=vw.hStartDate) 
   then vw.rEndDate
   -- периоды пересекаются в определенной области (2)
   when (vw.hStartDate > vw.rstartdate and IsNull(vw.hEndDate,'01-01-2099')<IsNull(vw.rEndDate,'01-01-2099')) then vw.hStartDate
   when ((vw.rStartDate between vw.hStartDate and vw.hEndDate) and (vw.hEndDate between vw.rStartDate and vw.rEndDate)) 
   then vw.rStartDate
   when ((vw.hStartDate between vw.rStartDate and vw.rEndDate) and (vw.rEndDate between vw.hStartDate and vw.hEndDate)) 
   then vw.hStartDate
   -- периоды полностью совпадают (1)
   when (vw.hStartDate=vw.rStartDate and vw.hEndDate=vw.rEndDate) 
   then vw.hStartDate
   -- периоды совпадают в одной точке и перекрываются (4)
   when (vw.hStartDate=vw.rStartDate and vw.rEndDate<vw.hEndDate) 
   then vw.hStartDate
   when (vw.hStartDate=vw.rStartDate and vw.hEndDate<vw.rEndDate) 
   then vw.hStartDate
   when (vw.hEndDate=vw.rEndDate and vw.rStartDate>vw.hStartDate) 
   then vw.rStartDate
   when (vw.hEndDate=vw.rEndDate and vw.rStartDate<vw.hStartDate) 
   then vw.hStartDate
   end as MainStartDate  
   ,case 
   -- периоды пересекаются только в одной точке (2)
   when (vw.hEndDate=vw.rStartDate) 
   then vw.hEndDate
   when (vw.rEndDate=vw.hStartDate) 
   then vw.rEndDate
   -- периоды пересекаются в определенной области (2)
   when (vw.hStartDate > vw.rstartdate and IsNull(vw.hEndDate,'01-01-2099')<IsNull(vw.rEndDate,'01-01-2099')) then vw.hEndDate
   when ((vw.rStartDate between vw.hStartDate and vw.hEndDate) and (vw.hEndDate between vw.rStartDate and vw.rEndDate)) 
   then vw.hEndDate
   when ((vw.hStartDate between vw.rStartDate and vw.rEndDate) and (vw.rEndDate between vw.hStartDate and vw.hEndDate)) 
   then vw.rEndDate
   -- периоды полностью совпадают (1)
   when (vw.hStartDate=vw.rStartDate and vw.hEndDate=vw.rEndDate) 
   then vw.hEndDate
   -- периоды совпадают в одной точке и перекрываются (4)
   when (vw.hStartDate=vw.rStartDate and vw.rEndDate<vw.hEndDate) 
   then vw.rEndDate
   when (vw.hStartDate=vw.rStartDate and vw.hEndDate<vw.rEndDate) 
   then vw.hEndDate
   when (vw.hEndDate=vw.rEndDate and vw.rStartDate>vw.hStartDate) 
   then vw.hEndDate
   when (vw.hEndDate=vw.rEndDate and vw.rStartDate<vw.hStartDate) 
   then vw.hEndDate
   end as MainEndDate
   from (select ...
   case when (IsNull(h.[EndDate],'01-01-2099')=r.[StartDate]) then '1 периоды пересекаются только в одной точке (2) h.[EndDate]=r.[StartDate]'
   when(IsNull(r.[EndDate],'01-01-2099')=h.[StartDate]) then '2 периоды пересекаются только в одной точке (2) r.[EndDate]=h.[StartDate]'
   -- периоды пересекаются в определенной области (2)
   when (h.startdate > r.startdate and IsNull(h.[EndDate],'01-01-2099')<IsNull(r.[EndDate],'01-01-2099')) then 'Период иерархии полностью входит в период территории'
   when((r.StartDate between h.StartDate and IsNull(h.[EndDate],'01-01-2099')) and (IsNull(h.[EndDate],'01-01-2099') between r.StartDate and IsNull(r.[EndDate],'01-01-2099'))) then 'периоды пересекаются в определенной области (2) 3 ((r.StartDate between h.StartDate and IsNull(h.[EndDate],01-01-2099)) and (IsNull(h.[EndDate],01-01-2099) between r.StartDate and r.EndDate))'
   when((h.startdate between r.startdate and IsNull(r.[EndDate],'01-01-2099')) and (IsNull(r.[EndDate],'01-01-2099') between h.Startdate and IsNull(h.[EndDate],'01-01-2099'))) then 'периоды пересекаются в определенной области (2) 4 ((h.startdate between r.startdate and r.enddate) and (r.EndDate between h.Startdate and IsNull(h.[EndDate],01-01-2099)))'
   -- периоды полностью совпадают (1)
   when(h.startdate=r.startdate and IsNull(h.[EndDate],'01-01-2099')=IsNull(r.[EndDate],'01-01-2099')) then 'периоды полностью совпадают (1) 5 (h.startdate=r.startdate and IsNull(h.[EndDate],01-01-2099)=r.enddate)'
   -- периоды совпадают в одной точке и перекрываются (4)
   when(h.startdate=r.startdate and IsNull(r.[EndDate],'01-01-2099')<IsNull(h.[EndDate],'01-01-2099')) then 'периоды совпадают в одной точке и перекрываются (4) 6 (h.startdate=r.startdate and r.enddate<IsNull(h.[EndDate],01-01-2099))'
   when(h.startdate=r.startdate and IsNull(h.[EndDate],'01-01-2099')<IsNull(r.[EndDate],'01-01-2099')) then 'периоды совпадают в одной точке и перекрываются (4) 7 (h.startdate=r.startdate and IsNull(h.[EndDate],01-01-2099)<r.enddate)'
   when(IsNull(h.[EndDate],'01-01-2099')=IsNull(r.[EndDate],'01-01-2099') and r.startdate>h.startdate) then 'периоды совпадают в одной точке и перекрываются (4) 8 (IsNull(h.[EndDate],01-01-2099)=r.enddate and r.startdate>h.startdate)'
   when(IsNull(h.[EndDate],'01-01-2099')=IsNull(r.[EndDate],'01-01-2099') and r.startdate<h.startdate) then 'периоды совпадают в одной точке и перекрываются (4) 9 (IsNull(h.[EndDate],01-01-2099)=r.enddate and r.startdate<h.startdate)'
   ... where ...
   (IsNull(h.[EndDate],'01-01-2099')=r.[StartDate]) 
   or (IsNull(r.[EndDate],'01-01-2099')=h.[StartDate])
   -- периоды пересекаются в определенной области (2)
   or ((r.StartDate between h.StartDate and IsNull(h.[EndDate],'01-01-2099')) and (IsNull(h.[EndDate],'01-01-2099') between r.StartDate and IsNull(r.[EndDate],'01-01-2099')))
   or ((h.startdate between r.startdate and IsNull(r.[EndDate],'01-01-2099')) and (IsNull(r.[EndDate],'01-01-2099') between h.Startdate and IsNull(h.[EndDate],'01-01-2099')))
   or (h.startdate > r.startdate and IsNull(h.[EndDate],'01-01-2099')<IsNull(r.[EndDate],'01-01-2099'))
   -- периоды полностью совпадают (1)
   or (h.startdate=r.startdate and IsNull(h.[EndDate],'01-01-2099')=IsNull(r.[EndDate],'01-01-2099'))
   -- периоды совпадают в одной точке и перекрываются (4)
   or (h.startdate=r.startdate and IsNull(r.[EndDate],'01-01-2099')<IsNull(h.[EndDate],'01-01-2099'))
   or (h.startdate=r.startdate and IsNull(h.[EndDate],'01-01-2099')<IsNull(r.[EndDate],'01-01-2099'))
   or (IsNull(h.[EndDate],'01-01-2099')=IsNull(r.[EndDate],'01-01-2099') and r.startdate>h.startdate)
   or (IsNull(h.[EndDate],'01-01-2099')=IsNull(r.[EndDate],'01-01-2099') and r.startdate<h.startdate)

   я даже знать не хочу, что в этом запросе на 140 строк происходит, и молю богу, чтобы это не ломалось, пока автор этого в декрете.
   форматирование убрал, а то в 6к символов не помещался
   отформатировать можно тут http://poorsql.com/

   Lokich, 02 Апреля 2014

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

9. bash / Говнокод #15642

   −128

   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

   # Now convert the arguments - kludge to limit ourselves to /bin/sh
   i=0
   for arg in "$@" ; do
       CHECK=`echo "$arg"|egrep -c "$OURCYGPATTERN" -`
       CHECK2=`echo "$arg"|egrep -c "^-"`                                 ### Determine if an option
   
       if [ $CHECK -ne 0 ] && [ $CHECK2 -eq 0 ] ; then                    ### Added a condition
           eval `echo args$i`=`cygpath --path --ignore --mixed "$arg"`
       else
           eval `echo args$i`="\"$arg\""
       fi
       i=$((i+1))
   done
   case $i in
       (0) set -- ;;
       (1) set -- "$args0" ;;
       (2) set -- "$args0" "$args1" ;;
       (3) set -- "$args0" "$args1" "$args2" ;;
       (4) set -- "$args0" "$args1" "$args2" "$args3" ;;
       (5) set -- "$args0" "$args1" "$args2" "$args3" "$args4" ;;
       (6) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" ;;
       (7) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" ;;
       (8) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" ;;
       (9) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" "$args8" ;;
   esac

   gradle wrapper

   Elvenfighter, 02 Апреля 2014

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

10. Си / Говнокод #15638

    +134

    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

    
    /* A C statement or statements which output an assembler instruction
       opcode to the stdio stream STREAM.  The macro-operand PTR is a
       variable of type `char *' which points to the opcode name in its
       "internal" form--the form that is written in the machine description.
    
       GAS version 1.38.1 doesn't understand the `repz' opcode mnemonic.
       So use `repe' instead.  */
    
    #undef ASM_OUTPUT_OPCODE
    #define ASM_OUTPUT_OPCODE(STREAM, PTR)	\
    {									\
      if ((PTR)[0] == 'r'							\
          && (PTR)[1] == 'e'						\
          && (PTR)[2] == 'p')						\
        {									\
          if ((PTR)[3] == 'z')						\
    	{								\
    	  fputs ("repe", (STREAM));					\
    	  (PTR) += 4;							\
    	}								\
          else if ((PTR)[3] == 'n' && (PTR)[4] == 'z')			\
    	{								\
    	  fputs ("repne", (STREAM));					\
    	  (PTR) += 5;							\
    	}								\
        }									\
      else									\
        ASM_OUTPUT_AVX_PREFIX ((STREAM), (PTR));				\
    }

    Костыль из GCC. Ассемблер GAS версии 1.38.1 не переваривает мнемоники repz и repnz. Эта макрохрень перекодирует их в repe и repne соответственно
    https://github.com/mirrors/gcc/blob/master/gcc/config/i386/gas.h#L81

    j123123, 02 Апреля 2014

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

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

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

    public function getPagination($page, $rows, $limit){
            $pageArray = array();
    
            if ($rows > $limit){
                $allPage = ceil($rows/$limit);
    
                if ($allPage <= 7){
                    for ($i=1;$i<=$allPage;$i++){$pageArray[]=$i;}
                }else{
                    if ($page < 5){
                        for ($i=1;$i<=($page+3);$i++){$pageArray[]=$i;}
                        $pageArray[]=$allPage;
                    }else{
                        if ($allPage <= ($page+3)){
                            $pageArray[]=1;
                            for ($i=($page-3);$i<=$allPage;$i++){$pageArray[]=$i;}
                        }else{
                            $pageArray[]=1;
                            for ($i=($page-3);$i<=($page+3);$i++){$pageArray[]=$i;}
                            $pageArray[]=$allPage;
                        }
                    }
                }
    
                return array(
                    'pages' => $pageArray,
                    'page' => $page
                );
            }else{
                return array(
                    'pages' => 1,
                    'page' => $page
                );
            }
        }

    My friend wrote pagination function. takes current page, number of rows (from query), and limit on each page. function works really good but :D govno kod :D

    tatocaster, 01 Апреля 2014

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

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

• 1971
• …
• 66
• 65
• 64
• 63
• 62
• 61
• 60
• 59
• 58
• …
• 1