По колено в коде — Говнокод.ру

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

+159

1
2
3
4
5
6
7
8
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)

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

+74

1
2
3
4
5
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)

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

+164

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
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)

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

+168

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
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)

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

+80

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
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)

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

+72

1
2
3
4
5

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

lotik, 10 Июня 2010

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

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

+66

1
2
3

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

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

lotik, 10 Июня 2010

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

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

+75

1
2
3

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

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

lotik, 10 Июня 2010

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

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

+137

1
2
3
4
5
6
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)

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

+155

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
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)

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

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

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

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

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

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

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

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

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

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

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