Как правильно сохранить модель в Laravel?

Question

[semki096]

Есть 2 связанные таблицы - например posts и comments. К посту я добавлю комментарий. То есть я могу записать коммент 2мя способами - либо сразу передать значения в модель комментария и сохранить

$comment = new Comment($data);
$comment->save();

либо сделать то же самое через модель поста

$comment = new Comment($data);
$post = Book::find($data['post_id']);
$post->comments()->save($comment);

Второй вариант взят из урока, я не понял зачем так делается если можно по первому простому варианту.

PS. Во втором варианте есть смысл если нам нужны данные из модели Post. Но в моем случае они не нужны, зачем же делать лишний запрос?

Comments

[JhaoDa]

А ты документацию почитай вместо уроков.

[semki096]

JhaoDa, я хочу понять в чем разница этих двух подходов

[JhaoDa]

semki096, вот в документации и написано, в чем разница.

[sim3x]

JhaoDa, Если вас не затруднит, ответьте линком на документацию

[JhaoDa]

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

[sim3x]

JhaoDa, понимаю
И прошу ето в надежде, что потом десяток-другой, незамеченных нами пользователей, все же загуглят и почитают инфу по линку

[semki096]

sim3x, жаль что удалили коммент где я прояснил подробности этого вопроса, поэтому продублирую. Во втором варианте есть смысл если нам нужны данные из модели Post. Но в моем случае они не нужны, зачем же делать лишний запрос?

[Алексей Скляров]

Не работал с laravel, но полагаю, что в первом случае вы просто создаете комментарий, без какой-либо привязки к посту, во втором случае в конкретный пост вы добавляете комментарий и сохраняете пост.

[semki096]

Алексей Скляров, здесь в $data['post_id'] у меня как раз поле связывающее коммент с постом

Answer 1

[Алексей Уколов]

Если вы следуете методологии REST, то у вас post_id не в $data, а прямо в URL.
И тогда метод контроллера будет выглядеть примерно так:

public function save(Post $post, Request $request) {
  $fields = $request->only(['field', 'other_field']);
  $post->comments()->save($fields);
}

Согласитесь, такой код гораздо проще читать? Самое главное в комментарии - это его связь с какой-то сущностью, даже не содержимое. Комментария не может быть без поста, поэтому и плясать нужно от поста.

Первый способ - процедурный. Связь между сущностями описана в "магическом" поле в каком-то массиве. Он имеет право на существование в каких-то технических консольных скриптах, например (и то не всегда).
Второй - объектно-ориентированный и легче читается. Этот код очень хорошо коррелирует с методологией REST, когда мы оперируем сущностями, а не массивами и абстрактными объектами.

Comments

[semki096]

Спасибо теперь понятно.

[Тимофей]

Только здесь как бы лишний запрос выполняется на вытягивание поста, так что это не самый оптимальный вариант.
Можно первый параметр заменить на int $postId, а при сохранении коммента тогда есть 2 аналогичных способа привязать этот id к нему:

$comment = new Comment($req->all()); // если что, у коммента можно определить $fillable, чтобы не париться с определением списка полей в контроллере
$comment->post_id = $postId;
// или
$comment->post()->associate($postId);
$comment->saveOrFail();

Если в миграциях нормально настроены внешние ключи, то при несуществующем $postId saveOrFail будет ругаться.

[semki096]

Тимофей, Cпасибо