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

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

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

1. PHP (5711)
2. C++ (2742)
3. Куча (2397)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1120)
8. Pascal (648)
9. Python (591)
10. 1C (540)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (44)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. Куча / Говнокод #17423

   +133

   1. 1

   http://www.youtube.com/watch?v=yRsT5wBSYZ0

   Советую

   LispGovno, 09 Января 2015

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

3. C# / Говнокод #17388

   +96

   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

   _catalogDb.Categories
                    .Where(w => w.Resources.Where(r => _catalogDb.Resources
                       .Where(a => _catalogDb.Persons.Where(b => b.EmployeeNumber == employeeNumber)
                           .Select(c => c.VIP).FirstOrDefault() || !a.VIP)
                    .Where(a => _catalogDb.Persons.Where(b => b.EmployeeNumber == employeeNumber)
                        .Select(c => c.InnerEmployee).FirstOrDefault() || !a.InnerEmployee)
                    .Where(a => !a.Paths.Any()
                        || a.Paths.Where(wer => wer.Name == null).Any()
                           || a.Paths.Where(pt => pt.Name != null && _catalogDb.Persons
                               .Where(b => b.EmployeeNumber == employeeNumber)
                                   .Select(c => c.Path).FirstOrDefault().Contains(pt.Name)).Any())
                    .Where(a => !a.Filials.Any() || a.Filials.Where(wer => wer.Code == null).Any() || a.Filials
                        .Where(pt => pt.Code != null && pt.Code == _catalogDb.Persons
                            .Where(b => b.EmployeeNumber == employeeNumber)
                               .Select(c => c.Filial).FirstOrDefault()).Any())
                    .Select(a => a.UNID).Contains(r.UNID)).Any())
                       .Select(s => s.ID).Distinct().ToList();

   Кто сказал что Entity framework облегчает жизнь...

   SuperCuke, 30 Декабря 2014

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

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

   −108

   1. 1
   2. 2

   if True: #зачем.
       ....

   На новом проекте. Радуют и код и комментарии)

   intestinalbrain, 20 Ноября 2014

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

5. C# / Говнокод #16205

   +143

   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

   return string.Format(templateStr, "",
                   _p1_Tb.Text,
                   _p2_Tb1.Text + ", " + _p2_Tb2.Text,
                   _p3_Tb.Text,
                   "",//_p4_Tb.Text,
                   _p5_Tb1.Text + " / " + _p5_Tb2.Text,
                   _p6_Tb.Text,
                  RblTxt(_p7_Rbl) + " " + _p7_Tb.Text,
                   b("Построен в ") + _p8_Tb1.Text + " году, в " + _p8_Tb2.Text + " году произведен "
                   + _p8_Tb3.Text + " ремонт. " + _p8_Tb4.Text + ", Количество корпусов " + _p8_Tb4.Text + ", Количество этажей в каждом корпусе " + _p8_Tb5.Text
                   + (_p8_Ch1.Checked ? (", " + _p8_Ch1.Text) : "")
                   + (_p8_Ch2.Checked ? (", " + _p8_Ch2.Text) : "")
                   + (_p8_Ch3.Checked ? (", " + _p8_Ch3.Text) : "") ,
                   FillTbl9() ,
                   b("Уровень средства размещения: ") + _p10_Tb.Text,
                   b("Объект находится рядом с ") + _p11_Tb.Text,
                   "принимаются  с " + _p12_Tb1.Text + " лет, " + _p12_Tb2.Text,
                   _p13_Tb.Text,
                   b("Период работы: ") + (_p14_Rb1.Checked ? ("С " + _p14_Tb1.Text + " По " + _p14_Tb2.Text) : "Круглогодично"),
                   "к " + _p15_Tb1.Text + " до " + _p15_Tb2.Text,
                   _p16_Tb.Text
                   + (_p16_Ch1.Checked ? ("<br/> От аэропорта " + _p16_Tb1.Text + " автобусом № " + _p16_Tb2.Text + " до остановки" + _p16_Tb3.Text) : "")
                   + (_p16_Ch2.Checked ? ("<br/> От жд вокзала " + _p16_Tb4.Text + " автобусом № " + _p16_Tb5.Text + " до остановки" + _p16_Tb6.Text) : "")
                   + (_p16_Ch3.Checked ? ("<br/> Другое " + _p16_Tb7.Text) : "")
                   ,
                   b("Типы номеров: ") + _p17_Tb.Text,
                   b("В номере: ")
                    ((_p23_Ch1.Checked ? (" " + _p23_Ch1.Text + ",") : "")
                   + (_p23_Ch2.Checked ? (" " + _p23_Ch2.Text + ",") : "")
                   + (_p23_Ch3.Checked ? (" " + _p23_Ch3.Text + ",") : "")
                   + (_p23_Ch4.Checked ? (" " + _p23_Ch4.Text + ",") : "")
                   + (_p23_Ch5.Checked ? (" " + _p23_Ch5.Text + ",") : "")
                   + (_p23_Ch6.Checked ? (" " + _p23_Ch6.Text + ",") : "")
                   + (_p23_Ch7.Checked ? (" " + _p23_Ch7.Text + ",") : "")
                   + (_p23_Ch8.Checked ? (" " + _p23_Ch8.Text + ",") : "")
                   + (_p23_Ch9.Checked ? (" " + _p23_Ch9.Text + ",") : "")
                   + (_p23_Ch10.Checked ? (" " + _p23_Ch10.Text + ",") : "")
                   + (_p23_Ch11.Checked ? (" " + _p23_Ch11.Text + ",") : "")
                   + (_p23_Ch12.Checked ? (" " + _p23_Ch12.Text + ",") : "")
                   + (_p23_Ch13.Checked ? (" " + _p23_Ch13.Text + ",") : "")
                   + (_p23_Ch14.Checked ? (" " + _p23_Ch14.Text + ",") : "")
                   + (_p23_Ch15.Checked ? (" " + _p23_Ch15.Text + ",") : "")
                   + (_p23_Ch16.Checked ? (" " + _p23_Ch16.Text + ",") : "")
                   + (_p23_Ch17.Checked ? (" " + _p23_Ch17.Text + ",") : "")
                   + (_p23_Ch18.Checked ? (" " + _p23_Ch18.Text + ",") : "")
                   + (_p23_Ch19.Checked ? (" " + _p23_Ch19.Text + ",") : "")
                   + (_p23_Ch20.Checked ? (" " + _p23_Ch20.Text + ",") : "")
                   + (_p23_Ch21.Checked ? (" " + _p23_Ch21.Text + ",") : "")
                   + (_p23_Ch22.Checked ? (" " + _p23_Ch22.Text + ",") : "")
                   + (_p23_Ch23.Checked ? (" " + _p23_Ch23.Text + ",") : "")
                   + (_p23_Ch24.Checked ? (" " + _p23_Ch24.Text + ",") : "")
                   + (_p23_Ch25.Checked ? (" " + _p23_Ch25.Text + ",") : "")
                   + (_p23_Ch26.Checked ? (" " + _p23_Ch26.Text + ",") : "")
                   + (_p23_Ch27.Checked ? (" " + _p23_Ch27.Text + ",") : "")
                   + (_p23_Ch28.Checked ? (" " + _p23_Ch28.Text + ",") : "")
                   + (_p23_Ch29.Checked ? (" " + _p23_Ch29.Text + ",") : "")
                   + (_p23_Ch20.Checked ? (" " + _p23_Ch30.Text + ",") : "")
                   + (_p23_Ch31.Checked ? (" " + _p23_Ch31.Text + ",") : "")
                   + (_p23_Ch32.Checked ? (" " + _p23_Ch32.Text + ",") : "")
                   + (_p23_Ch33.Checked ? (" " + _p23_Tb.Text) : "")).TrimEnd(new char[] { ' ', ',' }),
                   ((_p24_Ch1.Checked ? (" " + _p24_Ch1.Text + ",") : "")
                   + (_p24_Ch2.Checked ? (" " + _p24_Ch2.Text + ",") : "")
                   + (_p24_Ch3.Checked ? (" " + _p24_Tb.Text) : "")).TrimEnd(new char[] { ' ', ',' }),
                   FillTbl25(),
                   RblTxt(_p26_Rbl1) + " " + _p26_Tb.Text + ". " + RblTxt(_p26_Rbl2),
                   ((_p27_Ch1.Checked ? (" " + _p27_Ch1.Text + ",") : "")
                   + (_p27_Ch2.Checked ? (" " + _p27_Ch2.Text + ",") : "")
                   + (_p27_Ch3.Checked ? (" " + _p27_Ch3.Text + ",") : "")
                   + (_p27_Ch4.Checked ? (" " + _p27_Ch4.Text + ",") : "")
                   + (_p27_Ch5.Checked ? (" " + _p27_Ch5.Text + ",") : "")
                   + (_p27_Ch6.Checked ? (" " + _p27_Ch6.Text + ",") : "")
                   + (_p27_Ch7.Checked ? (" " + _p27_Tb.Text) : "")).TrimEnd(new char[] { ' ', ',' }),
                   _p28_Tb.Text,
                   b("Минимальная продолжительность заезда ") + _p29_Tb.Text + " дней",
                   "от " + _p30_Tb1.Text + " до " + _p30_Tb2.Text + " лет " + _p30_Tb3.Text,
                   ((_p31_Ch1.Checked ? (" " + _p31_Ch1.Text + " " + _p31_Tb1.Text) : "")
                   + "<br/>" + (_p31_Ch2.Checked ? (" " + _p31_Ch2.Text + " " + _p31_Tb2.Text) : "")
                   + "<br/>" + (_p31_Ch3.Checked ? (" " + _p31_Ch3.Text + " " + _p31_Tb3.Text) : "")

   Бэкенды крупнейшего российского туроператора. На отдел разработки уходит 1500000 рб/мес.

   tablecell, 23 Июня 2014

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

6. Си / Говнокод #16065

   +133

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

   typedef long bool_t;
   #define TRUE 1
   #define FALSE 0
   
   void f() {
           bool_t var;
           for (var = TRUE; var > FALSE; var++) { /* ... */ }
   }

   evg_ever, 28 Мая 2014

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

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

   +154

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

   var turn = 0;
   		function turn_over() {
   			var text = document.getElementById('turn-over-in').value;
   			var first_letter = text.charAt(0).toLowerCase();
   			var last_letter = text.charAt(text.length-1).toUpperCase();
   			var new_text = first_letter + text.substr(1,text.length-2) + last_letter;
   			var text_array = new_text.split('');
   			var inside_out = text_array.reverse();
   			var turn_over_text = inside_out.join('');
   			document.getElementById('turn-over-out').value = turn_over_text;
   <...>

   Задача: развернуть строку текста (Абв->Вба).

   gost, 21 Мая 2014

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

8. C++ / Говнокод #15978

   +16

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

   #include <windows.h>
    #include <iostream>
    int main ()
    {
        HINSTANCE result;
        result=ShellExecute(NULL,NULL,L"E:\\.mp3",NULL,NULL,SW_SHOWDEFAULT);
        if ((int)result<=32)
        std::cout << "Error!\nReturn value: " << (int)result << "\n";
        return 0;
    }

   Как написать mp3-плеер на с++ в 10 строк без использования сторонних библиотек?
   Гении с cyberforum знают ответ!
   
   http://www.cyberforum.ru/cpp-beginners/thread444490-page3.html

   gost, 13 Мая 2014

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

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

   +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

   function rawToStructuredDataTree($data) {
           $structured_array = array();
           foreach ($data as $cid => $node) {
               $data[$cid]['children'] = array();
               if ($node['parent_id'] == $cid || $node['parent_id'] == 0) {
                   $structured_array[$cid] = &$data[$cid];
               } else {
                   $data[$node['parent_id']]['children'][$cid] = & $data[$cid];
               }
           }
           return $structured_array;
     }

   Вот такое выдал мой ученик (школьник, 8 класс), когда его попросили из массива id - parent_id построить дерево.

   imissyouso, 20 Апреля 2014

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

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

    +154

    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

    function unpackLoginAndPas($signed,$iv, $dateSend){
        global $crypt, $cryptLast,  $defaultDate, $ivsId, $mysqlBaseName, $mysqlBase;
        //получаем текущую дату. 
        if (isset($dateSend) )
          $date = $dateSend;
        else
          $date = $defaultDate;
    
          $query = "SELECT id FROM ivs WHERE val='$iv';" ;
        $query = $mysqlBase->query($query);// or die (compactResult('error','криворукий программист!'));
        //есть ли такой маркер безопасности?
        if ($query->num_rows)//раз есть, то надо послать за другим маркером. Такой уже не годиться. 
            die (compactResult('error','bad iv marker,pls generate new iv'));
        //запоминаем такой маркер безопасности, и больше под ним не пускаем.
        //получаем id автоинкремент
        $myBase = $mysqlBase->query("SHOW TABLE STATUS FROM $mysqlBaseName") or die (compactResult('error','SystemTable not get (')); ;
        $result = false;
        //поиск времени изменения БД.
        $dateSrv = (int)( $date->getTimestamp() / 1000 );
        while($row = $myBase->fetch_assoc()){
          if ($row['Name'] !== "ivs" )
    	continue;
          $ivsId = $row["Auto_increment"];
        }
    
        $userIP = getUserIpAddr();
        $mysqlBase->query("INSERT INTO ivs(val,IP) VALUES('$iv','$userIP')") or die(compactResult('error','криворукий программист!'));
    
        //составляем строку из даты, которая у нас будет ключем. 1 раз в 5 минут она становиться другой.
        $decodeKey = /*формирование строки от даты*/
        $dateLast = $date->add( DateInterval::createFromDateString(' 5 minuts ' ) );
        $decodeKeyLast = /*формирование строки от даты*/
        //var_dump($decodeKey);
        $decodeKey = md5($decodeKey);
        $decodeKeyLast = md5 ( $decodeKeyLast );
        
        mcrypt_generic_init ($crypt,$decodeKey,$iv);
        mcrypt_generic_init ($cryptLast,$decodeKeyLast,$iv);
        //декриптуем
        $encDataLoginAndPass = rtrim (mdecrypt_generic($crypt,$signed));
        $encDataLoginAndPassLast = rtrim (mdecrypt_generic($cryptLast,$signed));
        //если правильный был и ключ(время не истекло) и 
        //var_dump($encDataLoginAndPass);
        $encDataLoginAndPass = json_decode ($encDataLoginAndPass,true)  or ( ($decodeKey = $decodeKeyLast ) and json_decode ($encDataLoginAndPassLast,true) )or die(compactResult('error','Key time out.'));
        //в поле secdata в POST будет лежать массив, декодированный
        mcrypt_generic_init ($crypt,$decodeKey,$iv);
    
        $temp = array_keys($encDataLoginAndPass);
        $encData = array('login'=>$temp[0], 'password'=>end($encDataLoginAndPass));
        $encPost = mdecrypt_generic($crypt, base64_decode( $_POST['secdata'] ) );
        $encData['post'] = json_decode (rtrim($encPost)) or die(compactResult('error','post uncompressed!'));
        //деинит
        mcrypt_generic_deinit ($crypt);
        //раз все прошло успешо, возвращаем массив с логином и паролем, который проверяем дальше. Тут даже использование вместо пароля, его хеша не поможет от хака. будем надеятся на алгоритм шифровки. Хотя есть взять, и знать что тут будет одинаковая информация, то вычислить её по паре перехваченных обращений не составит труда. Надо бы переделать на хеш от логина с паролем. Хотя и это врядли поможет. Надо взять что-то изменяемое. От чего можно будет отталкиваться. Но это надо обдумывать.
        return $encData;
    };

    попытка шифрования в 1-м проекте.
    Человек первый раз писал на php. Оно и видно...

    Dart_Sergius, 17 Апреля 2014

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

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

    +153

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

    var cont_els = section.el.getElementsByClassName('cont');
    for (var i = 0; i < cont_els.length; i++)
    {
        var node_els = cont.el.getElementsByClassName('node');
        for (var i = 0; i < node_els.length; i++)

    Вложенный цикл переписывает i внешнего, и так бесконечно.

    Itareo, 01 Апреля 2014

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

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

• 1976
• …
• 1564
• 1563
• 1562
• 1561
• 1560
• 1559
• 1558
• 1557
• 1556
• …
• 1