[MYSQL][PHP] Wysyłanie zapytania do mysql Opcje

[Koldy]

Siemanko mam takie pytanie: Czy to jest poprawne? w sensie takim czy tak się używa.

[PHP] pobierz, plaintext 
$ile = mysql_num_rows(mysql_query("SELECT cid FROM comments WHERE post='$jid'"));
[PHP] pobierz, plaintext 

Jak najlepiej robić zapytania i je wysyłać, bo jak tak sobie robie:

[PHP] pobierz, plaintext 
$q = "SELECT * FROM comments, users WHERE comments.user = users.uid AND post='$jid' ORDER BY cid DESC";
		$qid = mysql_query($q);
		$ile = mysql_num_rows($qid);
[PHP] pobierz, plaintext 

to się trochę syf robi, jak dbacie o to by wasz kod był przejżysty przy powtarzających się rzeczach?
podzielcie się (IMG:style_emoticons/default/smile.gif)

[Sephirus]

Nikt Cię nie rozniesie (IMG:style_emoticons/default/smile.gif)

Ale oczywiście to jest zły pomysł w dokładnie takiej formie... dlaczego? Bo każdorazowo łączy się z bazą...

Singleton jest powszechnie używany i sprawdzony (IMG:style_emoticons/default/smile.gif)

Co prawda można przerobić tą funkcję tak by to miało sens - ale nie ma co odkładać na bok sprawdzonych wzorców ;/

A dla tych co mają problem z obsługą wielu różnych połączeń do bazy i singletonem - mam dwa słowa - "fabryka singletonów" (IMG:style_emoticons/default/tongue.gif) - dziękuję (IMG:style_emoticons/default/smile.gif)

Zresztą wydaje mi się że każdy prosty DBAL jest zbudowany wstępnie w oparciu o singleton (IMG:style_emoticons/default/smile.gif)

Przykład pseudo fabryki singletonów do obsługi bazy danych:

[PHP] pobierz, plaintext 
class Db
{
    public static $connections = array();
	private static $instances = array();
 
	public static getInstance($connectionName = 'default')
	{
		$data = self::$connections[$connectionName];
 
		if(!isset(self::$instances[$connectionName]))
			self::$instances[$connectionName] = new PDO('mysql:host='.$data->host.';dbname='.$data->dbname, $data->user, $data->password);
		return self::$instances[$connectionName];
	}
 
	public function addConnection($host,$user,$password,$dbname,$connectionName = 'default')
	{
		self::$connections[$connectionName] = new stdClass;
		self::$connections[$connectionName]->host = $host;
		self::$connections[$connectionName]->user = $user;
		self::$connections[$connectionName]->password = $password;
		self::$connections[$connectionName]->dbname = $dbname;
	}
}
 
// przykład użycia:
 
// ustalamy dane do domyślnego połączenia:
Db::addConnection('localhost','user','pass','tabelka');
// i do drugiego:
Db::addConnection('123.123.123.123','user2','pass2','tabelkaX','drugie');
 
 
$db = Db::getInstance(); // pobieramy domyślne połączenie
 
// potem gdzie indziej możemy się do drugiego odwołać poprzez:
$db = Db::getInstance('drugie');
 
// itd...
 
[PHP] pobierz, plaintext 

EDIT: oczywiście przedstawiłem sam koncept bez obsługi wyjątków i sprawdzania błędów (IMG:style_emoticons/default/smile.gif)

Ten post edytował Sephirus 31.01.2012, 12:33:22