在一个控制器中使用2个模型是不好的做法

Question

We are using Laravel and we have multiple models controllers.

Ex:

Model1 and Model2 and they corresponding controllers Controller1 and Controller2. We need to make a new entry for Model2 every time we create new entry for Model1.

The question is, is it bad practice to make new entries in Model1 and Model2 in Controller1

public function store(Request $request)
{
    //Create new Model1
    $model1 = new Model1;
    $model1->bla = "Model1";
    $model1->save();

    //Create new Model2
    $model2 = new Model2;
    $model2->bla = "Model2";
    $model2->save();
}

Answer 1

I have multiple projects that are doing what you just have said, saving entries on a multiple tables at once. I believe that this is really happening on an app in a real world scenario.

Here's what I am doing though:

public function store(Request $request)
{
    $main = $this->saveModel1($request);
    $this->saveModel2($request);    

    return redirect()->route('my.route.name.for.edit', $main->id);
}

private function saveModel1($request)
{
    $model1 = new Model1;
    $model1->bla = "Model1";
    $model1->save();
    return $model1->id;
}

private function saveModel2($request)
{
    $model2 = new Model2;
    $model2->bla = "Model2";
    $model2->save();
}

UPDATE: This answer has been selected as correct, but I just want to add that on condition like this, it is safe to enclose your saving query into a transaction.

DB::beginTransaction();
try
{

}
catch (\PDOException $e)
{

}