Как облегчить запрос sql?

Question

[dima12s]

Здравствуйте! Скажите пожалуйста, как можно облегчить запрос:

$result_actors = $db->query("SELECT * FROM `actors` WHERE `post_id` = '".$row['id']."' ORDER BY `num` ASC");
if ($db->num_rows($result_actors)) {
while ($row_actor = $db->get_row($result_actors)) {
$full_actor = $db->super_query("SELECT `id`, `date`, `title`, `category`, `alt_name`, `xfields` FROM `".PREFIX."_post` WHERE `title` = '".$db->safesql($row_actor['name'])."' LIMIT 1");

$tpl3->set('{name}', $row_actor['name']);
}
}

Начинает грузить сайт, если запросов на странице больше 50

Comments

[tema_sun]

Про индексы в БД не забыли?

Answer 1

[Виталий]

• Почитайте про php.net/manual/ru/pdo.prepared-statements.php
  
• Не нужно делать запросы в циклах PHP
  
• Не нужно выгружать *, если не все поля используются
  
• Лучше SQL запросом собрать все что нужно и с ним уже работать.
  

Из этого куска не сильно понятно что надо получить то и что есть

Comments

[dima12s]

Я просто не понимаю как можно запрос super query вынести за цикл

$full_actor = $db->super_query("SELECT `id`, `date`, `title`, `category`, `alt_name`, `xfields` FROM `".PREFIX."_post` WHERE `title` = '".$db->safesql($row_actor['name'])."' LIMIT 1");

Тут связь с двумя таблицами идёт.

В первом запросе идет выборка из таблицы1, которая содержит инфу об "актерах"

А во второй идет вывод актеров, в котором сравнивается, есть ли актер из таблицы1 в таблице2

[dima12s]

как бы это сравнение грамотно вынести из цикла, чтоб избежать нагрузки

[Виталий]

Если надо вывести всех актеров, и у тех, о ком есть инфа - вывести инфу, то испольщуют Join

SELECT
a.id_actor,
a.actor_name,
ai.param_1
ai. ....
ai.param_x
FROM actor_table as a
LEFT JOIN actor_info as ai
ON a.id_actor = ai.id_actor

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

[dima12s]

Виталий: Так это будет один запрос вместо двух? Или вместо второго, который в цикле?

[Виталий]

dima12s: один , JOIN - объединение таблиц ... потратьте день на SQL и разберитесь как оно там работает в общем, и это потом сэкономит время

[dima12s]

Виталий: А вы сможете сделать за вознаграждение?)

[Виталий]

dima12s: до марта занят :/

[dima12s]

Виталий: Там структура готовая, все работает, только нагрузка создается из-за того что бд в цикле постоянно используется. Если будет время напишите ICQ 447774890

Answer 2

[ReFeRy]

Так здесь же элементарный inner join двух таблиц. Или left join, если во второй таблице может быть несколько записей с одинаковыми title.

Comments

[dima12s]

А как это реализовать?

У меня получается, если значения из первой и второй таблицы совпадают, то ссылкой выводится результат

Answer 3

[dima12s]

$result_actors = $db->query("SELECT `a`.`post_id`, `a`.`role`, `a`.`name`, `a`.`main_role`, `p`.`id`, `p`.`date`, `p`.`title`, `p`.`category`, `p`.`alt_name`, `p`.`xfields`
FROM `dle_post` AS `p`
INNER JOIN actors AS `a` ON (`a`.`post_id` = '".$row['id']."')
WHERE `a`.`post_id` = `p`.`id`
ORDER BY `num` ASC
");

if ($db->num_rows($result_actors)) {
$tpl3->load_template('actors.tpl');
while ($row_actor = $db->get_row($result_actors)) {
$tpl3->set('{vav1}', $row_actor['id']);
$tpl3->set('{name}', $row_actor['name']);
$tpl3->set('{role}', $row_actor['role']);

$tpl3->compile('actors');
}
$tpl3->clear();

$tpl->set('[actors]', '');
$tpl->set('[/actors]', '');
$tpl->set('{actors}', $tpl3->result['actors']);

} else {
$tpl->set_block('#\[actors\].*?\[\/actors\]#is', '');
}

Теперь вообще выводится как то странно.... Поле id получается дублируется везде, а поля актеры и роли как нужно выводятся.