Question

关于在类中使用全局函数的问题，我正在研究MVC系统。

以下方法是类的一部分，它调用类中的方法，例如模型和视图我的问题是我在那里有一个名为redirect（）的全局函数我将它设置在一个函数文件中加载到我的bootstrap文件中，因为所有控制器都可能需要访问它，这是一种不好的做法，或者我最好将其作为控制器类的方法，因为所有控制器都扩展了父控制器。

public function post($slug){

  if(!$slug){ redirect('blog'); }

  $data = $this->model->getPost($slug);
  $this->view->render('blog/single', $data);
}

或者静态类会更有意义吗？对于一个简单的重定向函数来说，它似乎有点过头了。

Answer 1

全球功能不一定是不好的做法。有两件事需要注意：

命名：您应该确保该函数的名称可以避免冲突，并且与其相关性显而易见。您的重定向功能，它将执行请求重定向。相反，如果您的方法与其他更具体的方法相关，那么需要明确的是通过预先添加一些上下文信息来实现这一点。

状态：全局函数应该是无状态的。这意味着该函数应始终为给定输入提供相同的结果。如果它根据一天中的时间，某些变量的值等做不同的事情......那么通常这是不好的。这可能看起来像一个明显的陷阱，但有更微妙的版本的问题。这条规则的明显例外实际上是要求一天中的时间......

看起来你的代码遵守这两条规则，所以完全可以接受这样做。

编辑存在可测试性问题，因为您正在引用类（或应该是）所需的全局函数，您无法模拟此问题。例如，在您的重定向示例中，您无法在不触发重定向功能的情况下正确测试该类，并且您无法使用该函数的模拟版本来检测是否已调用真正的重定向方法。

Answer 2

使用全局函数是不好的做法，因为它使您的类不可测试。您应该将所需的方法放在某个对象上，并将其作为依赖项传递给类。

Answer 3

正如你在评论中指出的那样，如果你使用PHP 5.3，那么使用lambda函数并传递它而不是创建一个全新的类，而不是全局会更好。

我会尝试设计代码，以便注入依赖项，并可能将引导程序暴露给控制器并在其中存储lambda然后只需将其调用到您想要的位置。

有关lambda的使用，请参阅http://fabien.potencier.org/article/17/on-php-5-3-lambda-functions-and-closures，DI示例也将提供一些有用的见解。

在类中使用全局函数是不好的做法吗？

3 个答案: