[Berti]

Hallo. Ich habe folgende Klasse:

PHP-Code:

<?php

    class Search
    {
        /**
         * Alle Produkte abrufen
         *
         * @access    public
         * @return    array $items
         */
        public function getAllItems()
        {
            global $db;
            global $smarty;

            $itemDB = $db->query("SELECT
                                      `ID`,
                                      `artID`,
                                      `title`,
                                      `isbn`,
                                      `publisher`,
                                      `author`
                                  FROM
                                      `bero__articles`
                                  WHERE
                                      `check` = '1'
                                  ORDER BY
                                      `ID`
                                  DESC");

            $items = array();
            for ($i = 0; $i < $db->numRows($itemDB); $i++)
            {
                $row              = $db->fetchAssoc($itemDB);
                $row['publisher'] = $this->getInfoByID($row['ID']);
                $items[]          = $row;
            }

            return $items;
        }


        /**
         * Kategorie-Informationen abrufen
         *
         * @access    public
         * @param     int $id
         * @return    string $pub
         */
        public function getInfoByID($id)
        {
            global $db;

            $pubDB = $db->query("SELECT
                                     `title`
                                 FROM
                                     `bero__categories`
                                 WHERE
                                     `ID` = '". $id ."'");

            $row = $db->fetchAssoc($pubDB);
            $pub = $row['title'];

            return $pub;
        }
    }

?>

Wenn ich mit dieser Klasse arbeite, wird mir immer nur der erste Datensatz ausgegeben, obwohl mehrere vorhanden wären. Es muss an dieser Zeile liegen, denn wenn ich diese entferne gehts, aber warum?

PHP-Code:

<?php

                $row['publisher'] = $this->getInfoByID($row['ID']);

?>

[WarrenFaith]

Na du hast ein $db-Object und sobald du getInfoByID() aufrufst, setzt du diesem Object ein neues Query vor die Nase und es liefert dir das Ergebnis davon zurück. Hast also das Query aus getAllItems() überschrieben.
Das Problem ist in dem Fall das "global".

[KingCrunch]

Wieso setzt du überhaupt $db global? Gerade wenn du mit Klassen arbeitest, bietet sich doch an dies als Klassenvariable zu speichern.

PHP-Code:

class abc {
  protected $_db = null;
  // usw
} 


[Xean]

ich seh das Problem hier:

PHP-Code:

$row = $db->fetchAssoc($pubDB);
$pub = $row['name'];
return $pub; 


ich denkmal bei $db->fetchAssoc($pubDB); kommt nur ein datensatz zurück, also kann in $pub dann auch nur einer stehen. "richtiger" wäre es so:

PHP-Code:

$info_names = array();
while($row = $db->fetchAssoc($pubDB)){
  $info_names[] = $row['name'];
}
return $info_names; 


da durch haben wir ein Array mit allen Namen

[Basti]

Nein, hier (bei den Katergorie-Infos) soll ja auch nur ein Wert ausgelesen werden, oder? Allerdings passt das alles irgendwie nicht, denn Du liest ja "title" aus und willst nachher auf den Index "name" zugreifen, den es ja gar nicht gibt (error_reporting mind. auf E_ALL setzen!). Und, es macht so ja null Sinn, bei einer so banalen Verknüpfung n+1 Abfragen zu starten, anstatt alles einfach in eine einzige Query zu setzen.

Basti

[Berti]

Hallo, danke Problem gelöst. Ich hab die Funktion einfach woanders erstellt und dann in der Klasse verwendet.