PHP OOP处理UI错误通知

Question

我正在寻找显示用户错误的最佳方式，我使用的是Class File（）和个人档案页面。类文件（）将处理以下事项： - 检查文件的扩展名 - 检查文件是否已经存在 - 爆炸并创建链接文件& &插入链接到数据库 - 上传文件到文件夹项目文件夹目录

代码工作正常，除了错误显示，例如，如果扩展名是错误的，它将显示最后一个错误，而不是停止执行并显示扩展错误。

此外，如果文件已经存在于数据库& &文件夹目录下的PHP函数move_uploaded_file将显示与错误“警告：未能打开流：权限被拒绝”

感谢您的帮助

class File { 

    private $file = array(); 
    private $file_up = ''; 
    private $pdo = ''; 
    private $error = ''; 
    private $regex_file = '/^[a-z0-9][a-z0-9]{4,20}.jpg|.jpeg|.png|.gif|.txt|.doc|.docx|.pdf|.xlsx|.xlm|.xls|.pub|.one|.pptx$i/'; 

    /** 
    * [__construct connection ] 
    * @param [int] $id [Unique user_id retrieved from database (from include/header.inc)] 
    * Construct database connection using PDO 
    * 
    */ 
    public function __construct($id) 
    { 
     $this->pdo = new PDO('mysql:host='.DB_HOST.';dbname='.DB_NAME, DB_USER, DB_PASS, array(PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION)); 
     $sql = 'SELECT `file_name`, `file_extention` FROM `users`' 
       .'JOIN `file`' 
       .'ON users.`user_id` = file.`user_id`' 
       .'WHERE users.`user_id` = :id'; 
     $stmt = $this->pdo->prepare($sql); 
     $stmt->execute(array(':id' => $id)); 
     while($row = $stmt->fetch(PDO::FETCH_ASSOC)) { 
      $this->file[] = $row['file_name'] . '.' . $row['file_extention']; 
     } 
    } 

    /** 
    * [displayFile extention] 
    * @return [link] [description] 
    */ 
    public function displayFile() 
    { 
     $output = ''; 
     $link = $this->file; 
     foreach($link as $row) { 
      $output .= '<table class="table table-bordered table-hover"'; 
      $output .= '<thead><tr>'; 
      $output .= '<th><a href="../file_user/'.$row.'><i class="fa fa-folder"></i>&nbsp&nbsp'.$row.'</a><br /></th>'; 
      $output .= '</tr></thead>'; 
      $output .= '</table>'; 
     } 
     return $output; 
    } 

    public function checkFile($file) 
    { 
     $this->file_up = strip_tags($file); 
     if(!preg_match($this->regex_file, $this->file_up)){ 
       $this->error = '<b style="color:red">Extention ou nom de fichier incorrect</b>'; 
     } 
     foreach($this->file as $row){ 
      if($this->file_up == $row){ 
       $this->error = '<b style="color:red">Fichier ou nom du fichier deja existant</b>'; 
      } 
     } 
     if($this->error){ 
      return $this->error; 
     }else{ 
      $this->file_up = explode(".", $this->file_up);// array 
      return $this->file_up; 
     }  
    } 

    public function getError() 
    { 
     if($this->error !== ''){ 
      return $this->error; 
     } 
    } 

    public function uploadFile($array, $id) 
    { 
     if(is_array($array)){ 
      array_push($array, $id); 
      $sql = 'INSERT INTO `file`(`file_name`, `file_extention`, `user_id`) VALUES (?, ?, ?)'; 
      $con = $this->pdo->prepare($sql); 
      $con->execute($array); 
     }else{ 
      $this->error = '<b style="color:red">Fichier ne peut etre telecharger</b>'; 
     } 
    } 

    public function mvFile($size, $name, $tmp_name) 
    { 
     $to = 'file_user/'; 
     if($size <= 2000000){ 
      move_uploaded_file($tmp_name, $to.$name); 
      $this->error = '<b style="color:green">Fichier téléchargé avec succes</b>'; 
     }else{ 
      $this->error = '<b style="color:red">Un probleme est survenue veuillez recommencer</b>'; 
     } 
    } 
} 

And testing.php来测试它：

require_once 'class/file.inc'; 
$error = ''; 
$id = 2; 
$file = new File($id); 

$name = $_FILES['file']['name']; 
$size = $_FILES['file']['size']; 
$type = $_FILES['file']['type']; 
$tmp_name = $_FILES['file']['tmp_name']; 



if(isset($_FILES['file'])){ 
     $file_up = $_FILES['file']['name']; 
     $file_up = $file->checkFile($file_up); 
     $file->uploadFile($file_up, $id); 
     //$file->mvFile($size, $name, $_FILES['file']['tmp_name']); 
    if($file->getError()){ 
     $error = $file->getError(); 
    } 

} // end isset 

if($error){ 
    echo $error; 
} 

Answer 1

两个单独的问题：

1. 如果您发现任何错误，你的checkFile（）方法，你想只得到了第一个错误（这是从我的角度来看奇数;你可以在数组中收集错误），你应该直接返回该错误，而不是继续执行其余的方法。此外，我会想到协调返回的值。你的方法应该返回错误还是文件名？到目前为止还不是很干净。

2. 您有关move_uploaded_file（）的问题似乎与权限相关。检查您的源（tmp上传）目录和目的地的权限。