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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2371)
4. JavaScript (2031)
5. C# (1928)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (587)
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. C# / Говнокод #8818

   +120

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

   for (; itemList.Parent != null; {
           Item parent;
           itemList = parent.Parent;
       }
   )
   {
       parent = itemList.Parent.Parent.Parent;
       list.Add((object) parent);
   }

   Crazzy, 14 Декабря 2011

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

2. SQL / Говнокод #8817

   −109

   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

   end;
   create or replace package body "MUT_TABLE" is
   type a_table is table of number index by pls_integer;
   position a_table;
   empty a_table;
   
   procedure SET_INITIAL_STATE
   as
   begin
   position:=empty;
   end SET_INITIAL_STATE;
   
   procedure BEF_TR( ID IN NUMBER, A IN NUMBER)
   as
   
   begin
   
   position (ID):= A;
   
   end BEF_TR;
   
   procedure AFT_TR
   is
   v number;
   
   begin
   v:=position.FIRST;
   while v is not null loop
   
   update A set
   A=1,
   B='!!!!!!!!!'
   where A = 1;
   v:= position.next(v); 
   end loop;
   
   end AFT_TR;
   
   end "MUT_TABLE";

   kikis, 14 Декабря 2011

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

3. SQL / Говнокод #8816

   −111

   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
   90. 90
   91. 91
   92. 92
   93. 93
   94. 94
   95. 95
   96. 96
   97. 97
   98. 98

   CREATE TABLE "A" 
   ( "ID" NUMBER, 
   "A" NUMBER, 
   "B" VARCHAR2(20), 
   CONSTRAINT "A_PK" PRIMARY KEY ("ID") ENABLE
   )
   /
   
   CREATE OR REPLACE TRIGGER "A_2" 
   AFTER
   insert or update on "A"
   for each row
   begin
   
   mut_table.bef_tr(:NEW.ID,:NEW.A);
   
   end;
   /
   ALTER TRIGGER "A_2" ENABLE
   /
   
   CREATE OR REPLACE TRIGGER "A_3" 
   AFTER
   insert on "A"
   declare
   b number;
   
   begin
   mut_table.aft_tr;
   end;
   /
   ALTER TRIGGER "A_3" ENABLE
   /
   
   CREATE OR REPLACE TRIGGER "A_AT" 
   BEFORE
   insert or update on "A"
   for each row
   declare 
   v number;
   PRAGMA AUTONOMOUS_TRANSACTION;
   begin
   
   :NEW.A:=:NEW.A+1;
   :NEW.B:='!!!!!!!!!!';
   
   commit;
   
   end;
   /
   ALTER TRIGGER "A_AT" DISABLE
   /
   
   CREATE OR REPLACE TRIGGER "A_I_S" 
   BEFORE
   insert or update on "A"
   begin
   mut_table.set_initial_state;
   end;
   /
   ALTER TRIGGER "A_I_S" ENABLE
   /
   
   CREATE OR REPLACE TRIGGER "A_T1" 
   BEFORE
   insert or update on "A"
   for each row
   
   begin
   
   update A set 
   A = :NEW.A+1,
   B = 'was incremented'
   where A = :NEW.A;
   end;
   /
   ALTER TRIGGER "A_T1" DISABLE
   /
   
   CREATE OR REPLACE TRIGGER "BI_A" 
   before insert on "A" 
   for each row 
   begin 
   if :NEW."ID" is null then 
   select "A_SEQ".nextval into :NEW."ID" from dual; 
   end if; 
   end; 
   
   /
   ALTER TRIGGER "BI_A" ENABLE
   /
   
   
   create or replace package MUT_TABLE as
   
   procedure set_initial_state;
   procedure bef_tr(ID number,A NUMBER);
   procedure aft_tr;

   kikis, 14 Декабря 2011

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

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

   +161

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

   <?
   if(!file_exists($abs_path)){
     $new = "Y";
     // ...
   }
   //...
   if(strlen($new) > 0  ){
   // ...
   ?>

   Угадай CMS по коду!

   belukov, 14 Декабря 2011

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

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

   +993

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

   if(pObject == NULL)
   {
           //code
   }else if(pObject != NULL && pObject->getProperty() == Class::VALUE)
   {
          //code
   }

   Идентификаторы надуманные, но суть такова

   google-moogle, 14 Декабря 2011

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

6. ActionScript / Говнокод #8813

   −126

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

   Config.createErrorSkin = function (data:*):IPlayerSkin
   {
   	return new PlayerSkinPromoDjVideoErrorLoadConfState(stage.stageWidth, stage.stageHeight, data);
   }

   kyzi007, 13 Декабря 2011

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

7. SQL / Говнокод #8812

   −114

   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

   -- Я ТУТ ИСПОЛЬЗУЮ 2 КУРСОРА, 
   -- причем 2рой вызывается изнутри первого, 
   -- причем этот 2рой курсор ПОЛНОСТЬЮ (в худшем случае) 
   -- перебирает все свои значения, чтобы подставить одно.
   
   -- ВОПРОС: как сделать так, чтоб работало так же, но чтоб не так яро грузилось???
   
   DECLARE 
     CURSOR get_col1 IS
       SELECT DISTINCT 
   	a.constraint_name, 
     a.constraint_type,
     a.table_name,
     a.r_constraint_name,
     b.column_name
       FROM all_constraints a, all_cons_columns b
       WHERE a.table_name = b.table_name
     AND a.constraint_type in ('P', 'R')
     AND a.constraint_name = b.constraint_name
       ORDER by 1;
     
     CURSOR get_col2 IS
       SELECT constraint_name, table_name, column_name 
       FROM all_cons_columns
       WHERE constraint_name in 
             (select r_constraint_name 
              from all_constraints 
              where constraint_type = 'R');
       
   BEGIN
     dbms_output.ENABLE;
     dbms_output.put_line(rpad('Имя ограничения', 30)
               ||rpad('Тип', 5)
               ||rpad('Имя столбца', 30)
               ||rpad('Имя таблицы', 30)
               ||rpad('Имя таблицы', 30)
               ||rpad('Имя столбца', 30));
     dbms_output.put_line(rpad('-', 155, '-'));
     
     FOR main_col IN get_col1 LOOP
       dbms_output.put(rpad(to_char(main_col.constraint_name), 30)
         ||rpad(to_char(main_col.constraint_type), 5)
         ||rpad(to_char(main_col.column_name), 30)
         ||rpad(to_char(main_col.table_name), 30));
       
       IF (main_col.constraint_type = 'R') THEN
         
            FOR sec_col IN get_col2 LOOP
                 
                 IF (main_col.r_constraint_name = sec_col.constraint_name) THEN
                   
                   dbms_output.put(rpad(to_char(sec_col.table_name), 30)
                                      ||rpad(to_char(sec_col.column_name), 30));
                 END IF;
                 
            END LOOP;
       
       END IF;
       
       dbms_output.new_line();  
     END LOOP;
   END;
   /

   7. Используя сведения из представлений словаря данных получить
   информацию о первичных и внешних ключах схемы: Номер по порядку,
   Имя ограничения целостности, Тип, Имя столбца, Имя таблицы.
   Кроме того, для внешних ключей указать Имя таблицы и Имя столбца
   на которые ссылаются эти ключи.

   kikis, 13 Декабря 2011

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

8. Куча / Говнокод #8811

   +129

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

   <div class="sliderItem">
       <a class="j-fullscreen-thumbnail" href="/events/night-life/2011/12/10/568385/pictures/18257298">
           <img src="about: blank"  lazysrc="http://files2.geometria.ru/pics/thumbnail/18257298.jpg" style="visibility: hidden;"  alt="Всемирный День Футбола!" width="110" height="110"/>
           <span class="j-fullscreenLink accessConfirmed" message="Чтобы посмотреть фото на весь экран, войдите на сайт" title="На весь экран"></span>
       </a>
   </div>

   С главной страницы социалки одной. Страница - ну просто памятник культу карго. Сто сорок загружаемых скриптов. С ними - двести запросов. Триста шестьдесят вот этих чудесных, использующих lazy loading, declarative data binding и хрен пойми чего ещё шаблонов.
   
   PHP-шник - он как эмбрион, по мере роста повторяет весь путь развития отрасли веб-разработки.

   actuator, 13 Декабря 2011

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

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

   +164

   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

   class PAGING
   {
     var $sql,$records,$pages;
     var $page_no,$total,$limit,$first,$previous,$next,$last,$start,$end;
     function PAGING($sql,$records=9,$pages=5)
     {   global $mysqli;
       if($pages%2==0) $pages++;
       $res = $mysqli->query($sql);
       $sqlquery++;
       $total=mysqli_num_rows($res);
       $page_no=isset($_GET["p"])?$_GET["p"]:1;
       $limit=($page_no-1)*$records;
       $sql.=" limit $limit,$records";
       $first=1;
       $previous=$page_no>1?$page_no-1:1;
       $next=$page_no+1;
       $last=ceil($total/$records);
       if($next>$last)
         $next=$last;
       $start=$page_no;
       $end=$start+$pages-1;
       if($end>$last)
         $end=$last;
       if(($end-$start+1)<$pages)
       {
         $start-=$pages-($end-$start+1);
         if($start<1)
           $start=1;
       }
       if(($end-$start+1)==$pages)
       {
         $start=$page_no-floor($pages/2);
         $end=$page_no+floor($pages/2);
         while($start<$first)
         {
           $start++;
           $end++;
         }
         while($end>$last)
         {
           $start--;
           $end--;
         }
       }
       $this->sql=$sql;
       $this->records=$records;
       $this->pages=$pages;
       $this->page_no=$page_no;
       $this->total=$total;
       $this->limit=$limit;
       $this->first=$first;
       $this->previous=$previous;
       $this->next=$next;
       $this->last=$last;
       $this->start=$start;
       $this->end=$end;
     }
     function show_paging($url,$params=""){...}
   }

   Вот так вот товарищ пейджинг организует ООО)

   aleXoid, 13 Декабря 2011

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

10. Куча / Говнокод #8809

    +137

    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

    function ShowRubricThread(parent_id)
    		{
     
    		myArray = new Array('menu1_child_of_0','menu2_child_of_1','menu3_child_of_2','menu4_child_of_2','menu18_child_of_2','menu21_child_of_2','menu23_child_of_21','menu25_child_of_21','menu28_child_of_21','menu200_child_of_21','menu30_child_of_21','menu32_child_of_21','menu33_child_of_2','menu180_child_of_2','menu202_child_of_2','menu46_child_of_1','menu131_child_of_46','menu132_child_of_131','menu137_child_of_131','menu138_child_of_131','menu47_child_of_46','menu48_child_of_46','menu50_child_of_46','menu51_child_of_46','menu52_child_of_51','menu53_child_of_51','menu54_child_of_51','menu55_child_of_46','menu59_child_of_1','menu61_child_of_59','menu62_child_of_59','menu17_child_of_0','menu184_child_of_17','menu185_child_of_184','menu186_child_of_184','menu187_child_of_184','menu188_child_of_184','menu189_child_of_184','menu190_child_of_184','menu191_child_of_184','menu192_child_of_184','menu197_child_of_184','menu194_child_of_184','menu63_child_of_17','menu64_child_of_63','menu65_child_of_63','menu66_child_of_63','menu67_child_of_63','menu68_child_of_63','menu69_child_of_63','menu70_child_of_63','menu71_child_of_63','menu84_child_of_63','menu73_child_of_63','menu74_child_of_17','menu77_child_of_17','menu81_child_of_17','menu124_child_of_0','menu125_child_of_124','menu126_child_of_124','menu129_child_of_124','menu130_child_of_124','menu182_child_of_124','menu198_child_of_124','menu141_child_of_0','menu142_child_of_141','menu143_child_of_141','menu144_child_of_141','menu146_child_of_141','menu177_child_of_141','menu147_child_of_141','menu148_child_of_141','menu149_child_of_141','menu150_child_of_141','menu151_child_of_141','menu152_child_of_141','menu153_child_of_141','menu154_child_of_141','menu155_child_of_141','menu156_child_of_141','menu157_child_of_141','menu201_child_of_141','menu8_child_of_0','menu160_child_of_8','menu161_child_of_8','menu162_child_of_8','menu163_child_of_8','menu164_child_of_8','menu165_child_of_8','menu166_child_of_8','menu203_child_of_0','menu204_child_of_203','menu11_child_of_0','menu105_child_of_11','menu181_child_of_11','menu108_child_of_11','menu109_child_of_11','menu110_child_of_11','menu111_child_of_11','menu12_child_of_0','menu199_child_of_12','menu168_child_of_12','menu179_child_of_12','menu115_child_of_12','menu116_child_of_12','menu117_child_of_12','menu170_child_of_117','menu167_child_of_12','menu178_child_of_12','menu13_child_of_0','menu174_child_of_13','menu14_child_of_0','menu120_child_of_14','menu121_child_of_14','menu122_child_of_14','menu123_child_of_14');
    					for(var i=0; i < myArray.length; i++)
    					{
     
    							reg_string = "_child_of_" + parent_id + "$";
    							reg = new RegExp(reg_string);
    							newArray = myArray[i].match(reg);
    							if( newArray != null)
    							{
    							document.getElementById(myArray[i]).style.display = "block";
    							}
     
    					}
     
    		ShowTread = true;
     
    		}

    http://chechnya.gov.ru/page.php?day=102&month=13&year=2222
    Формат мой.    и говнокода

    dos_, 13 Декабря 2011

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

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

• 2107
• …
• 864
• 863
• 862
• 861
• 860
• 859
• 858
• 857
• 856
• …
• 1