Почему save() срабытывает один раз?

Question

[Сергей Беловенцев]

есть массив $catID
вот его значения

array(2) { 
    	[0]=> string(1) "1" 
    	[1]=> string(1) "2"
    	 }

пытаюсь запустить его для сохранения через foreach вот код

foreach($catID as $cat){
                     $catSerial->id_cat=$cat;
                     $catSerial->id_serial=$model->id;
                     $catSerial->save();
                 }

проблема в том что save() отрабатывается один раз последний почему ? И как заставить работать дважды.

Answer 1

[333Serega333 333Serega333]

Что хотелось бы сказать в первую очередь. Вызов метода save() влечет за собой выполнение запроса в базу. Вы же вызываете этот метод в цикле, и это ну вообще не правильно. Я бы рекомендовал сделать нужный функционал другим способом.
Запросы sql в цикле делать нельзя !!!

Comments

[Евгений Бухарев]

плюс ещё то, что одну и ту же модель сохраняет в цикле, причем переписывая старые данные

[AlikDex]

вообще-то можно делать запросы в цикле, технически. Но это называется говнокод. Правильно будет сказать: - "настроятельно не рекомендуется, в целях повышения производительности"

[Сергей Беловенцев]

А как лучше ?

[333Serega333 333Serega333]

Сергей Беловенцев: stackoverflow.com/questions/452859/inserting-multi...

[Сергей Беловенцев]

333Serega333 333Serega333: а как мне это INSERT INTO MyTable ( Column1, Column2 ) VALUES
( Value1, Value2 ), ( Value1, Value2 ) сделать методом ActiveRecord?

[333Serega333 333Serega333]

Сергей Беловенцев: Пройдись циклом по своему массиву и составь запрос INSERT.
Не знаю что у тебя за фреймворк, но там по любому должен быть метод, который выполняет запрос в базу. Что то типа такого query($sql). Вызови этот метод и передай туда созданный тобой запрос.

[Сергей Беловенцев]

у мне фраимворк yii2

[333Serega333 333Serega333]

Сергей Беловенцев: та я уже понял. так как вопрос в разделе "Yii"))
Я давно с Yii не работал. Ищи что то такое:
$db->createCommand('Insert ... ')
->execute();

[Mylistryx]

Сергей Беловенцев: stackoverflow.com/questions/27355262/activerecord-...

[mitaichik]

333Serega333 333Serega333: Не согласен с вашем высказыванием по поводу запроса в цикле. Понятно что у автора что-то непонятное твориться, но вообще... bathInsert всего лишь оптимизирует запрос, но это само по себе далеко не всегда самое важное. Как правило, гораздо важнее прохождение валидации, обработка событий, выполение каких-либо бизнес правил и прочая бизнес логика при вставке/изменении. Поэтому правило "Запросы sql в цикле делать нельзя" на самом деле не всегда применимо, все зависит от контекста.

[333Serega333 333Serega333]

mitaichik: Вынужден Вас огорчить, но Вы неправы. Валидация, бизенс-правила и т.д. и т.п. это все очень важно, не спорю. Но запросы в цикле делать нельзя. Это применимо всегда и не может обсуждаться.

[Сергей Беловенцев]

batchInsert сработал спасибо всем

[Никита]

333Serega333 333Serega333: Можно, в одной транзакции, падения в производительности по сравнению с batch минимальные, если вообще есть.

[333Serega333 333Serega333]

Никита: При чем здесь транзакции. Если у него массив будет на 10 000 элементов, то такой скрипт будет работать пол часа, если вообще отработает.

Answer 2

[Slava Kryvel]

Смысле он срабатывает один раз?
Происходит только запись последнего элемента? или записываются все но только в конце?
какой primarykey у вас в catSerial ?

Comments

[Сергей Беловенцев]

происходит запись последнего элемента

[Slava Kryvel]

не видя кода сложно сказать, что именно Вы хотите, но попробуйте посмотреть на это
stackoverflow.com/questions/29272150/insert-multip...

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

[Сергей Беловенцев]

Slava Kryvel: да вы правы мне так и нужно. Но я не нашел решения вопроса.

[Сергей Беловенцев]

Slava Kryvel: вот такого плана есть запрос INSERT INTO MyTable ( Column1, Column2 ) VALUES
( Value1, Value2 ), ( Value1, Value2 ) как оформить

[Slava Kryvel]

Сергей Беловенцев: stackoverflow.com/questions/7048144/how-do-you-ins...
$values = '(null, "your string"), (null, "next string"), (null, "third string")';
$sql = 'INSERT INTO table_data (id, data) VALUES ' . $values;
$command = Yii::app()->db->createCommand($sql);
$command->execute();

либо вот так
stackoverflow.com/questions/27355262/activerecord-...

Answer 3

[mitaichik]

У вас скорее всего ошибка в логике приведенного кода. Такое чувство, что вы хотите создать 2 строки в БД, но приведенный код сделает одну, а затем изменит ее. Вы б лучше написали что вы хотите сделать.

Так же следует учесть: save вызывается в любом случае, но не всегда он будет сохранять. Например, если не прошла валидация, он не будет пытаться сохранить невалидную модель в базу.

Вполне возможно что у вас есть уникальный валидатор по cat_id и строка с таким id уже существует - save не удет сохранять а просто вернет false.

Если делаешь что-то критичное, результат save всегда нужно проверять! И либо бросать эксепшн, либо как-то это обрабатывать.