2015-05-19 59 views
1

我正在尋找顯示用戶錯誤的最佳方式,我使用的是Class File()和個人檔案頁面。類文件()將處理以下事項: - 檢查文件的擴展名 - 檢查文件是否已經存在 - 爆炸並創建鏈接文件& &插入鏈接到數據庫 - 上傳文件到文件夾項目文件夾目錄PHP OOP處理UI錯誤通知

代碼工作正常,除了錯誤顯示,例如,如果擴展名是錯誤的,它將顯示最後一個錯誤,而不是停止執行並顯示擴展錯誤。

此外,如果文件已經存在於數據庫& &文件夾目錄下的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; 
} 

回答

0

兩個單獨的問題:

  1. 如果您發現任何錯誤,你的checkFile()方法,你想只得到了第一個錯誤(這是從我的角度來看奇數;你可以在數組中收集錯誤),你應該直接返回該錯誤,而不是繼續執行其餘的方法。此外,我會想到協調返回的值。你的方法應該返回錯誤還是文件名?到目前爲止還不是很乾淨。

  2. 您有關move_uploaded_file()的問題似乎與權限相關。檢查您的源(tmp上傳)目錄和目的地的權限。

+0

1.是的,我的Class File()可能比這更好的編碼,所有的建議是值得歡迎的。對於錯誤,我希望代碼在檢測到錯誤後立即停止執行,並返回&&如果沒有錯誤返回文件名。 2.因此,如果權限問題,爲什麼我仍然可以上傳文件,如果這個文件不存在? – Nicks

+0

對於1.,只返回錯誤。這很基本。對於2.,你正在移動你的文件,這意味着複製和刪除。也許你不能刪除tmp上傳的文件。爲了解決這個問題,你可以使用copy(),然後使用tmp上傳文件的unlink()。 – jossif