[MySQL][PHP]Polaczenie z mysql OOP Opcje

[redelek]

Witam,

Przerobiłem troszkę materiału i proszę o poradę. Zanim się napiszę i potem wszystko będzie źle (IMG:style_emoticons/default/smile.gif)
Napisałem klasę do łączenia się z mysql i nawet działa. Pytanie mam takie czy jest to dobrze napisane ?
Czy na coś zwrócić uwagę i gdzieś mam jakieś błędy? które Wy widzicie ?

[PHP] pobierz, plaintext 
error_reporting(E_ALL);
ini_set('display_errors','1'); 
class mMySQL 
{
 
public $Serwer, $Baza, $Uzytkownik, $Haslo;
#-#############################################
# Nawiazywanie polaczenia z baza
function Polacz ($link=false) {
	$this->link_id=@mysql_connect($this->Serwer,$this->Uzytkownik,$this->Haslo,$link);
 
	if (!$this->link_id) {
		$this->blad("Nie mogę podłączyć się do serwera: <b>$this->Serwer</b>.");
		}
 
	if(!@mysql_select_db($this->Baza, $this->link_id)) {//no database
		$this->blad("Nie mogę wybrać bazy danych: <b>$this->Baza</b>.");
		}
	if(@mysql_select_db($this->Baza, $this->link_id)) {
		mysql_query("SET NAMES 'utf8'");
		mysql_query("SET CHARACTER_SET_CLIENT=utf8");
		mysql_query("SET CHARACTER_SET_RESULTS=utf8");
	}
}#-#Polacz()
#-#############################################
# zamykanie polaczenia z baza
function Zamknij() {
	if(!@mysql_close($this->link_id)){
		$this->blad("Nie można zamknąć połączenia z serwerem :(");
	}
}#-#zamknij()
#-#############################################
# komunikaty o błędach
function blad($msg='') {
	if($this->link_id>0){
		$this->error=mysql_error($this->link_id);
		$this->errno=mysql_errno($this->link_id);
	}
	else{
		$this->error=mysql_error();
		$this->errno=mysql_errno();
	}
	echo "<b>Błąd aplikacji</b><br />";
	echo "<hr>";
	echo "<b>Komunikat:</b>".$msg."<br />";
	echo "<b>MySQL błąd: </b>".$this->error."<br>";
	echo "<b>MySQL kod: </b>".$this->errno."<br>";
	echo "<br>";
}#-#blad()
 
}
## JAK TO DZIAŁA 
$SQL = new mMySQL();
$SQL->Serwer = 'localhost';
$SQL->Baza = 'mysql';
$SQL->Uzytkownik = 'root';
$SQL->Haslo = '';
$SQL->Polacz();
 
 
$SQL->Zamknij();
?>
[PHP] pobierz, plaintext 

Czy na przykład użyć protected Polacz; ?

Ten post edytował redelek 17.05.2010, 11:42:30

[lobopol]

No dla mnie to jest nieco dziwna klasa, czemu nie używasz konstruktorów/destruktorów i nie deklarujesz typów metod? Dodatkowo metody powinny zwracać jakieś wartości, a nie wyświetlać za pomocą echo czyli nie:

[PHP] pobierz, plaintext 
function metoda(){
echo "costam";
}
[PHP] pobierz, plaintext 

a

[PHP] pobierz, plaintext 
public function metoda(){
return $this->wartosc;
}
[PHP] pobierz, plaintext 

Dalej idą metody i zmienne do których nie powinno się mieć dostępu z zewnątrz (czyli login do bazy danych/hasło/itp.) powinny być ustawione na private albo protected (przy dziedziczeniu)


Sama klasa(prosta wersja) powinna wyglądać mniej więcej tak:

[PHP] pobierz, plaintext 
class Database{
    private $dbHandler;    
    public function  __construct($dbLogin, $dbPassword, $dbHost, $dbName) {
        $this->dbHandler = @mysql_connect($dbHost,$dbLogin,$dbPassword) or $this->sql_error(mysql_error());
        mysql_select_db($dbName,$this->dbHandler);
        mysql_query('SET NAMES utf8');
        return true;
    }
 
    public function query($query){
//tu kasowanie/update/dodawanie do sql jeżeli błąd to $this->sql_error(mysql_error());
        return true;
    }
 
    public function select($query){
//tu funkcja do wyciągania selecta z bazy danych
    }
 
    private function sql_error($error){
//tu funkcje raportowania błędów sql np. zapis do pliku nie powinno się raczej wyświetlać użytkownikom dokładnego błędu
die('Wystąpil problem z bazą danych.');
    }
   public function __destruct(){
//tu niszczenie połączenia bazy
}
 
}
a użycie:
 
$db = new Database($dbLogin, $dbPassword, $dbHost, $dbName); //tworzenie obiektu
$db->select($query); //zapytanie select do bazy
unset($db); //niszczenie obiektu
[PHP] pobierz, plaintext 

Ten post edytował lobopol 17.05.2010, 12:40:18