Forum PHP.pl

Witam
Nie mogę sobie poradzić z tym kodem:

[PHP] pobierz, plaintext 
<?php
class menuComponents extends sfComponents
{
    public function executeDefault()
    {
        $c = new Criteria();
        $c->add(GrupaPeer::ID_PARENT,0);
          $this->grupy = GrupaPeer::doSelect($c);
          foreach($this->grupy as $grupa)
          {
              $d = new Criteria();
              $d->add(GrupaPeer::ID_PARENT,$grupa->getIDGrupa());
              $this->podgrupy = GrupaPeer::doSelect($d);
          }
    }
}
?>
[PHP] pobierz, plaintext 

template

[PHP] pobierz, plaintext 
<ul>
    <?php foreach($grupy as $grupa): ?>
        <li>
        <?php echo link_to($grupa->getNazwaGrupaPl(),"towary/grupa?id=" . $grupa->getIdGrupa()); ?>
        <ul>
            <?php foreach ($podgrupy as $podgrupa): ?>
            <li>
            <?php echo link_to($podgrupa->getNazwaGrupaPl(),"towary/grupa?id=" . $podgrupa->getIdGrupa()); ?>
            </li>
            <?php endforeach ?>
        </ul>
        </li>
    <?php endforeach ?>
</ul>
[PHP] pobierz, plaintext 

W wyniku wyświetlają się wszystkie grupy główne (id_parent=0) i w każdej z nich podgrupa ostatniej grupy głównej.
Gdy dam echo w funcji wyświetla ok, przez template jak powyżej.

Ojej,.. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

1. Pobieranie obiektów z bazy danych (nawet mowiac bardziej ogolnie - wykonwywanie jakichkolwiek zapytanian do bazy w jakimkolwiek systemie) w petli niema sesu. To tak jak bys robil zakupy, biegajac miedzy domem a sklepem z kazdym pojedynczo zakupionym towarem..

2. Zapytanie do bazy (najlepiej jedno) tworzsz w pliku akcji komponentu. Wynikiem zapytania jest obiekt lub tablica obiektow.
W templecie umieszczasz petle w ktorej wyswietlasz wyniki obiekt po obiekcie poprzez petle foreach.

3. Wazne jest utworzenie prawidlowego MODELU. Jezeli masz Grupy i Podgrupy to pownienes miec 2 tabele w bazie i 2 odpowiedajace im obiekty,


4. Odnosnie braku wyswietlania podgrup:

[PHP] pobierz, plaintext 
<?php
$this->podgrupy = GrupaPeer::doSelect($d);
?>
[PHP] pobierz, plaintext 

Zmienna $this->podgrupy jest nadpisywana przy kazdej iteracji.

"Prawidlowe" rozwiazanie to:

[PHP] pobierz, plaintext 
<?php
//Tablica na obiekty z zapytan
 $podgrupyArray = array();
//Przypisujemy obiekty do tablicy
 $podgrupyArray[] =  GrupaPeer::doSelect($d);
//Wysylamy do template
 $this->podgrupyArray = $podgrupyArray ;
?>
[PHP] pobierz, plaintext 

Ten post edytował smentek 5.12.2008, 23:07:21

Może nieco odejdę od tematu ale:


A co w przypadku gdy będzie konieczność dołożenia 3-go poziomu zagłębień? Dodasz nową tabelę? Moim zdaniem rozwiązanie problem drzewiastego menu z jedną tabelą, w której klucz obcy wskazuje na nią samą, jest najlepszym pomysłem.

Masz racje tworzenie osobnych tabel moze faktycznie nie byc najlepszym rozwiazaniem, Zwlaszcza jesli struktura ma byc nieograniczona.



To juz zalezy od priorytetów, ktore obrales. Od strony wydajnosci na pewno nie jest to najlepsza metoda z mozliwych.

Dziękuję
Już myślałem że nikt nie odpowie.
Jednak bedę zawsze budował model struktur drzewiastych w jednej tabeli.
Zbieranie danych w tablicy jest rozwiązaniem tego problemu i zminiejsza ilość zapytań do bazy.
Pogrzebałem trochę i staram się użyć sfPropelActAsNestedSetBehaviorPlugin do listowania takich struktur.
Dzięki za pomoc

Pozdrawiam
Voytas