关于在类中使用全局函数的问题,我正在研究MVC系统.
下面的方法是类的一部分,它调用类中的方法,例如模型和视图我的问题是我在那里有一个名为redirect()的全局函数我将它设置在一个函数文件中,该文件已加载到我的bootstrap文件,因为所有控制器都可能需要访问它,这是一种不好的做法,或者我最好将其作为控制器类的方法,因为所有控制器都扩展了父控制器.
public function post($slug){
if(!$slug){ redirect('blog'); }
$data = $this->model->getPost($slug);
$this->view->render('blog/single', $data);
}
或者静态类会更有意义吗？对于一个简单的重定向函数来说,它似乎有点过头了.
解决方法:
全球职能不一定是不好的做法.有两件事需要注意：
命名：您应确保函数的名称避免冲突,并且与其相关的内容显而易见.您的重定向功能,它将执行请求重定向.相反,如果您的方法与其他更具体的方法相关,那么需要明确的是通过预先添加一些上下文信息来实现这种情况.
国家：全球职能应该是无国籍的.这意味着该函数应始终为给定输入提供相同的结果.如果它根据一天中的时间,某些变量的值等做不同的事情……那么通常这是不好的.这可能看起来像一个明显的陷阱,但有更微妙的版本的问题.这条规则的明显例外实际上是要求一天中的时间……
看起来你的代码遵循这两个规则,所以这样做是完全可以接受的.
编辑存在可测试性问题,因为您正在引用类(或应该是)所需的全局函数,您无法模拟它.例如,在您的重定向示例中,您无法在不触发重定向功能的情况下正确测试类,并且您无法使用函数的模拟版本来检测是否已调用实际重定向方法.
