Cześć,
mam tu taki przykładzik z artykułu z lipca 2022 i nie rozumiem dlaczego autor podał parametry klasy przy tworzeniu obiektu. W artykule nie ma tego wyjaśnione, natomiast Visual Studio Code mówi mi przy tworzeniu obiektu, że klasa nie ma żadnego konstruktora, więc zostanie wywołana bez parametru. Ok, jasne, ale więc dlaczego podano te parametry?

klasa:

[PHP] pobierz, plaintext 
<?php
 
class GermanShepherd
{
    //...
 
    private $eye_color;
    private $dob;
    private $does_shed = true;
    private $kingdom = "Animalia";
    private $phylum = "Chordata";
    private $class = "Mammalia";
    private $order = "Carnivara";
    private $family = "Canidae";
    private $genus = "Canis";
    private $species = "Canis Lupus";
    private $subspecies = "Canis Lupus Familiaris";
    private $breed = "German Shepherd Dog";
    private $fur_color;
 
    //...
 
    /**
     * @return string
     */
    public function getEyeColor(): string
    {
        return $this->eye_color;
    }
 
    /**
     * @param string $eye_color
     */
    public function setEyeColor(string $eye_color): void
    {
        $this->eye_color = $eye_color;
    }
 
    /**
     * @return string
     */
    public function getDob(): string
    {
        return $this->dob;
    }
 
    /**
     * @param string $dob
     */
    public function setDob(string $dob): void
    {
        $this->dob = $dob;
    }
 
    /**
     * @return bool
     */
    public function isDoesShed(): bool
    {
        return $this->does_shed;
    }
 
    /**
     * @param bool $does_shed
     */
    public function setDoesShed(bool $does_shed): void
    {
        $this->does_shed = $does_shed;
    }
 
    /**
     * @return string
     */
    public function getFurColor(): string
    {
        return $this->fur_color;
    }
 
    /**
     * @param string $fur_color
     */
    public function setFurColor(string $fur_color): void
    {
        $this->fur_color = $fur_color;
    }
 
    //...
}
[PHP] pobierz, plaintext 

Wywołanie:

[PHP] pobierz, plaintext 
$gs_dog = new GermanShepherd("Brown", "Jan 20, 2018", "Brown");
echo $gs_dog->getEyeColor();
$gs_dog->setEyeColor("Blue");
echo $gs_dog->getEyeColor();
[PHP] pobierz, plaintext 

W ogóle kiedy to sie odpali to wywala błąd:

No widac blad autora, pewnie zapomnial o konstruktorze.
Temu tez masz potem blad
Fatal error: Uncaught TypeError: GermanShepherd::getEyeColor(): Return value must be of type string, null returned
bo probujesz pobrac eyeColor, ktory jeszcze nie zostal okreslony. Jakbys to robil w konstruktorze, co sugeruje kod, to tego bledu bys nie mial

po tym komentarzu autora
"
In the example above, we use the constructor to create a new GermanShepherd with Brown eyes.
"

Widac, ze zapomnial poprostu dodac konstruktor w artykule

choc na koncu artykulu ten konstruktor juz jest

public function __construct(string $eye_color,
string $dob,
string $fur_color)
{
$this->eye_color = $eye_color;
$this->dob = $dob;
$this->fur_color = $fur_color;
}


czytac to konca

No i teraz jest to dla mnie jasne skąd te parametry i czemu w takiej kolejności:)
Dziękuję.

PS. Ileż ja czasu natraciłem i w łeb zachodziłem co robię źle przez wszelkie błędy, literówki i niedopatrzenia w różnych artykułach i książkach

EDIT:

Hej, tak się trochę bawię, celem lepszego zrozumienia konceptu i mam pewną zagwozdkę.

[PHP] pobierz, plaintext 
class Test
{
    private int $a;
    private int $b;
    private int $wynik;
 
    public function __construct(int $a, int $b)
    {
        $this->a = $a;
        $this->b = $b;
    }
 
    public function setA(int $a): void
    {
        $this->a = $a;
    }
 
    public function getA(): int
    {
        return $this->a;
    }
 
    public function setB(int $b): void
    {
        $this->b = $b;
    }
 
    public function getB(): int
    {
        return $this->b;
    }
 
    public function setWynik(): void
    {
        $this->wynik = $this->a + $this->b;
    }
 
    public function getWynik(): int
    {
        return $this->wynik;
    }
 
}
[PHP] pobierz, plaintext 

Przy tak zbudowanej klasie musze podać $a i $b przy tworzeniu obiektu a konstruktor je sam stworzy. Moge więc tworzyć obiekt w taki sposób:

[PHP] pobierz, plaintext 
$obj = new Test(10, 10);
echo $obj->getA();
echo $obj->getB();
$obj->setWynik();
echo $obj->getWynik();
[PHP] pobierz, plaintext 

W wyniku dostanę 10 10 20 ($a, $b, $wynik).

Ale mogę równierz zupełnie pozbyć się konstruktora, wtedy wywołanie będzie wyglądać jak poniżej, a wynik będzie ten sam:

[PHP] pobierz, plaintext 
$obj = new Test();
$obj->setA(10);
echo $obj->getA();
$obj->setB(10);
echo $obj->getB();
$obj->setWynik();
echo $obj->getWynik();
[PHP] pobierz, plaintext 

To czy w takim wypadku konstruktor jest niezbędny? Czy mozna z niego zrezygnować i będzie to zgodne ze sztuką? I pytanie w drugą stronę - czy jeśli mam konstruktor to powinienem robić setter/getter aby było zgodnie ze sztuką. Czy to tylko czysty wybór programisty?

Jeżeli parametry a i b mają być niezmienne w całym cyklu życia obiektu, to zostaw konstruktor i wywal settery dla nich.

Metoda `setWynik` nic mi nie mówi, więc musisz zmienić jej nazwę - na np. `calculate`. Po kodzie widać, że dodajesz, więc może lepiej `calculateSum`?
Co do gettera dla wyniku - zależy. Jeżeli operacje obliczenia są skomplikowane i czasochłonne, to przypisanie będzie dobrym rozwiązaniem. W innym przypadku, to metoda `calculate` powinna już coś zwracać.

Można jeszcze zrobić tak, że `calculate` będzie metodą prywatną, a `getWynik` będzie mieć if w środku i sprawdzać czy `->wynik` nie jest pusty. Aczkolwiek to rozwiązanie łąmie trochę SOLID.

Nie do końca dobrze jest to zapisane. Brakuje wartości domyślnej dlatego jeżeli wywołasz getter przed setterem to php rzuci błędem "Typed property Test::$wynik must not be accessed before initialization". Ale tak, możesz pozbyć się konstuktora. Czasami stosuje też zapis np Test::fromArray(array $values) który inicjuje hurtem jakieś wartości.

edit: Oczywiście jeśli parametry "mają być niezmienne w całym cyklu życia obiektu" to po to powstało readonly.

Rozumiem. Dzięki za wyjaśnienia.
Tak, rzeczywiście setWynik nie ma sensu bo ona liczy tylko a i b.

to nie lepiej użyć wtedy stałych?

Stalej nie okreslisz w konstruktorze,

Stała, to co innego niż niezmienny param klasy w całym cyklu życia obiektu.