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

Question

[Иван]

Добрый вечер. Грубо говоря, есть примерно такой код.

public function GenerateUser($first_name,  $last_name)
    {
        $isset = User::where('first_name', $first_name)
            ->where('last_name', $last_name)->first();
        $response = null;
        if (isset($isset)) {
            $response = User::where('first_name', $first_name)
                ->where('last_name', $last_name)->first();
        } else {
            $response = new User();
            $response->first_name = $first_name;
            $response->last_name = $last_name;
            $response->save();
        }
        return response()->json($response);
    }

Работать должно следующим образом. Приходят 2 параметры, проверяем в базе в таблице users есть ли там уже такой юзер. Если есть, возвращаем модель, ничего в базу не пишем. Если нет, возвращаем модель и пишем ее в базу. Работает этот кусок кода нормально, но надо сделать как то с помощью вышеперечисленных методов, чтоб уменшить все это дело и повысить читаемость. Пробовал по всякому, примерно так

public function GenerateUser($first_name, $last_name)
    {
        $response = User::firstOrNew([
                'first_name' => $first_name,
                'last_name' => $last_name,
            ]);
        $response->save();
        return response()->json($response);
    }

Возвращает модель, но и пишет в базу в есть ли там такая запись или нет. Без save() не сохраняет. Пробовал другие подобные методы, результат примерно такой же. Как это решить?

Answer 1

[Илья]

Ну так firstOrCreate же, нет? возвращает, либо создаёт и возвращает

public function GenerateUser($first_name, $last_name)
{
	$user = User::firstOrCreate([
		'first_name' => $first_name,
		'last_name' => $last_name,
	]);

	return response()->json($user);
}

Comments

[Иван]

Если там уже есть такая запись, он снова ее создает и возвращает. А надо чтоб если она там есть, возвращал, но новую не создавал.

[Илья]

Иван, Вы ошибаетесь. Метод firstOrCreate создаёт запись, только если не существует записи, подходящей под условия в первом параметре. Если запись существует, то он её и возвращает.

[Иван]

Ilyas Sarsenbaev, Да.. Небольшая нестыковка. У меня там есть вставка еще одного поля с генерацией случайного кода, поэтому и всегда создавало, так как этот код менялся. Как в этом случае можно сделать, чтоб проверяло только по этих двух параметрах, а по этому рандомному коду нет?

[Иван]

Илья, Да.. Небольшая нестыковка. У меня там есть вставка еще одного поля с генерацией случайного кода, поэтому и всегда создавало, так как этот код менялся. Как в этом случае можно сделать, чтоб проверяло только по этих двух параметрах, а по этому рандомному коду нет?

[jazzus]

Иван, доп.поля по которым не нужно искать передаются во втором массиве

$user = User::firstOrCreate([
    'first_name' => $first_name,
    'last_name' => $last_name,
],['slug' => $slug]);

для дебага на Тостере нужно выкладывать точный код.

[Иван]

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

[jazzus]

Иван, этот и есть правильный ответ. На заданный вопрос) доки https://laravel.com/docs/7.x/eloquent#other-creati...

[Иван]

jazzus, Я понял что это и есть ответ. Я имел ввиду если напишете в ответах а не в комментариях, отмечу решением. Если оно надо конечно.

[jazzus]

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