signup page created using pdo php it gives error

Question

I am learning oops concepts in php. I created Login/Signup/dbconfig pages using PHP PDO. DBconfig.php is working well but signup page gives below error.

Fatal error: Uncaught Error: Call to undefined method dBase::prepare() in D:\xampp\htdocs\phpoops\signup.php:26 Stack trace: #0 D:\xampp\htdocs\phpoops\signup.php(20): signUp->register() #1 D:\xampp\htdocs\phpoops\index.php(7): signUp->__construct(Object(dBase)) #2 {main} thrown in D:\xampp\htdocs\phpoops\signup.php on line 26

I created same code in procedural manner it works well. How can I solve this problem?

Here is my code:

Index.php

<?php  
    require("dbconfig.php");
    $db = new dBase();
    if(isset($_POST['submit'])){
    require_once("signup.php");
    $reg = new signUp($db);
    $reg->register();
   }
?>

<form action="" method="post">          
      <div class="top-row">
        <div class="field-wrap">
          <label>
            First Name<span class="req">*</span>
          </label>
          <input type="text" name="ufname" required autocomplete="off" />
        </div>        
        <div class="field-wrap">
          <label>
            Last Name<span class="req">*</span>
          </label>
          <input type="text" name="ulname" required autocomplete="off"/>
        </div>
      </div>
      <div class="field-wrap">
        <label>
          Email Address<span class="req">*</span>
        </label>
        <input type="email" name="uemail" required autocomplete="off"/>
      </div>          
      <div class="field-wrap">
        <label>
          Set A Password<span class="req">*</span>
        </label>
        <input type="password" name="upass" required autocomplete="off"/>
      </div>          
      <button type="submit" name="submit" class="button button-block"/>Get Started</button>          
      </form>

dbconfig.php

<?php 
class dBase
{
protected $uname;
protected $upass;
protected $host;
protected $dbname;
private   $dbcon;

function __construct()
{
     $this->dbcon  = false;
     $this->dbname = "oopsdb";
     $this->uname  = "root";
     $this->upass  = "";
     $this->host   = "localhost";
     //$this->connect();
     if(!$this->dbcon){
        try{
            $this->dbcon = new PDO('mysql:host='.$this->host.';dbname='.$this->dbname.'',$this->uname,$this->upass);
            $this->dbcon->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        }
        catch(PDOException $e){
            echo $e->getMessage();
            die();
        }
    }
    return $this->dbcon;    
 }  
}

Signup.php

<?php 
   class signUp
  {
   private $fname;
    private $lname;
   private $email;
   private $pass;
    private $db;
   function __construct($db)
   {    
    $this->fname = htmlspecialchars($_POST['ufname']);
    $this->lname = htmlspecialchars($_POST['ulname']);
    $this->email = htmlspecialchars($_POST['uemail']);  
    $this->pass = htmlspecialchars($_POST['upass']);
    $this->db = $db;
    $this->register();
}

function register(){        
    if($_SERVER["REQUEST_METHOD"] == "POST"){
    $sql = "INSERT INTO user(`fname`,`lname`,`email`,`pass`,`usr_update`) VALUES (:fname,:lname,:email,:pass,NOW())";
    $stmt = $this->db->prepare($sql);
    $stmt->bindParam(':fname',$this->fname);
    $stmt->bindParam(':lname',$this->lname);
    $stmt->bindParam(':email',$this->email);
    $stmt->bindParam(':pass',$this->pass);
    $stmt->execute();
    if($stmt == true){
        echo "<script>alert('Successfully Inserted');</script>";
    }else{
        echo "<script>alert('Not Inserted...please check logic once');</script>"; 
    }   

    }
  }
}

get rid of dBase class and go with raw PDO. Everything will be OK — Your Common Sense, Jan 10 '17 at 14:58
`$this->db->prepare($sql);` is calling your class `dBase`, which doesn't have any method called `prepare`. The error message says it all. — M. Eriksson, Jan 10 '17 at 14:59
your dBase class doesn't have that prepare() when you call it with `$stmt = $this->db->prepare($sql);` — Ardit Meti, Jan 10 '17 at 14:59
See this answer: http://stackoverflow.com/questions/130878/global-or-singleton-for-database-connection — Crouching Kitten, Jan 10 '17 at 15:21

Your Common Sense · Accepted Answer · 2017-01-10T15:18:36.070

-1

I am learning oops concepts in php.

That's fine.
However, OOP is not stands for creating as much classes as possible. Create a new class only if you have an idea what it should be used for.
For the moment, your dBase class doesn't do anything that vanilla PDO can't do. So just use PDO instead.

dbconfig.php

 <?php

 $dbname = "oopsdb";
 $uname  = "root";
 $upass  = "";
 $host   = "localhost";
 $charset = "utf8";

 $db = new PDO("mysql:host=$host;dbname=$dbname;charset=$charset",$uname,$upass);
 $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

then remove $db = new dBase(); from index and keep the rest as is

edited Jan 10 '17 at 15:18

answered Jan 10 '17 at 15:03

Your Common Sense

156,878
40
214
345

Thank..it works. I don't understand what is my mistake. – muraliniceguy97 Jan 10 '17 at 15:25
dBase class simply doesn't have a prepare method. – Your Common Sense Jan 10 '17 at 15:51

signup page created using pdo php it gives error

1 Answers1