[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

[redelek]

Hej,

Ja uparcie obiektowo, poprawiłem to na co zwróciliście mi uwagę i wygląda to tak
Proszę zobaczcie i powiedźcie czy jest dobrze lub czego brakuje

[PHP] pobierz, plaintext 
class mMySQL
{
private $polacz_db;	//połączenie do serwera
private $dbuzytkownik = "root1"; //uzytkownik nazy danych
private $dbhaslo = ""; // jego hasło
private $dbnazwa = "bz_txt"; //nazwa bazy
private $dbserwer = "localhost"; //nazwa serwera
private $dbwynik;	// wynik zapytania
 
public function __construct() {
	$this->polacz_db = @mysql_connect($this->dbserwer,$this->dbuzytkownik,$this->dbhaslo);
	if(!($this->polacz_db)) {
		return false;
	}
	else {
			mysql_query("SET NAMES 'utf8'");
			mysql_query("SET CHARACTER_SET_CLIENT=utf8");
			mysql_query("SET CHARACTER_SET_RESULTS=utf8");
			mysql_select_db($this->dbnazwa);
 
                        return true;
	}
}
 
public function __destruct() {
	@mysql_close($this->polacz_db);
}
 
public function query($sql) {
	$this->dbwynik = mysql_query($sql);
	return $this->dbwynik;
}
 
}
[PHP] pobierz, plaintext 

Będę wdzięczny za kolejne uwagi

Ten post edytował redelek 21.05.2010, 14:35:26