Dotaz: Problém s výpisem z třídy v php

[Trumbalo]

Ahoj, já už fakt nemůžu Může mi někdo prosím říct, že jsem idiot, že mám něco špatně? Pořád dostávám chybu

Catchable fatal error: Object of class FAV could not be converted to string in...

Mám třídy

class FAV {
 private $varFav;
 public function __construct() { $this->varFav  = 'blablabla'; $this->getVar(); }
 private function getVar() { return $this->varFav; }
}


class Book {
 private function getAutor()  { .. code .. return $string;     }
 private function getYear()   { .. code .. return $string;     }
 private function getFavo()   { $obj = new FAV(); return $obj; }
 private function getGenre()  { .. code .. return $string;     }
 public function __toString() { return $this->getAutor().
                                       $this->getYear().
                                       $this->getFavo().
                                       $this->getGenre();      }
}

$book = new Book();
echo $book;

Díky dík..

---

---

Odpovědi

[Kit]

V třídě FAV ti chybí metoda __toString().

[Trumbalo]

Díky, ale dyť je ve třídě Book která by měla vrátit string, nebo ne?

[Kit]

Volaná metoda $this->getFavo() vrací objekt, který je nutné převést na string, abys ho mohl zřetězit.

[Trumbalo]

Tak že to má být správně takto?

class FAV {
 private $varFav;
 public function __construct() { $this->varFav  = 'blablabla'; }
 private function getVar()     { return $this->varFav; }
 private function __toString() { return getVar(); }
}

Nebo takto ?

class FAV {
 private $varFav;
 public function __construct() { $this->varFav  = 'blablabla'; }
 private function __toString() { return varFav; }
}

Díky

[Kit]

Mělo by fungovat obojí, v obou případech tam však jsou chyby. Bližší je mi druhé řešení, které po patřičných úpravách vypadá takto:

class FAV {
    private $varFav;

    function __construct() {
        $this->varFav  = 'blablabla';
    }

    function __toString() {
        return $this->varFav;
    }
}

Ovšem pokud ten konstruktor nemá žádný parametr, úplně by stačila tato verze:

class FAV {
    function __toString() {
        return 'blablabla';
    }
}

Taková třída by však nebyla moc užitečná.

[Trumbalo]

Mělo by fungovat obojí, v obou případech tam však jsou chyby.

Ať hledám jak hledám žádná rozdíl jsem tam nenašel, tak v čem jsem měl chyby? Použití private a public před metodama je špatně?

[Kit]

public je v pořádku, ale tohle slovo je vlastně nadbytečné. Zkus si to s private. Zvenku je taková metoda nedostupná.

Dále ti tam chybí jedno $this->

[Trumbalo]

JJ už to vídím.. Díky

[Trumbalo]

Ještě prosím jedno upřesnění což mi taky hlava nebere. Ve tříde Book metody getFavo();

private function getFavo() {
 
 $class = 'FAV';
 
 if(class_exists($class, true)) {
  $obj = new $class();
  return $obj;
 }
 else {
  throw new Exception('Chyba!');
 }
}

Pořád mi to vrací chybu:

Fatal error: Method Book::__toString() must not throw an exception in...

a nevím jak to mám jinak kontrolovat... Děkuji ještě za pomoc..

[Kit]

Mně zase hlava nebere, k čemu vlastně potřebuješ tu hromadu zbytečných get...(). Gettery se přece nedělají, protože porušují zapouzření objektů.

Udělej to jednodušší:

private function getFavo() {
    $class = 'FAV';
    return new $class();
}

[Trumbalo]

Mně zase hlava nebere, k čemu vlastně potřebuješ tu hromadu zbytečných get...(). Gettery se přece nedělají, protože porušují zapouzření objektů.

Jako tohle je špatně?

private function getAutor() { .. }
private function getYear()  { .. }
private function getFavo()  { .. }
private function getGenre() { .. }

pokud ano, tak jak mají být pojmenovány ty metody?

[Kit]

Záleží na tom, co má ta třída dělat. Přece není jen skladištěm čtyř atributů, ale ta aplikace má nějaká data produkovat. Tím chci říct, že tyto čtyři metody tam vůbec nemají být, že jsou tam zbytečné.

[Trumbalo]

Ty metody samo něco dělají, ty řetězce tam jsou jen pro ukázku, že je výsledek string a ne objekt..

Kite a s tím tu mi můžeš poradit? Děkuji

[Kit]

Když já nevím. Takovou šílenou konstrukci jsem nikdy nedělal. Vzor Factory sice používám, ale místo testování existence třídy zachytávám fatální chyby.

[Trumbalo]

Vzor Factory sice používám, ale místo testování existence třídy zachytávám fatální chyby.

A prozradíš jak na to? Díky

[Kit]

register_shutdown_function("fatal_handler");

[Kit]

Co tedy dělá metoda getAutor()? Dělá i něco jiného, než {return $this->author;} ?

[Trumbalo]

Cílem metody getAutor() bude prohledat DB a vrátit veškeré id knih které jsou od autora XY.

[Kit]

Podle mne by bylo výhodnější místo nicneříkajících id knih vracet přímo ty knihy, protože samotné id knih jsou k ničemu. Potom by už ale název metody getAutor() byl zavádějící.