Dotaz: Návrh třídy v PHP

[Greener]

Zdravím, mám třídu, která po zavolání zpracuje v konstruktoru všechny metody a poslední metoda __toString() mi returnem vrátí celý výsledek a echo vypíše vytvořenou instanci této třídy. Všechno funguje super, ale dal jsem se na rozšíření chyb během zpracování celé třídy. Mimo jiné je tam i metoda chyba(), která se volá v případě nějaké chyby, no problém je v tom, že pokud je tato metoda zavolána, zpracují se potřebná data ale už nevím, jak bych mohl přinutit __toString() k okamžité akci, bez toho až na ni dojde řada, chci totiž v případě chyby přeskočit nejen veškeré následující metody uvnitř konstruktoru, ale chci taky ukončit ihned po zavolání průběh aktuálně zpracované metody. Neporadí nějaký profík? Děkuji

Odpovědi

[Kit]

throw new Exception('Chybové hlášení');

[Greener]

Díky. Exception používám, ale to neřeší můj dotaz.

[Kit]

Řeší, jen to možná neumíš použít.

[Greener]

Potřebuji při chybě zpracovat spoustu věcí, navíc nechci zobrazit na bílé stránce chybovou zprávu, chci zprávu zobrazit v nějaké html šabloně, což v tomto případě není jednoduše možné.

[Kit]

Ale je to možné. Jenom zatím nevíš, jak pohodlně se to dá pomocí výjimek udělat.

[Greener]

Tak to je super, ale ano, opravdu vůbec netuším jak na to

[Kit]

Z dotazu to není patrné: Chceš aby vznikl konstruovaný objekt, u kterého vznikla v nějaké metodě chyba, s nějakými náhradními nebo prázdnými hodnotami? Nebo aby ten objekt vůbec nevznikl?

[Greener]

Pokud vznikne v nějaké metodě chyba, tak chci tu metodu ihned ukončit a spustit metodu set_layout() která upraví výstupní data a následně spustit metodu __toString(), to mi ke spokojenosti zcela stačí

[Kit]

Promiň, to mi hlava nějak nebere. Nějaký příklad by nebyl? Blok try...catch nemusí být ve stejné metodě jako throw. Ve větvi catch si můžeš upravovat data co hrdlo ráčí.

[Greener]

Jasně, tady je... Děkuji

class Forms{
     private $input    = array();
     private $textarea = array();
     private $layout;
     public function __construct(){
          try{
               $this->getInput();
               $this->getTextarea();
               $this->getButton();
               $this->setLayout();
          }catch (PDOException $e){
               $this->setLayout();
          }
     }
     private function getInput(){
          if($a === true) //Vyvoláme chybu
          throw new Exception('Chyba!');
     }
     private function getTextarea(){}
     private function getButton(){}
     private function setLayout(){
          $this->layout = $input[0].
                          $input[1].
                          $input[2].
                          $textarea[0].
                          $textarea[1];
     }
     public function __toString(){
          return $this->layout();
     }
}

$forms = new Forms();
echo $forms;

[Tarmaq]

je tam dost chyb, ktere pravdepodobne vznikly orezanim ukazky - neslo by sem vlozit celou tridu bez orezavani? Neni mi z prikladu jasne co se ma dosadit do promenne $this->layout pokud nektera z metod vyhodi vyjimku..

[Kit]

Napadá mě, že $input, $textarea a případně i $button mají být vnořené objekty. V tom případě by to vypadalo mnohem lépe.

[Greener]

Nevím co myslíš tím "vnořené objekty". O tom jsem nikdy nečetl ani neslyšel

[Kit]

Uvnitř třídy máš příkazy

private $input = array();
private $textarea = array();
// v konstruktoru
$this->getInput();

Zřejmě metodou getInput() plníš pole $input. Jenže to tak být nemusí. Co třeba takto?

private $input;
private $textarea;
// v konstruktoru
$input = new Input();

a v metodě __toString() ty dva objekty spojíš do jednoho stringu.

[Kit]

Podle mne by proměnná $this->layout vůbec neměla existovat. Přece se i v OOP dělá normalizace. Nebo snad ne?

[Greener]

Souhlas, ale nedokázal jsem přimět metodu __toString() při chybě pracovat.

[Tarmaq]

jasne, ja se proste jen snazil dopidit co ma vlastne ta instance vypsat pri konverzi na string pokud v nejake z tech metod dojde k chybe, jeho pofiderni implementaci jsem neresil..

[Greener]

Jaké chyby myslíš? Do $this->layout jsem chtěl dosadit standardní input, textbox atd..

[Tarmaq]

napr.:

1. v metode __toString() se vola neexistujici metoda $this->layout()
2. v metode setLayout() jsou pouzity nedefinovane lokalni promenne $input a $textarea
3. v metode getInput() je pouzita nedefinovana lokalni promenna $a

[Greener]

1. ano, závorky tam nepatří, čili to má  být takto $this->layout;
2. no to jsou pole, ne proměnné
3. ano, vypadl řádek $a = true;

[Kit]

2. Pole je také proměnná, ale když to chceš přesně, tak:

"v metode setLayout() jsou pouzita nedefinovaná lokalni pole $input a $textarea"

Je to tak lepší?

[Greener]

Je, už to vidím...

[Kit]

Do té větve catch jsem dal výmaz naplněných dat, ale nemusí tam být, pokud mají zůstat. Zrušil jsem metodu setLayout(), protože do atributů objektu se dávají jen atomické a strukturované údaje, nikoli agregáty.

class Forms {
    private $input    = array();
    private $textarea = array();

    public function __construct(){
        try {
            $this->getInput();
            $this->getTextarea();
            $this->getButton();
        } catch (PDOException $e){
            $input    = array();
            $textarea = array();
        }
    }   

    private function getInput() {
        if($a === true) //Vyvoláme chybu
            throw new Exception('Chyba!');
    }       
    
    private function getTextarea() {}
    
    private function getButton() {}
    
    public function __toString(){
        return implode('', $this->input) .
            implode('', $this->textarea);
    }
}

$forms = new Forms();
echo $forms;

[Greener]

Pokud zakomentuji ve tvém příkladu řádek:

//if($a === true) //Vyvoláme chybu

abych vyvolal chybu kterou odchytím a pokračuji dál dle nastaveni, tak dostávam chybu:

Fatal error:  Uncaught exception 'Exception' with message 'Chyba!'

Proč? Díky

[Kit]

Toho jsem si všiml až dodatečně. Jen jsem upravoval tvůj program, ve kterém vyvoláváš Exception, ale zachytáváš PDOException. Pokud tu předponu "PDO" vyhodíš, bude to fungovat.

[Greener]

A odkud ji mám vyhodit? Nikde PDO nezachytávám hm, hm..

[Kit]

Děláš si srandu? Místo

} catch (PDOException $e){

napiš jen

} catch (Exception $e){

Takovou kravinu sis mohl klidně opravit sám. Měl jsi to tak chybně v originále a pouze jsem to přehlédl.

[Greener]

Ne, fakt jsem to neviděl!!!

[Tarmaq]

vzdyt to mas v konstruktoru:

    public function __construct(){
          try{
               $this->getInput();
               $this->getTextarea();
               $this->getButton();
               $this->setLayout();
          }catch (PDOException $e){
               $this->setLayout();
          }
     }

myslel tim, ze misto zachytavani PDOException tam mas mit obecnou Exception, takze tim zachytis jakoukoliv vyjimku, ne jen vyjimky ktere jsou instancemi PDOException

[Greener]

protože do atributů objektu se dávají jen atomické a strukturované údaje, nikoli agregáty.

Můžeš mi prosím tohle objasnit?

[Kit]

Konstruktor objektu analyzuje vstupní data a nastaví podle nich atributy objektu. Metoda __toString() má za úkol z těchto dat vytvořit nějaký řetězec reprezentující objekt. Funguje tedy podobně jako obecná funkce reduce(). Neber metodě __toString() její práci. Ušetříš paměť i čas procesoru.

[Greener]

Tak že

try {
..
} catch (Exception $e){
..
}

a volání metod patří do __toString() ?

[Kit]

Možná. Záleží na tom, co ty metody dělají. pokud analýzu vstupu, tak do konstruktoru. Pokud syntézu výstupu, tak do metody __toString(). Pokud mají jen modifikovat/číst vnitřní stav objektu, tak se volají extra.

[Greener]

Když to shrnu, tak jsem rád, že jsem se zeptal, jelikož je to jednodušší řešení než jsem požadoval. Tak že velké dík!

Nevím, jestli mám založit nové vlákno nebo se může ještě zeptat zde. Když tak ať to admin opraví. Děkuji Stálo by za zmíňku, jak mám odchytit ostatní chyby? Pokud existuje takové jednoduché řešení jako je tohle, tak určitě existuje jednoduché řešení i pro takový základ jako je zpracvání všech chyb.

try {
 echo $a; // tohle by mělo vyhodit chybu, že není inicializovaná prom. $a
} catch (Exception $e){
 echo "$e"; // a zprávu o té chybě bych chtěl dostat sem
}

A pak už bych byl velký frajem a změnil bych si i NICK

[Kit]

Takové chyby by za běhu neměly vznikat. Zkus si funkci isset($a).

[Greener]

Tak to je jen příklad, nabízí se různé nezdary s include/require, práce s mysql atd.. to mám úplně všechno ošetřit sólo? To se z toho ale poseju. Klasika, potřebný kód 10 řádků, ošetření chyb + 200 řádků navíc

[Kit]

A proč? include mám v každém projektu jen 1×, víc jich není potřeba. Ošetření také. Kde máš těch +200 řádek navíc? Možná 20. I kdyby to bylo 200 řádek, tak se to v celém projektu ztratí.

[Josef Kufner]

Chyby v kódu tam mít nemáš. Chyby, se kterými se počítá, vyhodí výjimku.

Pokud něco v kódu přecejen poděláš, použij set_error_handler. V takových případech ale má stránka chcípnout s HTTP kódem 500 nebo vyšším. Pro vývoj existují také různé ladicí nástroje, které umí ukázat podrobnosti, aby se chyby lépe lovily.

Doporučuju také přečíst phptherightway.com.