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

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

+164

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

for($i = 0; $i < 1000; $i++) {
        if(isset($_POST['discount_sum_'.$i]))
        {
            $a_discount_id_service[] = htmlspecialchars(@$_POST['id_service_'.$i]);
            $a_discount_sum[]        = htmlspecialchars(@$_POST['discount_sum_'.$i]);
            $a_discount_type[]       = htmlspecialchars(@$_POST['discount_type_'.$i]);
            $a_discount_time_type[]  = htmlspecialchars(@$_POST['discount_time_type_'.$i]);
        }
        else {
            break;
        }
}

По POST'у передаётся некоторое количество 'discount_sum_'.$i, 2 или 3, например 'discount_sum_1', 'discount_sum_2', 'discount_sum_3', в зависимости от точно известного количества записей в одной из таблиц, и передаваемое количество точно известно в момент передачи, но ведь лучше перебрать от 1 до 1000! :))

hardcoder, 06 Января 2010

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

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

+173.5

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

function load_config ()
  {
    $res = mysql_query ('select * from configurations');
    while ($row = mysql_fetch_assoc ($res))
    {
      $n = $row[name];
      $v = $row[value];
      $str = '$_config[\'' . $n . '\'] = \'' . $v . '\';';
      eval ($str);
    }
    return $_config;
  }

Загрузка конфига из базы данных.
Продукт, где содержится _это_, продаётся за ~300$

miami, 06 Января 2010

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

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

+157.7

1

$has_url = ($categories->pageID>0 || strlen(trim($categories->caturl))>0) ? true : false;

ryurik, 06 Января 2010

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

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

+158.9

1
2
3
4
5

<?php
/* ... */
if (isset($_GET['page'])) @include 'pages/' . $_GET['page'] . '.txt';
/* ... */
?>

Вот так :)

noisiaman, 05 Января 2010

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

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

+158.2

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

<?php
/**
 * @author DanxilLsPro.net16.net
 * @copyright 2010
 * @email [email protected]
 */
 session_start();
//require_once "include/auth.php";
require_once "include/config.php";
require_once "include/function.php";
if(is_numeric($_GET['theme']) and $_GET['theme'] >= 1){
$theme = getTheme($_GET['theme']);
if($theme == false)show_error();
}
top("Просмотр всех тем - Знания - народу!",$style,$bg);
echo<<<FOO
    <TD ROWSPAN=2 bgcolor="122745"> 
      <p>
FOO;
if(is_numeric($_GET['theme']) and $_GET['theme'] >= 1)
showTheme($theme->name,$theme->time,$theme->author,$theme->text,$theme->voted,$theme->view,$style);
else
echo show_all_theme();
echo "</p>";
bottom("Знания принадлежат народу!",$style);
?>

Это полный как бе ппц. К чему приводят бессонные ночи в количестве более одной ^^

DanxilLs, 05 Января 2010

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

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