🚀 go-pugleaf

Dnia Sun, 14 Apr 2019 22:27:45 +0200, Marek S napisał(a):

> Prościej byłoby w konstruktorze przypisać do zmiennej prywatnej:
> 
> $this->prv=new MarekService();
> 
> ...i dalej w kodzie posługiwać się $this->priv zamiast każdorazowo 
> wstrzykiwać coś.

Autowiring ma za zadanie dostarczyć Ci gotowy obiekt. Jeśli potrzebujesz go
w wielu miejscach, to rzecz jasna najwięcej sensu ma wstrzyknięcie go w
konstruktorze.

Z kolei powyższy przykład nie ma sensu - jeśli potrzebujesz nowy obiekt, to
go utwórz, do tego autowiring nie jest potrzebny.

> Po drugie, przy wstrzykiwaniu powstała sprzeczność. Popatrzcie na metodę 
> prywatną $this->getRandom(). Wstrzykuję do niej MarekService, a z 
> drugiej strony chcę ją użyć bez przekazywania parametrów (na dole 
> skryptu), bo nie ma po co ich przekazywać. Metoda ma mi coś zwrócić i 
> koniec, kropka. Gdybym bazował na $this->priv, to taka sprzeczność nie 
> miałaby miejsca.

Po pierwsze to ta metoda ma wyjątkowo nieczytelną nazwę. getRandom...co?

Najpierw określ, co ta metoda (i usługa jako taka) ma w ogóle robić, bo na
razie zwraca tablicę?/kolekcję? gifts bez żadnych zmian. A MarekService to
już w ogóle z niczym się nie kojarzy - ale to jak rozumiem po prostu testy.

-- 
Borys Pogoreło
borys(#)leszno,edu,pl