اشكال اين كلاس چيه از لحاظ اصولي و ...

[Vahid Faraji]

اين يك كلاسه براي وصل شدن به database. به die توجه كنيد اگه مشكلي پيش بياد من error رو چاپ مي كنم در ضمن من از error_reporting(0); استفاده كردم تا كاربر هيچ error ي نبينه. بحث اين كه تابعي بنويسم تا كوئري رو به كلاس بدم باشه برا بعد.
مي خوام بدونم كد از لحاظ اصولي درسته يا نه
اين كه از die استفاده شده براي كنترل خطا
اين كه از error_reporting استفاده شده تا كاربر خطاها را نبينه
متد سازنده و open و close و مخرب چطور، مشكلي نداره
روشي كه از كلاس شي ساخته شده و متداش فراخواني شده، مشكلي نداره
ممنون از شما


<?php
error_reporting(0);
class db
{
private $hostname;
private $username;
private $password;
private $database;
private $connect;
private $select_db;

function __construct($hostname, $username, $password, $database)
{
$this->hostname = $hostname;
$this->username = $username;
$this->password = $password;
$this->database = $database;
}

public function open_connection()
{
$this->connect = mysql_connect($this->hostname,$this->username,$this->password) or die(error);
$this->select_db = mysql_select_db($this->database) or die(error);
}

public function close_connection()
{
mysql_close($this->connect);
}

function __destruct()
{
$this->close_connection();
}
}
?>




<?php
require_once('config.php');
require_once('connectToDB.php');

$db = new db(dbhost, dbuser, dbpass, dbname);
$db->open_connection();

$query = "SELECT * FROM users";
$sql = mysql_query($query);
if($sql)
{
while($r = mysql_fetch_array($sql))
{
echo $r['username'] . '<br />';
}
}
mysql_free_result($result);
$db->close_connection();
?>

[rapidpich]

اینکه error رو نشون نمیدی چیز خوبیه!
ولی die بده باید exception بندازی

[Vahid Faraji]

ولی die بده باید exception بندازی

die دادم كه. exception بندازم؟