LaravelのControllerにおけるテーブルの指定方法

Question

LaravelのControllerでは基本の基本では以下のように書くとおもいます。

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Facades\DB;

class Frameworks extends Model
{
  protected $table = 'frameworks';

  protected $guarded = array('id');

  public $timestamps = false;

  public function getData()
  {
    $data = DB::table($this->table)->get();

    return $data;
  }
}

出典

例えば、一つのコントローラの別のクラスで別のテーブルを使いたい場合、以下のような指定方法があるようですが、これは正しいのでしょうか？

動作自体はしますが同様の記述方式を用いたサイトが他に見つけられず、何か他の記法があるのではないかお伺いしたいです。

<?php
 
namespace App\Http\Controllers;
 
use Illuminate\Http\Request;
use App\User;  // Userモデルを呼び出す
 
class UserController extends Controller
{
    public function index()
    {
       $user = User::first();　// Userモデル（usersテーブル）で一番最初にヒットするデータをコレクションで返す
 
       dd($user->name); // nameの値をデバッグ
    }
}

出典

<?php
 
namespace App\Http\Controllers;
 
use Illuminate\Http\Request;
use App\User;  // Userモデルを呼び出す
use App\Hoge;  // Userモデルを呼び出す
 
class UserController extends Controller
{
    public function index()
    {
       $user = User::first();　// Userモデル（usersテーブル）で一番最初にヒットするデータをコレクションで返す
 
       dd($user->name); // nameの値をデバッグ
    }

    public function hoge()
    {
       $user = Hoge::first();　// Userモデル（usersテーブル）で一番最初にヒットするデータをコレクションで返す
 
       dd($user->content); // nameの値をデバッグ
    }

}

Comments

[退会済みユーザー]

もらった回答にリアクションもしないのですか？

Answer 1

突っ込みどころが多すぎて困っちゃいますね。

LaravelのControllerでは基本の基本では以下のように書くとおもいます。

提示されたModelから色々指摘事項が・・・

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Facades\DB;

// class Frameworks extends Model
class Framework extends Model // モデル名は通常単数形
{
  // protected $table = 'frameworks';　// クラス名を複数形にするのがデフォルトなので、書かなくていい

  // protected $guarded = array('id');
  protected $guarded = ['id']; // array の指定は [] で。

// public $timestamps = false; // public だめ
  protect $timestamps = false;

// Framework::all() で同じことできるしかも定義済み
//  public function getData()
//  {
//    $data = DB::table($this->table)->get();
//
//    return $data;
//  }
}

Controller に持つ、メソッドは以下の形が、デフォルトになっている。

https://readouble.com/laravel/5.8/ja/controllers.html

基本的には、以下に示した8個のメソッドでCRUDが作成されます。

上記のモデルを作成した後、

php artisan make:controller FrameworkController -m Models/Framework -r

を実行してコントローラーを作成すれば、上記のメソッドが全て出力され、以下のように実装すれば、CRUDが機能します。

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Models\framework;
use Illuminate\Http\RedirectResponse;
use Illuminate\View\View;

/**
 * Class frameworkController
 * @package App\Http\Controllers
 */
class FrameworkController extends Controller
{
    /**
     * @return View
     */
    public function index()
    {
        $frameworks = Framework::query()->paginate();
        return \view('frameworks.index', compact('frameworks'));
    }

    /**
     * @return View
     */
    public function create()
    {
        return \view('frameworks.create');
    }

    /**
     * @param Request $request
     */
    public function store(Request $request)
    {
        $framework = new Framework();
        $framework->fill(
            $request->all()
        )->save();
        return \redirect()->route('frameworks.show', compact('framework'));
    }

    /**
     * @param Framework $framework
     * @return View
     */
    public function show(Framework $framework)
    {
        return \view('frameworks.show', compact('framework'));
    }

    /**
     * @param Framework $framework
     * @return View
     */
    public function edit(Framework $framework)
    {
        return \view('frameworks.edit', compact('framework'));
    }

    /**
     * @param Request $request
     * @param framework $framework
     * @return RedirectResponse
     */
    public function update(Request $request, Framework $framework)
    {
        $framework->fill(
            $request->all()
        )->save();

        return \redirect()->route('frameworks.show', compact('framework'));
    }

    /**
     * @param Framework $framework
     * @return RedirectResponse
     * @throws \Exception
     */
    public function destroy(Framework $framework)
    {
        $framework->delete();
        return \redirect()->route('frameworks.index');
    }
}

Answer 2

すでに出ていますが逆にこの書き方はほとんどしないかなと

  public function getData()
  {
    $data = DB::table($this->table)->get();

    return $data;
  }

プロジェクトにもよるかもしれませんが以下の書き方のほうが一般的なきがします

$user = User::first();

ちなみに参考する際はリーダブルが参考になりますよ
データベース：クエリビルダ 5.8 Laravel
Eloquent：利用の開始 5.8 Laravel

Answer 3

まずこんな使い方は誰もしてない。基本でもなんでもなくLaravelでは完全に間違ったやり方。そのサイトは間違いだらけなので見てはいけない。

  public function getData()
  {
    $data = DB::table($this->table)->get();

    return $data;
  }

後半のこっちは正しい。
https://udemy.benesse.co.jp/development/web/laravel-model.html

Hogeのほうはただの書き間違いだろうけど

    public function hoge()
    {
       $hoge = Hoge::first();

       dd($hoge->content);
    }

Answer 4

動作していたら問題ないのでは？それは「正しい」と言えるのでは？
ひとつのことを実現するための手段はひとつではありませんし、「他の記述方法」をどのレベルで求めているのかわかりません。

発想次第で如何様にもなるものなので、どこかの知らない誰かが書くとは限りませんし、あったとして鵜呑みにしてはいけません。

あと、結局のところ要件次第です。
「なぜそうしたいのか」「なにをしたいためなのか」
もっも具体的に記載してください。