Klasa do połączeń i zapytań SQL Opcje

[Zagiewa]

Witam. Chcę stworzyć klasę do połączenia SQL w której będą też metody do zapytań jednak zależy mi na stworzeniu bardzo wydajnej klasy, a co więcej, żeby cały kod był dobrej jakości dlatego też ja mam pomysł i jakiś zalążek wiedzy, wy za to duże doświadczenie więc liczę, że mi trochę pomożecie (IMG:style_emoticons/default/smile.gif) Głównym motywem na chwile obecną jest stworzenie metod, które będą generowane w zależności od tego co będziemy chcieli osiągnąć. Poniżej zamieszczę kod, który do tej pory napisałem, a pod nim pytania.

[PHP] pobierz, plaintext 
<?php
 
class connection {
 
	private $DB;
 
	function __construct($DB_in) {
 
		$this->DB = $DB_in;
	}
 
	public function SQL_connect() {
 
		include('static.php');
 
		$sql = mysql_connect(SQL_HOST, SQL_USER, SQL_PASS) or
		die("Sprawdź połączenie z serwerem! " . mysql_error());
		mysql_query("SET NAMES 'utf8'");
 
		mysql_select_db($this->DB, $sql) or
		die("Sprawdź bazę! " . mysql_error());
	}
 
	public function select_table($what_in, $from_in) {
 
		$pobierz = "SELECT $what_in FROM $from_in";
		$result = mysql_query($pobierz) or
		die("Sprawdź rezultat! " . mysql_error());
 
		while($row = mysql_fetch_array($result)) {
 
			$a = $row[$what_in];
			echo $a. "<br />";
		}
	}
}
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
	include('class.php');
	$connect = new connection("nazwa_bazy");
	$connect->SQL_connect();
	$connect->select_table("id", "nazwa_tabeli");
 
?>
[PHP] pobierz, plaintext 

1. Chciałem aby można było za pomocą jednej metody łączyć się z różnymi bazami dlatego zanim wywołamy metodę SQL_connect() dzięki konstruktorowi podajemy, która baza nas interesuje. Teraz pytanie czy rozwiązanie to jest dobre? Czy może lepiej było by usunąć konstruktor i nazwę bazy podać bezpośrednio do metody czyli: SQL_connect("nazwa_bazy")?
2. Jak wiadomo zanim wykonamy jakiekolwiek operacje na tabelach w bazie trzeba się połączyć z bazą więc łączenie z SQL MUSI się odbyć prawda? więc zastanawiam się czy by nie zrezygnować z metody SQL_connect() i całość kodu wstawić w konstruktor wyglądało by to tak:

[PHP] pobierz, plaintext 
function __construct($DB_in) {
 
		$this->DB = $DB_in;
 
		include('static.php');
 
		$sql = mysql_connect(SQL_HOST, SQL_USER, SQL_PASS) or
		die("Sprawdź połączenie z serwerem! " . mysql_error());
		mysql_query("SET NAMES 'utf8'");
 
		mysql_select_db($this->DB, $sql) or
		die("Sprawdź bazę! " . mysql_error());
	}
[PHP] pobierz, plaintext 

Ma to sens? Mam jeszcze pytania odnośnie metody select_table() ale to innym razem. Jeśli macie ew. jakieś zastrzeżenia co do tego kodu to piszcie. Z góry dzięki (IMG:style_emoticons/default/smile.gif)

[Zagiewa]

Dziś już jestem zbyt zmęczony na dalsze pisanie kodu więc wolę jutro się tym zająć ale odnośnie tego co napisaliście:

mg2 - "Z bazą łączysz się, jeżeli chcesz wykonać zapytanie a nie jesteś z nią połączony;)" masz racje, źle to sformułowałem.
Napisałeś też abym nie robił klas tylko bezpośrednio korzystał z PDO masz na myśli żebym wklepywał PDO bezpośrednio w kod wszędzie tam gdzie będę potrzebował wysłać zapytanie do bazy? jeśli tak to takie rozwiązanie jest myślę mało efektowne bo gdy okaże się, że mam błąd to będę musiał uruchamiać "x" plików w których użyłem PDO aby poprawić błąd. Gdy będzie on w klasie bądź nawet funkcji którą includuje wystarczy, że zmienię raz.

"... Już chyba lepiej byłoby te dane bazy podać konstruktorowi tej klasy jako argumenty." - czy nie jest bezpieczniej gdy one leżą sobie gdzieś w pliku jako stałe?

darko - "Dziedzicz po klasie PDO dodając jakieś pożądane funkcjonalności, a nie robisz kompozycję." Nie bardzo rozumiem co masz na myśli. A co do mysql_real_escape_string() to by tłumaczyło dlaczego wyskakiwał mi ten błąd (IMG:style_emoticons/default/smile.gif)