[zend] wyswietlanie polaczonych danych z dwoch tabell Opcje

[nieraczek]

Odnośnie tego co pisał batman i thm w innym moim poście chciałbym zrobić teraz poprawne pod względem technicznym wyświetlanie połączonych danych z dwóch tabel.

Powiem co zrobiłem:

W folderze 'models' utworzyłem pliki:

Ksiazki.php:

Kod

<?php

class Ksiazki extends Zend_Db_Table
{
    protected $_name = 'ksiazki';

    protected $_primary = 'id_ksiazki';
}

Autorzy.php:

Kod

<?php

class Autorzy extends Zend_Db_Table
{
    protected $_name = 'autorzy';

    protected $_primary = 'id_autora';
}

Natomiast w pliku IndexController.php mam:

Kod

    public function indexAction()
    {
        $db = Zend_Registry::get('dbAdapter');

        $select = $db->select()
             ->from(array('k' => 'ksiazki'))
             ->join(array('a' => 'autorzy'),
             'k.id_autora = a.id_autora');

        $this->view->lista = $db->fetchAll($select);
    }

A w innym pliku wyświetlam wyniki zapytania. Wszystko działa, ale jak widzicie w pliku 'IndexController.php' nie robię chyba zbyt poprawnym sposobem przynajmniej takie odniosłem wrażenie po wypowiedzi batmana i thm??

Więc jak to powinno wyglądać prawidłowo ? Jak rozumiem powinno być w pliku IndexController.php coś takiego:

Kod

    public function indexAction()
    {
               $ksiazkiTabela = new Ksiazki();
               $autorzyTabela = new Autorzy();

    }

Ale co dalej - jak zrobić SELECTA ? Bo jak zrobię tak jak na tej stronie: http://framework.zend.com/manual/en/zend.d...t.building.join

// Create the Zend_Db_Select object
$select = $db->select();

to wrócę przecież do tego sposobu wyświetlania, który mam obecnie, a który nie jest podobno poprawny.

Ten post edytował nieraczek 23.10.2008, 18:25:41

[batman]

Jak to powinno wyglądać u Ciebie.
1. W kontrolerze tworzysz obiekt modelu Książki
2. W modelu Książki dodajesz metodę getBooksWithAuthors.
3. W nowej metodzie wykonujesz zapytanie i zwracasz wyniki.

[PHP] pobierz, plaintext 
<?php
public function getBooksWithAuthors() {
    $this->select()->join(.......);
    return $this->fetchAll($select);
}
?>
[PHP] pobierz, plaintext 

[nieraczek]

W modelu Ksiazki zrobiłem tak:

Kod

public function getBooksWithAuthors()
    {
           $this->select()
             ->from(array('k' => 'ksiazki'))
             ->join(array('a' => 'autorzy'),
                    'k.id_autora = a.id_autora');
           return $this->fetchAll($select);
    }

W kontrolerze tak:

Kod

$ksiazkiTabela = new KsiazkiTable();
        
$this->view->lista = $ksiazkiTabela->getBooksWithAuthors();

Jednak

Kod

var_dump($this->lista);

zwraca mi NULL. W tej funkcji getBooksWithAuthors() chyba coś nie tak z zapytaniem ?


A może jednak to niemożliwe żeby w modelu, który odpowiada za tabelę Ksiazki pobrać takze dane z tabeli Autorzy ? Bo na tej stronie jest opisana metoda taką jak robiłem poprzednio i określona mianem 'najlepszej praktyki': http://wiki.ekini.net/main/Zend_db

Kod

Best Practices
[edit] What is the best way to do queries in Zend Framework?

Please see this blog post http://blog.ekini.net/2008/02/12/what-is-the-safest-way-to-do-database-queries-in-zend-framework/

//Roman Nestertsov's way of doing it.
$db = Zend_Registry::get(’dbAdapter’);
$sql = $db->quoteInto(”SELECT * FROM Table1 WHERE id = ?”, $target_id);

//Erik's way of doing it.
$db = Zend_Registry::get(’dbAdapter’);

$select = $db->select()->
          from(’Table1′, ‘*’)->
          where(’id = ?’, $id)->
          where(’url = ?’, $url);

$row = $db->fetchRow($select);

Czyli tak jak robiłem poprzednio. Ale z drugiej strony wtedy nie potrzebne byłyby modele, czyli jednak jakoś powinno się to dać zrobić w klasie, tylko jak ?

[batman]

W modelu Ksiazki zrobiłem tak:

Zwraca Ci NULL ponieważ do fetchAll przekazujesz zmienną, która nie zawiera zapytania. Dziwne, że nie zgłosiło Ci błędu lub ostrzeżenia.
Zrób tak:

[PHP] pobierz, plaintext 
<?php
public function getBooksWithAuthors() {
    $select = $this->select()
             ->join(array('a' => 'autorzy'), 'k.id_autora = a.id_autora');
    return $this->fetchAll($select);
}
?>
[PHP] pobierz, plaintext 

Jeśli nadal będzie zwracało NULL, to przed return wstaw echo $select i zobacz co Ci za SQL-a skleiło.

[nieraczek]

[PHP] pobierz, plaintext 
<?php
An error occurred
Application error
Exception information:
 
Message: Select query cannot join with another table
?>
[PHP] pobierz, plaintext 

Jak zamiast aliasu 'k' użyję całej nazwy tabeli to też jest ten sam błąd:

Kod

public function getBooksWithAuthors()
    {
            $select = $this->select()
            ->join(array('a' => 'autorzy'), 'ksiazki.id_autora = a.id_autora');
           return $this->fetchAll($select);
    }

Może faktycznie nie można w klasie odpowiedzialnej za tabelę używać zapytania z JOIN'em. Tylko wówczas oznaczałoby to poważne ograniczenie w modelach ZF i to, że nadają się one tylko do zwracania wyników zapytania tej tabeli której dotyczy dany model hm......

[batman]

Sądziłem, że ten sposób zadziała. Skoro nie działa, to zadziała ten

[PHP] pobierz, plaintext 
<?php
$select = $this->getAdapter()->select();
$select->from(array('k' => 'ksiazki'))->join(array('a' => 'autorzy'), 'k.id_autora = a.id_autora');
?>
[PHP] pobierz, plaintext 

[nieraczek]

hm.......

Obojętnie czy mam:

[PHP] pobierz, plaintext 
<?php
public function getBooksWithAuthors()
    {
 
         $select = $this->getAdapter()->select();
$select->from(array('k' => 'ksiazki'))->join(array('a' => 'autorzy'), 'k.id_autora = a.id_autora');
 
           return $this->fetchAll($select);
    }
?>
[PHP] pobierz, plaintext 

czy

[PHP] pobierz, plaintext 
<?php
$st = $this->getAdapter()->query("SELECT * FROM ksiazki k JOIN autorzy a ON k.id_autora=a.id_autora");
return $this->fetchAll($st);
?>
[PHP] pobierz, plaintext 

czy

[PHP] pobierz, plaintext 
<?php
$st = $this->getAdapter()->query('SELECT * FROM ksiazki');
return $this->fetchAll($st);
?>
[PHP] pobierz, plaintext 

to i tak dostaję komunikat:

Kod

An error occurred
Application error
Exception information:

Message: SQLSTATE[42000]: Syntax error or access violation: 1064 Something is wrong in your syntax obok '' w linii 1

[batman]

Zrób echo ze zmiennej $select i wykonaj je w pgadminie/phpmyadminie. Nie możliwe, by to nie działało.

[nieraczek]

Dziwne, bo jeden sposób nie działa, a drugi który opisałem niżej działa

Więc w modelu mam:

[PHP] pobierz, plaintext 
<?php
$select = $this->getAdapter()->select();
$select->from(array('k' => 'ksiazki'))->join(array('a' => 'autorzy'), 'k.id_autora = a.id_autora');
 
return $this->fetchAll($select);
?>
[PHP] pobierz, plaintext 

W kontrolerze:

[PHP] pobierz, plaintext 
<?php
$ksiazki = new Ksiazki();
$this->view->lista = $ksiazki->getBooksWithAuthors();
?>
[PHP] pobierz, plaintext 

echo selecta dało wynik:

Kod

SELECT `k`.*, `a`.* FROM `ksiazki` AS `k` INNER JOIN `autorzy` AS `a` ON k.id_autora = a.id_autora

Ale jak w modelu zrobię tak:

[PHP] pobierz, plaintext 
<?php
$select = $this->getAdapter()->query("SELECT * FROM ksiazki k JOIN autorzy a ON k.id_autora=a.id_autora");
return $select;
?>
[PHP] pobierz, plaintext 

A w kontrolerze tak:

[PHP] pobierz, plaintext 
<?php
$ksiazki = new Ksiazki();
$this->view->lista = $ksiazki->getBooksWithAuthors()->fetchAll();
?>
[PHP] pobierz, plaintext 

To wszystko działa i poprawnie się wyświetla, jednak nie rozumiem czemu tamten sposób nie działa, nawet jak w tamtym sposobie tak jak tu zwrócę $select i w kontrolerze tak jak tu zrobię fetchAll() to i tak nie działa, ta funkcja select() ZF ma jakieś błędy czy co ?

Ten post edytował nieraczek 24.10.2008, 08:54:41

[bigZbig]

Bo za pierwszym razem próbujesz wykonać

Kod

$this->fetchAll($select);

a za drugim wykonujesz

Kod

$select->fetchAll();

[batman]

Masz jakieś dziwne problemy. U mnie działa to bez żadnych cyrków
Możliwe, że jest jakiś błąd w ZF, możliwe jest też to, że z powodu błędu w kodowaniu danych wysyłanych do bazy i samej bazy. Powodów może być wiele.


edit

@bigZbig
To nie ma znaczenia, w takim sensie jak Ty to podajesz. Jednak Twój post nakierował mnie na problem.

Wstaw do modelu i będzie działać. Nie zauważyłem, że robisz fetchAll na obiekcie modelu, a nie na adapterze.

[PHP] pobierz, plaintext 
<?php
$select = $this->getAdapter()->select();
$select->from(array('k' => 'ksiazki'))->join(array('a' => 'autorzy'), 'k.id_autora = a.id_autora');
return $this->getAdapter()->fetchAll($select);
?>
[PHP] pobierz, plaintext 

[nieraczek]

lol - faktycznie, zrobiłem tak jak napisał batman i teraz działa - dzięki batman i bigZbig - dam Wam obu 'pomogl'

Ten post edytował nieraczek 24.10.2008, 09:06:13

[MakeArt]

Witam mam problem...wzorowałem się na tym wątku i w modelu mam

[PHP] pobierz, plaintext 
<?php
class Projekty extends Zend_Db_Table {
       protected $select = 'projekty';
       
    public function getProjektyRodzajProjektu() { 
              
        $select = $this->getAdapter()->select();
        $select->from(array('p' => 'projekty'))->join(array('r' => 'rodzajprojektu'), 'p.IdRodzaju = r.IdRodzaju');
        return $this->getAdapter()->fetchAll($select);
        
    }
}
?>
[PHP] pobierz, plaintext 

a w kontrolerze akcje details

[PHP] pobierz, plaintext 
<?php
function detailsAction() {
      $this->view->title = "Szczegóły";
              $model= new Projekty();
            $this->view->projekty = $model->getProjektyRodzajProjektu();
   }
?>
[PHP] pobierz, plaintext 

i nic pusto... zwraca pustą tablice... zapytanie generowane przez model działa w phpmyadmin. Zrobiłem jakiś oczywisty błąd?? Poprzednie sposoby także zwracają mi pustą tablice....

[batman]

Zamiast

[PHP] pobierz, plaintext 
<?php
protected $select = 'projekty';
?>
[PHP] pobierz, plaintext 

powinno być

[PHP] pobierz, plaintext 
<?php
protected $_name = 'projekty';
?>
[PHP] pobierz, plaintext 

. Ale to nie powinno być przyczyną problemów.
Wstaw przed return

[PHP] pobierz, plaintext 
<?php
echo $select;
?>
[PHP] pobierz, plaintext 

. Wówczas wykonaj zapytanie, które się wyświetli w phpmyadmin/pgadmin i zobacz czy też zwróci pusty wynik.

[MakeArt]

już to robiłem... zapytanie jest dobre.

[batman]

zapytanie jest dobre

Czyli zwróciło jakieś wiersze? Jeśli tak, to

[PHP] pobierz, plaintext 
<?php
Zend_Debug::dump($model->getProjektyRodzajProjektu());
?>
[PHP] pobierz, plaintext 

i zobacz co się wyświetli.

[MakeArt]

Dzieki. kurcze zwraca...

[PHP] pobierz, plaintext 
<?php
array(2) {
  [0] => array(5) {
    ["IdProjektu"] => string(1) "1"
    ["Tytul"] => string(51) "Aplikacja wspomagająca badania literaturowe BadLit"
    ["DataUtworzenia"] => string(10) "2008-10-15"
    ["IdRodzaju"] => string(1) "1"
    ["NazwaRodzaju"] => string(29) "Praca dyplomowa - in?ynierska"
  }
  [1] => array(5) {
    ["IdProjektu"] => string(2) "23"
    ["Tytul"] => string(19) "Suwnica przemys?owa"
    ["DataUtworzenia"] => string(10) "0000-00-00"
    ["IdRodzaju"] => string(1) "2"
    ["NazwaRodzaju"] => string(29) "Praca dyplomowa - licencjacka"
  }
}
?>
[PHP] pobierz, plaintext 

Mam podpięte SMARTY, może w tym jest problem i muszę wyświetlać to innym sposobem niż do tej pory
zwykłe $model->fetchAll().


Namieszałem ze smartym dlatego nie wyświetlało. Dzięki za pomoc!!