1

Есть вот такой код:

class m160115_201537_insert_data_in_phrases extends Migration
{
    public function exists($table_name, $fields)
    {
        $query = new \yii\db\Query();
        $model = $query
            ->select(["*"])
            ->from($this->db->getTableSchema($table_name)->name)
            ->where($fields)
            ->exists();

        return $model;
    }

    public function save($table_name, $fields, $exists = null)
    {
        if (is_null($exists)) {
            $exists = $fields;
        }

        if ($this->exists($table_name, $exists)) {
            $this->update($table_name, $fields);
        } else {
            $this->insert($table_name, $fields);
        }
    }

    public function up()
    {
        $phrases_array = require Yii::getAlias('@t10s/phrases.php');

        var_dump($phrases_array);

        foreach ($phrases_array as $name => $array) {
            if (count($array) != 3) {
                Console::say("Не все переводы для поля %s были добавлены! Пожалуйста, добавь все переводы!", [$name]);
                return false;
            }
            $this->save('{{%phrases}}', [
                'name' => $name,
                'locale' => 'ru',
                'text' => htmlspecialchars($array[0]),
            ], [
                'name' => $name,
                'locale' => 'ru',
            ]);
            echo $name;
        }
    }

    public function down()
    {
        echo "Для того, чтоб применить новые переводы просто запустите миграцию еще раз\n";
        return true;
    }
}

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

ОШИБКА: повторяющееся значение ключа нарушает ограничение уникальности

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

Почему так происходит? Как это справить? Как можно проще это реализовать?

18
  • Уникальные значения на то и уникальные, что не должны быть......не уникальными.. 17 янв 2016 в 11:11
  • А что вы конкретно хотите исправить? Чтобы были одинаковые значения? Вы уже ответили сами на этот вопрос - убрать неуникальность. Или в случае повторяющегося элемента хотите пропускать итерацию?... зачем при save() вы пишите два раза 'name' => $name, 'locale' => 'ru', ? 17 янв 2016 в 11:13
  • Исправить - довольно просто, в миграциях не должно быть данных, и уж тем более миграция не должна применяться дважды. Почему проверка фальшивит - не могу сказать.
    – etki
    17 янв 2016 в 11:19
  • 1
    @Razzwan А зачем столько ненужных функций?) Почему бы не воспользоваться стандартной on duplicate key?? $this->dbConnection->createCommand('INSERT INTO table` (field1, field2) VALUES ("value1", "value2") ON DUPLICATE KEY UPDATE field1 = VALUES(field1);')->execute();` или воспользоваться batchInsert с тем же параметром ON DUPLICATE KEY UPDATE или уж если на то пошло, просто взять модель phases наверняка она есть и сделать проверку сразу.... примерно как тут : php-mysql-highload.blogspot.ru/2013/07/… ..... 17 янв 2016 в 12:28
  • 1
    Скорее всего дело в том, что exists() проверяет совпадение данных по всем полям, а ключ определен на подмножестве из них. Загляните в описание таблицы и увидите так это или нет. Заодно проверьте, первичный ключ с автоинкрементом или нет.
    – Nick
    17 янв 2016 в 16:29

1 ответ 1

1

Я считаю у вас слишком много манипуляций для совершения такого, в принципе, нехитрого дела.

Если не выходит так, как сейчас есть, можно воспользоваться стандартной вещью от БД: ON DUPLICATE KEY UPDATE

ON DUPLICATE KEY UPDATE - Если вы указываете ON DUPLICATE KEY UPDATE (новшество в MySQL 4.1.0), и производится вставка строки, которая вызывает ошибку дублирующегося первичного (PRIMARY) или уникального (UNIQUE) ключа, то вполняется UPDATE старой строки.

C помощью createCommand это делается так:

Yii::$app->db->createCommand('INSERT INTO {{%table}} (field1, field2) VALUES ("value1", "value2") ON DUPLICATE KEY UPDATE field1 = VALUES (field1);')->execute();

,где после ON DUPLICATE KEY UPDATE можно задать любые поля и любое количество, которые надо перезаписать.


Можно воспользоваться методом batchInsert (метод для пакетной вставки данных) которую предлагает сам фреймворк (Описание метода) с тем же параметром ON DUPLICATE KEY UPDATE

$db = Yii::$app->db;
$sql = $db->queryBuilder->batchInsert($table, $fields, $rows);
$db->createCommand($sql . ' ON DUPLICATE KEY UPDATE')->execute();

,где $table - таблица, $fields - имена полей, $rows - массив данных


Ну и еще один способ, сразу манипулировать моделью. Возможно это не совсем корркетно и таит некоторые подводные камни, но на примете держать можно.

Пример (не забываем подключить namespace нужной модели):

use yii\db\Schema;
use yii\db\Migration;
use common\models\Test;

class m160118_064910_test extends Migration {

    public function up() {
        $user = Test::find()
            ->where(['field1' => 'value1'])
            ->one();

        if(!$user){
            $user = new User;
        }

        $user->attributes=array(
            'field1' => 'value1',
            'field2' => 'text 111',
        );

        $res = $user->save();
    }

    public function down() {
        // some migration here      
    }
}
3
  • Модели в миграциях - это беда будет, модели рано или поздно меняются (или вообще удаляются), а миграции старые остаются и при деплое с нуля все ломается. Менять миграции под модели тоже не вариант, там легко могут быть зависимые друг от друга цепочки и нужно проверять все вдоль и поперек. В общем я как-то пришел к выводу, что лучше без моделей...
    – kroder
    18 янв 2016 в 8:59
  • 1
    @kroder поэтому я не настаиваю на данном пункте и убрал его вниз и написал, что это не совсем корректный способ)) А вообще если модели меняются, то это должно быть отражено в тех миграциях, которые меняют эти самые модели. А так как название миграции должна отражать суть, то надо бы перед мигрированием удостоверится, что эта миграция не связана с другой. В прочем, это тема для большой дискуссии. Но в целом да, желательно не привязываться к ним 18 янв 2016 в 9:04
  • Для справки, в RoR (RubyOnRails) модели используются в миграциях и это подано в официальной документации, как рекомендуемый метод: active_record_migrations Так что все больше склоняюсь к свободному использованию моделей в миграциях. И не вижу в этом ничего очень уж плохого. Как вариант, можно продублировать в отдельной папке, скажем deleted_models те модели, которые были удалены, но могли использоваться в миграциях. Если миграции требуют какие-то модели, которых уже нет - их можно найти в этой папке.
    – Razzwan
    18 янв 2016 в 19:32

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.