1. Список говнокодов пользователя Khvorostin

    Всего: 15

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

    +153

    1. 1
    2. 2
    3. 3
    if($servid>0){
        $servid=$servid;
    }

    А если $servid == 0?

    Khvorostin, 30 Июля 2014

    Комментарии (2)
  3. PHP / Говнокод #16447

    +155

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    $q=mysql_query($que.$limit)or die(mysql_error());
    $numr=mysql_num_rows($q);
    $q2 = mysql_query($que)or die(mysql_error());
    $numrows = mysql_num_rows ($q2);
    $pages = ceil($numrows/$step);

    Здесь: $que - тяжелый SQL-запрос, который собирался в течение предыдущих сотен строк (десятки IF... ELSE...). $limit - лимит: выбрать 30 записей с такой по этакую. Чтобы орагнизовать на сайте пагинацию, автор выполняет запрос $que дважды. Во второй раз - чтобы просто посчитать общее число записей. При этом в MySQL второй запрос убивается парой SELECT SQL_CALC_FOUND_ROWS... / SELECT FOUND_ROWS().

    Khvorostin, 30 Июля 2014

    Комментарии (6)
  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)