你有一个基本的想法,但你的代码有一些问题.有些人正在停止工作,有些人只是传统问题.
首先,这是一个belongsTomany关系(你有一个数据透视表),所以你必须将关系的两边定义为belongsToMany(即使hasMany是你想到它的一面或两面的方式).这是因为Laravel期望某种数据库结构具有两种不同的关系类型.
另一个问题(您自己发现)是您在实际保存帖子之前将标签添加到关系中(via – > tag() – > sync().您必须先保存帖子(以便laravel知道要为post_id添加到数据透视表的ID,然后添加关系.如果你担心标签部分失败然后有一个不一致的数据库,你应该使用事务.
最后,您遇到的“约定”错误是,根据定义,属于多对的关系涉及结果集合.因此,标签和帖子分别是标签和帖子.
所以这是我的代码的重写版本:
class Post extends Eloquent
{
public function tags()
{
return $this->belongsToMany('Tag');
}
}
class Tag extends Eloquent
{
public function posts()
{
return $this->belongsToMany('Post');
}
}
class PostController extends BaseController
{
public function addPost()
{
// assume it won't work
$success = false;
DB::beginTransaction();
try {
$post = new Post;
// maybe some validation here...
$post->title = Input::get('post_title');
$post->content = Input::get('post_content');
if ($post->save()) {
$tag_ids = Input::get('tags');
$post->tags()->sync($tag_ids);
$success = true;
}
} catch (\Exception $e) {
// maybe log this exception, but basically it's just here so we can rollback if we get a surprise
}
if ($success) {
DB::commit();
return Redirect::back()->withSuccessMessage('Post saved');
} else {
DB::rollback();
return Redirect::back()->withErrorMessage('Something went wrong');
}
}
}
现在很多控制器代码都以交易内容为中心 – 如果你不太关心它那么你就可以删除它.还有几种方法可以做这种交易的东西 – 我已经选择了一个不理想的方法,但只需要少量的代码即可.