كلاس connection براي وصل شدن به mysql

من يه كلاس connection براي وصل شدن به mysql به صورت زير نوشتم
کد:
<?php
error_reporting(0);
class db
{
     private $hostname;
     private $username;
     private $password;
     private $database;
     private $connect;
     private $select_db;
 
     public function db()
     {
          $this->hostname = dbhost;
          $this->username = dbuser;
          $this->password = dbpass;
          $this->database = dbname;
     }
      
     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);
     }
}
?>
dbhost, dbuser, dbpas, dbname, error
از فايل config خونده مي شن
به صورت زير هم استفاده مي كنم
کد:
<?php
require_once('config.php');
require_once('connectToDB.php');
    
function pageInit()
{
    global $db;
    $db = new db();
    $db->open_connection();
}

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

function pageTerminate() 
{
    global $db;
    $db->close_connection();
}
pageInit();
select();
pageTerminate();
?>
مي خواستم بدونم مشكلي وجود داره از لحاظ اصول برنامه نويسي، اگه نكته اي وجود داره ، لطف كنيد بيان كنيد.
 

sajadmaz

Member
فانکشن open_connection اررور های احتمالی که اگر نتونه وصل بشه رو سایلنت کن و براش شرط بزار هندل تره

و همینطور به جای اینکه از global استفاده کنی مقدار هایی رو که میخوایی global کنی بصورت آرگومان به فانکشنت بده

.....
 
فانکشن open_connection اررور های احتمالی که اگر نتونه وصل بشه رو سایلنت کن و براش شرط بزار هندل تره
من از die استفاده کردم، شما منظورتون چیز دیگه ای، می شه بگین کدش چیه؟
همینطور به جای اینکه از global استفاده کنی مقدار هایی رو که میخوایی global کنی بصورت آرگومان به فانکشنت بده
منظورتون اینه که از pageinit استفاده نکنم و از کلاس db رو تو تابع select، شی بسازم می شه روشن تر بگید.
 

جدیدترین ارسال ها

بالا