Я пытаюсь создать простой в использовании одноэлементный класс для подключения к базе данных mysql и выполнения запросов, код работает нормально, и у меня не было никаких проблем с ним, но, поскольку я новичок в ООП, яИнтересно, это плохая практика или нет.
Вот класс
class Database {
private $databaseName = 'dbname';
private $host = 'localhost';
private $user = 'user';
private $password = 'pass';
private static $instance; //store the single instance of the database
private function __construct(){
//This will load only once regardless of how many times the class is called
$connection = mysql_connect($this->host, $this->user, $this->password) or die (mysql_error());
$db = mysql_select_db($this->databaseName, $connection) or die(mysql_error());
echo 'DB initiated<br>';
}
//this function makes sure there's only 1 instance of the Database class
public static function getInstance(){
if(!self::$instance){
self::$instance = new Database();
}
return self::$instance;
}
public function connect() {
//db connection
}
public function query($query) {
//queries
$sql = mysql_query($query) or die(mysql_error());
return $sql;
}
public function numrows($query) {
//count number of rows
$sql = $this->query($query);
return mysql_num_rows($sql);
}
}
//Intantiate the class
$database = Database::getInstance();
, и когда я захочу использовать класс, я сделаю:
$query = "SELECT * FROM registrations";
echo $database->numrows($query);
$sql = $database->query($query);