Тема срачей, говнокода и путей их преодоления

[SuLTaN]

Посмотреть вложение 3363

Кто-то не знает NOT IN (...).

AND `a`.`v2` NOT IN (201,211,233,238,245,248,249,254,255,260,284,285,29,180,283,267,32,257,30,139,342,343,251,250,252,31,188,229,217,269,276,277,278,279,228,229,222,296,297) Так)

 

[Insallah]

.. и тут я натыкаюсь на этот обалденный метод!

/*
    @ Метод определения пустое значение или нет
    @ TYPE: 0 - число, 1 - текст
    */
    public function emptyVal( $val, $type ) {
        $r = true;
        if( !isset($val) ) {
            $r = false;
        }elseif( $type == 0 ) {
            if( $val == 0 ) {
                $r = false;
            }
        }elseif( $type == 1 ) {
            $val = str_replace( ' ', '', str_replace( '    ', '', $val ) );           
            if( $val == '' ) {
                $r = false;
            }
        }
        return $r;
    }

 

[Bayer]

.. и тут я натыкаюсь на этот обалденный метод!

/*
    @ Метод определения пустое значение или нет
    @ TYPE: 0 - число, 1 - текст
    */
    public function emptyVal( $val, $type ) {
        $r = true;
        if( !isset($val) ) {
            $r = false;
        }elseif( $type == 0 ) {
            if( $val == 0 ) {
                $r = false;
            }
        }elseif( $type == 1 ) {
            $val = str_replace( ' ', '', str_replace( '    ', '', $val ) );         
            if( $val == '' ) {
                $r = false;
            }
        }
        return $r;
    }

*empty просто существует*
Чел решивший написать это:
ты недостоин моего кода!

 

[Insallah]

elseif ($usr['no_ip'] == 'trupojor' && $usr['level'] == 10 && ($u->info['level'] < 10 || $u->info['level'] > 11) && $u->info['level'] != 7 && $u->info['level'] != 8 && $u->info['level'] != 9 && $u->info['level'] != 11 && $u->info['level'] != 12) {
        ...
    if ($usr['level'] == 10) {
        ...
    }
}

Горбатый! Горбатый, я сказал!!!!

 

[Insallah]

Там же.

 

[Bayer]

Посмотреть вложение 3387
Там же.

Level !=9 && Level != 11 && Level == 10
И в правду уникально. === с одним условием для слабых!!

 

[Bayer]

$sp = mysql_query('SELECT * FROM `items_users` WHERE `item_id` = "' . ((int)$tims[0]) . '" AND `uid` = "' . $this->info['id'] . '" AND (`delete` = "0" OR `delete` = "1000") AND `inShop` = "0" AND `inOdet` = "0"');
while ($pl = mysql_fetch_array($sp)) {
   $num_rows++;
}
if ($num_rows < (int)$tims[1]) {
$trn = 0;
}

SELECT COUNT покинул чат

 

[DarksLight2]

$sp = mysql_query('SELECT * FROM `items_users` WHERE `item_id` = "' . ((int)$tims[0]) . '" AND `uid` = "' . $this->info['id'] . '" AND (`delete` = "0" OR `delete` = "1000") AND `inShop` = "0" AND `inOdet` = "0"');
while ($pl = mysql_fetch_array($sp)) {
   $num_rows++;
}
if ($num_rows < (int)$tims[1]) {
$trn = 0;
}

SELECT COUNT покинул

Переменные прикольные, sp pl trn🙂 и хер пойми что там творится)

 

[Insallah]

А помните массовый мор с LIMIT 1 в запросах, потому что оптимизация? Штамповали чуть ли не автозаменой не задумываясь, поэтому везде и постоянно всякое 'select * from ... where id = $id' . 'LIMIT 1'.

 

[Insallah]

Переменные прикольные, sp pl trn🙂 и хер пойми что там творится)

sp и pl - временные. туда можно, какие-то temp_1 и temp_2. а trn флаг. иногда фиг пойми как их правильно называть.

 

[Bayer]

Переменные прикольные, sp pl trn🙂 и хер пойми что там творится)

Да переменные то ладно, я орнул с того как чел знает что такое циклы, но не знает что это делается 1 запросом к базе.
Я как представил что нужно например 50к записей так посчитать аж плохо стало.