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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2380)
4. JavaScript (2033)
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. Куча / Говнокод #4298

   +165

   1. 1

   Говнокод.ру требуется ваша помощь!

   У нас небольшие проблемы с сервером, говнокод.ру может стать недоступным в любой момент. Срочно розыскиваем новый сервер. Или на время, пока не найдем достойную замену или навсегда. Возможна оплата с моей стороны. Отписывайтесь в комментарии к этому говнокоду или лично мне по следующим контактам:
   Электронная почта: [email protected]
   Jabber: страйкер@jabber.ru
   ICQ: 257073504

   striker, 21 Сентября 2010

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

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

   +172

   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

   // подключение библиотек и конфига
   require_once($_SERVER["DOCUMENT_ROOT"]."/config.php");
   require_once($_SERVER["DOCUMENT_ROOT"]."/content/lib/tree.php");
   require_once($_SERVER["DOCUMENT_ROOT"]."/content/lib/func.php");
   extract($_GET);
   if (isset($d1)&&(f_del_date($d1)!="")) {
      $arr[0]=f_n_date($d1); // очистка первой переменной
      if (isset($d2)&&(f_del_date($d2)!="")) {
   	$arr[1]=f_n_date($d2); // очистка второй переменной
   	if (isset($d3)&&(f_del_date($d3)!="")) {
   	   $arr[2]=f_n_date($d3); // очистка третьей переменной
   	   if (isset($d4)&&(f_del_date($d4)!="")) {
   		$arr[3]=f_n_date($d4); // очистка четвертой переменной
   		if (isset($d5)&&(f_del_date($d5)!="")) {
   		   $arr[4]=f_n_date($d5); // очистка пятой переменной
   		   if (isset($d6)&&(f_del_date($d6)!="")) {
   			$arr[5]=f_n_date($d6); // очистка шестой переменной
   			if (isset($d7)&&(f_del_date($d7)!="")) {
   			   $arr[6]=f_n_date($d7); // очистка седьмой переменной
   			   if (isset($d8)&&(f_del_date($d8)!="")) {
   				$arr[7]=f_n_date($d8); // очистка восьмой переменной
   				if (isset($d9)&&(f_del_date($d9)!="")) {
   				   $arr[8]=f_n_date($d9); // очистка девятой переменной
   				   if (isset($d10)&&(f_del_date($d10)!="")) {
   					$arr[9]=f_n_date($d10); // очистка десятой переменной
   					if (isset($d11)&&(f_del_date($d11)!="")) {
   					   $arr[10]=f_n_date($d11); // очистка одинадцатой переменной
   
   					}
   				   }
   				}
   			   }
   			}
   		   }
   		}
   	   }
   	}
      }
   }

   Это чудесная CMS Black Crystal (Кристалл-чернуха), админка у нее такая же, как и сам код.

   nakham, 20 Сентября 2010

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

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

   +168

   1. 1

   if(substr($_SERVER['QUERY_STRING'],0,9)==='act=const')

   Вот так мы получаем GET данные из адресной строки

   Shevron, 19 Сентября 2010

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

5. C++ / Говнокод #4275

   +169

   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

   if(typeid(*Line1)==typeid(TLine)) ; else
   if(typeid(*Line2)==typeid(TLine)) {cLine=Line1; Line1=Line2; Line2=cLine;} else
   if(typeid(*Line1)==typeid(TRay)) ; else
   if(typeid(*Line2)==typeid(TRay)) {cLine=Line1; Line1=Line2; Line2=cLine;} else
   if(typeid(*Line1)==typeid(TLineSegment)) ; else
   if(typeid(*Line2)==typeid(TLineSegment)) {cLine=Line1; Line1=Line2; Line2=cLine;}
   
   if(typeid(*Line1)==typeid(TLine)) {
   	if(typeid(*Line2)==typeid(TLine))  return 1; else
   	if(typeid(*Line2)==typeid(TRay)) {
   		if(
   			(( ((TRay*)Line2)->X1<=((TRay*)Line2)->X2 && ((TRay*)Line2)->X1<=Point.x )  ||
   			 ( ((TRay*)Line2)->X1>=((TRay*)Line2)->X2 && ((TRay*)Line2)->X1>=Point.x )) &&
   			(( ((TRay*)Line2)->Y1<=((TRay*)Line2)->Y2 && ((TRay*)Line2)->Y1<=Point.y )  ||
   			 ( ((TRay*)Line2)->Y1>=((TRay*)Line2)->Y2 && ((TRay*)Line2)->Y1>=Point.y ))
   		  )
   			return 1;
   		else
   			return -1;
   	} else
   	if(typeid(*Line2)==typeid(TLineSegment)) {
   		if(
   			(( ((TLineSegment*)Line2)->X1<=Point.x && Point.x<=((TLineSegment*)Line2)->X2 )||( ((TLineSegment*)Line2)->X2<=Point.x && Point.x<=((TLineSegment*)Line2)->X1 )) &&
   			(( ((TLineSegment*)Line2)->Y1<=Point.y && Point.y<=((TLineSegment*)Line2)->Y2 )||( ((TLineSegment*)Line2)->Y2<=Point.y && Point.y<=((TLineSegment*)Line2)->Y1 ))
   		  )
   			return 1;
   		  else
   			return -1;
   	}
   } else
   if(typeid(*Line1)==typeid(TRay)) {
   	if(typeid(*Line2)==typeid(TRay)) {
   		if(
   			((( ((TRay*)Line1)->X1<=((TRay*)Line1)->X2 && ((TRay*)Line1)->X1<=Point.x )  ||
   			  ( ((TRay*)Line1)->X1>=((TRay*)Line1)->X2 && ((TRay*)Line1)->X1>=Point.x )) &&
   			 (( ((TRay*)Line1)->Y1<=((TRay*)Line1)->Y2 && ((TRay*)Line1)->Y1<=Point.y )  ||
   			  ( ((TRay*)Line1)->Y1>=((TRay*)Line1)->Y2 && ((TRay*)Line1)->Y1>=Point.y )))
   			&&
   			((( ((TRay*)Line2)->X1<=((TRay*)Line2)->X2 && ((TRay*)Line2)->X1<=Point.x )  ||
   			  ( ((TRay*)Line2)->X1>=((TRay*)Line2)->X2 && ((TRay*)Line2)->X1>=Point.x )) &&
   			 (( ((TRay*)Line2)->Y1<=((TRay*)Line2)->Y2 && ((TRay*)Line2)->Y1<=Point.y )  ||
   			  ( ((TRay*)Line2)->Y1>=((TRay*)Line2)->Y2 && ((TRay*)Line2)->Y1>=Point.y )))
   		  )
   			return 1;
   		else
   			return -1;
   	} else
   	if(typeid(*Line2)==typeid(TLineSegment)) {
   		if(
   			((( ((TRay*)Line1)->X1<=((TRay*)Line1)->X2 && ((TRay*)Line1)->X1<=Point.x )  ||
   			  ( ((TRay*)Line1)->X1>=((TRay*)Line1)->X2 && ((TRay*)Line1)->X1>=Point.x )) &&
   			 (( ((TRay*)Line1)->Y1<=((TRay*)Line1)->Y2 && ((TRay*)Line1)->Y1<=Point.y )  ||
   			  ( ((TRay*)Line1)->Y1>=((TRay*)Line1)->Y2 && ((TRay*)Line1)->Y1>=Point.y )))
   			&&
   			((( ((TLineSegment*)Line2)->X1<=Point.x && Point.x<=((TLineSegment*)Line2)->X2 )||( ((TLineSegment*)Line2)->X2<=Point.x && Point.x<=((TLineSegment*)Line2)->X1 )) &&
   			((  ((TLineSegment*)Line2)->Y1<=Point.y && Point.y<=((TLineSegment*)Line2)->Y2 )||( ((TLineSegment*)Line2)->Y2<=Point.y && Point.y<=((TLineSegment*)Line2)->Y1 )))
   		  )
   			return 1;
   		else
   			return -1;
   	}
   	return 1;
   } else
   if(typeid(*Line1)==typeid(TLineSegment)) {
   	if(
   		((( ((TLineSegment*)Line1)->X1<=Point.x && Point.x<=((TLineSegment*)Line1)->X2 )||( ((TLineSegment*)Line1)->X2<=Point.x && Point.x<=((TLineSegment*)Line1)->X1 )) &&
   		((  ((TLineSegment*)Line1)->Y1<=Point.y && Point.y<=((TLineSegment*)Line1)->Y2 )||( ((TLineSegment*)Line1)->Y2<=Point.y && Point.y<=((TLineSegment*)Line1)->Y1 )))
   		&&
   		((( ((TLineSegment*)Line2)->X1<=Point.x && Point.x<=((TLineSegment*)Line2)->X2 )||( ((TLineSegment*)Line2)->X2<=Point.x && Point.x<=((TLineSegment*)Line2)->X1 )) &&
   		((  ((TLineSegment*)Line2)->Y1<=Point.y && Point.y<=((TLineSegment*)Line2)->Y2 )||( ((TLineSegment*)Line2)->Y2<=Point.y && Point.y<=((TLineSegment*)Line2)->Y1 )))
   	  )
   		return 1;
   	else
   		return -1;
   }

   Имеются классы:
   class TLine; // Линия
   class TRay: public TLine; // Луч
   class TLineSegment: public TRay; // Отрезок
   Функция записывает в Point точку пересечения прямых и возвращает 0 если прямые параллельны, -1 если пересекаются продолжения отрезков и 1 если пересекаются отрезки.
   Выше представлен кусок кода где проводится проверка, принадлежит точка пересечения отрезкам или их продолжениям.

   sanchousf, 17 Сентября 2010

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

6. PHP / Говнокод #4198

   +160

   1. 1
   2. 2
   3. 3

   <?php
   
   if (preg_match("/\.[gjpi][ipnc][fgo]/i", $_SERVER['REQUEST_URI'])) exit;

   user654321, 08 Сентября 2010

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

7. Куча / Говнокод #4185

   +144

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

   Что такое скрипт Интернет магазина?
   Скрипт интернет магазина – это набор файлов, позволяющий на своем сайте
   организовать онлайн магазин.
   
   Готовый скрипт интернет магазина
   Скрипт php магазина - это готовое решение для создания интернет магазина с
   максимальной экономией времени и денег. Если Вам нужна быстрая и качественная
   разработка интернет магазина, то готовый скрипт электронного магазина именно то,
   что Вам нужно.

   Источник: http://shopcms-design.ru/
   Думаю, что это всё же говнокод. На языке "русский" =)

   hybroid, 06 Сентября 2010

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

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

   +69

   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

   /**
        * Инициализация набора действий, определенных для контейнера.
        * @param context
        * @throws DfException
        */
       protected void initializeTransitions(EditorContext context) throws DfException{
           AbstractDocumentModel documentModel = getDocumentModel();
           AbstractState state = documentModel.getLifeCyclePolicy(context).getState(context);
           
           if (!documentModel.isNew())
           {
               addTransition(TransitionName.CREATE_DETACHED_COMMISSION, TransitionHandler.CREATE_DETACHED_COMMISSION);
               addTransition(TransitionName.PRINT, TransitionHandler.PRINT);
           }
           AbstractTaskModel taskModel = context.getTaskModel();
           if (taskModel  instanceof ReviewDocumentTaskModel && !(taskModel  instanceof ExpeditionaryProcessingDocumentTaskModel)){
   //                 && !(taskModel instanceof AttachFileAndReviewDocumentTaskModel)){
   
                     
           if ((taskModel instanceof AttachFileToDocumentTaskModel || taskModel instanceof WaitForScanningTaskModel) && ((documentModel instanceof InternalDocumentModel) || (documentModel instanceof OutgoingDocumentModel) || (documentModel instanceof IncomingDocumentModel)))
               addTransition(TransitionName.PRINT_BARCODE, TransitionHandler.PRINT_BARCODE);
   
           if (taskModel instanceof AbstractDocumentTaskModel){
               if ( state instanceof AbstractDraftState ){
                   if( state instanceof NewProtocolDraftState)
                       addTransition(ContainerAction.ACTION_NAME, TransitionName.SEND_DRAFT, TransitionName.REGISTER, TransitionHandler.SEND_DRAFT);
                   else if (state instanceof NewDraftState)
                       addTransition(TransitionName.SEND_DRAFT, TransitionHandler.SEND_DRAFT);
                   if (state instanceof AbstractApprovalDraftState)
                       addTransition(TransitionName.REWORK_DRAFT, TransitionHandler.REWORK_DRAFT);
                   if (state instanceof AnalyseDraftState)
                       addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_ANALYSE, TransitionHandler.SEND_DRAFT);
                   if (state instanceof ControlDraftState)
                       addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_CONTROL, TransitionHandler.SEND_DRAFT);
                   if (state instanceof ApproveDraftState)
                       addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_APPROVAL, TransitionHandler.SEND_DRAFT);
                   if (state instanceof SignDraftState)
                       addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_SIGN, TransitionHandler.SEND_DRAFT);
                   if (state instanceof ExternalApproveDraftState)
                       addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_APPROVAL, TransitionHandler.SEND_DRAFT);
                   if (state instanceof ControlApproveDraftState)
                       addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_CONTROLED, TransitionHandler.SEND_DRAFT);
                   if (state instanceof ConfirmationDraftState)
                       addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_CONFIRM, TransitionHandler.SEND_DRAFT);
               }
             }
           }
           
   .................. несколько страниц такого кода ...........................
         
           if (state instanceof AbstractApprovalDraftState ||
               state instanceof ReworkDraftState ||
               state instanceof NewDocumentState ||
               state instanceof RegisterDocumentState ||
               state instanceof ReviewDocumentState ||
               state instanceof ExecuteDocumentState){
           	addTransition(TransitionName.SAVE, TransitionHandler.SAVE);
           }
   
           super.initializeTransitions(context);
       }

   Автор решил захардкодить в одном месте логику переходов между всеми состояниями всех жизненных циклов всех документов, реализованных в системе документооборота. Получился говнокод-BPM-движок. А внешне код очень даже солидный: с константами и комментариями... и названиями классов вроде: ExpeditionaryProcessingDocumentTaskModel

   dab84, 06 Сентября 2010

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

9. Куча / Говнокод #4171

   +134

   1. 1

   http://youex.ru/skins/default/pages/search/

   В продолжение =)

   hybroid, 05 Сентября 2010

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

10. SQL / Говнокод #4147

    −856

    1. 1

    SELECT i.*, p.link as p_link, p.title as p_title, p.p_id FROM `items` i, `places` p WHERE i.public='1' and i.place=p.id and  (i.place='1' or i.place='1972' or i.place='2' or i.place='3' or i.place='4' or i.place='5' or i.place='6' or i.place='7' or i.place='8' or i.place='9' or i.place='10' or i.place='11' or i.place='12' or i.place='13' or i.place='14' or i.place='15' or i.place='16' or i.place='17' or i.place='18' or i.place='19' or i.place='20' or i.place='21' or i.place='22' or i.place='23' or i.place='24' or i.place='25' or i.place='26' or i.place='27' or i.place='28' or i.place='29' or i.place='30' or i.place='31' or i.place='32' or i.place='33' or i.place='34' or i.place='35' or i.place='36' or i.place='37' or i.place='38' or i.place='39' or i.place='40' or i.place='41' or i.place='42' or i.place='43' or i.place='44' or i.place='45' or i.place='46' or i.place='47' or i.place='48' or i.place='49' or i.place='50' or i.place='51' or i.place='52' or i.place='53' or i.place='54' or i.place='55' or i.place='56' or i.place='57' or i.place='58' or i.place='59' or i.place='60' or i.place='61' or i.place='62' or i.place='63' or i.place='64' or i.place='65' or i.place='66' or i.place='67' or i.place='68' or i.place='69' or i.place='70' or i.place='71' or i.place='72' or i.place='73' or i.place='74' or i.place='75' or i.place='76' or i.place='77' or i.place='78' or i.place='79' or i.place='80' or i.place='81' or i.place='82' or i.place='83' or i.place='84' or i.place='85' or i.place='86' or i.place='87' or i.place='88' or i.place='89' or i.place='90' or i.place='91' or i.place='92' or i.place='93' or i.place='94' or i.place='95' or i.place='96' or i.place='97' or i.place='98' or i.place='99' or i.place='100' or i.place='101' or i.place='102' or i.place='103' or i.place='104' or i.place='105' or i.place='106' or i.place='107' or i.place='108' or i.place='109' or i.place='110' or i.place='111' or i.place='112' or i.place='113' or i.place='114' or i.place='115' or i.place='116' or i.place='117' or i.place='118' or i.place='119' or i.place='120' or i.place='121' or i.place='122' or i.place='123' or i.place='124' or i.place='125' or i.place='126' or i.place='127' or i.place='128' or i.place='129' or i.place='130' or i.place='131' or i.place='132' or i.place='133' or i.place='134' or i.place='135' or i.place='136' or i.place='137' or i.place='138' or i.place='139' or i.place='140' or i.place='141' or i.place='142' or i.place='143' or i.place='144' or i.place='145' or i.place='146' or i.place='147' or i.place='148' or i.place='149' or i.place='150' or i.place='151' or i.place='152' or i.place='153' or i.place='154' or i.place='155' or i.place='156' or i.place='157' or i.place='158' or i.place='159' or i.place='160' or i.place='161' or i.place='162' or i.place='163' or i.place='164' or i.place='165' or i.place='166')  ORDER BY i.lastupdated DESC LIMIT 18160, 10;

    есть ещё примерно такой же, но с JOINами

    wiz, 01 Сентября 2010

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

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

    +144

    1. 1

    exit; ?>

    Самая последняя строчка в скрипте.

    7ion, 25 Августа 2010

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

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

• 1973
• …
• 898
• 897
• 896
• 895
• 894
• 893
• 892
• 891
• 890
• …
• 1