Forum PHP.pl

Witam,

Walczę z projektem pierwszy raz używając mvc.
Dużo udało mi się już zwalczyć, ale teraz mam problem próbuję zrobić coś w stylu dwóch zależnych od siebie zapytań do sqla. Problem w tym, że kiedy ustawiam model na to drugie - to już wychodzi mi z pierwszego.
Z resztą.. oto cały kod:

[PHP] pobierz, plaintext 
<?php
$this->model->listRozKol($id);
			$kolumny[]=array("kolumna"=>'Rozmiary:');
			while ( $rozmiar=$this->model->getProduct() ) {
			$kolumny[]=$rozmiar;
			}
 
			$rozmiary[]=$kolumny;
			$kolumny=array();
			$this->model->listRozWiersz1($id);
			while ( $wiersz1=$this->model->getProduct() ) {  $i++;
			$kolumny[]=array("kolumna"=>$wiersz1["rozmiar_wiersz"]);
			$this->model->listRozWiersz2($id, $wiersz1["id_rozmiarw"]);
			while ( $wiersz2=$this->model->getProduct2() ) {
			$kolumny[]=$wiersz2;
			}
			$rozmiary[]=$kolumny;
			}
?>
[PHP] pobierz, plaintext 

Niestety zawsze widać tylko 1 rekord.. mimo że $this->model->listRozWiersz2($id, $wiersz1["id_rozmiarw"]); powinno wyświetlać ich więcej.

Przypuszczam, że trzeba zakombinować coś w klasie model.. ale sam na chwile obecną chyba sobie z tym nie poradzę - dlatego proszę o podpowiedź.

Przy okazji - są jakieś alternatywy dla MVC? Bo na dłuższą metę to nie wiem czy się z nim dogadam - szczególnie ciężko mi się przyzwyczaić do tych zapytań w Modelu..

Pozdrawiam i jeszcze raz bardzo proszę o podpowiedź.
Jacek

Daj jakiś bardziej reprezentatywny kod, bo ciężko coś wywnioskować z tego co wkleiłeś.

[PHP] pobierz, plaintext 
<?php
while ( $wiersz1=$this->model->getProduct() ) {  $i++;
?>
[PHP] pobierz, plaintext 

Po co to $i++?

Po za tym 2 razy zapełniasz tablicę $rozmiary, a na pewno nie używasz jej za pierwszym razem.

edit:


Owszem, są. Poczytaj o frameworku PRADO i jaka jest jego zasada działania.

Ten post edytował LBO 22.06.2007, 13:18:34

Witam,
Dzięki za reakcję.. Za pomocą tego $i sprawdzałem ile razy kręci się ta pętla.

W tym momencie:
$this->model->listRozWiersz1($id);
Wywołuję funkcję:

function listRozWiersz1($id) {
$this->dao->fetch("SELECT ...... ;");
}

Liczba rekordów w tym zapytaniu jest > 1 natomiast ten while:
while ( $wiersz1=$this->model->getProduct() ) { ....
przekręca się tylko raz.

Wydaje mi się że przyczną tego jest ustawienie w nim: $this->model->listRozWiersz2($id, $wiersz1["id_rozmiarw"]);
ponieważ listRozWiersz2 to znowu $this->dao->fetch("SELECT ...... ;"); ...

Nie rozumiem dlaczego pętla obraca się raz skoro rekordów wynikających z zapytania jest więcej..
Co jeszcze wkleić?

Pozdrawiam,
Jacek

Sorry, ale coraz mniej z Tego rozumiem.

Nie powiem Ci dokładnie co jest nie tak - przyczyną są dwa główne problemy, piszesz tak jakbym znał Twoja implementację i używasz najgorszego możliwego nazewnictwa: zróbTo1(), zróbTo2() etc.

Na pewno wiem, że coś nie tak w samym modelu jest - źle agreguje wyniki zapytania.

Ten post edytował LBO 22.06.2007, 13:45:00

OK, teraz postaram się kompleksowo:
Jeżeli zrobię tak (pominę drugie zapytanie):

[PHP] pobierz, plaintext 
<?php
$kolumny=array();
			$this->model->listRozWiersz1($id);
			while ( $wiersz1=$this->model->getProduct() ) {  $i++;
			$kolumny[]=array("kolumna"=>$wiersz1["rozmiar_wiersz"]);
		  #$this->model->listRozWiersz2($id, $wiersz1["id_rozmiarw"]);
		  #while ( $wiersz2=$this->model->getProduct2() ) {
		  #$kolumny[]=$wiersz2;
		  #}
 
			$rozmiary[]=$kolumny;
			$kolumny=array();
			}
?>
[PHP] pobierz, plaintext 

To wtedy {$rozmiary} Array (2) . Czyli jest OK.
Natomiast Kiedy odhaszuje drugie wywołanie modelu i while:

[PHP] pobierz, plaintext 
<?php
$kolumny=array();
			$this->model->listRozWiersz1($id);
			while ( $wiersz1=$this->model->getProduct() ) {  $i++;
			$kolumny[]=array("kolumna"=>$wiersz1["rozmiar_wiersz"]);
		  $this->model->listRozWiersz2($id, $wiersz1["id_rozmiarw"]);
		  while ( $wiersz2=$this->model->getProduct2() ) {
		  $kolumny[]=$wiersz2;
		  }
 
			$rozmiary[]=$kolumny;
			$kolumny=array();
			}
?>
[PHP] pobierz, plaintext 

I teraz już: {$rozmiary} Array (1)

Funkcje z klasy Model:

[PHP] pobierz, plaintext 
<?php
function listRozWiersz1($id) {
	$this->dao->fetch("SELECT DISTINCT rozmiar_wartosc.id_rozmiarw, rozmiar_wiersz.rozmiar_wiersz  FROM `rozmiar_wartosc` LEFT JOIN rozmiar_wiersz ON (rozmiar_wiersz.id_rozmiarw=rozmiar_wartosc.id_rozmiarw) WHERE id_produkt = '".$id."';");
	}
 
	function listRozWiersz2($id, $idwiersz) {
	$this->dao->fetch("SELECT DISTINCT rozmiar_wartosc.wartosc AS kolumna FROM `rozmiar_wartosc` ,
 `rozmiar_kolumna` WHERE id_produkt = '".$id."' AND rozmiar_wartosc.id_rozmiark = rozmiar_kolumna.id_rozmiark AND rozmiar_warto
sc.id_rozmiarw='".$idwiersz."' ORDER BY rozmiar_kolumna.rozmiar_liczba;");
	}
 
 function getProduct() {
		if ( $product=$this->dao->getRow() )
			return $product;
		else
			return false;
	}
 
		function getProduct2() {
		if ( $product=$this->dao->getRow() )
			return $product;
		else
			return false;
	}
?>
[PHP] pobierz, plaintext 

W klasie Connect (obiekt dao) mam:

[PHP] pobierz, plaintext 
<?php
function fetch($sql) {
		$this->query=mysql_unbuffered_query($sql,$this->db) or die(mysql_error()); // Perform query here
	}
 
   function getRow () {
		if ( $row=mysql_fetch_array($this->query,MYSQL_ASSOC) )
			return $row;
		else
			return false;
	}
?>
[PHP] pobierz, plaintext 

Teraz jaśniej?
Nie wiem nawet co jeszcze mógłbym powklejać..
Pozdrawiam,
Jacek

No i wszystko jasne. To wina sposobu w jakim pracujesz na $dao.
Pobierasz dane metodą

[PHP] pobierz, plaintext 
<?php
listRozWiersz1()
?>
[PHP] pobierz, plaintext 

a do przechodzenia po kolejnych rekordach

[PHP] pobierz, plaintext 
<?php
getProduct1()
?>
[PHP] pobierz, plaintext 

i wszystko super, tylko, że metody

[PHP] pobierz, plaintext 
<?php
listRozWiersz2()
?>
[PHP] pobierz, plaintext 

i

[PHP] pobierz, plaintext 
<?php
getProduct2()
?>
[PHP] pobierz, plaintext 

Robia to samo na tej samej instancji $dao.

Innymi słowy przechodząc po $dao w pierwszej pętli - zawierającego rekordy z listRozWiersz1() - modyfikujesz go, tak, że zawiera całkiem inny zestaw rekordów (pobranych w listRozWiersz2()).

OK, tego się spodziewałem że coś sobie gdzieś nadpisuję - a mogę prosić jakąś podpowiedź jak temu zapobiec?

Najlepiej - moim daniem - stworzyć klasę specjalnie do trzymania rekordów. Klasa taka (Rowset) była by wracana jako wynik np. listRozWiersz1(). Jakbyś odpowiednio zbudował, to mógłbyś osiągnąć taki efekt:

[PHP] pobierz, plaintext 
<?php
$rozmiary = $this->model->listRozWiersz1($id);
foreach ($rozmiary as $wiersz) {
  // Wypisujesz rekordy z zapytania w metodzie listRozWiersz1().
  print $wiersz['id_rozmiarw'] . ",t" . $wiersz['rozmiar_wiersz'] . "n";
}
?>
[PHP] pobierz, plaintext 

Bardzo ładnie pod tym względem prezentuje się komponent Zend_Db_Table z Zend Framework'a.

Ten post edytował LBO 22.06.2007, 14:40:35

Niestety nowa klasa to jak dla mnie za dużo na raz
Dorobiłem sobie do klasy z której jest $dao takie coś:

[PHP] pobierz, plaintext 
<?php
function assoc($sql) {
	$this->query = mysql_query($sql);
	while ($rek=mysql_fetch_assoc($this->query)) {
	$return[]=$rek;
	}
	return $return;
	}
?>
[PHP] pobierz, plaintext 

No i później już:

[PHP] pobierz, plaintext 
<?php
$naglowki = $this->model->dao->assoc("SELECT DISTINCT CONCAT(rozmiar_kolumna.rozmiar_kolumna,'(',rozmiar_liczba,')') AS kolumna FROM `rozmiar_kolumna` LEFT JOIN rozmiar_wartosc ON (rozmiar_wartosc.id_rozmiark=rozmiar_kolumna.id_rozmiark) WHERE rozmiar_wartosc.id_produkt='".$id."' ORDER BY rozmiar_kolumna.rozmiar_liczba;");
			$kolumny[]='Rozmiary:';
			foreach ($naglowki as $naglowek) { 
			$kolumny[]=$naglowek["kolumna"];
			}
?>
[PHP] pobierz, plaintext 

Wiem że to pewnie nie bardzo obiektowe i w ogóle do.... ale działa
A co do PRADO - wolałbym coś mniej 'automatycznego' - tak żebym mógł sobie dalej używać QuickForm'a i Smarty - chodzi mi tylko o jakiś sprytny przykład poskładania tego wszystkiego w klasy.
Pozdrawiam i bardzo dziękuję za podpowiedzi,
Jacek