捕錯誤 - 參數必須是PDO

Question

的實例

嘿，大家好我得到這個錯誤：

Catchable fatal error: Argument 1 passed to Users::__construct() must be an instance of PDO, none given, called in C:\xampp\htdocs\xampp\ooplogin\register.php on line 46 and defined in C:\xampp\htdocs\xampp\ooplogin\class\Users.php on line 9 

說真正的代碼工作，沒有問題，但你知道，如果我想使乾淨的代碼它shouldbt是有任何錯誤。

我的代碼：

<?php 
class Users { 
    /* Deklarovanie premennych */ 
    public $name = null; 
    public $email = null; 
    public $phone = null; 
    public $message = null; 

    public function __construct(PDO $db_con) 
    { 
     $this->db_con = $db_con; 
    } 

    public function storeFormValues($data) 
    { 
     if(isset($data['name'])) $this->name = stripslashes(strip_tags($data['name'])); 
     if(isset($data['email'])) $this->email = stripslashes(strip_tags($data['email'])); 
     if(isset($data['phone'])) $this->phone = stripslashes(strip_tags($data['phone'])); 
     if(isset($data['message'])) $this->message = stripslashes(strip_tags($data['message'])); 
     return $this; 
    } 

    public function message() 
    { 
     $correct = false; 
     try { 
      $sql = "INSERT INTO user(name, email, phone, message) VALUES (:name, :email, :phone, :message)"; 

      $stmt = $this->db_con->prepare($sql); 
      $stmt->bindValue("name", $this->name, PDO::PARAM_STR); 
      $stmt->bindValue("email", $this->email, PDO::PARAM_STR); 
      $stmt->bindValue("phone", $this->phone, PDO::PARAM_STR); 
      $stmt->bindValue("message", $this->message, PDO::PARAM_STR); 
      $stmt->execute(); 
      return 'Sprava bola uspesne odoslana!'; 
     }catch(PDOException $e) { 
      return $e->getMessage(); 
     } 
    } 

    public function displayAll() 
    { 
     try{ 
      $sql = "SELECT * FROM users LIMIT 10"; 
      $stmt = $this->db_con->prepare($sql); 
      $stmt->execute(); 
      return $this->$stmt;  
     }catch(PDOException $e) { 
      return $e->getMessage(); 
     } 
    }  
} 

try { 
    $db_con = new PDO(DB_HOST, DB_USER, DB_PASS); 
    $db_con->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
    $db_con->setAttribute(PDO::ATTR_EMULATE_PREPARES, false); 
}catch(PDOException $e) { 
    return $e->getMessage(); 
} 


$Users = new Users($db_con); 
$Users->storeFormValues($_POST); 

echo $Users->message(); 
?> 

而且我的index.php樣子：

<?php if(!(isset($_POST['send']))) { ?> 

HTML FORM HERE 

<?php 
} else { 
    $Users = new Users; 
    $Users->storeFormValues($_POST); 
    if($Users->message()) { 
     echo "Sprava bola úspešne odoslana";  
    } else { 
     echo "Sprava nebola odoslana";  
    } 
} 

?> 

這裏是一個機會，有人可以幫我解決這個問題或解釋爲什麼它不工作？謝謝！

Answer 1

你沒有像傳遞錯誤信息那樣傳遞PDO實例。

$Users = new Users; 

編輯：所有的 首先，不要複製/粘貼代碼，並期望它的工作。我會告訴你該怎麼做，但不會給你任何代碼。

當您在構造函數中使用參數創建一個類（沒有默認值）時，您必須傳遞參數才能使其工作。這就是你得到這個錯誤的原因。我向你展示的是發生錯誤的地方。用戶類需要一個PDO對象，但你什麼都沒有傳過。雖然你在類文件做到了：

$Users = new Users($db_con); 

五言你需要，你還需要爲你在你的類文件做初始化$ db_con變量。

這將解決您的問題。我也有一些建議。

1. 把你的每個類放在一個單獨的文件中，不要在你的類中執行任何代碼在這些文件中。
2. 將您的連接（$ db_conn初始化）放入一個單獨的文件中，以便您可以在需要時再次調用它。
3. 見include和require功能，以瞭解如何從其他文件中的代碼申請1和2

這些都是很基本的建議。有更復雜但更好的方法來做同樣的事情，比如自動加載和IoC容器。但是當開始編碼時，爲了您的理智，您可以留下這些概念以便後續學習。 :)