بهتره برای رعایت استانداردها و جلوگیری از بروز خطاهای ناخواسته در ویرایشهای بعدی کدها (ازقبیل اضافه شدن دستورات بلاک if و...)، Bracket ها رو هم حذف نکنید. حتی اگه یک دستور دارین.
از REQUEST_$ هم استفاده نکنید و مشخص کنید که با GET میخواین کار کنین یا POST
درمورد خطای این کد هم:
$this->link= @ mysql_connect($host, $user, $pass);
if(!$this->link) {
$this->error();
return;
}
بهتره اینطوری کار کنید:
class DB
{
private $link;
public function connect($host, $user, $pass, $db)
{
if(!$this->link) {
if(!($this->link = mysql_connect($host, $user, $pass))) {
$this->error('Connection error');
}
elseif(!mysql_select_db($db, $this->link)) {
$this->error('Database error');
}
else {
mysql_query('SET NAMES \'utf8\'');
}
}
}
public function query($query)
{
$this->connect();
return mysql_query($query, $this->link);
}
// ...
}