Форумы портала PHP.SU :: Версия для печати :: sql rand()

Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770
Форумы портала PHP.SU :: Версия для печати :: sql rand()

Форумы портала PHP.SU » » Вопросы новичков » sql rand()

Страниц (1): [1]

1. jonston - 19 Августа, 2016 - 19:29:55 - перейти к сообщению

Всем здравствуйте.Скажите насколько оптимизирован данный код с точки зрения быстроты выполнения.Будет ли тормозить если таблица будет 10-20 тыс записей?

PHP:
скопировать код в буфер обмена

 
    function getFeaturedProducts($limit = 3){
        $sql = "SELECT COUNT(*) as total FROM products WHERE featured = 1";
        $query = $this->db->query($sql);
        $total = $query->row()->total;
        if($limit > $total) $limit = $total;
 
        $offset = array();
        $offset_str = '';
        $products = array();
 
        for($i = 0; $i < $limit; $i++){
            if($offset) $offset_str = " AND id NOT IN (".implode(',', $offset).")";
            $sql = "SELECT * FROM products WHERE featured = 1 " . $offset_str . " ORDER BY rand() LIMIT 1";
            $result = $this->db->query($sql)->row_array();
            $products[] = $result ? $result : $result;
        }
        return $products;
    }
 

2. Мелкий - 19 Августа, 2016 - 20:11:58 - перейти к сообщению

jonston пишет:

if($offset)

Всегда ложно

jonston пишет:

$products[] = $result ? $result : $result;

ммм, что?

Нет, это будет на больших таблицах всегда медленно из-за rand(), а вы ещё помножили время выполнения на величину $limit.

CODE (SQL):
скопировать код в буфер обмена

SELECT * FROM products WHERE featured = 1 ORDER BY rand() LIMIT $limit

На порядки проще и уже в $limit+1 раз быстрее при полном сохранении логики выполнения.

Что делать с медлительным order by rand() - думать. Серебряной пули нет, подпорки разной степени наркоманства широко описаны в интернетах.
(Добавление)

Мелкий пишет:

при полном сохранении логики выполнения.

А, точно, не полное соответствие. У вас, хоть и с обратно пропорциональной размеру выборки вероятностью, записи могут дублироваться. Но вряд ли это желаемое поведение.

3. jonston - 19 Августа, 2016 - 22:08:48 - перейти к сообщению

Я исключать из выборки id.Если проще спросить, то время на 10 запросов в цикле и один с LIMIT 10 c ORDER BY rand() будут примерно одинаковыми?

4. Мелкий - 19 Августа, 2016 - 22:36:38 - перейти к сообщению

jonston пишет:

Я исключать из выборки

По представленному коду видно, что вы формируете $limit тождественно идентичных запросов. Хоть в query cache их класть - да вызов rand некешируемым делает запрос.

jonston пишет:

время на 10 запросов в цикле и один с LIMIT 10 c ORDER BY rand() будут примерно одинаковыми?

10 запросов потребуют примерно в 10 раз больше времени. Найти следующее число куда дешевле получения всех строк, генерации случайного числа для каждой из них и поиска максимального значения.

Powered by ExBB FM 1.0 RC1