为什么在调用 Eloquent 模型中的方法时出现“不应静态调用非静态方法”？

Question

我试图在我的控制器中加载我的模型并尝试了这个：

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();

}

在控制器中加载模型然后返回其内容的正确方法是什么？

Answer 1

您将方法定义为非静态方法，并尝试将其调用为静态方法。 那说...

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方法。 之后，所有其他方法都被称为实例方法。

Answer 2

为什么不尝试添加范围？ 作用域是 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();

Laravel 文档中的 Eloquent #scopes

Answer 3

TL;博士。 您可以通过将您的查询表达为MyModel::query()->find(10);来解决这个问题MyModel::query()->find(10); 而不是MyModel::find(10); .

据我所知，对于MyModel::where() 、 MyModel::find()等方法，启动PhpStorm 2017.2代码检查失败（检查此线程）。 这可能会很烦人，当您尝试（假设）在提交代码之前使用PhpStorm 的 Git 集成时， PhpStorm不会停止抱怨这些静态方法调用警告。

解决此问题的一种优雅方法 (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_smart', false)
    ->get();

好的

格式良好的代码，没有抱怨

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

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

Answer 4

以防万一这对某人有帮助，我收到此错误是因为我完全错过了在调用本地范围时不得使用范围前缀的陈述事实。 因此，如果您在模型中定义了一个局部范围，如下所示：

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

你应该这样称呼它：

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

请注意，调用中不存在前缀scope 。

Answer 5

你可以这样给

public static function getAll()
{

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

}

当你在你的控制器函数中静态调用时..

Answer 6

在我的案例中，我实际上刚刚得到了答案。 我正在创建一个实现了 create 方法的系统，所以我收到了这个实际错误，因为我访问的是被覆盖的版本，而不是来自 Eloquent 的版本。

希望有帮助？

Answer 7

检查您是否没有在模型中声明 getAll() 方法。 这会导致控制器认为您正在调用非静态方法。

Answer 8

使用类似return Post::getAll();的语法你应该在你的类中有一个神奇的函数__callStatic来处理所有静态调用：

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

Answer 9

原始问题的解决方案

您静态调用了非静态方法。 要将模型中的公共函数设为静态，应如下所示：

public static function {
  
}

一般来说：

Post::get()

在这个特定的例子中：

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

在定义关系和范围时要注意的一件事是，当它们命名相同时，我遇到了导致“不应静态调用非静态方法”错误的问题，例如：

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

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

当我执行以下操作时，出现非静态错误：

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

问题在于，Laravel 使用的是我的称为类别的关系方法，而不是我的类别范围（scopeCategory）。 这可以通过重命名范围或关系来解决。 我选择重命名关系：

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

请注意，我定义了外键 (category_id)，否则 Laravel 会查找 cat_id 而不会找到它，因为我在数据库中将其定义为 category_id。