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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2374)
4. JavaScript (2032)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (588)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (199)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (31)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. Pascal / Говнокод #3445

   +104

   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

   procedure TForm1.save_to_file_coordinates;
   label
   f,d;
   begin
    f:begin
    if str1=0 then
     begin
      memo1.items.Strings[0]:=memo3.Lines.Strings[0];
      str1:=str1+1;
     end;
    if str1=b then
     begin
       goto d;
     end;
    begin
      memo1.items.Strings[strtoint(floattostr(str1))]:=memo3.Lines.Strings[strtoint(floattostr(str1))];
      str1:=str1+1;
    end;
    goto f;
    end;
     d:begin
         begin
            if b=i then
             begin
               exit;
             end;
             begin
               memo2.items.Strings[strtoint(floattostr(str2))]:=memo3.Lines.Strings[strtoint(floattostr(b))];
               b:=b+1;
               str2:=str2+1;
               end;
     goto d;
   end;
   end;
   
   end;

   mistmax, 10 Июня 2010

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

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

   +157

   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

   function month_to_number(a) {
   	if (a == 'Jan') {
   		a = '01';
   	}
   	if (a == 'Feb') {
   		a = '02';
   	}
   	if (a == 'Mar') {
   		a = '03';
   	}
   	if (a == 'Apr') {
   		a = '04';
   	}
   	if (a == 'May') {
   		a = '05';
   	}
   	if (a == 'Jun') {
   		a = '06';
   	}
   	if (a == 'Jul') {
   		a = '07';
   	}
   	if (a == 'Aug') {
   		a = '08';
   	}
   	if (a == 'Sep') {
   		a = '09';
   	}
   	if (a == 'Oct') {
   		a = '10';
   	}
   	if (a == 'Nov') {
   		a = '11';
   	}
   	if (a == 'Dec') {
   		a = '12';
   	}
   	return a;
   };

   eval, 10 Июня 2010

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

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

   +159

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

   echo "<tr><td></td><td><small><font color=red>$sum0</font>/<font color=blue>$sum00</font></small></td>
   	<td> </td><td align=center><strong><!--$duse,--> $sum1</strong></td><td align=right><strong>".str_replace(".",",",c_mon($sum2))."</strong></td><td align=right><strong>".str_replace(".",",",c_mon($sum3))."</strong></td>\n
   	<td align=right><strong>".str_replace (".", ",", $sum4)."</strong></td>
   	<td align=right><strong>".str_replace (".", ",", sprintf("%01.2f",$straf/1024/1024))."</strong></td>
   	<td align=right><strong>".str_replace(".",",",c_mon($sum_pl))."</strong></td>
   	<td align=right nowrap><strong id=red>".str_replace(".",",",c_mon($sum_mn))."</strong></td>
   	<td> </td>
   	<td align=right><strong>".str_replace (".", ",", $sum5)."</strong></td>
   	<td>  </td></tr>";

   Точки и запятые в числах

   Werdn, 10 Июня 2010

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

4. Java / Говнокод #3442

   +74

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

   protected void setPhones(ArrayList<MBPhone> phones) {
       if(phones!=null) 
           this.phones=phones;
       else 
           phones.clear();
   }

   Код из одного западного вэб-сервиса для профессионалов в сфере недвижимости.
   Мораль: не надо игнорировать подсказки IDE. Автор на самом деле хотел очистить this.phones. И Intelij Idea подсказывает, что в этом месте может быть брошен NullPointerException. Увидеть эту проблему можно было только используя аннотацию @SuppressWarnings. Потому что варнингов так много, что различить среди них опасные очень сложно.

   vladimir.loshchin, 10 Июня 2010

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

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

   +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

   <?
   $mult=$mult+0;
   if ($mult<1 and $mult>38) $mult="001";
   $mult = substr("00".$mult, 0,3);
   //echo "mult=".$mult."<br>";
   $title="Мультик №".$mult;
   include ('../../novost/l1.htm');
   $minmult=1+1;
   $maxmult=38-1;
   $multa= "00".($mult-1);
   $multb= "00".($mult+1);
   $multa = substr($multa, -3);
   $multb = substr($multb, -3);
   
   if ($mult>$maxmult) $multb="001";
   if ($mult<$minmult) $multa="038";
   
   ?>
   <!-- ********************************************** -->
   <SCRIPT SRC="embedFlash.js" LANGUAGE="JavaScript" TYPE="text/javascript"></SCRIPT>
   <CENTER>
   <SCRIPT LANGUAGE="JavaScript">embedFlash("<? echo $mult; ?>.swf","100%","100%");</SCRIPT>
   <br>
   <a href="play.php?mult=<? echo $multa; ?>">Предыдущий</a> |
   <a href="./">Список</a> |
   <a href="play.php?mult=<? echo $multb; ?>">Следующий</a>
   </CENTER>
   <!-- ********************************************** -->
   <?php include ('../../novost/l3.htm'); ?>

   код моего друга админа)

   nur, 10 Июня 2010

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

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

   +168

   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

   function func()
     	{
   		$func = array(
   			"*3jd9asl4ksh8kl" => "Core::psUser->updateSelf",
   			"*9dkvls3kfrj54a" => "Core::psUser->setPassword",
   			"*3lso35ne9d0kek" => "Core::psUser->login",
   			"*487fj409z91dkv" => "Core::psUser->register_step1",			
   			"*ns252lo3a8ps11" => "Core::psUser->register_step2",
   			"*ale93kvnwo4pdz" => "Core::psUser->closeRequest",
   			"*ble93kv1wo4pdz" => "Core::psUser->checkEmail",
   			"*wpv48k9k2054k1" => "Core::psUser->recover",
   			"*3lsd8cjfdl3mdk" => "Core::Exchange->startExchange",
   			"*42sd9cjfd24mdk" => "Core::Exchange->startExchangeNew",
   			"*53se9cjfd35mdk" => "Core::Exchange->ExchangeStep2",
   			"*64se0cjfd46mdk" => "Core::Exchange->ExchangeStep3",
   			"*4164-4622-1270" => "Core::Exchange->dest1",
   			"*8273-7121-7643" => "Core::Exchange->dest2",
   			"*8710-4879-0216" => "Core::Exchange->dest3",
   			"*5566-0506-6230" => "Core::Exchange->dest4",
   			"*0635-1830-1345" => "Core::Exchange->dest5",
   			"*3265-0565-4871" => "Core::Exchange->dest6",
   			"*0914-1921-7194" => "Core::Exchange->dest7",
   			"*8126-3901-3136" => "Core::Exchange->dest8",
   			"*2711-0035-9992" => "Core::Exchange->dest9",
   			"*2846-4302-4032" => "Core::Exchange->dest10",
   			"*3879-1991-4154" => "Core::Exchange->dest11",
   			"*8557-5244-6506" => "Core::Exchange->dest12",
   			"*9455-7047-8778" => "Core::Exchange->dest13",
   			"*7329-2394-7939" => "Core::Exchange->dest14",
   			"*4168-0400-0156" => "Core::Exchange->dest15",
   			"*4672-6100-6653" => "Core::Exchange->dest16",
   			"*4930-5681-4263" => "Core::Exchange->dest17",
   			"*2028-3904-4406" => "Core::Exchange->dest18",
   			"*3344-7461-9263" => "Core::Exchange->dest19",
   			"*9282-3030-8891" => "Core::Exchange->dest20",
   			"*8198-3749-8009" => "Core::Exchange->dest21",
   			"*7254-2894-6243" => "Core::Exchange->dest22",
   			"*7969-8488-0540" => "Core::Exchange->dest23",
   			"*0202-7596-3531" => "Core::Exchange->dest24",
   			"*3321-4063-6580" => "Core::Exchange->dest25",
   			"*8319-8427-1264" => "Core::Exchange->dest26",
   			"*9606-3792-8045" => "Core::Exchange->dest27",
   			"*0242-6362-5886" => "Core::Exchange->dest28",
   			"*9362-4435-1538" => "Core::Exchange->dest29",
   			"*9332-4235-1337" => "Core::Exchange->dest30",
   			"*5362-2352-1350" => "Core::Exchange->dest31",
   			"*3262-4356-1670" => "Core::Exchange->dest32",
   			"*4562-4786-9440" => "Core::Exchange->dest33",
   			"*3222-3454-1430" => "Core::Exchange->dest34",
   			"*6573-3226-5683" => "Core::Exchange->dest35",
   			"*6543-9807-2573" => "Core::Exchange->dest36",
   			"*3455-7639-6268" => "Core::Exchange->dest37",
   			"*4523-6563-0978" => "Core::Exchange->dest38",
   			"*3485-7339-6298" => "Core::Exchange->dest39",
   			"*3489-8339-6238" => "Core::Exchange->dest40",
   			"*5485-9339-6295" => "Core::Exchange->dest41",
   			"*3445-7369-6248" => "Core::Exchange->dest42",
   			"*3945-7769-3248" => "Core::Exchange->dest43",
   			"*5945-7369-3568" => "Core::Exchange->dest44",
   			"*8945-7769-3868" => "Core::Exchange->dest45",
   			"*5948-1369-3578" => "Core::Exchange->dest46",			
   			"*9362-4435-1539" => "Core::Order->setStatusByUser",
   			"*9864-4532-2579" => "Core::Feedback->saveFeedback"
   		);
   		return $func;
   	}

   Передача action'а с обфускацией.

   hdkeeper, 10 Июня 2010

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

7. Java / Говнокод #3439

   +80

   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

   private JMyForm frm;
   
   public void view() 
   {
       if (((JOurComponent) frm).getAction() == Actions.Action1 
       {
           viewList(Actions.Action1);
       } 
       else
       if (((JOurComponent) frm).getAction() == Actions.Action2
       {
           viewList(Actions.Action2);
       }
       ...
       else
       if (((JOurComponent) frm).getAction() == Actions.Action117
       {
           viewList(Actions.Action117);
       }
   }
   
   private void viewList(String action) {
       if (action == Actions.Action1)
       {
           //
       }
       ...
       if (action == Actions.Action117)
       {
           //
       }
   }

   Что тут может радовать? ООП отменяется. И паттерны туда же.

   lotik, 10 Июня 2010

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

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

   +72

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

   if (doSomething() == true) {
       if (doSomething2()) {
           //...
       }
   }

   lotik, 10 Июня 2010

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

9. Java / Говнокод #3437

   +66

   1. 1
   2. 2
   3. 3

   if (xobj.getStringProp() == "N") {
       //do it
   }

   Ну а что тут скажешь?

   lotik, 10 Июня 2010

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

10. Java / Говнокод #3436

    +75

    1. 1
    2. 2
    3. 3

    if ((getProductId() != null) && (getProductId().equals(new String("5")))) {
      //do it
    }

    ProductId читается один раз при старте приложения, а таких проверок около 50-и.

    lotik, 10 Июня 2010

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

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

• 2108
• …
• 339
• 338
• 337
• 336
• 335
• 334
• 333
• 332
• 331
• …
• 1