1. PHP / Говнокод #16440

    +156

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    if(!$_SESSION['id'])
        {?>
        <ul class="login"><?php
            }
            elseif($_SESSION['id'])
            {?>
            <ul class="login"><?php
                }?>
// всякий код подобный тому что выше
</ul>

    История одного проекта

    reilag, 29 Июля 2014

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

  2. PHP / Говнокод #16438

    +158

    1. 1
    2. 2
    3. 3
    4. 4
    $rs=mysql_query("
update f_subsdetails 
set UDate='".date ( "Y-m-d H:i:s", mktime (date("H"),date("i"),date("s"),date("m"),date("d"),date("Y")))."' 
where username='$username'");

    Прогер наверное дорвался до изучения стандартных функции дат и времени и наверное решил закрепить знания. Наверное изучает по-алфавиту, вcкоре изучит time(), а там возможно и прочитает о втором аргументе date().

    reddevil, 29 Июля 2014

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

  3. PHP / Говнокод #16436

    +154

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    public function quizAction(){
        $gsession = new Zend_Session_Namespace('global');
        $showQuiz = true;
        if($gsession->__isset('quizCompleted')) {
            if($this->getParam('restart-quiz', false)) {
                $gsession->__unset('quiz');
                $gsession->__unset('quizCompleted');

            }else{ /* ... */ }
        }
        // ...
    }

    Вызов магических методов напрямую

    baldrs, 28 Июля 2014

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

  4. PHP / Говнокод #16435

    +160

    1. 1
    if ($sess_gr != 7 || $sess_gr == 4 || $sess_gr == 14 || $sess_gr == 15) { // ... }

    Избыточное условие.

    Khvorostin, 28 Июля 2014

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

  5. PHP / Говнокод #16434

    +155

    1. 1
    2. 2
    3. 3
    <?  if($sess_gr==1||$sess_gr==7||$sess_gr==11||$sess_gr==2||$sess_gr==3){?>
<?      if($sess_gr==1||$sess_gr==7||$sess_gr==11||$sess_gr==2||$sess_gr==3){?><td rowspan="2"></td><?}?><td rowspan="2"></td>
<?  }?>

    Контрольный IF, на случай, если PHP с первого раза не понял

    Khvorostin, 28 Июля 2014

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

  6. PHP / Говнокод #16424

    +157

    1. 1
    2. 2
    3. 3
    4. 4
    $PriceProduct = $Price;
$Skidka = $Skidka;
$PriceSkidka = $PriceSkidka;
$Price = $PriceItog;

    Самописная CRM. Коллега метко назвал этот кусок кода считалочкой. Для полноты картины осталось заменить последнюю строку на $Price = $PriceProduct;

    Khvorostin, 25 Июля 2014

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

  7. PHP / Говнокод #16422

    +163

    1. 1
    //проверяем, не извлечены ли данные пользователя из базы. Если нет, то он не вошел, либо пароль в сессии неверный. Выводим окно для входа. Но мы не будем его выводить для вошедших, им оно уже не нужно.

    Не замысловатый комментарий из истории одного проекта

    reilag, 25 Июля 2014

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

  8. PHP / Говнокод #16421

    +158

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    $dbhost = DB_HOSTNAME;
 $dbuser = DB_USERNAME;
 $dbpass = DB_PASSWORD;
 $dbdatabase = DB_DATABASE;
 $dbprefix = DB_PREFIX;

    Из платного модуля для опенкарта.

    SOKOJI, 25 Июля 2014

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

  9. PHP / Говнокод #16413

    +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
    if($s["type"] == 9){
    if(in_array($ch_newprice, array("1220","1185","1244"))){
        if(in_array($ch_oldprice,array("1237","1239","1238"))){
            $status = delete_DW_Customer($s["login"]);
            if($status == 0) {
                echo "ERROR!!! Can't delete user in server DrWEB\n";
                //break;
            }
                if(in_array($ch_oldprice,array("1239"))){
                  delete_DW_Customer($s["login"]."_android");
                }
        }
    }
}

    IF Driven Development + Magic Numbers

    ptaag, 24 Июля 2014

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

  10. PHP / Говнокод #16412

    +154

    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
    for ($i = 0; $i < count($total_data); $i++) {
                    if ($total_data[$i]['title'] == "Сумма") {
                        $sum = (float) $total_data[$i]['value'];
                        $tdata = array();
                        $tdata[count($total_data)]['title'] = "Сумма со скидкой";
                        $tdata[count($total_data)]['code'] = "discount_total";
                        $tdata[count($total_data)]['value'] = $sum;
                        $tdata[count($total_data)]['text'] = $this->currency->format($sum);
                        $tdata[count($total_data)]['sort_order'] = "2";
                        $total_data = array_merge($total_data, $tdata);
                        $id_sum = $i;
                        $sub_total = $sum;
                    }
                    if ($total_data[$i]['title'] == "Итого") {
                        $id_discount = count($total_data) - 1;
                        $total_data[$i]['value'] = (float) $total_data[$i]['value'] - (float) $total_data[$id_sum]['value'] + (float) $total_data[$id_discount]['value'];
                        $total_data[$i]['text'] = $this->currency->format($total_data[$i]['value']);
                    }
                }

    true2trance, 24 Июля 2014

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