1. 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)

  2. 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)

  3. 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)

  4. 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)

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

    +72

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (doSomething() == true) {
    if (doSomething2()) {
        //...
    }
}

    lotik, 10 Июня 2010

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

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

    +66

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

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

    lotik, 10 Июня 2010

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

  7. 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)

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

    +137

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    // Javascript
function updateDownloadSpeed(){
	$("download_speed").update(random(4000,6000)+" KB/s");
	setTimeout("updateDownloadSpeed()",random(1000,4000));
}
// Html
<div class="dowload-speed"id="download_speed">4 293 KB/s</div>

    http://maxzakachka.ru/
    Блок справа со средней закачкой.

    This is obvious, 10 Июня 2010

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

  9. Куча / Говнокод #3434

    +155

    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
    <!DOCTYPE html>
<html>
<head>
<style>
	*{ margin: 0; padding: 0; }
	html,body{
		height: 300px;
		width: 100%;
		border: 1px solid #C0C0C0;
	}
</style>
</head>
<body>
<table height=100%>
	<tr>
		<td><div style="border: 1px solid green;">fasdfadsf<br />fasdfadsf<br />fasdfadsf<br />fasdfadsf<br /></div></td>
	</tr>
	<tr height=100%>
		<td height=100%><div style="height:100%; border: 1px solid red;">fadsfadsfasd</div></td>
	</tr>
</table>
<!--
<div style="height: 100%; border: 1px solid blue; ">
	<div style="border: 1px solid green;">fasdfadsf<br />fasdfadsf<br />fasdfadsf<br />fasdfadsf<br /></div>
	<div style="height:100%; border: 1px solid red;">fadsfadsfasd</div>
</div>
-->
</body>
</html>

    в IE выходит за границы которые заданны в body.

    g0xff, 10 Июня 2010

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

  10. VisualBasic / Говнокод #3433

    −86

    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
    99. 99
    10 GOSUB 300
30 INPUT "Долгота (град)";B5,L5: INPUT "Часовая зона (час)";H
50 L5=L5/360: Z0=H/24
60 GOSUB 1170: T=(J-2451545)+F
70 TT=T/36525+1: ' TT = столетия, начиная с 1900.0
90 GOSUB 410: T=T+Z0
120 GOSUB 910: A(1)=A5: D(1)=D5 '  Получаем положение солнца
130 T=T+1
140 GOSUB 910: A(2)=A5: D(2)=D5
150 IF A(2)<A(1) THEN A(2)=A(2)+P2
160 Z1=DR*90.833: ' Вычисление зенита
170 S=SIN(B5*DR): C=COS(B5*DR)
180 Z=COS(Z1): M8=0: W8=0: PRINT
190 A0=A(1): D0=D(1)
200 DA=A(2)-A(1): DD=D(2)-D(1)
210 FOR C0=0 TO 23
220 P=(C0+1)/24
230 A2=A(1)+P*DA: D2=D(1)+P*DD
240 GOSUB 490
250 A0=A2: D0=D2: V0=V2
260 NEXT
270 GOSUB 820: ' Вывод информации
280 END
300 DIM A(2),D(2)      ' Константы
320 P1=3.14159265: P2=2*P1: DR=P1/180: K1=15*DR*1.0027379: S$="Заход солнца в "
330 R$="Восход солнца в ": M1$="В этот день солнце не восходит"     
370 M2$="В этот день солнце не заходит": M3$="Солнце заходит весь день"           
390 M4$="Солнце восходит весь день"          
400 RETURN                                   
410 T0=T/36525 '     Получение часового пояса
430 S=24110.5+8640184.813*T0
440 S=S+86636.6*Z0+86400*L5
450 S=S/86400: S=S-INT(S)
460 T0=S*360*DR
470 RETURN
490 L0=T0+C0*K1: L2=L0+K1  '  Просматриваем возможные события на полученный час
510 H0=L0-A0: H2=L2-A2
520 H1=(H2+H0)/2: '  Часовой угол,
530 D1=(D2+D0)/2: '  наклон в получасе
550 IF C0>0 THEN 570
560 V0=S*SIN(D0)+C*COS(D0)*COS(H0)-Z
570 V2=S*SIN(D2)+C*COS(D2)*COS(H2)-Z
580 IF SGN(V0)=SGN(V2) THEN 800
590 V1=S*SIN(D1)+C*COS(D1)*COS(H1)-Z
600 A=2*V2-4*V1+2*V0: B=4*V1-3*V0-V2
610 D=B*B-4*A*V0: IF D<0 THEN 800
620 D=SQR(D)
630 IF V0<0 AND V2>0 THEN PRINT R$;
640 IF V0<0 AND V2>0 THEN M8=1
650 IF V0>0 AND V2<0 THEN PRINT S$;
660 IF V0>0 AND V2<0 THEN W8=1
670 E=(-B+D)/(2*A)
680 IF E>1 OR E<0 THEN E=(-B-D)/(2*A)
690 T3=C0+E+1/120: ' Округление
700 H3=INT(T3): M3=INT((T3-H3)*60)
710 PRINT USING "##:##";H3;M3;
720 H7=H0+E*(H2-H0)
730 N7=-COS(D1)*SIN(H7)
740 D7=C*SIN(D1)-S*COS(D1)*COS(H7)
750 AZ=ATN(N7/D7)/DR
760 IF D7<0 THEN AZ=AZ+180
770 IF AZ<0 THEN AZ=AZ+360
780 IF AZ>360 THEN AZ=AZ-360
790 PRINT USING ",  азимут ###.#";AZ
800 RETURN
820 IF M8=0 AND W8=0 THEN 870 ' Процедура вывода информации
840 IF M8=0 THEN PRINT M1$
850 IF W8=0 THEN PRINT M2$
860 GOTO 890
870 IF V2<0 THEN PRINT M3$
880 IF V2>0 THEN PRINT M4$
890 RETURN
910 L=.779072+.00273790931*T ' Фундаментальные константы (Van Flandern & Pulkkinen, 1979)
950 G=.993126+.0027377785*T
960 L=L-INT(L): G=G-INT(G): L=L*P2: G=G*P2
980 V=.39785*SIN(L): V=V-.01000*SIN(L-G)
1000 V=V+.00333*SIN(L+G): V=V-.00021*TT*SIN(L)
1020 U=1-.03349*COS(G): U=U-.00014*COS(2*L)
1040 U=U+.00008*COS(L): W=-.00010-.04129*SIN(2*L)
1060 W=W+.03211*SIN(G): W=W+.00104*SIN(2*L-G)
1080 W=W-.00035*SIN(2*L+G): W=W-.00008*TT*SIN(G)
1120 S=W/SQR(U-V*V) '    Вычисление солнечных координат
1130 A5=L+ATN(S/SQR(1-S*S))
1140 S=V/SQR(U):D5=ATN(S/SQR(1-S*S))
1150 R5=1.00021*SQR(U)
1160 RETURN
1170 INPUT "Год, Месяц, День";Y,M,D ' Календарь --> JD
1190 G=1: IF Y<1583 THEN G=0
1200 D1=INT(D): F=D-D1-.5
1210 J=-INT(7*(INT((M+9)/12)+Y)/4)
1220 IF G=0 THEN 1260
1230 S=SGN(M-9): A=ABS(M-9)
1240 J3=INT(Y+S*INT(A/7))
1250 J3=-INT((INT(J3/100)+1)*3/4)
1260 J=J+INT(275*M/9)+D1+G*J3
1270 J=J+1721027+2*G+367*Y
1280 IF F>=0 THEN 1300
1290 F=F+1: J=J-1
1300 RETURN

    Вот такой вот реликт. Вы всё ещё пользуетесь GOTO? Тогда мы идём к вам.
    Программа вычисляет время восхода и захода солнца по дате (с точностью до минуты) в пределах нескольких текущих столетий. Производит корректировку, если географическая точка находится в арктичиском или антарктическом регионе, где заход или восход солнца на текущую дату может не состояться. Вводимые данные: положительная северная широта и отрицательная западная долгота. Часовой пояс указывается относительно Гринвича (например, 5 для EST и 4 для EDT). Алгоритм обсуждался в "Sky & Telescope" за август 1994, страница 84.
    P. S. VisualBasic такой Visual...

    Stalker, 10 Июня 2010

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