【Laravel 5.8】直接のリレーションを持っていないモデルの保存をしたい

Question

前提・実現したいこと

以下のテーブル構成において、problemモデルを新規保存したい

users
    id - integer
    name - string

drills
    id - integer
    user_id - integer
    name - string

problems
    id - integer
    drill_id - integer
    name - string

該当のソースコード

UserとDrillのモデルは以下の通り。

class User extends Authenticatable
{
    public function drills()
    {
        return $this->hasMany('App\Drill');
    }
    public function drillProblem()
    {
        return $this->hasManyThrough('App\Problem', 'App\Drill');
    }
}

class Drill extends Model
{
    public function problems()
    {
        return $this->hasMany('App\Problem');
    }
}

以下のコードを書くと、それぞれコメントアウトしたエラーが発生してしまいます。
user、drillに紐付いたproblemを保存するにはどうしたら良いでしょうか。

use App\Drill;
use App\Problem;
use Illuminate\Support\Facades\Auth;

class DrillsController extends Controller
{
    public function create(Request $request)
    {
        $drill = new Drill;
        $problem = new Problem;

        Auth::user()->drills()->save($drill->fill($request->all())); //保存可能
        Auth::user()->drills()->problems()->save($drill->fill($request->all())); //Call to undefined method Illuminate\Database\Eloquent\Relations\HasMany::problems()
        Auth::user()->drillProblem()->save($drill->fill($request->all())); //Call to undefined method Illuminate\Database\Eloquent\Relations\HasOneThrough::save()
    }
}

Comments

[mix-peach]

$request->all()　は、どのような内容ですか？

[wijiwiji]

情報不足で申し訳ございません！

$request->all()は、Drill（およびProblem）登録ページのformにてsubmitした内容です。
このページでは、Drillのname、および複数のProblemのnameを登録するイメージです。

つまり、＄requestには、drill_name、problem0_name、problem1_name、problem2_name,,,などが入いる予定です。

Answer 1

Auth::user()->drills();

これで戻ってくるのは、「user　は　drill を複数持ちます」というリレーション設定(hasMany)のインスタンスなので、drillのModelインスタンスではありません。

その為、Auth::user()->drills()->problems()　という使い方は出来ません。（problems()を持っているのは、drillのModelインスタンスです。）

Auth::user()->drillProblem();

こちらは、「user　は　drill 経由で、problemを複数持ちます」というリレーション設定（hasManyThrough）です。Auth::user()->drills()は、直接のリレーションを持っているので、save()することが出来ますが、Auth::user()->drillProblem()は、経由のdrillなしにはデータを持ち得ませんので、直接save()することはできません。

---

上記を踏まえつつ。。。

多分ちょっと難しく考えすぎなのかなと思いました。

user → problem の関係性は、drillありきで成り立つものなので、

①　userに所属するdrill

②　①で作成したdrillに所属する、problem（複数）

の2手順でデータを作成すれば、自動的に　user → problem　の関係が出来上がります。

つまり、「直接のリレーションを持っていないモデルの保存をしたい」という処理自体は、実は必要ないのです。

---

私ならこんな感じにするかなという参考コードです。
（※動作検証しておりませんので、スペルミス・不具合等ありましたらご容赦を）

Modelインスタンスは最初に定義しなくても、リレーション設定を使ってデータを作成することで取得可能です。

drillの作成

//これでuserに紐づいたdrillのModelインスタンスが戻ってきます
$drill = Auth::user()->drills()->create(['name' => $request->input['drill_name']]);

problemの作成

$problem_names = [];
$idx = 0;

//problemのnameがいくつ渡されるのか分からないので、とりあえず取得出来る限り保存するつもりでこんな書き方にしましたが、実際の取得した結果で上手くように調整してください。
while($proble_name = $request->input[sprintf('problem%d_name', $idx++)]){
  $problem_names[] = ['name' => $proble_name ];
}

//取得したproblem_namesを一括で保存してみます。
//problemsの中身は、Problemのモデルインスタンスを沢山持つCollectionです。
$problems = $drill->problems()->createMany($problem_names);

以上、ご参考までに。。。