Zmienna nazwa zwracanej wartości Opcje

[Wieloryb]

A więć, mam sobie taką prostą funkcję

[PHP] pobierz, plaintext 
public function sql($q) {
		$this->res = mysql_query ($q);
		return $this->res;
	}
[PHP] pobierz, plaintext 

tylko teraz, zalozmy ze w skrypcie chcialbym wywolac w petli inne zapytanie z petla czyli

[PHP] pobierz, plaintext 
$s->sql("zapytanie");
 
while($row = my..fe..as($s->res)) {
 
   $s->sql("zapytanie2"); 
 
   while($row = my..fe..as($s->res)) {
 
  }
}
[PHP] pobierz, plaintext 

To 2 wyniki z SQLa myli i sie gubi. Chcialem dodac identyfikacje do zapytania czyli

[PHP] pobierz, plaintext 
public function sql($q, $id="") {
 
		$this->res.$id = mysql_query ($q);
 
		return $this->res.$id;
	}
[PHP] pobierz, plaintext 

i przy wywolywaniu odpowiednio to nazywac sobie np

[PHP] pobierz, plaintext 
$s->sql("zapytanie", 1);
 
while($row = my..fe..as($s->res1)) {
 
   $s->sql("zapytanie2", 2); 
 
   while($row = my..fe..as($s->res2)) {
 
  }
}
[PHP] pobierz, plaintext 

Ale dostaje blad

Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /users/modasu/htdocs/_sites/pg3/class/class.sites.php on line 12

Czy mój pomysł jest w ogóle dobry i ma sens ? Co jest nie tak ?

Z góry dzieki
Pozdrawiam

Ten post edytował Wieloryb 22.04.2010, 20:58:55

[mdco]

parameter id jest nie potrzebny, problem masz w sekwencji:

[PHP] pobierz, plaintext 
while($row = my..fe..as($s->res)) {
$s->sql("zapytanie2");
while($row = my..fe..as($s->res)) {
}
}
[PHP] pobierz, plaintext 

zmienną $row z pierwszego zapytania nadpisujesz zmienną $row z drugiego zapytania, poza tym nadpisujesz sobie zmienną $s->res w podpętli,
na mój gust powinno to wyglądać tak:

[PHP] pobierz, plaintext 
 
class S {
   public var $res;
   public __construct() {
    //......
   }
   public function sql($q) {
      $this->res = mysql_query ($q);
   }
 
}
 
$s = new S();
$s->sql("zapytanie1");
while($row1 = my..fe..as($s->res)) {
   $s = new S();
   $s->sql("zapytanie2");
   while($row2 = my..fe..as($s->res)) {
   }
}
[PHP] pobierz, plaintext 

[mortus]

Drugi parametr funkcji rzeczywiście jest niepotrzebny. Natomiast nadpisywanie zmiennej $row nie ma tutaj nic do rzeczy, choć to zależy od tego, czy w drugiej pętli while (lub w ogóle po nadpisaniu zmiennej $row) wykorzystujesz (chcesz wykorzystać) dane pobrane za pomocą pierwszego zapytania. Niewątpliwie problemem jest nadpisywanie obiektu $s, gdyż obiekt ten przechowuje odpowiedni zasób zwrócony przez zapytanie SQL. Kiedy pierwsza pętla while wykona już jeden przebieg, to próbuje uzyskać dostęp do zmiennej $res obiektu $s, aby pobrać następny rekord. Jednak obiekt ten został nadpisany i nie zawiera już wyników pierwszego zapytania, a zawiera wyniki zapytania drugiego (tutaj się gubi, ale nie druga pętla, tylko pierwsza). Wyjaśnię dobitniej w komentarzach do kodu:

[PHP] pobierz, plaintext 
// tworzymy obiekt $s
$s->sql("zapytanie"); // wykonujemy zapytanie, a identyfikator zasobu zapisujemy w zmiennej $res obiektu $s
 
while($row = my..fe..as($s->res)) {   // pobieramy wiersz z zasobu przypisanego do zmiennej $res obiektu $s
 
   // tutaj możemy wykorzystać dane pobrane z pierwszego zapytania i zapisane w zmiennej $row
 
   $s->sql("zapytanie2"); // wykonujemy zapytanie, a identyfikator zasobu zapisujemy w zmiennej $res obiektu $s
                                    // i tutaj pojawia się problem, ponieważ pierwsza pętla while nie będzie się poruszać po zasobie zwróconym
                                    // przez pierwsze zapytanie, gdyż został on nadpisany, a będzie się poruszać po zasobie zwróconym przez drugie zapytanie
 
 
   while($row = my..fe..as($s->res)) {
 
    // tutaj już nie możemy wykorzystać danych pobrany z pierwszego zapytania, gdyż teraz w zmiennej $row znajdują się dane pobrane z drugiego zapytania
 
    // wykorzystanie tutaj drugiej pętli while w praktyce oznacza, że pierwsza pętla while kończy działanie, już po pierwszym przebiegu, ponieważ ukryty wskaźnik wyniku został przesunięty do końca zasobu zwróconego przez drugie zapytanie, po którym to zasobie teraz porusza się również pierwsza pętla
 
   }
   // tutaj również nie możemy wykorzystać danych pobranych z pierwszego zapytania, ponieważ w zmiennej $row mamy w tej chwili dane z ostatniego wiersza zwróconego przez drugie zapytanie
}
[PHP] pobierz, plaintext 

Jeszcze słowo odnośnie rozwiązania z identyfikatorem wyniku. W praktyce nie stosuje się takiego czegoś. Zazwyczaj jeden obiekt przechowuje wyniki jednego zapytania, a inny obiekt wyniki innego zapytania. Jednak nie jest to niewykonalne. Kwestią do rozstrzygnięcia jest odpowiednia budowa klasy.

@mdco: piszesz o nadpisywaniu i sam nadpisujesz

[PHP] pobierz, plaintext 
class S {
   public var $res;
   public __construct() {
    //......
   }
   public function sql($q) {
      $this->res = mysql_query ($q);
   }
 
}
 
$s = new S();
$s->sql("zapytanie1");
while($row1 = my..fe..as($s->res)) {
   $s = new S();
   $s->sql("zapytanie2");
   while($row2 = my..fe..as($s->res)) {
   }
}
[PHP] pobierz, plaintext