Question

我需要将一些数据传递给我的视图。我有两个模型用户和提示模型。

User模型有一个返回用户hasMany（Tip :: Class）的方法，Tip模型有一个返回tipTo（User :: class）的方法。

我正在为用户创建个人资料页面，并使用路由模型绑定在访问个人资料时返回用户模型。

public function tipsterProfileShow(User $tipster)
{
  if (!$tipster->isTipster())
  {
    return redirect()->route('home');
  }

  return view('profile.index')->with([
    'tipster' => $tipster,
    'tips' => $tipster->tips(),
  ]);
}

我想显示一些数据，例如提示表中状态列指示的正确提示量。

目前在刀片视图中我正在使用

{{$tips->where('status','Won')->count()}}

我觉得这不是最佳做法，但我可能错了。

做下面的事情会更好吗？

public function tipsterProfileShow(User $tipster)
{
  if (!$tipster->isTipster())
  {
    return redirect()->route('home');
  }
  return view('profile.index')->with([
    'tipster' => $tipster,
    'tips' => $tipster->tips(),
    'wins' => $tipster->tips()->where('status', 'Won')->count()
  ]);
}

这样我就会将查询保留在视图之外。我对laravel和“最佳实践”非常陌生。所以试着得到一些建议。

Answer 1

你问的是最佳做法，这通常是不赞成的，但这确实是每个初学者都应该学习的必备条件，所以我仍然认为这是值得回答的。

简而言之：是的！你正在做的是保持代码逻辑分离的第一步。您的视图应负责显示数据，您的控制器应负责处理数据到视图。您的控制器是否应该实际负责计算数据是另一个主题，也是一个经常争论的主题。

那就是说，如果你应用一些其他逻辑，你可以把它归结为控制器中的一行：

public function tipsterProfileShow(User $tipster)
{
  return view('profile.index', compact('tipster'));
}

第一步是为User模型添加一个方法，如下所示：

public function winCount()
{
    return $this->tips()->where('status', 'Won')->count();
}

现在，您可以从视图中访问$tipster->winCount()了。您也可以在视图中直接访问$tipster->tips() - 大多数人都认为这非常好。

第二步是将非提示者的重定向调用提取到中间件中，您可以在此处阅读：https://laravel.com/docs/5.3/middleware

您可以从那里采取进一步措施，但这是一个很好的起点。祝好运！：）

Answer 2

我建议您创建一个 POPO（普通旧PHP对象），其中包含与用户个人资料同义的用户的完整描述。

laravel中的模型表示表中的一行，它将延迟加载与其相关的任何关系。

因此，如果您有 POPO ，您将能够定义与用户相关的所有内容并将其传递给视图，而无需在视图中进行查询。

以下面为例：

Class UserPorfile{
  private $id;
  private $username;
  private $tips;

  public function setId($id){
     $this->id = $id;
  }

  public function getId(){
     return $this->id;
  }

  public function setUsername($username){
      $this->username = $username;
  }

  public function getUsername(){
     return $this->username;
  }

  public function setTips(array $tips){
     $this->tips = $tips;
  }

  public function getTips(){
     return $this->tips;
  }
}

将此类的对象传递给您的视图似乎更好

Answer 3

是的，你是对的。鉴于您正在尝试遵循MVC软件开发模式，您应该避免将业务逻辑放在您的视图中。

这行代码：

{{$tips->where('status','Won')->count()}}

实际上做了两件事：

使用特定条件查询所有对象的模型
统计结果

通过遵循MVC原则，它应该是您的控制器将命令发送到模型，而不是视图。

Laravel最佳实践 - 查询

3 个答案: