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

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)

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

+166.4

1
2
3

$videoname = explode(".",$video_name);
unset($videoname[count($videoname)-1]);
$videoname = implode(".",$videoname);

Вот такое вот милое отрезание расширения файла = )

Johnny, 24 Декабря 2009

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

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

+173.4

1
2
3
4
5
6
7
8
9

if ($_POST["cc"]["status"]==2 && (int)current (mysql_fetch_row (mysql_query ('SELECT COUNT(*) FROM test_vc_log LEFT JOIN test_cc ON test_cc.uid=test_vc_log.cc_id WHERE test_vc_log.event_id=2 AND test_cc.user_id='. $user->data["uid"]. ' AND test_vc_log.created>"'. date ("Y-m-d H:i:s", max (
	strtotime (is_in (mysql_fetch_row (mysql_query ('SELECT test_vc_log.created FROM test_vc_log LEFT JOIN test_cc ON test_cc.uid=test_vc_log.cc_id WHERE test_vc_log.event_id=1 AND test_cc.user_id='. $user->data["uid"]. ' ORDER BY created DESC LIMIT 0,1')), 0)),
	strtotime (is_in (mysql_fetch_row (mysql_query ('SELECT created FROM test_rating WHERE event_type=18 AND user_id='. $user->data["uid"]. ' ORDER BY created DESC LIMIT 0,1')), 0)),
	strtotime ("2009-02-06")
)). '"'))) > 2){

....

}

Я бы этого любителя писать всю программу в if-ах... Да еще и с такими запросами... Урод.

IHateBidloKod, 24 Декабря 2009

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

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

+163.1

1
2
3
4
5
6
7

$hash = $sql->Execute("INSERT INTO `x_client_info` 
					VALUES(null, '".$cli['ip']."', '".$cli['fio']."', '".$cli['org_name']."', 
					'".$cli['phone']."', '".$cli['email']."', '".$cli['icq']."', '".$cli['skype']."', 
					'".$cli['passport']."', '".$cli['airline_order']."', '".$cli['train_order']."', '".$cli['car_order']."', '".$cli['hotel_order']."', 
					'".$cli['cash']."', '".$cli['invoice']."', '0', 'w_proc', '0', '0', 
					'".time()."', '".@date('d.m.Y H:i:s')."', '".guid()."')")
                      ->Assoc();

по мне - так говнокод. Как от него избавиться - хз. Вечно бесило такое обилие кавычек

nolka4, 24 Декабря 2009

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

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