Forum PHP.pl

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)

1. Jeżeli pobierasz dane tylko po to aby zliczyć wiersze to używaj COUNT() zamiast mysql_num_rows()
2.

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

3. Tego typu rzeczy robi się za pomocą funkcji aby jak to już sam zauważyłeś nie brudzić kodu.

[PHP] pobierz, plaintext 
$select = $db -> fetchRow("SELECT *, COUNT(*) as count FROM comments, users WHERE comments.user = users.uid AND post='$jid' ORDER BY cid DESC");
echo $select['count'];
[PHP] pobierz, plaintext 

A w ogóle było by cudownie gdybyś przerzucił się na PDO.

Ten post edytował CuteOne 31.01.2012, 05:25:31

Ad. 2

Dla bardziej skomplikowanych (tak naprawdę po prostu dłuższych) zapytań osobiście polecam wyrzucenie zapytania poza ciało funkcji tak jak zaproponował autor:

[PHP] pobierz, plaintext 
$query = "SELECT a.costam1, a.costam2, a.costam3 FROM tabelka1 AS a " .
              "LEFT JOIN tabelka2 AS b ON a.id1 = b.id2 " .
              "WHERE a.jakies_pole LIKE 'to jest jakis tekst%' " .
              "ORDER BY a.jakies_pole ASC LIMIT 0,100";
 
$query = mysql_query($query);
 
// dalsze instrukcje
[PHP] pobierz, plaintext 

Dla dłuższych zapytań niezależnie od tego czy uzywa się funkcji z rodziny mysql_* czy PDO czy czegokolwiek innego (może poza ORM) to dobry zwyczaj bo kod staje się o wiele czytelniejszy a poprawne podzielenie zapytania na linie może jedynie pomóc w jego zrozumieniu przez osoby trzecie, które kiedyś mogą na dany kod zerknąć...

I teraz odnosząc się do tego co napisał mój przedmówca - weź sobie do serca tą radę z PDO Autorze - to raz a dwa:

mysql_num_rows() jak i jego odpowiednik w PDO czyli pdostatement::rowCount() służą owszem do pobrania liczby rekordów i można z nich korzystać ale tylko gdy pobierasz rekordy w celu ich wyświetlenia (wszystkich) a dodatkowo potrzebujesz znać ich liczbę... Tak jak napisał przedmówca jeśli potrzebujesz z bazy jedynie liczby rekordów spełniającej dane kryteria bardzo mile widziane jest COUNT()

(IMG:style_emoticons/default/wink.gif)

Ten post edytował Sephirus 31.01.2012, 08:52:24

Wziąć, wziąłem sobie do serca, bo od rana siedzę przy PDO, tyle że już jestem w pierwszym rowie..
mianowicie wcześniej mogłem wysyłać zapytania "globalnie", teraz nie widzę PDO w innych funkcjach, szukałem coś i znalazłem singleton, ale dużo osób to krytykuje..

więc jak? w każdej funkcji mam tworzyć nowe PDO?

[PHP] pobierz, plaintext 
$dbh = new PDO('mysql:host=localhost;dbname='.DB_NAME, DB_USER, DB_PASSWORD);
[PHP] pobierz, plaintext 

like that?

Ten post edytował Koldy 31.01.2012, 11:25:59

Hmm kto krytykuje singleton?? O_o

Zrób singleton - zwracający obiekt PDO już połączony i po krzyku... Nie wiem w czym problem

[PHP] pobierz, plaintext 
class MyPDO
{
      private static $instance = null;
 
      public static getInstance()
      {
            if(!(self::$instance instanceOf MyPDO)) self::$instance = new PDO('mysql:host=localhost;dbname='.DB_NAME, DB_USER, DB_PASSWORD);
            return self::$instance;
      }
 
      private __construct() {}
}
[PHP] pobierz, plaintext 

I tam gdzie potrzebujesz dajesz np:

[PHP] pobierz, plaintext 
$db = MyPDO::getInstance();
 
$st = $db->prepare('SELECT * FROM users WHERE id = ?');
$st->execute(array(123));
$user= $st->fetchObject();
 
// itd... gdzie tylko potrzebujesz bazy...
[PHP] pobierz, plaintext 

Ten post edytował Sephirus 31.01.2012, 11:38:57

Czytałem kilka opini gdzie narzekali na Singleton, że się nie da na kilku bazach pracować itp.

Pewnie teraz mnie rozniesiecie, ale:

[PHP] pobierz, plaintext 
    class MyPDO
    {
    private static $instance = null;
 
    public static getInstance()
    {
    if(!(self::$instance instanceOf MyPDO)) self::$instance = new PDO('mysql:host=localhost;dbname='.DB_NAME, DB_USER, DB_PASSWORD);
    return self::$instance;
    }
 
    private __construct() {}
    }
[PHP] pobierz, plaintext 

Czy zamiast tego mógłbym zrobić:

[PHP] pobierz, plaintext 
function db() {
	try {
		$dbh = new PDO('mysql:host=localhost;dbname='.DB_NAME, DB_USER, DB_PASSWORD);	
	} catch (PDOException $e) {
   		print "Error!: " . $e->getMessage() . "<br/>";
  		die();
	}
	return $dbh;
}
[PHP] pobierz, plaintext 

i to nie było by to samo?

E tam dupa. Moim zdaniem singleton jest czasem dobrym wyjściem, a co do połączenia singletona i kilku baz danych to Kohana 2 obsługuje właśnie taki mechanizm, więc jak widać da się.

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

Sephirus: nie zgodzę się z Tobą w kwestii wstępnego zapisu zapytania do zmiennej. Jaki to ma sens?

[PHP] pobierz, plaintext 
$query = "SELECT a.costam1, a.costam2, a.costam3 FROM tabelka1 AS a " .
              "LEFT JOIN tabelka2 AS b ON a.id1 = b.id2 " .
              "WHERE a.jakies_pole LIKE 'to jest jakis tekst%' " .
              "ORDER BY a.jakies_pole ASC LIMIT 0,100";
$select = mysql_query($query);
 
$select = mysql_query("SELECT a.costam1, a.costam2, a.costam3 
                                   FROM tabelka1 AS a 
                                   LEFT JOIN tabelka2 AS b ON a.id1 = b.id2
                                   WHERE a.jakies_pole LIKE 'to jest jakis tekst%' 
                                   ORDER BY a.jakies_pole ASC LIMIT 0,100");
[PHP] pobierz, plaintext 

?

Koldy: zastanów się czym jest singleton i kiedy się go wykorzystuje a dojdziesz do prostego wniosku, że jest idealny do obsługi bazy danych (IMG:style_emoticons/default/smile.gif)


ps. również Zend wykorzystuje singleton do obsługi bazy danych (IMG:style_emoticons/default/wink.gif)

Ano choćby taki, że możesz zrobic echo $query; i sobie sprawdzić co masz nie tak w zapytaniu gdy pluje ci błędem.