laravel マスタの子テーブルのセレクトボックス表示方法

Question

Laravel5.5で開発しています。

商品マスタの子テーブルとしてメーカーマスタを定義していますが
マスタメンテナンス画面の作り方が、いまいちしっくりしていません。

現状やっていることは、コントローラー内でメーカーを取得して、ビューに渡してセレクトボックスを作っています。
モデルでリレーションを定義しているので、コントローラー内でいちいち取得せずに、もっと簡潔にできないかな？
と思っています。
Laravelの開発もそうですが、フレームワークを利用した開発自体経験がすくないのでできるかどうかすら判断ができません。
どうかご教授願います。

以下に抜粋しているソースを記載します。
モデル

class Product extends Model
{
	use SoftDeletes;

	protected $table = 'products';

	public function maker() {
		return $this->belongsTo(Maker::class);
	}
}

class Maker extends Model
{
	use SoftDeletes;

	protected $table = 'makers';
}

コントローラー(createのみ)

class ProductController extends Controller
{
    public function create(){
        $makers = Maker::query()->get();
        return view('products.create')->with(compact('makers'));
   }
}

ビュー（maker部分のみ）

<div class="panel panel-default">
		<div class="panel-heading">メーカー情報</div>
			<div class="panel-body">
				<div class="form-group">
					<div class="col-sm-6">
						<label>メーカー</label> {!! Form::select('maker_id',['' => '選択してください']+array_pluck($makers, 'name', 'id') ,old('maker_id'),array('class'=>'form-control','id'=>'maker_id')) !!}
					</div>
				</div>
			</div>
		</div>
</div>

どなたも貴重なご意見ありがとうございます。
質問の意図がわかりずらいようなので少し追記いたします。
商品マスタですが、実際はメーカーだけでなく複数の子テーブルと紐付いております。
メーカーだけの場合は、上記のようにコントローラー内でwithでわたすのは1つで済むのですが、これが増えると
下記の用になってしまいます。
この記述がスッキリしないので質問しました。
この記述が無難なのか、普通はもっとスマートな書き方があるのかをしりたいです。

class ProductController extends Controller
{
    public function create(){
        $makers = Maker::query()->get();
        $hoge1= Hoge::query()->get();
        $hoge2= Hoge::query()->get();
        $hoge3= Hoge::query()->get();
        $hoge4= Hoge::query()->get();
        $hoge5= Hoge::query()->get();

        return view('products.create')
                ->with(compact('makers'))
                ->with(compact('hoge1'))
                ->with(compact('hoge2'))
                ->with(compact('hoge3'))
                ->with(compact('hoge4'))
                ->with(compact('hoge5'));
   }
}

Answer 1

具体的な質問の意図がつかみ取りづらかったので、気づいた点を回答します。
まず、

class Product extends Model
{
    //（中略）

    public function maker() {
        return $this->belongsTo(Maker::class);
    }
}

class Maker extends Model
{
    //（中略）

    public function products()
    {
        return $this->hasMany(Product::class);
    }
}

のようにすれば、Maker→Product(s)、Product→Makerが参照できるようになります。
※ belongsTo() の逆は hasMany() です。

また、 $makers = Maker::query()->get(); と書かれている部分は、

$query = Maker::query();
$query->with('products');
$query->where('name', 'ほげほげ');
$query->orderBy('price');
$makers = $query->get();

のように行を分けてクエリを書く際に便利ですが、必要なければ単に $makers = Maker::get(); 等でいいです。
ちなみに、 hasMany() や belongsTo() は get() 時にリレーションが解決されませんので、必要に応じて、$query->with('products')のように結合取得を強制しておくとよいかと思います。

単にメーカーの一覧を全て取得したいのであれば、上記のように全メーカーを取得してビューに引き渡せばよいですね。

Answer 2

リレーションは、何らかのレコードについて紐づいている別テーブルのレコードを取得できますが、
create アクションの場合はレコードがまだ無く紐づいているレコードもありませんので、取得できません。

ローカルスコープを使って再利用可能なクエリーを Maker モデルに定義するのはいかがでしょうか。

---

追記

view に変数を渡す部分は

return view('products.create', compact('makers', 'hoge1', 'hoge2', 'hoge3', 'hoge4', 'hoge5'));

とするしかないですね。

Comments

[n.Taro]

やはり、そうなりますね。
ありがとうございます。

Answer 3

$makers = Maker::query()->get();
クエリビルダ(Eloquent?)のquery()メソッドがよくわからないのですが、動きますか？
知識不足で申し訳ないです。
$makers = Maker::all();
が好きです。

Comments

[n.Taro]

nuchaさん
Maker::query()->get();で取得できています。
Maker::all();でももちろん問題ないんですが、条件指定があるときのため今の所all()で取得しないようにしているだけです。

[退会済みユーザー]

おっとそういうことか。すいません説明よく読んでませんでした。model でくえりを完結させてはどうでしょうか。

User クラスに
public function getMakers () {
return this ->query()->get();
}

でコントローラ側では
$makers = Maker->getMakers()
で呼び出す
って動くかなこれ。出先からなんですいません。