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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2341)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   В номинации:

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

   За время:

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

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

   +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

   function displayFilterElements(typeList){
   	_filter.form.elements["name1"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name2"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["nam3"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name4"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name5"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name6"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name7"].closest("tr").hidden = (typeList === "Type1") ? false : true;
   	_filter.form.elements["name8"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name9"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name10"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name11"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name12"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name13"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name14"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name15"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name16"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name17"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name18"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name19"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name20"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name21"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name22"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name23"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name24"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name25"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name26"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name27"].closest("tr").hidden = (typeList === "Type2") ? false : true;
   	_filter.form.elements["name28"].closest("tr").hidden = (typeList === "Type3") ? true : false;
   	_filter.form.elements["name29"].closest("tr").hidden = (typeList === "Type3") ? false : true;
   };
   	
   function stateFilterElements(){
   	let select = false;
   	["name1", "name2", "name3", "name4", "name5", "name6", "name7", "name8", "name9", "name10", "name11", "name12", "name13"].forEach(function(fieldName){
   		if ($.query.hashJSON.has(fieldName))
   			select = true;
   	});
   	return select;
   }

   Получил в доработку модуль сотрудника. Идентификаторы изменены для публикации, не в них суть.

   torbasow, 17 Марта 2015

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

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

   +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

   foreach (scandir(DIR.'app'.SLASH.'lib') as $filename) {
       $path = DIR.'app'.SLASH.'lib'.SLASH.$filename;
       if (is_file($path) && substr($filename,strlen($filename)-4,4)=='.php') {
           require_once($path);
       }
   }
   foreach (scandir(DIR.'app'.SLASH.'models') as $filename) {
       $path = DIR.'app'.SLASH.'models'.SLASH.$filename;
       if (is_file($path) && substr($filename,strlen($filename)-4,4)=='.php') {
           require_once($path);
       }
   }
   foreach (scandir(DIR.'app'.SLASH.'ext') as $filename) {
       $path = DIR.'app'.SLASH.'ext'.SLASH.$filename;
       if (is_file($path) && substr($filename,strlen($filename)-4,4)=='.php') {
           require_once($path);
       }
   }

   Начался разбор проблемы со слов заказчика - "сайт долго грузится"

   Crawdaunt, 22 Февраля 2015

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

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

   +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

   if ($sub_sub_sub_sub_category_id !== null) {
   
   													if ($this->core->countUrl > 5) {
   														header('Location: /404/');
   													}
   
   													foreach ($categories[$category_id][$sub_category_id][$sub_sub_category_id][$sub_sub_sub_category_id][$sub_sub_sub_sub_category_id] as $k1 => $v1) {
   														if ($k1 !== 0) {
   															foreach ($v1 as $k2 => $v2) {
   																if ($k2 !== 0) {
   																	if ($this->core->translit($v2[0]) == $sub_sub_sub_category) {
   																		$sub_sub_sub_sub_sub_category_id = $k1;
   																		$sub_sub_sub_sub_sub_sub_category_id = $k2;
   																		break;
   																	}
   																}
   															}
   														}
   													}
   
   													$this->core->title($categories[$category_id][$sub_category_id][$sub_sub_category_id][$sub_sub_sub_category_id][$sub_sub_sub_sub_category_id][$sub_sub_sub_sub_sub_category_id][$sub_sub_sub_sub_sub_sub_category_id][0]);
   													$bread_crumbs[] = array(
   														'text' => $categories[$category_id][$sub_category_id][$sub_sub_category_id][$sub_sub_sub_category_id][$sub_sub_sub_sub_category_id][$sub_sub_sub_sub_sub_category_id][$sub_sub_sub_sub_sub_sub_category_id][0],
   														'url' => $location['alias'] . '/' . $this->core->translit($categories[$category_id][0]) . '/' . $this->core->translit($categories[$category_id][$sub_category_id][$sub_sub_category_id][0]) . '/' . $this->core->translit($categories[$category_id][$sub_category_id][$sub_sub_category_id][$sub_sub_sub_category_id][$sub_sub_sub_sub_category_id][0]) . '/' . $this->core->translit($categories[$category_id][$sub_category_id][$sub_sub_category_id][$sub_sub_sub_category_id][$sub_sub_sub_sub_category_id][$sub_sub_sub_sub_sub_category_id][$sub_sub_sub_sub_sub_sub_category_id][0])
   													);
   
   													$query_add = "
   														AND (`market_item`.`category_key` = '" . $category_id . "')
   														AND  `market_item`.`param` REGEXP '(\"" . $sub_sub_sub_sub_category_id . "\")'
   													";
   													
   													$vip_query_add = "
   														AND `market_item`.`category_key` = '" . $category_id . "'
   													";
   
   												}

   qualitycode, 16 Февраля 2015

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

5. PHP / Говнокод #17645

   +160

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

   <?php
   namespace MultipleHashMapQueueMessageRetrieverCriterias\Domain\MultipleHashMapQueueMessageRetrieverCriterias\Adapters\Exceptions;
   use MultipleHashMapQueueMessageRetrieverCriterias\Domain\Exceptions\MultipleHashMapQueueMessageRetrieverCriteriaException;
   
   final class CannotConvertMultipleHashMapQueueMessageRetrieverCriteriaToMultipleQueueMessageRetrieverCriteriaException extends MultipleHashMapQueueMessageRetrieverCriteriaException {
       const CODE = 1;
       public function __construct($message, MultipleHashMapQueueMessageRetrieverCriteriaException $parentException = null) {
           parent::__construct($message, self::CODE, $parentException);
       }
   }

   http://www.reddit.com/r/lolphp/comments/2vrgr6/irestfulmultiplehashmapqueuemessageretri evercriter/
   https://github.com/irestful/MultipleHashMapQueueMessageRetrieverCrit erias/blob/master/MultipleHashMapQueueMessageRetrieverCrit erias/Domain/MultipleHashMapQueueMessageRetrieverCrit erias/Adapters/Exceptions/CannotConvertMultipleHashMapQueueMessage RetrieverCriteriaToMultipleQueueMessageR etrieverCriteriaException.php
   
   Алсо, обратите внимание на структуру гитхабовских репозиториев

   myaut, 15 Февраля 2015

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

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

   +160

   1. 1
   2. 2

   var hgh = $(document).height() - 320 - 180 - 440;
   $(".message-form").height(  hgh );

   шта?

   heyzea1, 11 Февраля 2015

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

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

   +160

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

   <?php // if you delete this the sky will fall on your head
   endwhile; //endwhile have_posts()
   else: //have_posts()
   ?>
   <p>Sorry, no posts matched your criteria.</p>
   <?php endif; ?>
   <!-- // this is just the end of the motor - don't touch that line either :) -->

   ...мм, Ок!

   elvis, 04 Февраля 2015

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

8. JavaScript / Говнокод #17561

   +160

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

   this.alphabet = ['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z'];
   this.identityDefence = this.alphabet[Math.floor(Math.random() * (this.alphabet.length - 1))] + Math.floor(Math.random() * 1000) 
   + this.alphabet[Math.floor(Math.random() * (this.alphabet.length - 1))] 
   + this.alphabet[Math.floor(Math.random() * (this.alphabet.length - 1))] + Math.floor(Math.random() * 1000) 
   + this.alphabet[Math.floor(Math.random() * (this.alphabet.length - 1))] + Math.floor(Math.random() * 1000);

   А как вы генерируете случайный id объектов?

   SuperCuke, 02 Февраля 2015

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

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

   +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

   'idate' => 
           array (
             'UNIX' => '1346688000',
             'datetime' => '03.09.2012 19:00',
             'time' => '19:00',
             'hour' => '19',
             'second' => '00',
             'date' => '03.09.2012',
             'datename' => '03 September 2012',
             'year' => '2012',
             'y' => '12',
             'd' => '03',
             'd0' => '3',
             'm' => '09',
             'm0' => '9',
             'month' => 'September',
             'day' => 'Monday',
             'monthr' => 'сентября',
             'datenamer' => '03 сентября 2012',
             'ago' => '2 года назад',
           ),

   Правильное хранение даты. Работаю недавно и сам рак еще тот - но это слишком. Переношу БД из в свой проект и встречаю это...

   VladDelec, 28 Января 2015

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

10. JavaScript / Говнокод #17463

    +160

    1. 1

    $('div.body').css('height', '100%').parent('body').css('height', '100%').parent('html').css('height', '100%');

    Долой css и здравый смысл!

    hasesanches, 17 Января 2015

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

11. JavaScript / Говнокод #17280

    +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

    var someThing = new Object({
        isVariant1: function() {
            return (this._type() == "var1") ? true : false;
        },
        isVariant2: function() {
            return (this._type() == "var2") ? true : false;
        },
        isVariant3: function() {
            return (this._type() == "var3") ? true : false;
        },
        isVariant4: function() {
            return ((this._type().indexOf("vvvar") >= 0)) ? true : false;
        }
    });

    Легаси код одного известного проекта.
    Все имена функций и содержимое строк изменены.

    pater, 09 Декабря 2014

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

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

• 1571
• …
• 1310
• 1309
• 1308
• 1307
• 1306
• 1305
• 1304
• 1303
• 1302
• …
• 1