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

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

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

1. PHP (5700)
2. C++ (2734)
3. Куча (2331)
4. JavaScript (2028)
5. C# (1926)
6. Java (1501)
7. Си (1097)
8. Pascal (647)
9. Python (582)
10. 1C (532)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (29)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. JavaScript / Говнокод #19139

   +1

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

   try {
           renderer = new THREE.WebGLRenderer({ antialias: true, preserveDrawingBuffer: !!navigator.userAgent.match(/Trident.*rv[ :]*11\./) });
       } catch (mammothFeces) {
           renderer = new THREE.CanvasRenderer();
       }

   клиент тестирует на сафари 5. а я думал, ие11 (/Trident.*rv[ :]*11\./) это худшее, что случится на проекте

   makc3d, 04 Декабря 2015

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

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

   0

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

   $a=array();
   for($i=0;$i<100;$i++){
    $t=rand(1,100);
    $a[$t]=$t;
   }

   Решение задачи из теста на пригодность в джуны.
   Сама задача была поставлена так:
   
   "Нужно объявить массив, в цикле от 1 до 100 заполнить его случайными цифрами (тоже от 1го до 100), и в итоге получить массив с уникальными значениями, реализовать минимальным кол-вом строк. "
   
   Шедевральная наркомания.

   Twissel, 04 Декабря 2015

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

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

   +5

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

   for(var i=0; i<data.length; i++) {
   	…
   	self._markerList[data[i].r_osm_id].GOZ.arrRowNum[self._markerList[data[i].r_osm_id].GOZ.arrRowNum.length] = parseInt(i);
   	self._markerList[data[i].r_federal_osm_id].GOZ.arrRowNum[self._markerList[data[i].r_federal_osm_id].GOZ.arrRowNum.length] = parseInt(i);
   	self._markerList[0].GOZ.arrRowNum[self._markerList[0].GOZ.arrRowNum.length] = parseInt(i);
   }

   Это parseInt умилительно. Нет, со счётчиком ничего внутри цикла не делается. Но а вдруг?

   torbasow, 04 Декабря 2015

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

4. Куча / Говнокод #19131

   +2

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10

   Del1 = fun(K,[A|B],F,Acc) ->
   if
   not(is_list(A)) and (B==[]) and (A rem K == 0) -> Acc;
   not(is_list(A)) and (B==[]) -> [A|Acc];
   not(is_list(A)) and is_list(B) and (A rem K == 0) -> F(K,B,F,Acc);
   not(is_list(A)) and is_list(B) -> F(K,B,F,[A|Acc])
   end
   end.
   D=[10,22,34,45,52,60,75].
   Del1(5,D,Del1,[]).

   Студент сдал по Функциональному программированию (написано на Erlang, работает).
   Функция удаляющая из списка эл-ты кратные 1му аргументу.

   vt244, 04 Декабря 2015

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

5. JavaScript / Говнокод #19130

   0

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

   $(".tabs_nav-item").click ->
       if($(this).hasClass("_active")) 
       else 
         $this = $(this)
         target = $(this).attr("href")
         $this.addClass("_active").siblings("._active").removeClass("_active")
         $(target).addClass("_active").siblings("._active").removeClass("_active")

   LOL
   P.S. coffeescript

   Crazyzubr, 03 Декабря 2015

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

6. JavaScript / Говнокод #19129

   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

   jQuery.AdImage = function(props) {
   				li = $('<li>');
   				li.css({ position: 'relative' });
   				li.css('text-align', 'center');
   				img = $('<img src="'+Path.small+props.lamp+props.index+'.jpg" class="'+props.orientation+'" />');
   				img.css('display', 'inline');
   				img.css('height', '180px');
   				img.css('margin', '10px');
   				imga = $('<a>');
   				imga.attr('title', props.title);
   				imga.attr('rel', 'lightbox');
   				imga.attr('href', 'javascript:;');
   				imga.click(function() {
   					$.easybox(Path.large+props.lamp+props.index+'.jpg', props.title, {});
   				});
   				imga.append(img);
   				li.append(imga);
   				div = $('<div>');
   				div.text(props.title);
   				li.append(div);
   				$('.img_group:first').append(li);
   	hidden = $('<input type="hidden" />');
   	hidden.attr('value', props.id);
   				li.append(hidden);
   	togglePic = function() {
   		x = $(this);
   		b = $(this).closest('li');
   		h = b.children(':input');
   		n = h.attr('value')
   		jQuery.ajax('/lamper/delpic/'+n).done(function(data) {
   			obj = jQuery.parseJSON(data);
   			if (obj.status == 'deleted') {
   				b.addClass('hidden');
   				x.text('Восстановить');
   			}
   			if (obj.status == 'restored') {
   				b.removeClass('hidden');
   				x.text('Удалить');
   			}
   		});
   	};
   	btn_del = $('<button>').css('float', 'right');
   	btn_del.text(props.exists ? 'Удалить' : 'Восстановить');
   	//btn_del.css('display', 'none');
   	li.append(btn_del);
   	if (!props.exists) li.addClass('hidden');
   	btn_del.click(togglePic);
   	return li;
   }

   Из старой админки. Управление отображением товара на сайте. Префикс Ad, насколько я помню, сокращение от Admin

   kgm-rj, 03 Декабря 2015

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

7. JavaScript / Говнокод #19128

   +2

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

   for (var i = this.request.languages.length; i < this.MAX_LANGUAGES; i++) {
         if (!this.request.languages) {
             this.request.languages = [];
          }
   
          this.request.languages.push(this.createEmptyLanguage());
   }

   Когда сначала ничего не понял, а потом как заорал в голосину

   Disorder, 03 Декабря 2015

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

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

   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

   <?php
   
   namespace Oro\Bundle\ActivityListBundle\Controller;
   
   use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
   use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
   
   use Symfony\Bundle\FrameworkBundle\Controller\Controller;
   use Symfony\Component\EventDispatcher\EventDispatcherInterface;
   use Symfony\Component\HttpFoundation\Request;
   use Symfony\Component\Translation\TranslatorInterface;
   
   use Oro\Bundle\ActivityListBundle\Event\ActivityConditionOptionsLoadEvent;
   
   /**
    * @Route("/activity-list/segment")
    */
   class SegmentController extends Controller
   {
       /**
        * @Route("/activity-condition")
        * @Template
        */
       public function activityConditionAction(Request $request)
       {
           $params = $request->attributes->get('params', []);
           $conditionOptions = [
               'activityConditionOptions' => [
                   'listOption'     => $this->forward(
                       'OroActivityListBundle:Api/Rest/ActivityList:getActivityListOption',
                       [],
                       ['_format' => 'json']
                   )->getContent(),
                   'entitySelector' => sprintf('#%s', $params['entity_choice_id']),
                   'fieldsLoaderSelector' =>  sprintf(
                       '[data-ftid=%soro_api_querydesigner_fields_entity]',
                       $params['entity_choice_id']
                   ),
                   'fieldChoice' => [
                       'select2' => [
                           'placeholder' => $this->getTranslator()->trans(
                               'oro.query_designer.condition_builder.choose_entity_field'
                           ),
                       ],
                   ],
                   'extensions' => [],
               ]
           ];
   
           $dispatcher = $this->getEventDispatcher();
           if (!$dispatcher->hasListeners(ActivityConditionOptionsLoadEvent::EVENT_NAME)) {
               return $conditionOptions;
           }
   
           $event = new ActivityConditionOptionsLoadEvent($conditionOptions['activityConditionOptions']);
           $dispatcher->dispatch(ActivityConditionOptionsLoadEvent::EVENT_NAME, $event);
   
           return [
               'activityConditionOptions' => $event->getOptions(),
           ];
       }
   
       /**
        * @return EventDispatcherInterface
        */
       protected function getEventDispatcher()
       {
           return $this->get('event_dispatcher');
       }
   
       /**
        * @return TranslatorInterface
        */
       protected function getTranslator()
       {
           return $this->get('translator');
       }
   }

   Странная передача параметра в виджет.

   HiNeX, 03 Декабря 2015

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

9. PHP / Говнокод #19125

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

   require_once './_конфигурация.php';
   
   	$обращеніе = Обращеніе::экземпляръ();
   	
   	/* @var $отъдѣлъ Отъдѣлъ\На\Английскомъ */
   	
   	if (isset($обращеніе->параметры->{'учётный-номер'}) == false) $отъдѣлъ->управленіе->перенаправить('страницы.php');
   	$учётный_нумеръ = $обращеніе->параметры->цѣлочисленное('учётный-номер');
   	
   	$вѣсть = $отъдѣлъ->фабрика->вѣсть->сотворить($учётный_нумеръ);
   	/* @var $вѣсть Отъдѣлъ\_\Объектъ\Вѣсть */
   	
   	if (false) {
   		var_dump($вѣсть->иллюстрація->на->страницѣ->вѣсти->ссылка);
   		$вѣсть->извлечь();
   		var_dump($вѣсть->иллюстрація->на->страницѣ->вѣсти->ссылка);
   		die();
   	} else {
   		$вѣсть->извлечь();
   	}
   	
   	if (false) {
   		var_dump($вѣсть->иллюстрація->большая->имя);
   		var_dump($вѣсть->иллюстрація->малая  ->имя);
   		var_dump($вѣсть->иллюстрація->на->страницѣ->вѣсти ->имя);
   		var_dump($вѣсть->иллюстрація->на->страницѣ->вѣстей->имя);
   		var_dump($вѣсть->иллюстрація->на->страницѣ->мѣтки ->имя);
   		var_dump($вѣсть->иллюстрація->въ->блокѣ           ->имя);
   		die();
   	}
   
   	if ($обращеніе->является->утвердительнымъ == true) {
   		
   		$вѣсть->учётный->нумеръ->страницы->вѣстей = $обращеніе->вводъ->цѣлочисленное('страница-новостей->');
   		$вѣсть->важная      = $обращеніе->вводъ->двоичное ('важная'    );
   		$вѣсть->заголовокъ  = $обращеніе->вводъ->строчное ('заголовок' );
   		$вѣсть->разъясненіе = $обращеніе->вводъ->текстовое('содержимое');
   		
   		$вѣсть->иллюстрація->большая->загрузить('иллюстрация-большая');
   		$вѣсть->иллюстрація->малая  ->загрузить('иллюстрация-малая'  );
   		if (empty($вѣсть->иллюстрація->большая->собственное->имя) == false) {
   			$вѣсть->иллюстрація->на->страницѣ->вѣсти ->копировать->изъ($вѣсть->иллюстрація->большая);
   		}
   		if (empty($вѣсть->иллюстрація->малая->собственное->имя) == false) {
   			$вѣсть->иллюстрація->на->страницѣ->вѣстей->копировать->изъ($вѣсть->иллюстрація->малая);
   			$вѣсть->иллюстрація->на->страницѣ->мѣтки ->копировать->изъ($вѣсть->иллюстрація->малая);
   			$вѣсть->иллюстрація->въ->блокѣ           ->копировать->изъ($вѣсть->иллюстрація->малая);
   		}
   		
   		\База\Данныхъ\Транзакція::экземпляръ()->начать();
   		$вѣсть->мѣтки->уничтожить();
   		\База\Данныхъ\Транзакція::экземпляръ()->утвердить();
   		
   		$отъдѣлъ->множество->мѣтокъ->вѣстей->извлечь();
   		foreach ($обращеніе->вводъ->массива->цѣлочисленныхъ('тег') as $учётный_нумеръ_мѣтки) {
   			foreach ($отъдѣлъ->множество->мѣтокъ->вѣстей->элементы as $мѣтка) {
   				if ($мѣтка->учётный->нумеръ->мѣтки != $учётный_нумеръ_мѣтки) continue;
   				$вѣсть->мѣтки->дополнить($мѣтка);
   				continue(2);
   			}
   		}
   		
   		if ($обращеніе->вводъ->двоичное('опубликовать') == true) $вѣсть->часъ->обнародованія = date('Y-m-d H:i:s');
   		
   		\База\Данныхъ\Транзакція::экземпляръ()->начать();
   		$вѣсть->измѣнить();
   		\База\Данныхъ\Транзакція::экземпляръ()->утвердить();
   		
   		unset($вѣсть);
   		$вѣсть = $отъдѣлъ->фабрика->вѣсть->сотворить($учётный_нумеръ);
   		/* @var $вѣсть Отъдѣлъ\_\Объектъ\Вѣсть */
   		$вѣсть->извлечь();
   		\База\Данныхъ\Транзакція::экземпляръ()->начать();
   		$вѣсть->измѣнить();
   		\База\Данныхъ\Транзакція::экземпляръ()->утвердить();
   		
   		$отъдѣлъ->управленіе->перенаправить('новости.php');
   	}
   	
   	$отъдѣлъ->множество->страницъ      ->извлечь();
   	$отъдѣлъ->множество->мѣтокъ->вѣстей->извлечь();
   	$вѣсть->мѣтки->извлечь(); ?>
   <html>
   	<head><?php
   		require_once директорія_включеній . 'html/заглавие.php'; ?>
   		<title>
   			Новость: <?php print $вѣсть->заголовокъ; ?>
   		</title>
   	</head>

   VitaliiPuhach, 03 Декабря 2015

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

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

    +1

    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

    $RECORDS = array();
    $qRecords = ' SELECT `id`, `original_name`, `urn_name` FROM `ac_brands` ORDER BY `original_name` ASC ';
    $rqRecords = mysql_query($qRecords, $mysql) or die(mysql_error());
    
    // общее количество объявлений в БД
    $totalAmt = 0;
    
    while($RECORD = mysql_fetch_assoc($rqRecords))
    {
    	// получаем количество объявлений в БД для данного бренда
    	$qAmtAds = ' SELECT `count_ads` FROM `ac_brands` WHERE `id` = "'.intval($RECORD['id']).'"';
    	$rqAmtAds = mysql_query($qAmtAds, $mysql) or die(mysql_error());
    	
    	$RECORD['amt_ads'] = mysql_fetch_object($rqAmtAds)->{"count_ads"};
    	$totalAmt += $RECORD['amt_ads'];
    	
    	$RECORDS[] = $RECORD;
    }

    Что ОН, б**ть, делает? =)
    Походу чувак не знает про COUNT() и GROUP BY...

    Lander, 03 Декабря 2015

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

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

• 2097
• …
• 1613
• 1612
• 1611
• 1610
• 1609
• 1608
• 1607
• 1606
• 1605
• …
• 1