0

私はPHP oopのことは初めてです。

クラスデータベースを作成し、そこから他のクラスを呼び出そうとしています。私はそれを正しい方法でやっていますか？

クラス データベース:

class database extends mysqli {

private $classes = array();

public function __construct() {
parent::__construct('localhost', 'root', 'password', 'database');
    if (mysqli_connect_error()) {
    $this->error(mysqli_connect_errno(), mysqli_connect_error());
    }
}

public function __call($class, $args) {
    if (!isset($this->classes[$class])) {
    $class = 'db_'.$class;
    $this->classes[$class] = new $class(); 
    }
return $this->classes[$class];
}

private function error($eNo, $eMsg) {
die ('MySQL error: ('.$eNo.': '.$eMsg);
}

}

クラス db_users:

class db_users extends database {

public function test() {
echo 'foo';
}

}

そして、私はそれをどのように使用していますか

$db = new database();
$db->users()->test();

それは正しい方法ですか、それとも別の方法で行うべきですか?

ありがとうございました。

4

2 に答える 2

2

あなたはそのようにそれを行うことができます、それは何も悪いことではありません (私はよく似たようなことをします)。私が提案する唯一のことは、ダイの代わりに例外を使用することです（そうすれば、エラーを安全に処理できます）...

protected function error($eNo, $eMsg, $extra = '') {
    throw new Exception('MySQL error: ['.$eNo.'] '.$eMsg.': '.$extra);
}

さらに、クエリメソッドもオーバーロードすることをお勧めします

public function query($sql, $result_mode = MYSQLI_STORE_RESULT) {
    $result = parent::query($sql, $result_mode);
    if ($result === false) {
         $this->error($this->errno, $this->errstr, $sql);
    }
    return $result;
}

$db オブジェクトのコピーを子クラス内に格納することもお勧めします。そう：

class db_users extends database {
    protected $db = null;

    public function __construct(Database $db) {
        $this->db = $db;
    }

    public function test() {
        echo 'foo';
    }
}

次に、__call で:

if (!isset($this->classes[$class])) {
    $class = 'db_'.$class;
    $this->classes[$class] = new $class($this); 
}
于 2010-05-19T13:25:57.977 に答える
1

クラスを作成するためのこのファクトリ スタイルには何の問題もありません。その中に少し例外処理を入れます。

私の他の唯一の懸念は、サブクラスでデータベースを拡張することです。

したがって、次のように変更します。

public function __call($className, $args) {
    if (!isset($this->classes[$class])) {
        if(include_once('db_'.$class)) {
            $class = 'db_'.$class;
            $this->classes[$class] = new $class($this);
        } else {
            throw new Exception("Db class not found");
        }
    }

    return $this->classes[$class];
}

そしてユーザークラスは次のとおりです。

public class db_users {
    private $db;

    public __constructor($db) {
        $this->db = $db;
    }

    public function test() {
        return 'Foo';
    }
}
于 2010-05-19T15:32:55.037 に答える