delete с select в условии

[achyorny]

можно ли это:

$result = $this->db->query("SELECT * FROM product_to_store WHERE `product_id` NOT IN (SELECT `product_id` FROM product)");
foreach ($result->rows as $item)
	$this->db->query( "DELETE FROM product_to_store WHERE `product_id` ='".(int)$item['product_id']."'" );

заменить этим:

$this->db->query( "DELETE FROM product_to_store WHERE `product_id` NOT IN (SELECT `product_id` FROM product)" );

?

 

[С.]

A у самого Mysql какое мнение по этому поводу?

 

[achyorny]

а как проверить?

 

[prolis]

а как проверить?

create table test as (select * from product_to_store)
DELETE FROM test WHERE `product_id` NOT IN (SELECT `product_id` FROM product)

 

[achyorny]

надо выполнить эти два запроса?

 

[prolis]

надо выполнить эти два запроса?

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

 

[achyorny]

ок, спс а по нагрузке на сервер если и php и мускл стоят на одном серваке какой вариант лучше?

 

[Adelf]

Второй

 

[achyorny]

а что скажете насчет

DELETE `t1`
FROM `product_to_store` `t1` LEFT JOIN `product` `t2` ON `t1`.`product_id` = `t2`.`product_id`
WHERE `t2`.`product_id` IS NULL

 

[Adelf]

achyorny
Скажу, что ты сейчас занимаешься ненужной херней
Особенно когда "снижаешь нагрузки" на сервере, где их пока нет.

 

[achyorny]

да я люблю такой хернёй заниматься)) снииииииииижааааааааем наааааагррррууууууууууузкккккккииииииии!!!!!!!!!!
в идеале хочу, чтобы в инетмагазе было 2 запроса на страницу!!!!!!!!!!!!!!

 

[achyorny]

и те кешировать!!!!!


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

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

 

[korvin0]

Как вариант, можно и без подзапросов сделать, обойдясь только left join'ом, и сбором всех IDшников в один массив, и дальнейшим удалением нужных записей одним запросом
$result = $this->db->query("
SELECT product_to_store.*
FROM product_to_store
left join product on product.product_id = product_to_store.product_id
WHERE product.product_id IS NULL
");
$ids = array();
foreach ($result->rows as $item) $ids[] = intval($item['product_id']);
if (count($ids) > 0) $this->db->query( "DELETE FROM product_to_store WHERE `product_id` IN(".implode(',', $ids).")" );

Простой и надежный вариант

 

[achyorny]

зачем, если можно один запрос выполнить и всё

 

[korvin0]

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

 

[achyorny]

это да

 

[prolis]

Простой и надежный вариант

этот вариант умрет уже на тысяче записей

 

[Adelf]

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