Как производить поиск по имени, что бы получить id?

Question

[ITwoolf]

Всем привет. У меня 2 таблицы. Авторы и Книги (каждый со своими полями). В таблице с книгами есть поле ид автора, которое отображалось. Я через вюшку сделал так, что бы вместо ид автора в данной таблице отображалась имя автора из таблицы с авторами. Сделал экспорт таблицы Книг. Как и полагалось он экспортировал вместе с другими полями имя автора из др таблицы, а не ид.
Теперь я делаю импорт таблицы.
И мне нужно производить поиск в бд по имени и найти ид автора, подставив его.
Или цитирую слова своего препода

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

нет. ничего не надо убирать. ты грузишь данные из файла. вместо ид автора у тебя его имя. ты берешь это имя, делаешь запрос к бд (ищешь по имени автора), получаешь данные по этому автору и берешь из них его ид

Вот код импорта. Нужно изменить строку с $model-> id_avtor1 = $row [4];

public function actionUpload()
    {
        $model = new UploadForm();
        if (Yii::$app->request->isPost ) {
            $model->fName = UploadedFile::getInstance($model, 'fName');
            if ($fName =$model->upload()) {
                    //путь к файлу

                    if (($handle = fopen($fName, 'r')) !== false) {


                        while (($row = fgetcsv($handle, 1000, ',')) !== false) {
                            if ($row[0]=='ID' and $row[0]=='#'){
                                continue;}
                            $model = new Kniga1();
                            $model->name = $row [2];
                            $model-> id_avtor1  =  $row [4];
                            $model->creation_date = $row [5];
                            
                            if ($model->validate()) {
                                $model->save();

                            } else {
                                $model->save();
                                print_r($model->errors);
                            }
                        }
                        fclose($handle);
                    };
                    //... код после импорта
            }

        }
        return $this->render('upload', ['model' => $model]);
    }

Answer 1

[Дмитрий]

Доброе утро.
Во это неверно.

if ($model->validate()) {
    $model->save();
  } else {
    $model->save();
    print_r($model->errors);
  }

Если атрибуты не прошли валидацию, то зачем пытаться опять сохранить модель? Ничего не произойдёт. Просто выводите ошибки.

Нужно изменить строку с $model-> id_avtor1 = $row [4];

Если в $row[4] у Вас хранится имя автора, то сделайте метод, который будет искать в таблице `author` автора по его имени и возвращать его id.
Как-то так, статический метод в модели Author:

public static function getIdAuthor($name){
    return Author::find()->select('id')->where('name=:name',[':name' => $name])->column();
}

И подставляете в строку
$model-> id_avtor1 = Author::idAuthor($row[4]);

Comments

[ITwoolf]

Спасибо большое.
А что за поле idAuthor ? у меня просто такого поле нету.
есть id_avtor, но он находится в книгах. а в таблице с автором есть только следующие поля:
'id' => 'ID',
'name' => 'Name',
'creation_date' => 'Creation Date',
'date_of_change' => 'Date Of Change',

[Дмитрий]

ITwoolf, это статический метод.
Смотрите внимательней ответ.

[ITwoolf]

slo_nik, большое спасибо. все в порядке и работает.
Можете объяснить что тут происходит? where('name=:name',[':name' => $name])->column();
Не люблю тупо копировать код. Хочу понять

[Дмитрий]

ITwoolf, подготовленный запрос, ради безопасности.
Тут смотрите.

[ITwoolf]

slo_nik, спасибо!)

[Дмитрий]

ITwoolf, пожалуйста.

[ITwoolf]

slo_nik, Кстати по поводу кода

if ($model->validate()) {
    $model->save();
  } else {
    $model->save();
    print_r($model->errors);
  }

Валидация не работает.
убрал $model->save(); после else.
Импортирую файл. выбирай файл с компа и нажимаю submit. В итоге пишет 'Файл не выбран'
С чем это может быть связано ?

[Дмитрий]

ITwoolf, думаю, что проблема с моделью загрузки, точнее с правилами валидации.

[ITwoolf]

slo_nik, Я попробовал сделать сохранение без валидации, стерев if и else, и, написав вместо них $model->save(false)
Все так же

[ITwoolf]

slo_nik, Проблему решил. Вроде все правильно импортируется. И ошибок никаких нет. Но у меня создаются в бд много пустых строк. бесконечно.

[Дмитрий]

ITwoolf, думаю, что проблема в цикле while() и его условии.
Вообще-то правильней было бы заменить while() на foreach()

[ITwoolf]

slo_nik, хорошо. тогда куда мне деть код внутри цикла while, а именно

while (($row = fgetcsv($handle, 1000, ',')) !== false)

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

[ITwoolf]

slo_nik, Проблему с циклом решил. Все работает хорошо. Кроме поля id_avtor1. Ваш код правильный, но где-то там нужно что-то поменять. Так как при загрузке файла он пишет Array ( [id_avtor1] => Array ( [0] => Id Avtor1 is invalid. ) )
Сам я пока в поисках проблемы, но пока не нашел его источник.
В голову пока пришло сделать наподобие такого варианта $model = Avtor1::find()->where("name")= $row [4];

Или ваш донастроить. Что скажете ?

[Дмитрий]

ITwoolf, что-то я не понял, что Вы хотите добиться этим?
$model = Avtor1::find()->where("name")= $row [4];
Тем более, что запрос написан неверно, с ошибкой.

[ITwoolf]

slo_nik, уже ничего. я хочу поменять ваш код. что бы не вылазила ошибка Array ( [id_avtor1] => Array ( [0] => Id Avtor1 is invalid. ) )
Но пока не очень удается

[Дмитрий]

ITwoolf, где она вылазит? Сначала же код работал? Теперь Вы его меняете и получаете ошибку?

[ITwoolf]

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

[Дмитрий]

ITwoolf, всё-равно ничего не понял)))
Без кода тут не разобраться)))

[ITwoolf]

slo_nik, Смотрите вот код импорта, в котором так же есть ваш код, который решает мою выше описанную проблему в 1 посту. )

public function actionUpload()
    {
        $model = new UploadForm();
        if (Yii::$app->request->isPost ) {
            $model->fName = UploadedFile::getInstance($model, 'fName');
            if ($fName =$model->upload()) {
                    //путь к файлу

                    if (($handle = fopen($fName, 'r')) !== false) {


                        while (($row = fgetcsv($handle, 1000, ',')) !== false)  {

                            if ($row[1]==='ID'){
                                continue;
                            }
                            $modell = new Kniga1();
                            $modell->name = $row [2];
                            $modell-> id_avtor1 = Avtor1::idAuthor($row[4]);
                            $modell->creation_date = $row [6];
                            
                            if ($modell->validate()) {
                                $modell->save();

                            } else {
                                print_r($modell->errors);
                            }
                        }
                        fclose($handle);
                        return $this->redirect(['index']);
                    };
                    //... код после импорта
            }

        }
        return $this->render('upload', ['model' => $model]);
    }

Так же я создал в моделях Avtor1 статическую функцию, как вы выше указали

public static function IdAuthor($name){
        return Avtor1::find()->select('id')->where('name=:name',[':name' => $name])->scalar();
    }

Только я в конце поставил scalar(), ибо без него выдавала какую-то ошибку.

Ну и моя модель загрузки файлов

class UploadForm extends Model
{
    /**
     * @var UploadedFile
     */
    
    public $fName;
    
    public function rules()
    {
        return [
            [['fName'], 'safe'],
        ];
    }

    public function upload()
    {
        if ($this->validate()) {
            $path = \Yii::getAlias('@runtime').'/uploads/' . $this->fName->baseName . '.' . $this->fName->extension;
            $this->fName->saveAs($path);

            return $path;
        } else {
            return false;
        }
    }
}

При импорте файла браузер выдает ошибку, связанную со строчкой id_avtor1.

[Дмитрий]

ITwoolf, ну а зачем Вы поставили scalar? Чем Вам column не угодил?

[ITwoolf]

slo_nik, вот этим )

[Дмитрий]

ITwoolf, так column() здесь ни при чём. Причина во внешнем ключе. Вы пытаетесь обновить то, что не можете обновить из-за целостности внешнего ключа.

[Дмитрий]

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

[ITwoolf]

slo_nik, саму логику кода импорта в контроллере ?

[Дмитрий]

ITwoolf, ну почти.
Опять же у Вас проблемы с именованием переменных.
Вы используете $model для разных моделей, для UploadForm и Kniga1. Это вносит пунаницу.
Не логичней ли было сделать так?

$upload = new UploadForm();
$book = new Kniga1();

Теперь дальше.
У Вас настроен внешний ключ между Kniga1 и Avtor1? Вот в этом проблема.

[Дмитрий]

ITwoolf, что Вам возвращает метод Avtor1::idAuthor($row[4])?

[ITwoolf]

slo_nik, мы же в этом методе делаем поиск в БД по имени, находим ид и подставляем его. Вы сами написали этот метод )

[Дмитрий]

ITwoolf, он может ничего не вернуть, например такого автора нет в базе данных.
Метод должен возвращать id.
Что он возвращает на самом деле?

[ITwoolf]

slo_nik, он возвращает 'ничего'?

[Дмитрий]

ITwoolf, ну вот Вам и причина ошибки.
Почему он Вам возвращает "ничего"?
Нужный автор есть в таблице?

[Дмитрий]

ITwoolf, Вы делаете импорт в свою базу данных.
Поэтому необходимого автора может не быть в Вашей таблице.

[ITwoolf]

slo_nik, Все проблема решена ) были ошибки в контроллере. еще раз спасибо!

[Дмитрий]

ITwoolf, как решили, что за ошибки?

[ITwoolf]

slo_nik, Добрый день. Все оказалось проще, чем думал.
В контроллере, где было

if ($modell->validate()) {
                                $modell->save();

                            } else {
                                print_r($modell->errors);
                            }

Убрал print_r($modell->errors);

Ну и в модели автора изменил

public static function IdAuthor($name){
        return Avtor1::find()->select('id')->where('name=:name',[':name' => $name])->scalar();
    }

На

public static function IdAuthor($name){
        return
            Avtor1::find()->where(['name' => $name])->scalar();
    }

И Бинго!
Все работает как надо)

[Дмитрий]

ITwoolf, scalar то зачем?

[ITwoolf]

slo_nik, Мне нужно было получить скалярное значение 1 столбца 1 строки )

[Дмитрий]

ITwoolf, Вам надо получить id)))

[ITwoolf]

slo_nik, ну я его и так получаю)
ведь в контроллере в коде импорта я прописал

if ($row[1]==='ID'){
       continue;
 }

вместо

if ($row[0]=='ID' and $row[0]=='#'){
      continue;
}

Который у меня был в прошлом варианте)