Forum PHP.pl

Witam

Przechodząc do sedna sprawy. Chciałbym zoptymalizować poniższe zapytania.

[PHP] pobierz, plaintext 
        $query = mysql_query("SELECT * FROM stc WHERE id_st=1 AND symbol='G'");
		$con .= '<div id="first">';
 
		while($smth = mysql_fetch_array($query))
		{
		   $con.= $smth['field'];
		}
 
		$con.= '</div>';
 
	        $query = mysql_query("SELECT * FROM stc WHERE id_st=2 AND symbol='G'");	
		$con .= '<div id="second">';		
		while($smth = mysql_fetch_array($query))
		{
		   $con .= $smth['field'];
		}	
		$con.= '</div>';
[PHP] pobierz, plaintext 

Nie liczę na gotowca, lecz bardziej na wskazówki, informacje na temat funkcji, którymi powinienem się zainteresować.

Ten post edytował solid23 18.11.2009, 19:06:31

Możesz połączyć zapytania mysql i warunek w phpie, tylko nie wiem czy ma to sens.

Po co robić dwa zapytania, skoro możesz to zrobić na jednym?

Te zapytania niczym od siebie się nie różnią zgadza się? Oczywiście poza id_st, to może usuń te drugie zapytanie i daj

[PHP] pobierz, plaintext 
(...) WHERE  id_st IN(1,2)
[PHP] pobierz, plaintext 

W końcu będzie tylko jedno zapytanie :-)

Ten post edytował maly_pirat 18.11.2009, 19:26:35

A więc tak...
1) Pierwsze w warunku zawsze powinno być te, które najbardziej okroi wyniki. Im mniej masz po pierwszym warunku wyników, tym lepiej, bo każdy kolejny AND leci już po mniejszej puli wyników.
2) Wyciągasz z bazy tylko pole o nazwie 'field', więc tylko je walnij do zapytania za SELECT, a nie *. To zmniejsza liczbę przesyłanych danych
3) Jeśli masz kilka opcji tych samych a tylko jedną różną to możesz wrzucić to jako jedno zapytanie z warunkiem zmiennym ujętym i wyszczególnionym w IN... tutaj miałbyś

[SQL] pobierz, plaintext 
WHERE id_st IN (1, 2) AND symbol = 'G'
[SQL] pobierz, plaintext 

Choć symbol mocniej by Ci zjechał wynik więc pchnąłbym go jako pierwszy zaraz po WHERE, a nie jako drugi (nie znam rozkładu danych w bazie, ale zgaduję, że tych z symbol = 'G' jest o wiele mniej niż z id_st = 1 czy id_st = 2) (IMG:style_emoticons/default/smile.gif) Obrobić te dane w PHP można już jak chcesz sprawdzając poprzez wrzucene do wyniku nie tylko field, ale i id_st.

Dzięki za pomoc. Jednak wystąpił kolejny problem. Aktualnie kod wygląda tak (wszystkie pobrane rekordy wyświetlają się w divie o id "first", a powinny tylko te gdzie id_st=1):

[PHP] pobierz, plaintext 
$query = mysql_query("SELECT id_st,field FROM stc WHERE symbol='G' AND id_st IN (1,2)");
$smth = mysql_fetch_array($query);
($smth['id_st'] == 1)? $con .= '<div id="first">' : $con .= '<div id="second">';
while($smth = mysql_fetch_array($query)) $con.= $smth['field'];	
$con.= '</div>';
[PHP] pobierz, plaintext 

Głównie zależy mi na tym, aby rekordy gdzie id_st jest równe "1" wylądowały w divie o id "first", a reszta (id_st=2) w divie o id "second".

Próbowałem też inaczej - wrzuciłem warunek w pętle - wszystko działało prawidłowo, lecz w każdym divie był wyświetlany tylko jeden rekord.

Ten post edytował solid23 19.11.2009, 18:10:01

Masz na myśli coś takiego? Nie wiem czy dobrze zrozumiałem.

[PHP] pobierz, plaintext 
$query = mysql_query("SELECT id_st,field FROM stc WHERE symbol='G' AND id_st IN (1,2) ORDER BY id_st ASC");
$smth = mysql_fetch_array($query);
$id_st = 1;
$con .= '<div id="first">';
while($smth = mysql_fetch_array($query)) 
{
   if($smth['id_st'] != $id_st) $con .= '</div><div id="second">'; 
   $con.= $smth['field'];
}
$con.= '</div>';
[PHP] pobierz, plaintext 

Poprawka: if($smth['id_st'] == 1)

Nie wiem czy dokładnie o to chodziło thekowi, ale Twoje rozwiązanie, maly_swd jak najbardziej działa. Dzięki za pomoc, dostajecie "pomógł" (IMG:style_emoticons/default/smile.gif)

"Poprawka: if($smth['id_st'] == 1)", no tak -> moje przeoczenie

Widzę, że mój pomysł został dobrze zinterpretowany. Tylko zapomniano dopisać drobiazgu, o którym myślałem, że będzie odczytany między wersami. Przy każdym zmieniającym DIVy przejściu pętli $id_st się nadpisuje wartością $smth['id_st'], bo inaczej po zmianie wartości na 2 każdy kolejny wers będzie nowym div, a ma to nastąpić tylko w przypadku zmiany id_st. Dla dwóch różnych id_st algorytm maly_swd będzie moim zdaniem lepszym rozwiązaniem. W przypadku nieograniczonej liczby id_st mała modyfikacja mojego sprawi, że będzie on uniwersalny.

[PHP] pobierz, plaintext 
$query = mysql_query("SELECT id_st,field FROM stc WHERE symbol='G' AND id_st > 0 ORDER BY id_st ASC");
$smth = mysql_fetch_array($query);
$id_st = 1;
$con .= '<div id="kolejny_'.$id_st.'">';
while($smth = mysql_fetch_array($query)) 
{
   if($smth['id_st'] != $id_st) {
      $con .= '</div><div id="kolejny_'.$smth['id_st'].'">';
      $id_st = $smth['id_st'];
   }
   $con.= $smth['field'];
 
}
$con.= '</div>';
[PHP] pobierz, plaintext 

Sam zerknij jak teraz działa dla G, niezależnie od id_st. A wymyślanie wielkiej ilości IF-ów staje się zbędne (IMG:style_emoticons/default/smile.gif)

Ten post edytował thek 20.11.2009, 11:23:40