Всем доброго дня. Для выборки из таблицы случайной записи, набрасал код ниже. Есть 2 таблицы: table1 и table2. В table1 - множество str(записей), из неё, в зависимости от IP обратившегося, необходимо выводить случайную запись(str) лишь один раз. Т.е. одна и таже, для конкретного IP выводиться не должна! В table2 - сохраняются все ранее выведенные из table1 записи. Из неё выбираются все id'ы ранее выведенных для определённого IP и исключаются из sql условия. Код собственно работает. Но вот в оптимизации я не дальновиден. Всё ли на Ваш взгляд корректно? Спасибо за внимание =) Код: Спойлер PHP: <?php $cnnect = mysql_connect('localhost', 'root', 'PaSSwOD'); mysql_select_db('dbase'); $code = mysql_query("SELECT id FROM table2 WHERE IP='".$_SERVER["REMOTE_ADDR"]."'"); $id = array(); while($rows = mysql_fetch_row($code)){ $id[] = $rows[0]; } if (!empty($id)){ $query = "SELECT id,str FROM table1 WHERE id NOT IN ( '" . implode($id, "', '") . "' ) ORDER BY RAND() LIMIT 1"; } else { $query = 'SELECT id,str FROM table1 ORDER BY RAND() LIMIT 1'; } $result = mysql_query($query); if ($result) { $per = mysql_fetch_array($result); if (!empty($per['str'])) { mysql_query("INSERT INTO table2 (id,ip,str,time) VALUES('".$per['id']."','".$_SERVER["REMOTE_ADDR"]."', '".$per['str']."', NOW());"); } } mysql_free_result($result); mysql_close($cnnect); ?>
Во первых, ты мог спокойно засунуть первый select в подзапрос внутри in(). Прокачивать данные через пхп это напрасная трата времени. Во вторых, Not in() считается тяжёлым решением, хотя и оно наглядно. Выгоднее сделать left join и условие is null.
главный тормоз на больших объемах будет из-за ORDER BY RAND() LIMIT 1 ранее это уже обсуждалось https://php.ru/forum/threads/optimizirovannyj-order-by-rand.46932/