Как обновить таблицу со сложным условием?

Question

[Антон Штинов]

Есть таблица с двумя полями: id, active
И есть список id (id_list), которые не надо обновлять. Как обновить только те записи в таблице, которые не попадают в список id_list, и у которых active=1

Я пока придумал такие варианты:
1) делаю выборку нужных элементов, а потом обновляю их по одному.

$data = Model::find()->where(['not in', 'id', $id_list])->andWhere(['active' => 1])->all();
foreach($data as $line) {
	// обновить элемент
}

2) задаю условие напрямую

Model::updateAll(['active' => 0], 'active = 1 AND id NOT IN ('.implode(',', $id_list).')');

Но что-то ни один из этих вариантов мне не нравится.

Comments

[Дениска Петров]

www.yiiframework.com/doc-2.0/yii-db-activerecord.h...
" The conditions that will be put in the WHERE part of the UPDATE SQL. Please refer to yii\db\Query::where() on how to specify this parameter."
Второй параметтр updateAll, попробуйте в него задать условие.

[Дениска Петров]

Model::updateAll(['active' => 0], ['and', ['active' => 1], ['not in', 'active', $id_list]]); Что то вроде такого должно получится, если не ошибаюсь.

[Антон Штинов]

Дениска Петров: да, так работает. Жаль, что в мануале не привели пример сложных условий для выборки.

[jacksparrow]

Поясните, чем именно, вам не нравятся варианты. Первый вариант, кроме точного задания что актив =1 удобно читается.
Второй будет быстрее.

[Антон Штинов]

jacksparrow: первый вариант - много запросов к базе. второй вариант - может получится дырка для SQL-injection

[jacksparrow]

Антон Штинов: Отсюда и вопрос, что вы хотите получить. По поводу Sql иньекции, достаточно айди к инту привести, при условие что они числовые.

[Антон Штинов]

jacksparrow: ответ уже дали выше. Model::updateAll(['active' => 0], ['and', ['active' => 1], ['not in', 'active', $id_list]]);