[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

[wookieb]

Wszystkiego.
Ziomuś co my mamy oceniać? Wpakowanie na siłę w obiekt mysql_query i mysql_connect? Co mamy ocenić?
Pomysł? brak
Jakość? brak
Czy jesteś pewien, że wykonanie dwóch dodatkowych zapytań po "set names" jest potrzebne czy wyczytałeś tak "w necie" ?
Ludzie nie chwalcie się takimi rzeczami bo nie ma czym, a jedynie robicie śmietnik!!!

Zresztą PDO oferuje wystarczająco dużo, żeby takie klasy jak ta nie powstawały.
Ciekawostka przy 10 000 wywołaniach pdo przegrywa z mysql o jedna sekunde. A jak mysql opakujesz w klase to nie zrobi roznicy.

Ten post edytował wookieb 21.05.2010, 14:40:06