выход php (); функция предупреждения в кодировке
при использовании 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 показывает это …
любые альтернативы или предложения, чтобы избежать этого, были бы полезны.
Решение
Codacy не отображается ошибки, в смысле проблем ты необходимость чинить; это анализирует качественный вашего кода, и предполагая, что exit появление в этой должности не является хорошей практикой, поэтому вы можете хочу починить это.
Во-первых, прикладные инфраструктуры часто разрабатываются так, чтобы иметь единую точку входа, обрабатывать некоторую логику, а затем возвращать результат в точку входа, которая будет выводить и очищать. Выход из разных точек кода затрудняет прогнозирование потока, поскольку целые разделы кода могут выглядеть достижимыми, но на самом деле они появляются после выхода из программы.
Во-вторых, такой код может быть использован для отладки, чтобы прервать поток выполнения в определенной точке, чтобы отобразить некоторые промежуточные данные или смоделировать конкретный сбой. В этом случае его появление в анализируемом коде предполагает, что вы оставили код отладки случайно.
Другие решения
Других решений пока нет …
detector