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

    Всего: 14

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

    +156

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    $varians_ids = array();
    $getvaluesvariants=mysql_query("SELECT  `id` FROM `values` group by `id`") or die(mysql_error());;
    while($valuesvariants = mysql_fetch_array($getvaluesvariants, MYSQL_ASSOC))
    {
    $varians_ids[]= $valuesvariants['id'];
    }
    mysql_free_result($getvaluesvariants)or die("Query failed: " . mysql_error());
    for ($i = 0;  $i < count($varians_ids); $i++)
    {
      $setvaluesnames = mysql_query("UPDATE `option_values` SET `option_value_ru` = (SELECT  `option_value_ru` FROM `values` WHERE `id` = '$varians_ids[$i]') WHERE  `id` = '$varians_ids[$i]'") or die(mysql_error());;
    }

    Наткнулся на этот шедевр. Отсутствие использования foreach кажется мелочью по сравнению с тем, как производится операция, которую можно сделать одним запросом:
    $setvaluesnames = mysql_query("UPDATE `option_values`, `values`
    SET
    `option_values`.`option_value_ru` = `values`.`option_value_ru`
    WHERE `option_values`.`id` = `values`.`id`") or die(mysql_error());

    P. S.: там еще в БД пишется пробел вместо пустой строки или NULL...

    CRRaD, 20 Июля 2011

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

    +169

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    if(!$this->first_name && !$this->last_name && !$this->countryID && !$this->zoneID && !$this->zip && !$this->state && !$this->city && !$this->address_street && !trim(strip_tags($strAddress))){
            return '';
    }
    if(!$this->first_name && !$this->last_name && !$this->countryID && !$this->zoneID && !$this->zip && !$this->state && !$this->city && !$this->address_street && !$this->address_home && !trim(strip_tags($strAddress))){
    	return '';
    }
    if(!$this->first_name && !$this->last_name && !$this->countryID && !$this->zoneID && !$this->zip && !$this->state && !$this->city && !$this->address_street && !$this->address_home && !$this->address_houses && !trim(strip_tags($strAddress))){
    	return '';
    }
    if(!$this->first_name && !$this->last_name && !$this->countryID && !$this->zoneID && !$this->zip && !$this->state && !$this->city && !$this->address_street && !$this->address_home && !$this->address_houses && !$this->address_houseroom && !trim(strip_tags($strAddress))){
    	return '';
    }

    Найдено в одной CMS

    CRRaD, 12 Июля 2011

    Комментарии (17)
  4. SQL / Говнокод #6966

    −116

    1. 1
    SELECT COUNT(DISTINCT p.productID) as cnt FROM (SELECT DISTINCT p.productID FROM SC_products p AND collection!=2 ) AS p

    мало того, что тут синтаксические ошибки, так еще и суть просто шедевральная

    CRRaD, 16 Июня 2011

    Комментарии (13)
  5. PHP / Говнокод #6766

    +165

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    class email_selector {
    
    private static $number_per_page = 30;//
    
    public function getNumberPerPage() {
        return self::$number_per_page;
    }
    
    }

    понятно, что это только кусок класса. увидев, просто не смог не запостить....

    CRRaD, 27 Мая 2011

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