OOPHPデータベース接続スクリプトの入力

Question

私はOOPHPを学んでいます。経験豊富なスクリプト作成者からの質問をしたかっただけです。このスクリプトはうまくいきますが、私が最初からすぐにそれを学ぶなら、より良いことです。

これは私のconnect.php

<?php 
include ("classes/mysql.class.php"); 

$mysql = new Connect(); 
$mysql->mysql_Connect('localhost', 'root', 'root', 'test');?> 

であり、これは任意のコメントはofcourseのウェルコムある私のmysql_class.php

<?php 
class Connect { 
    var $serverhost, $username, $password, $db_name; 

    public function mysql_Connect($host, $user, $pass, $db) { 
     $this->serverhost = $host; 
     $this->username = $user; 
     $this->password = $pass; 
     $this->db_name = $db; 

     @mysql_connect($this->serverhost, $this->username, $this->password) or die("Kan geen verbinding maken met de database!"); 
     @mysql_select_db($this->db_name) or die("Kan geen verbinding maken met de geselecteerde database naam"); 
    } 
}?> 

です！

Answer 1

私の最初の助言は、ホイールを再開発しないことです.PHPはすでに、PDOやMySQLi拡張のクラスバージョンなど、いくつかのデータベースインタフェースオブジェクトクラスを提供しています。

@を使用してエラーメッセージを表示しないようにすることもお勧めします。スクリプトをデバッグする必要がある場合に後で戻ってきて、スクリプトのパフォーマンスに悪影響を及ぼします。

あなたのOOPへのアプローチがかなり狭いので、まだコメントすることはあまりありません。あなたのクラスのプロパティを宣言するためにvarを使用するのではなく、適切な可視性を設定する代わりにpublic、protectedまたはprivateを使用することをお勧めします。 varキーワードはPHP 4への後戻りであり、最終的に削除される可能性があります。また、これは公開のエイリアスです。一般的な経験則として、パブリックプロパティを必要とせず、パブリックプロパティを保護または非公開にし、getterおよびsetterを実装して他のクラスにアクセスできるようにしたいとします。これは、パブリックプロパティを持っている場合、オブジェクトの内部状態を壊すコードの別のpiexeを停止するものは何もないからです。カプセル化とデータ隠蔽はOOPの重要な概念ですので、それらを調べることをお勧めします。

Answer 2

@を使用してエラーを抑制することは、実際には良いことではありません。 はまた、クラスの金型の使用も本当に悪い習慣（それはPHPUnitでコードがテスト可能ではない可能）で、あなたの代わりに

http://www.php.net/manual/en/language.exceptions.php

PHPはすでにデータベースを管理するために、このようにいくつかのクラスを持っている例外を使用する必要があります。

あなたのクラスは本当に簡単なので、なぜ

http://www.php.net/manual/en/class.mysqli.php

のMysqliクラスを使用しないPDO WICH私はお勧め： http://ca2.php.net/manual/en/class.pdo.php

データベース接続を管理します。機能を追加したい場合は、PDOまたはMysqliをサブクラス化することができます。