如何在 Laravel 中进行即时重定向

Question

我想在 Laravel 的控制器中进行即时重定向。

我知道我可以使用

public function show($page) 
{   
   return Redirect::url('http://example.com');
}

但我想在许多添加条件的控制器中重复此代码，例如我想做这样的事情：

public function show($page) 
{
  $totalPages = 100; // here calculating maximum page

  if ($page < 2 || $page > $totalPages)  {
     return Redirect::url('http://example.com');
  }    
  // rest of code here - should be run if condition is false
}

但我不想在每个控制器中重复代码。

如果我尝试将重定向代码放在其他方法中（可能存在于基本控制器中），它将无法正常工作，因为它不会在主控制器中返回任何内容：

public function show($page) 
{
  $totalPages = 100; // here calculating maximum page
  $this->checkPages($page, $totalPages, 'http://example.com');   

 // rest of code here - should be run if condition is false   
}

public function checkPages($page, $totalPages, $url) 
{
  if ($page < 2 || $page > $totalPages)  {
     return Redirect::url($url);
  }    
}

我该如何解决这个问题？

Answer 1

经过一段时间的挖掘，您似乎应该使用来自Symfony\Component\HttpFoundation 的send() 方法（Laravel RedirectResponse 继承自此类）。

所以你可以这样修改checkPages方法：

public function checkPages($page, $totalPages, $url) 
{
  if ($page < 2 or $page > $totalPages)  {
     Redirect::url($url)->send();
  }    
}

它会立即重定向。

Answer 2

您必须在基本控制器中返回方法的结果。

return $this->checkPages($page, $totalPages, 'http://example.com');

Answer 3

我可能会建议您的 checkPages 方法试图做太多事情。我会期望checkPages（可能应该是validatePageNumber() 或类似的）返回一个redirect？不，我希望它返回 true 或 false。

这就是你的逻辑：

/**
 * Make sure the current page is between 1 and $lastPage
 *
 * @param int $page
 * @param int $lastPage
 * @return bool
 */
protected function validatePageNumber( $page, $lastPage ) {
    return $page < 2 || $page > $lastPage;
}

现在你的控制器逻辑很简单：

public function show( $page ) {
    $totalPages = 100;
    if ( ! $this->validatePageNumber( $page, $totalPages ) ) {
        return Redirect::url( 'http://example.com' );
    }   

    // rest of code here - should be run if condition is false   
}

---

现在，如果我们使用准英语阅读您的代码，您有一个公共方法 show($page)，它显示一个页面：它设置总页数，检查 $page 是否有效，如果不是，则重定向到某个地方.所有这些都是控制逻辑，也只是控制逻辑，这是控制器应该做的。

您的 validatePageNumber() 现在可以抽象为一个模型，或者更好的是一个 ServiceProvider，它只做一项工作：验证一个数字是否介于 1 和 n 之间。