PHP:Mysqli_real_escape_string

Question

i'm having an issue with this piece of code and i cant see where the problem .

    <?php 
require_once("config.php");
class MySQLDatabase{
    private $connection;
    function _construct(){
        $this->open_connection();
    }
    public function open_connection(){
        $this->connection=mysqli_connect(DB_SERVER,DB_USER,DB_PASS,DB_NAME);
        if(!$this->connection){
            Die("Database Connection Problem : ".mysqli_connect_errno()." : ".mysqli_connect_error());
        }
        }

    public function close_connection(){
        if(isset($this->connection)){
            mysqli_close($this->connection);
            unset($this->connection);
        }
    }
    public function query($sql){
        $sql=mysql_prep($sql);
        $result=mysqli_query($this->connection,$sql);
        confirm_query($result,$sql);
        return $result;
    }
    public function mysqli_prep($string){
        global $connection;
        $safe_string=mysqli_real_escape_string($this->connection,$string);
        return $safe_string;
    }
    private function confirm_query($result,$sql){
        if(!$result){
            Die("Database query Problem : ".$sql);
        }
    }

}
$database=new MySQLDatabase();
$db=& $database;
 ?>

And i try to run the code bellow on Index.php

<?php
require_once("../includes/database.php");
echo "Is this working?";
echo $database->mysqli_prep("sss");
?>

But I receive this Error

Warning: mysqli_real_escape_string() expects parameter 1 to be mysqli, null given in \includes\database.php on line 29

You should learn to use prepared statements rather than escaping strings. — Barmar
– Barmar, Commented May 23, 2017 at 20:50

rickdenhaan · Accepted Answer · 2017-05-23 20:45:50Z

1

The error is saying that the $this->connection you use in MySQLDatabase::mysqli_prep() is null. That's because it was never initialized.

Your class constructor needs to start with two underscores instead of one:

function __construct(){
    $this->open_connection();
}

answered May 23, 2017 at 20:45

rickdenhaan

11.4k32 silver badges42 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

PHP:Mysqli_real_escape_string

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related