Как выполнит одним запросом несколько действий MYSQL?

Question

[Maxim Plotnikov]

Доброго времени суток, я в коде выполняю работу с базой данных.
И у меня там выполняется сразу 4 запроса.
Как мне все это сделать в одном запросе, для снижения нагрузки на базу данных.
Сами запрос:

$db->query("DELETE FROM timer WHERE vk_id = $ids");
$db->query("INSERT INTO timer (vk_id, tim) VALUES ('$ids', '$time_now2')");
$db->query("UPDATE score SET score = score + $klik push = push + 1 WHERE vk_id = $ids LIMIT 1");
$db->query("UPDATE score SET score = score + $klik push = push + 1 WHERE vk_id = $ids LIMIT 1");

Заранее спасибо ;)

Answer 1

[FanatPHP]

Вот ты и подрос, наконец, чтобы узнать главное правило программирования:

Преждевременная оптимизация - корень всех зол.

Никакой нагрузки тут нет, снижать нечего. Поэтому расслабьтся, забудь эти фантазии и займись чем-нибудь полезным.

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

Comments

[Северное Сияние]

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

[FanatPHP]

Lander скорее всего не будет работать. У него 100% mysqli. У таких оптимизаторов всегда mysqli

Answer 2

[ThunderCat]

Как выполнит одним запросом несколько действий MYSQL?

Никак, это разные запросы с разными таблицами и разными задачами.

для снижения нагрузки на базу данных.

Неужели у вас уже более 1000 запросов в секунду и необходима срочная оптимизация?

$db->query("UPDATE score SET score = score + $klik push = push + 1 WHERE vk_id = $ids LIMIT 1");
$db->query("UPDATE score SET score = score + $klik push = push + 1 WHERE vk_id = $ids LIMIT 1");

во первых - не хватает запятых между присваиваниями, во вторых SET score = score + $klik*2, push = push + 2 вот вам минус 1 запрос. В третьих - вроде используете pdo, а переменные вставляете без prepared statements...

Все это можно:
1) обернуть в транзакцию - профит от этого - целостность выполнения операций(на скорости особо не отразится).
2) Запустить одним скриптом, разделив запросы семиколоном ";" (профита практически никакого)

Comments

[Maxim Plotnikov]

ThunderCat, ага, спасибо большое)

[FanatPHP]

2) противоречит рекомендации использовать подготовленные запросы.

[ThunderCat]

FanatPHP, угу, есть такое, но если нельзя, но очень хочется, то можно :)

[FanatPHP]

И как вообще связаны оптимизация при 1000 запросов в секунду и пихание всех запросов в один?

[ThunderCat]

FanatPHP,

для снижения нагрузки на базу данных.

Собсно вопрос с замахом на оптимизацию(весьма преждевременную), на самом деле предлагалось "найти" мегазапрос который бы волшебным образом все делал за один проход не напрягая систему.

Answer 3

[Stalker_RED]

Можно вместо

$db->query("UPDATE score SET score = score + $klik push = push + 1 WHERE vk_id = $ids LIMIT 1");
$db->query("UPDATE score SET score = score + $klik push = push + 1 WHERE vk_id = $ids LIMIT 1");

обновить score за один раз

$db->query("UPDATE score SET score = score + $klik + $klik, push = push + 2 WHERE vk_id = $ids LIMIT 1");

Можно вместо DELETE FROM timer и INSERT INTO timer сделать один раз update (если у вас таблице timer нет повторяющихся записей с одинаковым vk_id).

А объединять запросы к timer и score смысла нет, скорее всего.

Comments

[FanatPHP]

Вот это я называю вдумчивым ответом

Answer 4

[Северное Сияние]

поставить индекс на timer на vk_id
поставить индекс на score на vk_id
больше ничего не трогать
никакой нагрузки тут нет

Comments

[Maxim Plotnikov]

Северное Сияние, как мне это выполнить в одном запросе? Таков был мой вопрос.

[Северное Сияние]

Maxim Plotnikov, зачем?

[Lander]

Maxim Plotnikov, Ну если вам прям сильно принципиально, то можете:

$db->query("DELETE FROM timer WHERE vk_id = $ids; INSERT INTO timer (vk_id, tim) VALUES ('$ids', '$time_now2'); UPDATE score SET score = score + $klik push = push + 1 WHERE vk_id = $ids LIMIT 1; UPDATE score SET score = score + $klik push = push + 1 WHERE vk_id = $ids LIMIT 1");

[Boris Köln]

Lander, SQL-запросов как было 4, так и осталось.

[Lander]

Boris Korobkov, Ну да. :)