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

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/topic.php on line 737
Форумы портала PHP.SU :: sql rand()

 PHP.SU

Программирование на PHP, MySQL и другие веб-технологии
PHP.SU Портал     На главную страницу форума Главная     Помощь Помощь     Поиск Поиск     Поиск Яндекс Поиск Яндекс     Вакансии  Пользователи Пользователи


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

> Без описания
jonston
Отправлено: 19 Августа, 2016 - 19:29:55
Post Id



Посетитель


Покинул форум
Сообщений всего: 455
Дата рег-ции: Март 2011  


Помог: 3 раз(а)




Всем здравствуйте.Скажите насколько оптимизирован данный код с точки зрения быстроты выполнения.Будет ли тормозить если таблица будет 10-20 тыс записей?
PHP:
скопировать код в буфер обмена
  1.  
  2.     function getFeaturedProducts($limit = 3){
  3.         $sql = "SELECT COUNT(*) as total FROM products WHERE featured = 1";
  4.         $query = $this->db->query($sql);
  5.         $total = $query->row()->total;
  6.         if($limit > $total) $limit = $total;
  7.  
  8.         $offset = array();
  9.         $offset_str = '';
  10.         $products = array();
  11.  
  12.         for($i = 0; $i < $limit; $i++){
  13.             if($offset) $offset_str = " AND id NOT IN (".implode(',', $offset).")";
  14.             $sql = "SELECT * FROM products WHERE featured = 1 " . $offset_str . " ORDER BY rand() LIMIT 1";
  15.             $result = $this->db->query($sql)->row_array();
  16.             $products[] = $result ? $result : $result;
  17.         }
  18.         return $products;
  19.     }
  20.  


-----
$i = 0;
$i = $i++ + ++$i; ?
 
 Top
Мелкий Супермодератор
Отправлено: 19 Августа, 2016 - 20:11:58
Post Id



Активный участник


Покинул форум
Сообщений всего: 11926
Дата рег-ции: Июль 2009  
Откуда: Россия, Санкт-Петербург


Помог: 618 раз(а)




jonston пишет:
if($offset)

Всегда ложно

jonston пишет:
$products[] = $result ? $result : $result;

ммм, что?


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

CODE (SQL):
скопировать код в буфер обмена
  1. SELECT * FROM products WHERE featured = 1 ORDER BY rand() LIMIT $limit

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

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

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


-----
PostgreSQL DBA
 
 Top
jonston
Отправлено: 19 Августа, 2016 - 22:08:48
Post Id



Посетитель


Покинул форум
Сообщений всего: 455
Дата рег-ции: Март 2011  


Помог: 3 раз(а)




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


-----
$i = 0;
$i = $i++ + ++$i; ?
 
 Top
Мелкий Супермодератор
Отправлено: 19 Августа, 2016 - 22:36:38
Post Id



Активный участник


Покинул форум
Сообщений всего: 11926
Дата рег-ции: Июль 2009  
Откуда: Россия, Санкт-Петербург


Помог: 618 раз(а)




jonston пишет:
Я исключать из выборки

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

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

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


-----
PostgreSQL DBA
 
 Top
Страниц (1): [1]
Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)
« Вопросы новичков »


Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.
 



Powered by PHP  Powered By MySQL  Powered by Nginx  Valid CSS  RSS

 
Powered by ExBB FM 1.0 RC1. InvisionExBB