PHP — Говнокод.ру

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

+162.3

1
2
3
4
5
6

$comments = (isset($_POST['lici-'.$login->id.'-comments']))?"yes":"no";

if($comments === "yes")
{
$xml .= "<nocomment>1</nocomment>\r\n";
}

Код из плагина кросспостера для wordpress.
Автор, видимо, не знает про булев тип. Все бы было хорошо, если бы плагин не закрывал комментарии, когда их нужно было открывать.

gunya, 31 Декабря 2009

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

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

+184.4

1

$happy = new Year();

C наступающим ;)

miami, 31 Декабря 2009

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

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

+144.9

1
2
3

foreach ($quote as $quote_key => $quote_value) {
$tpl[$key] = $value;
}

$quote = $tpl; как бы слишком коротко..

Deromanok, 29 Декабря 2009

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

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

+155.9

1
2
3
4
5
6

<?php 

$Fnm = $mosConfig_absolute_path .'/components/'.$option.'/cron.php';
$inF = fopen($Fnm,"w");
fwrite($inF,'<?php $last_cron_date='.$last_cron_date.';?>');
fclose($inF);

всеми любимая джумла... а точнее один из её компонентов... строка 1846-ая...

почему собственно гадость:
1. а если нету доступа на запись...
2. не безопасно
3. если в файле что-нибудь будет не так - обвалится весь сайт, потому как ошибко php
4. я настороженно отношусь к файлам по 2246 - строк ... уж больно много всего...

nicolay-punin, 29 Декабря 2009

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

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

+168.1

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

function check($pass) 
{ if (strpos($pass,'0') || strpos($pass,'1') 
    || strpos($pass,'2') || strpos($pass,'3') 
    || strpos($pass,'4') || strpos($pass,'5') 
    || strpos($pass,'6') || strpos($pass,'7') 
    || strpos($pass,'8') || strpos($pass,'9')) 
    { 
     if (strpos($pass,'a') || strpos($pass,'b') || strpos($pass,'c') 
         || strpos($pass,'d') || strpos($pass,'e') || strpos($pass,'f') 
         || strpos($pass,'g') || strpos($pass,'h') || strpos($pass,'i') 
         || strpos($pass,'j') || strpos($pass,'k') || strpos($pass,'l') 
         || strpos($pass,'m') || strpos($pass,'n') || strpos($pass,'o') 
         || strpos($pass,'p') || strpos($pass,'q') || strpos($pass,'r') 
         || strpos($pass,'s') || strpos($pass,'t') || strpos($pass,'u') 
         || strpos($pass,'v') || strpos($pass,'w') || strpos($pass,'x') 
         || strpos($pass,'y') || strpos($pass,'z')) 
         { 
          if (strpos($pass,'A') || strpos($pass,'B') || strpos($pass,'C') 
              || strpos($pass,'D') || strpos($pass,'E') || strpos($pass,'F') 
              || strpos($pass,'G') || strpos($pass,'H') || strpos($pass,'I') 
              || strpos($pass,'J') || strpos($pass,'K') || strpos($pass,'K') 
              || strpos($pass,'M') || strpos($pass,'N') || strpos($pass,'O') 
              || strpos($pass,'P') || strpos($pass,'Q') || strpos($pass,'R') 
              || strpos($pass,'S') || strpos($pass,'T') || strpos($pass,'U') 
              || strpos($pass,'V') || strpos($pass,'W') || strpos($pass,'X') 
              || strpos($pass,'Y') || strpos($pass,'Z')) 
              {                if (ctype_lower(substr($pass,0,1)) || ctype_upper(substr($pass,0,1)) || is_numeric(substr($pass,0,1))) 
                   {                        if (ctype_lower(substr($pass,0,1))) 
                           {                             if (!ctype_lower(substr($pass,-1))) 
                                {                                  return $pass;                                 }                            } 
                       if (ctype_upper(substr($pass,0,1))) 
                           { 
                            if (!ctype_upper(substr($pass,-1))) 
                                { 
                                 return $pass; 
                                } 
                           } 
                       if (is_numeric(substr($pass,0,1))) 
                           { 
                            if (!is_numeric(substr($pass,-1))) 
                                { 
                                 return $pass; 
                                } 
                           }                    } 
              } 
         } 
    } }

без коментариев. (из одного вап чата)

[email protected], 28 Декабря 2009

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

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

+162.5

1
2
3

public function run() {
     $this->_stop();
}

К концу рабочего дня у себя такое обнаружил = ) Заработался.

Johnny, 28 Декабря 2009

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

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

+148

01
02
03
04
05
06
07
08
09
10
11
12

<? if (!$isLogged) {
    echo '
	<table class="br-table" align="center" cellspacing="0" cellpadding="0">
			<tr>
			<td class="corner-lt"></td>
			<td class="left-br-top"></td>
			<td class="corner-rt"></td>
			</tr>
			<tr>
			<td colspan="3" class="box">
			<h1 class="h1-title" ><img src="img/star.png">';
блаблабла

Да, и <? if(): ?> <?endif;?> их тоже не учили :(

azzz, 28 Декабря 2009

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

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

+144.5

1
2
3
4
5

<?php if (!$mode || true) { ?>
<div class="yiiForm">
<?php } else { ?>
<div class="yiiForm">
<?php } ?>

последствия удаления ненужных частей кода

Bethrezen, 28 Декабря 2009

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

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

+171.6

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

if ( !empty( $page ) )
{
    if ( $page == "login" )
    {
        $spage = "Login.inc";
    }
    else if ( $page == "rules" )
    {
        $spage = "rules.inc";
    }
    else if ( $page == "help" )
    {
        $spage = "help.inc";
    }
    else if ( $page == "wm" )
    {
        $spage = "wm.inc";
    }
    else if ( $page == "game" 
    {
        $spage = "game.inc";
    }
    else if ( $page == "webmoney" )
    {
        $spage = "webmoney.inc";
    }
    else if ( $page == "egold" )
    {
        $spage = "egold.inc";
    }
    else if ( $page == "cashin" )
    {
        $spage = "cashin.inc";
    }
    else if ( $page == "ballans" )
    {
        $spage = "ballans.inc";
    }
    else if ( $page == "remind" )
    {
        $spage = "remind.inc";
    }
    else if ( $page == "contact" )
    {
        $spage = "contact.inc";
    }
    else if ( $page == "reg" )
    {
        $spage = "reg.inc";
    }

:(((

azzz, 25 Декабря 2009

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

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

+154.8

1

substr ("1111117495". ereg_replace ("[^0-9]", "", $user1->data["mobile_phone"]), -10) == substr ("1111117495". ereg_replace ("[^0-9]", "", $user2->data["mobile_phone"]), -10)

Сравнение двух мобильных телефонов.

IHateBidloKod, 24 Декабря 2009

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

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