Eloquentモデルでメソッドを呼び出すときに、「非静的メソッドを静的に呼び出すべきではない」というメッセージが表示されるのはなぜですか？

Question 1

モデルをコントローラーにロードしようとして、これを試しました：

return Post::getAll();

エラーが発生しました Non-static method Post::getAll() should not be called statically, assuming $this from incompatible context

モデルの関数は次のようになります。

public function getAll()
{

    return $posts = $this->all()->take(2)->get();

}

モデルをコントローラーにロードして、その内容を返す正しい方法は何ですか？

Question 2

メソッドを非静的として定義し、静的として呼び出そうとしています。そうは言っても...

1.静的メソッドを呼び出す場合は、::を使用してメソッドを静的として定義する必要があります。

// Defining a static method in a Foo class.
public static function getAll() { /* code */ }

// Invoking that static method
Foo::getAll();

2.それ以外の場合、インスタンスメソッドを呼び出す場合は、クラスをインスタンス化する必要があります->。を使用します。

// Defining a non-static method in a Foo class.
public function getAll() { /* code */ }

// Invoking that non-static method.
$foo = new Foo();
$foo->getAll();

注：Laravelでは、ほとんどすべてのEloquentメソッドがモデルのインスタンスを返すため、以下に示すようにメソッドをチェーンできます。

$foos = Foo::all()->take(10)->get();

そのコードでは、Facadeを介してメソッドを静的に呼び出していallます。その後、他のすべてのメソッドがインスタンスメソッドとして呼び出されます。

Question 3

スコープを追加してみませんか？スコープはEloquentの非常に優れた機能です。

class User extends Eloquent {

    public function scopePopular($query)
    {
        return $query->where('votes', '>', 100);
    }

    public function scopeWomen($query)
    {
        return $query->whereGender('W');
    }

}

$users = User::popular()->women()->orderBy('created_at')->get();

LaravelDocsのEloquent＃scopes

Question 4

TL; DR。のMyModel::query()->find(10);代わりにクエリを表現することで、これを回避できますMyModel::find(10);。

私の知る限りでは、開始PhpStorm 2017.2コード検査は、次のような方法で失敗したMyModel::where()、MyModel::find()など（このチェックスレッドを）。コードをコミットする前にPhpStormのGit統合を使用しようとすると、PhpStormはこれらの静的メソッド呼び出しの警告について文句を言うのをやめません。

これを回避するための1つのエレガントな方法（IMOO）は、意味のある場所に明示的に呼び出す::query()ことです。これにより、無料のオートコンプリートと優れたクエリフォーマットのメリットを享受できます。

例

検査が静的メソッド呼び出しについて文句を言うスニペット

$myModel = MyModel::find(10); // static call complaint

// another poorly formatted query with code inspection complaints
$myFilteredModels = MyModel::where('is_beautiful', true)
    ->where('is_not_smart', false)
    ->get();

苦情のない適切にフォーマットされたコード

$myModel = MyModel::query()->find(10);

// a nicely formatted query with no complaints
$myFilteredModels = MyModel::query()
    ->where('is_beautiful', true)
    ->where('is_not_smart', false)
    ->get();

Question 5

これが誰かを助ける場合に備えて、ローカルスコープを呼び出すときにスコーププレフィックスを使用してはならないという事実を完全に見逃したため、このエラーが発生していました。したがって、モデルで次のようにローカルスコープを定義した場合：

public function scopeRecentFirst($query)
{
    return $query->orderBy('updated_at', 'desc');
}

あなたはそれを次のように呼ぶべきです：

$CurrentUsers = \App\Models\Users::recentFirst()->get();

プレフィックスscopeは呼び出しに存在しないことに注意してください。

Question 6

あなたはこのように与えることができます

public static function getAll()
{

    return $posts = $this->all()->take(2)->get();

}

また、コントローラー関数内で静的に呼び出す場合も同様です。

Question 7

私の場合、文字通り答えにたどり着きました。createメソッドを実装したシステムを作成しているので、Eloquentからのものではなく、オーバーライドされたバージョンにアクセスしていたため、この実際のエラーが発生していました。

それがお役に立てば幸いですか？

Question 8

モデルでメソッドgetAll（）を宣言していないかどうかを確認してください。これにより、コントローラーは、非静的メソッドを呼び出していると見なします。

Question 9

次のような構文を使用するには、すべての静的呼び出しを処理return Post::getAll();するマジック関数__callStaticをクラスに含める必要があります。

public static function __callStatic($method, $parameters)
{
    return (new static)->$method(...$parameters);
}

Question 10

元の質問の解決策

非静的メソッドを静的に呼び出しました。モデルでパブリック関数を静的にするには、次のようになります。

public static function {
  
}

一般に：

Post::get()

この特定の例では：

Post::take(2)->get()

関係とスコープを定義するときに注意すべきことの1つは、「非静的メソッドを静的に呼び出さないでください」というエラーが発生するという問題が発生したことです。たとえば、名前が同じである場合です。

public function category(){
    return $this->belongsTo('App\Category');
}

public function scopeCategory(){
    return $query->where('category', 1);
}

次の手順を実行すると、非静的エラーが発生します。

Event::category()->get();

問題は、Laravelがカテゴリスコープ（scopeCategory）ではなく、categoryと呼ばれるリレーションシップメソッドを使用していることです。これは、スコープまたは関係の名前を変更することで解決できます。関係の名前を変更することにしました。

public function cat(){
    return $this->belongsTo('App\Category', 'category_id');
}

外部キー（category_id）を定義したことに注意してください。そうしないと、Laravelは代わりにcat_idを検索し、データベースでcategory_idとして定義したため、それを見つけることができませんでした。