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

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

全球职能不一定是不好的做法.有两件事需要注意:

命名:您应确保函数的名称避免冲突,并且与其相关的内容显而易见.您的重定向功能,它将执行请求重定向.相反,如果您的方法与其他更具体的方法相关,那么需要通过预先添加一些上下文信息来明确这种情况.

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

看起来你的代码遵循这两个规则,所以这样做是完全可以接受的.

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