php退出（）；代码中的功能警告

Question

在使用 Codacy 分析我的 PHP 代码时，我发现了一些由 exit(); 引起的错误。功能。这是一个功能，

public function saveCssForm(){

      $data = $_POST;
      if(!$data){
        // is a direct acess 
        $this->index();exit();
      }
      // update the data
      $this->csssettingmodel->updateCSS($data);
      // save the notifications 
      $this->notify_update($data['site_id'],$data['lang_key']);
      // set the success message
      $this->session->set_flashdata('edit_item', 'edited');
      // redirect to the view page 
      $baseUrl = $this->config->item('base_url');
      redirect($baseUrl.'index.php/cssSettings/view/'.$this->session->userdata("languageabbr"));
  }
 public function index()
      {
        // Denay Direct Access
          echo "<hr><h1><center>NO DIRECT ACCESS</h1> </center>";
          echo "<center>You are not permitted to access this page </center>";
      }

codacy 结果显示了这一点...

任何避免这种情况的替代方法或建议都会有所帮助。

Answer 1

Codacy 未显示错误，就您需要修复的问题而言；它正在分析您代码的质量，并建议出现在这个位置的exit不是一个好的做法，所以您可能想要修复它。

首先，应用程序框架通常设计为具有单个入口点，处理一些逻辑，然后 return 将结果发送到入口点，该入口点将输出并清理。从代码中的不同点退出使得预测流程变得更加困难，因为代码的整个部分可能看起来可以到达，但实际上是在程序退出后才到达。

其次，此类代码可用于调试，在特定点中断执行流程以显示一些中间数据或模拟特定故障。在这种情况下，它出现在分析代码中表明您不小心遗漏了调试代码。

php退出（）；代码中的功能警告

php exit(); function warning in codacy

php

codacy