PDO Insert-Anweisung mit Variablen

Question

Ich versuche, eine Insert-Anweisung über PDO mit Variablen zu erstellen, aber ich immer Fehler folgende

Fatal error: Uncaught Error: Call to a member function prepare() on null in /Applications/XAMPP/xamppfiles/htdocs/Jobber/user.php:9 Stack trace: #0 /Applications/XAMPP/xamppfiles/htdocs/jobber/usercheck.php(3): Users->__construct('steve', 'testt@test.com', 'test', 1) #1 {main} thrown in /Applications/XAMPP/xamppfiles/htdocs/Jobber/user.php on line 9 

dies ist der Code, den ich für sie

mit

class Users{ 
     private $connect ; 

     function __construct($name,$email,$password,$usertype){ 
      include("connect.php"); 
      $startDb = new Connect(); 
      $this->connect = $startDb->getDb(); 
      $sqlquery = $this->connect->prepare("INSERT INTO `user`(`Username`, `Email`, `Active`, `Password`, `usertype`) VALUES (" + $name + "," +$email +",0," + $password + "," + $usertype + ")"); 
      $sqlquery->execute(); 
     }  
    } 

Gibt es ein Problem mit der Verbindung oder ist es etwas anderes mit den Variablen?

dies ist die Verbindungsklasse

class Connect{  
     private $host = "localhost:8080"; 
     private $user = "root"; 
     private $password = "test"; 
     private $dbName = "jobber"; 
     private $db; 

     function __construct(){ 
      try{ 
       $connection = new PDO('mysql:host=localhost;dbname=jobber',$this->user,$this->password); 
      }catch(PDOException $ex){ 
       echo $ex; 
      } 
     } 

     function getDb(){ 
      if($this->db instanceof PDO){ 
       return $this->db; 
      } 
     } 
    } 

Answer 1

In Ihrem Connect Klasse, Sie haben private $db; erklärt, und Sie kehren $this->db im getDb() Methode, aber Sie haben es nie ein Wert zugewiesen. Es scheint, dass Sie das wahrscheinlich im Konstruktor tun wollten.

Statt

$connection = new PDO('mysql:host=localhost;dbname=jobber',$this->user,$this->password); 

Verwendung

$this->db = new PDO('mysql:host=localhost;dbname=jobber',$this->user,$this->password); 

---

In Ihrer Users Klasse, ein richtiger Weg, um die vorbereitete Anweisung zu verwenden, wäre eine SQL-Anweisung mit Platzhalter vorzubereiten. Sie können dann entweder binden Ihre Variablen in die vorbereitete Anweisung manuell mit bindValue, oder sie in die execute Methode in einem Array übergeben wie folgt aus:

$sqlquery = $this->connect->prepare("INSERT INTO `user` 
     (`Username`, `Email`, `Active`, `Password`, `usertype`) VALUES (?, ?, 0, ?, ?)"); 
$sqlquery->execute([$name, $email, $password, $usertype]); 

aber, um ehrlich zu sein, die Tatsache, dass dies in geschieht in der Konstruktor scheint mir ziemlich seltsam.