[ZendFramework]Mysql i relacje Opcje

[ShadowD]

Trochę ułatwię sprawę na potrzeby stwierdzenie "czego najlepiej użyć", mamy następujące tabele:

PAGE
- id
- idStatus
- name
- value

STATUS
- id
- statusName

Jak najlepiej pobrać całość w jednym zapytaniu?

1. Jojn w zapytaniu
2. Coś takiego - http://blog.wilgucki.pl/2010/11/zenddb-i-relacje.html tutaj wydaje mi się że jest to bardziej do relacji 1:n
3. Pod zapytanie

Mam takie trzy możliwości, w każdej jest to do wykonania, ale nie wiem która z nich będzie najbardziej poprawna przy relacji 1:n wybrał bym #1 lub #2 w zależności od skomplikowania (z przewagą dla #2) a tutaj za skarby nie mogę się zdecydować, problem może nawet nie zendowski więc jak ukochani moderatorzy postanowią przenieść nie pogniewam się. (I nie ma tutaj sarkazmu :])

[mortus]

Niestety pod względem wydajności najsłabiej wypadną relacje tworzone przez ZF, a to dlatego, że wydobycie statusów dla każdej strony to w tym przypadku wykonywanie odrębnych zapytań SELECT dla każdej ze strony. Potwierdzeniem niech będzie odpowiedni fragment kodu (linia 54):

[PHP] pobierz, plaintext 
    /**
     * Query a dependent table to retrieve rows matching the current row.
     *
     * @param string|Zend_Db_Table_Abstract  $dependentTable
     * @param string                         OPTIONAL $ruleKey
     * @param Zend_Db_Table_Select           OPTIONAL $select
     * @return Zend_Db_Table_Rowset_Abstract Query result from $dependentTable
     * @throws Zend_Db_Table_Row_Exception If $dependentTable is not a table or is not loadable.
     */
    public function findDependentRowset($dependentTable, $ruleKey = null, Zend_Db_Table_Select $select = null)
    {
        $db = $this->_getTable()->getAdapter();
 
        if (is_string($dependentTable)) {
            $dependentTable = $this->_getTableFromString($dependentTable);
        }
 
        if (!$dependentTable instanceof Zend_Db_Table_Abstract) {
            $type = gettype($dependentTable);
            if ($type == 'object') {
                $type = get_class($dependentTable);
            }
            require_once 'Zend/Db/Table/Row/Exception.php';
            throw new Zend_Db_Table_Row_Exception("Dependent table must be a Zend_Db_Table_Abstract, but it is $type");
        }
 
        // even if we are interacting between a table defined in a class and a
        // table via extension, ensure to persist the definition
        if (($tableDefinition = $this->_table->getDefinition()) !== null
            && ($dependentTable->getDefinition() == null)) {
            $dependentTable->setOptions(array(Zend_Db_Table_Abstract::DEFINITION => $tableDefinition));
        }
 
        if ($select === null) {
            $select = $dependentTable->select();
        } else {
            $select->setTable($dependentTable);
        }
 
        $map = $this->_prepareReference($dependentTable, $this->_getTable(), $ruleKey);
 
        for ($i = 0; $i < count($map[Zend_Db_Table_Abstract::COLUMNS]); ++$i) {
            $parentColumnName = $db->foldCase($map[Zend_Db_Table_Abstract::REF_COLUMNS][$i]);
            $value = $this->_data[$parentColumnName];
            // Use adapter from dependent table to ensure correct query construction
            $dependentDb = $dependentTable->getAdapter();
            $dependentColumnName = $dependentDb->foldCase($map[Zend_Db_Table_Abstract::COLUMNS][$i]);
            $dependentColumn = $dependentDb->quoteIdentifier($dependentColumnName, true);
            $dependentInfo = $dependentTable->info();
            $type = $dependentInfo[Zend_Db_Table_Abstract::METADATA][$dependentColumnName]['DATA_TYPE'];
            $select->where("$dependentColumn = ?", $value, $type);
        }
 
        return $dependentTable->fetchAll($select);
    }
[PHP] pobierz, plaintext 

W osobnym temacie natomiast toczymy dyskusję na temat wyższości JOIN-ów nad PODZAPYTANIAMI, czy też odwrotnie i moje wstępne testy wykazały, że złączenia są szybsze/bardziej wydajne. Osobiście tego się właśnie spodziewałem i w związku z tym faktem polecam złączenia.

[ShadowD]

Ok, zdecydowałem się na połączenie jein'em, rozumiem że joinLeft() będzie najlepszym rozwiązaniem, mamy jeszcze join() i inne pochodne.

Mam tylko jeden problem, może ktoś zna obejście problemu.

Kod modelu

[PHP] pobierz, plaintext 
    protected $_name = 'page';
 
    public function getPage($name)
    {
        $select = $this->select()
            ->from('page',array('name', 'template', 'time', 'title', 'value'))
            ->where('name="'.$name.'"')
            ->setIntegrityCheck(false)
            ->joinLeft(array('status'=>'pageStatus'), 'page.idStatus = status.id',array('statusName'=>'name', 'statusActive'=>'active'))
            ->joinLeft(array('seo'=>'pageSeo'), 'page.idSeo = seo.id',array('seoTitle'=>'title', 'seoDescription'=>'description', 'seoKeywords'=>'keywords', 'seoFallow'=>'follow', 'seoIndex'=>'index'));
 
        return $this->fetchRow($select)->toArray();
    }
}
[PHP] pobierz, plaintext 

W tabeli page i pageStatus mam pole o takiej samej nazwie "name" próbuję zmienić jego nazwę na "statusName" (w tabeli status) dostaję komunikat o błędzie:
Message: SQLSTATE[23000]: Integrity constraint violation: 1052 Column 'name' in where clause is ambiguous

Po zmianie nazwy pola na "nazwa2" wszystko działa jak powinno, jest opcja by zachować takie same nazwy pól i jakoś przy pobieraniu je zmienić?

[Psajkus]

Masz w dwóch tabelach pole o takiej samej nazwie. W Twoim zapisie nie wiadomo więc do którego pola odnosi się warunek. Wystarczy, że podasz nazwę tabeli:

[PHP] pobierz, plaintext 
->where('page.name LIKE ?', $name)
[PHP] pobierz, plaintext 

lub też

[PHP] pobierz, plaintext 
->where('pageStatus.name LIKE ?', $name)
[PHP] pobierz, plaintext 

[ShadowD]

Próbowałem błąd identyczny był. :-<

[mortus]

Przed return-em wypisz zapytanie (w linii 11):

[PHP] pobierz, plaintext 
echo $select;
[PHP] pobierz, plaintext 

I powinieneś zobaczyć, co jest nie tak.