Dotaz: Oprava PHP kódu

[Jakub]

Ahoj, nevidí prosím někdo chybu? Díky za pomoc

     class Zelenina {

          protected $nazev;
          protected $zeme;
          protected $popis;

          public function getNazev() {
               return $this->nazev;
          }

          public function getZeme() {
               return $this->zeme;
          }


          public function getPopis() {
               return $this->popis;
          }
     }

     class Melouny extends Zelenina {

          public function __construct() {

               $this->nazev = 'Meloun vodní';
               $this->zeme  = 'Italy';
               $this->popis = 'Barva melounu je zelená, neobsahuje pecky!';
               return; //nevím jestli sem patří return když plní konstruktor jen proměnné a nic nevraci
          }
     }

     $obj = new Zelenina();

     $nazev = $obj->getNazev();
     $zeme  = $obj->getZeme();
     $popis = $obj->getPopis();

     var_dump($nazev);
     var_dump($zeme);
     var_dump($popis);

Výsledek: NULL NULL NULL

Odpovědi

[Jakub]

OPRAVA! Někde se mi zašantročil konstruktor :D

class Zelenina {

          protected $nazev;
          protected $zeme;
          protected $popis;

          public function __construct() {
               new Melouny();
          }

          public function getNazev() {
               return $this->nazev;
          }

          public function getZeme() {
               return $this->zeme;
          }


          public function getPopis() {
               return $this->popis;
          }
     }

     class Melouny extends Zelenina {

          public function __construct() {

               $this->nazev = 'Meloun vodní';
               $this->zeme  = 'Italy';
               $this->popis = 'Barva melounu je zelená, neobsahuje pecky!';
               return; //nevím jestli sem patří return když plní konstruktor jen proměnné a nic nevraci
          }
     }

     $obj = new Zelenina();

     $nazev = $obj->getNazev();
     $zeme  = $obj->getZeme();
     $popis = $obj->getPopis();

     var_dump($nazev);
     var_dump($zeme);
     var_dump($popis);

[DarkKnight]

vytvaris v rodicovske tride instanci tridy, ktera z ni dedi, return tam nepatri

class Zelenina {

          protected $nazev;
          protected $zeme;
          protected $popis;

          public function __construct() {}

          public function getNazev() {
               return $this->nazev;
          }

          public function getZeme() {
               return $this->zeme;
          }


          public function getPopis() {
               return $this->popis;
          }
     }

     class Meloun extends Zelenina {

          public function __construct() {

               $this->nazev = 'Meloun vodní';
               $this->zeme  = 'Italy';
               $this->popis = 'Barva melounu je zelená, neobsahuje pecky!';
          }
     }

     $obj = new Meloun();

     $nazev = $obj->getNazev();
     $zeme  = $obj->getZeme();
     $popis = $obj->getPopis();

     var_dump($nazev);
     var_dump($zeme);
     var_dump($popis);

[Jakub]

Děkuji, no a když budu volat přímo třídu Meloun() tak to znamená, že už nemůžu mít u jiné třídy s názvem Atributy() extends Zelenina() a plnit tak další třídou s názvem Atributy() proměnné ve třídě Zelenina() ?

Myslím to nějak takto, děkuji

class Zelenina {

          protected $nazev;
          protected $zeme;
          protected $popis;

          protected $vaha;
          protected $cena;
          protected $baleni;

          public function __construct() {}

          public function getNazev() {
               return $this->nazev;
          }

          public function getZeme() {
               return $this->zeme;
          }

          public function getPopis() {
               return $this->popis;
          }

          public function getVaha() {
               return $this->vaha;
          }

          public function getCena() {
               return $this->cena;
          }

          public function getBaleni() {
               return $this->baleni;
          }
     }

     class Meloun extends Zelenina {

          public function __construct() {

               $this->nazev = 'Meloun vodní';
               $this->zeme  = 'Italy';
               $this->popis = 'Barva melounu je zelená, neobsahuje pecky!';
          }
     }

     class Atributy extends Zelenina {

          public function __construct() {

               $this->vaha  = '5 Kg';
               $this->cena  = '10,- Kč/1Kg';
               $this->baleni = '50 Kg/krabice';
          }
     }


     $obj = new Meloun();

     $nazev  = $obj->getNazev();
     $zeme   = $obj->getZeme();
     $popis  = $obj->getPopis();
     $vaha   = $obj->getVaha;
     $cena   = $obj->getVaha;
     $baleni = $obj->getCena;

     var_dump($nazev);
     var_dump($zeme);
     var_dump($popis);
     var_dump($vaha);
     var_dump($cena);
     var_dump($baleni);

[DarkKnight]

ne, nemuzes, na podobne veci slouzi interface, "schopnosti" tridy

nicmene u tohoto by to bylo taky spatne... kdyz mas zeleninu a predpokladas, ze u ni budes mit vahu, cenu, baleni, nazev, zemi, popis, tak to dej do jednoho, treba primo do konstruktoru zeleniny, ktery zavolas z konstruktoru melounu

nebo si vytvor settery danych atributu a tim to nastavuj

[axel]

No v melounu plnis tri a v atributech jine tri, tak kde by se vzaly v tom melounu?. Mozna te mate nesmyslna volba nazvu - dej si misto Atributy "Okurka" a hned ti to bude jasne, proste jsi vytvoril dva druhy zeleniny, ktere spolu nemaji nic spolecneho.

[Kit]

Máš tam napsáno, že třída "Atributy" je Zelenina. Co je to za blbost?

[Jakub]

Není to blbos, jsou to další atributy přímo a jen pro tu třídu Zelenina().

[DarkKnight]

tak proc jsou v jine tride?

[Kit]

Takže podle tebe je atribut zeleninou. Hmm. Meloun je zelenina, ale atribut není zelenina. Kde že rostou atributy? U tebe na zahradě?

[Jakub]

Chlapy já si nesmírně Vážim Vaší snahy o pomoc, ale asi se zle vyjadřuji, zapoměňme na zeleninu, udělal jsem jiný příklad, snad bude demostrovat přesně to, co potřebuji.

     class Data {

          protected $a1;
          protected $a2;
          protected $a3;

          protected $b1;
          protected $b2;
          protected $b3;

          protected $c1;
          protected $c2;
          protected $c3;

          public function a1() { return $this->a1; }
          public function a2() { return $this->a2; }
          public function a3() { return $this->a3; }

          public function b1() { return $this->b1; }
          public function b2() { return $this->b2; }
          public function b3() { return $this->b3; }

          public function c1() { return $this->c1; }
          public function c2() { return $this->c2; }
          public function c3() { return $this->c3; }
     }


     class A { public function __construct() { $this->a1 = 'Chci naplnit proměnnou a1 v class Data';
                                               $this->a2 = 'Chci naplnit proměnnou a2 v class Data';
                                               $this->a3 = 'Chci naplnit proměnnou a3 v class Data'; } }

     class B { public function __construct() { $this->b1 = 'Chci naplnit proměnnou b1 v class Data';
                                               $this->b2 = 'Chci naplnit proměnnou b2 v class Data';
                                               $this->b3 = 'Chci naplnit proměnnou b3 v class Data'; } }

     class C { public function __construct() { $this->c1 = 'Chci naplnit proměnnou c1 v class Data';
                                               $this->c2 = 'Chci naplnit proměnnou c2 v class Data';
                                               $this->c3 = 'Chci naplnit proměnnou c3 v class Data'; } }

     $obj = new ?????????;

     echo $obj->a1().$obj->a2().$obj->a3();
     echo $obj->b1().$obj->b2().$obj->b3();
     echo $obj->c1().$obj->c2().$obj->c3();

Děkuji ;)

[Kit]

Ta zelenina byla lepší. Na té se to dalo vysvětlit, ale na této zrůdě to nepůjde.

[Jakub]

Tak tady je zelenina

     class ZeleninaDetaily {

          public $vaha_1_kus;
          public $vaha_1_baleni;
          public $vaha_1_kontejner;

          public $cena_nakup;
          public $cena_prodej;
          public $cena_zisk;

          public $nazev;
          public $puvod;
          public $popis;

          public function vaha_1_kus()       { return $this->vaha_1_kus; }
          public function vaha_1_baleni()    { return $this->vaha_1_baleni; }
          public function vaha_1_kontejner() { return $this->vaha_1_kontejner; }

          public function cena_nakup()  { return $this->cena_nakup; }
          public function cena_prodej() { return $this->cena_prodej; }
          public function cena_zisk()   { return $this->cena_zisk; }

          public function nazev() { return $this->nazev; }
          public function puvod() { return $this->puvod; }
          public function popis() { return $this->popis; }
     }


     class Vaha { public function __construct() { $this->vaha_1_kus       = 'Chci naplnit proměnnou vaha_1_kus v class Data';
                                                  $this->vaha_1_baleni    = 'Chci naplnit proměnnou vaha_1_baleni v class Data';
                                                  $this->vaha_1_kontejner = 'Chci naplnit proměnnou vaha_1_kontejner v class Data'; } }

     class Cena { public function __construct() { $this->cena_nakup  = 'Chci naplnit proměnnou cena_nakup v class Data';
                                                  $this->cena_prodej = 'Chci naplnit proměnnou cena_prodej v class Data';
                                                  $this->cena_zisk   = 'Chci naplnit proměnnou cena_zisk v class Data'; } }

     class Info { public function __construct() { $this->nazev = 'Chci naplnit proměnnou nazev v class Data';
                                                  $this->puvod = 'Chci naplnit proměnnou puvod v class Data';
                                                  $this->popis = 'Chci naplnit proměnnou popis v class Data'; } }

     $obj = new ??????????();

echo $obj->vaha_1_kus();
echo $obj->vaha_1_baleni();
echo $obj->vaha_1_kontejner();
echo $obj->cena_nakup();
echo $obj->cena_prodej();
echo $obj->cena_zisk();
echo $obj->nazev();
echo $obj->puvod();
echo $obj->popis();

[Jakub]

Sakra, edit!

class ZeleninaDetaily {

          public $vaha_1_kus;
          public $vaha_1_baleni;
          public $vaha_1_kontejner;

          public $cena_nakup;
          public $cena_prodej;
          public $cena_zisk;

          public $nazev;
          public $puvod;
          public $popis;

          public function vaha_1_kus()       { return $this->vaha_1_kus; }
          public function vaha_1_baleni()    { return $this->vaha_1_baleni; }
          public function vaha_1_kontejner() { return $this->vaha_1_kontejner; }

          public function cena_nakup()  { return $this->cena_nakup; }
          public function cena_prodej() { return $this->cena_prodej; }
          public function cena_zisk()   { return $this->cena_zisk; }

          public function nazev() { return $this->nazev; }
          public function puvod() { return $this->puvod; }
          public function popis() { return $this->popis; }
     }


     class Vaha { public function __construct() { $this->vaha_1_kus       = 'Chci naplnit proměnnou vaha_1_kus v class ZeleninaDetaily';
                                                  $this->vaha_1_baleni    = 'Chci naplnit proměnnou vaha_1_baleni v class ZeleninaDetaily';
                                                  $this->vaha_1_kontejner = 'Chci naplnit proměnnou vaha_1_kontejner v class ZeleninaDetaily'; } }

     class Cena { public function __construct() { $this->cena_nakup  = 'Chci naplnit proměnnou cena_nakup v class ZeleninaDetaily';
                                                  $this->cena_prodej = 'Chci naplnit proměnnou cena_prodej v class ZeleninaDetaily';
                                                  $this->cena_zisk   = 'Chci naplnit proměnnou cena_zisk v class ZeleninaDetaily'; } }

     class Info { public function __construct() { $this->nazev = 'Chci naplnit proměnnou nazev v class ZeleninaDetaily';
                                                  $this->puvod = 'Chci naplnit proměnnou puvod v class ZeleninaDetaily';
                                                  $this->popis = 'Chci naplnit proměnnou popis v class ZeleninaDetaily'; } }

     $obj = new ??????????();

echo $obj->vaha_1_kus();
echo $obj->vaha_1_baleni();
echo $obj->vaha_1_kontejner();
echo $obj->cena_nakup();
echo $obj->cena_prodej();
echo $obj->cena_zisk();
echo $obj->nazev();
echo $obj->puvod();
echo $obj->popis();

[Kit]

Je to sice značně anemické řešení, ale až doplníš potřebné metody (zejména __toString()), tak to bude i sloužit.

class ZeleninaDetaily {
    private $info;
    private $vaha;
    private $cena;

    public function __construct(Info $info, Vaha $vaha, Cena $cena) {
        $this->info = $info;
        $this->vaha = $vaha;
        $this->cena = $cena;
    }

    public function vaha_1_kus()       { return $this->vaha->getKus(); }
    public function vaha_1_baleni()    { return $this->vaha->getBaleni(); }
    public function vaha_1_kontejner() { return $this->vaha->getKontejner(); }

    public function cena_nakup()  { return $this->cena->getNakup(); }
    public function cena_prodej() { return $this->cena->getProdej(); }
    public function cena_zisk()   { return $this->cena->getZisk(); }

    public function nazev() { return $this->info->getNazev(); }
    public function puvod() { return $this->info->getPuvod(); }
    public function popis() { return $this->info->getPopis(); }
}

class Info {
    private $nazev;
    private $puvod;
    private $popis;

    public function __construct($nazev, $puvod, $popis) {
        $this->nazev = $nazev;
        $this->puvod = $puvod;
        $this->popis = $popis;
    }

    public function getNazev() { return $this->nazev; }
    public function getPuvod() { return $this->puvod; }
    public function getPopis() { return $this->popis; }
}

class Vaha {
    private $kus;
    private $baleni;
    private $kontejner;

    public function __construct($kus, $baleni, $kontejner) {
        $this->kus       = $kus;
        $this->baleni    = $baleni;
        $this->kontejner = $kontejner;
    }

    public function getKus()       { return $this->kus; }
    public function getBaleni()    { return $this->baleni; }
    public function getKontejner() { return $this->kontejner; }
}

class Cena {
    private $nakup;
    private $prodej;
    private $zisk;

    public function __construct($nakup, $prodej, $zisk) {
        $this->nakup  = $nakup;
        $this->prodej = $prodej;
        $this->zisk   = $zisk;
    }

    public function getNakup()  { return $this->nakup; }
    public function getProdej() { return $this->prodej; }
    public function getZisk()   { return $this->zisk; }
}

$obj = new ZeleninaDetaily(
    new Info('Meloun', 'Portugalsko', 'Prvotřídní melouny z Portugalska'),
    new Vaha(5, 20, 500),
    new Cena(8, 20, 12)
);

echo $obj->vaha_1_kus();
echo $obj->vaha_1_baleni();
echo $obj->vaha_1_kontejner();
echo $obj->cena_nakup();
echo $obj->cena_prodej();
echo $obj->cena_zisk();
echo $obj->nazev();
echo $obj->puvod();
echo $obj->popis();

[Jakub]

Díky za názornou ukázku...

Tak tohle je to slavné OOP ? Rychlý a přehledný kód ? Tak to je tedy pěkná sračka! To vážně někdo tak programuje? To si děláte prdel, ne? No jestli tohle dokáže nekdo po někom přečíst, tak to vážně smekám, ale já se teda takový nepřehledný a nelogický balast ani nebudu snažit pohopit, a nebudu se ho začínat ani učit. Ty jo, teď jsi mě fakt dostal na kolena! A mě někdo bude tvrdit, že procedurální programovaní je nepřehledné a nelogické, v procedurálním programování nemusím aní psát žádné komentáře, ovšem k tomuhle je potřeba snad i manuál....... Mám dost, jdu se vyblít...

PS: Myšleno všeobecně!

[DarkKnight]

rychly a prehledny kod pri dodrzovani OOP vznika, ty pises srackoidni kod, respektive se snazis dosahnout neceho, co by slo vyresit uplne jinak

[Jakub]

rychly a prehledny kod pri dodrzovani OOP vznika, ty pises srackoidni kod, respektive se snazis dosahnout neceho, co by slo vyresit uplne jinak

Proto jsem chtěl proniknout do tajů OOP abych nepsal "srackoidni kod" Vzdycky existuje x způsobů jak něco udělat lépe/hůře prostě jinak

[Kit]

Jenom jsem opravil tvůj sračkoidní kód, aby fungoval. Ty jsi udělal chybu v návrhu.

[Jakub]

Kit-e díky moc, právě jsi mi ušetřil pár let života, které jsem chtěl věnovat OOP v PHP. Jsem rád, že jsi poskytl názornou ukázku OOP programování.

[Kit]

Nepochopil jsi to. Udělal jsi špatnou analýzu a já jsem na jejím základě udělal anemický program. Je to tvoje chyba. Udělej lepší analýzu, popiš, co má být vstupem, co má být výsledkem a já napíšu lepší skript. Definoval jsi příšerně košaté rozhraní a já ho implementoval. Máš, co jsi chtěl.

[Jakub]

Nepochopil jsi to. Udělal jsi špatnou analýzu a já jsem na jejím základě udělal anemický program. Je to tvoje chyba.

Ano, udělal jsem špatnou analízu, přiznávám, je to moje chyba.

Udělej lepší analýzu, popiš, co má být vstupem, co má být výsledkem a já napíšu lepší skript.

Lepší analízu neudělám, ale napíšu prvotní představu jak jsem to chtěl udělat. připadala mi jako nejlepší možná.

1) Mám class Zelenina a tu volám
$obj = new Zelenina();
echo($obj);

2) Uvnitř té class "Zelenina" mám 3 proměnné
private $nazev;
private $puvod;
private $popis;

3) Dále uvnitř té class "Zelenina" potřebuji zavolat další class s názvem zeleniny. Každá taková class nastaví buď
  a) nastaví tyto tři proměnné v class Zelenina, se kterýma můžu dále pracovat
nebo
  b) nastaví tyto tři proměnné ve své class a z class Zelenina získám nějakým způsobem obsah těch proměnných

4) Ty class které se budou volat z class Zelenina musí být primitivní a malinké, nejlépe bez metod které by vypisovali ty proměnné

Bod 1 a 2 je pevně dán, s tím nelze nic dělat.

Díky

[potato]

To, co hledáš, je asi

(a) dědičnost. Tedy vy věci ‚uvnitř‘ třídy Zelenina by ve skutečnosti měly být podtřídy a konstruovat by se měla ta konkrétní zelenina (které is-a Zelenina).

(b) factory, nicméně je to v rozporu s požadavkem, že si zeleninu vyrobíš ručně sám a pak si ji teprve necháš smysluplně inicializovat zvenku.

[potato]

Nebo možná hledáš ORM.

Ale ve všech případech je přístup ‚nejdřív si vyrobím neinicializovaný objekt, a pak si ho nechám zvnějšku nějak inicializovat‘ zdroj problémů.

[Šangala]

1) class nevoláš, ale vytváříš instanci (class „popisuje“ objekt, tedy v tvém případě class zelnina je popis jako http://cs.wikipedia.org/wiki/Zelenina a pomocí new si pořídíš jeden kus této zeleniny).

2) Může být, nicméně to jsou obecnější informace, myslím, že by to mělo být s píše v něčem jako „Produkt“ z čehož se to odvodí. Pokud je Zelenina to prvotní s čím pracuješ a neexistuje nic obecnějšího, tak je to v pohodě, jak to máš (ale nevím reálně čím by šlo začít už tak specificky).

3) Mám radu: „Běž do obchodu (stačí jen virtuálně) a podívej se co je zelenina a co ní lze říct a co s ní lze přímo může udělat (respektive co sama ze sebou umí udělat).“ Pak tě to třeba nebude napadat. Ano může mýt Zelenina metodu RozmnožSe(), která může vrátit další instanci (další kus) Zeleniny.

4) To jsem úplně nepobral, takže nic

Každý takový(následný) příklad kulhá minimálně na jednu nohu, ale zde je:

<?php

abstract class Produkt{
  private $nazev;
  private $puvod;
  private $popis;
  function __construct ($nazev, $puvod, $popis = ''){
    $this->nazev = $nazev;
    $this->puvod = $puvod;
    $this->setPopis($popis);
  }
  public function getNazev(){
    return $this->nazev; 
  }
  public function getPuvod(){
    return $this->puvod; 
  }
  public function getPopis(){
    return $this->popis; 
  }
  public function setPopis($popis){
    $this->popis = $popis; 
  }
  abstract public function RozmnozSe();
};

class Zelenina extends Produkt{
  function __construct ($nazev, $puvod, $popis = ''){
    parent::__construct($nazev, $puvod, $popis);
  }
  public function __toString(){
    return "Zelenina: ".$this->getNazev()." puvodem: " . $this->getPuvod();
  }
  public function RozmnozSe(){
    return new Zelenina($this->getNazev(),$this->getPuvod(),$this->getPopis());
  }
};

class Ovoce extends Produkt{
  function __construct ($nazev, $puvod, $popis = ''){
    parent::__construct($nazev, $puvod, $popis);
  }
  public function __toString(){
    return "Ovoce: ".$this->getNazev()." puvodem: " . $this->getPuvod();
  }
  public function RozmnozSe(){
    return new Ovoce($this->getNazev(),$this->getPuvod(),$this->getPopis());
  }
};

$zelenina = new Zelenina("Cesnek","China","Nelecivy.");
$ovoce    = new Ovoce("Jablko","Italie","Jako amaroun.");
$zelenina_mlade = $zelenina->RozmnozSe();
$zelenina_mlade->setPopis($zelenina_mlade->getPopis() . " Malucky.");

echo $zelenina," popis: ",$zelenina->getPopis(),"<br />\n";
echo $zelenina_mlade," popis: ",$zelenina_mlade->getPopis(),"<br />\n";
echo $ovoce," popis: ",$ovoce->getPopis(),"<br />\n";

echo "Poznamka: ",$zelenina->getNazev()," z ",$zelenina->getPuvod()," stoji za ho[vn]{2}no.";

PS: V tomto případě (a i obecně) lze rozmnožovat pěstitelem pomocí clone.

[Šangala]

S tím, že můžu napsat různé věci, jsem se už smířil, ale jako můžu naspat, že Zelenina může umývat nějakou metodu RozmnožSe, fakt nevím, asi zaskočím za Freud-em…

[Josef Kufner]

PHP nevolá rodičovské konstruktory automaticky. Přidej na začátek každého konstruktoru (vyjma toho u kořenové třídy) toto:

       parent::__construct();

Viz dokumentace.