Ocena kodu OOP - rozwiazania, Prosze o uwagi do implementacji koszyka wg zadanego interfejsu Opcje

[Lechus]

Szanowni profesjonaliści,

Proszę, w wolnej chwili, o przejrzenie kodu i udzielenie cennych wskazówek jak ten kod można ulepszyć.

Kod na Githubie

Plik READ ME.txt - tu jest opis co miało być zaimplementowane.
Instrukcja implementacji interfejsu

[Crozin]

Trochę większy przykład:

[PHP] pobierz, plaintext 
class MyService {
  protected $container;
 
  public function __construct($container) {
    $htis->container = $container;
  }
 
  public function doSth() {
    ...
    $mailer = $this->container->get('my.mailer.service');
    $mailer->doSth();
  }
 
  public function doSthElse() {
    ...
    $repo = $this->container->get('my.repo');
    $obj = $repo->create(..);
 
    ...
 
    $repo->save($obj);
  }
}
[PHP] pobierz, plaintext 

1. Tutaj na dobrą sprawę w ogóle nie mamy właściwie DI. Co prawda sam kontener zależności jest przekazywany w sposób typowy dla DI, ale nie on nas tutaj interesuje.
2. Jakie zależności ma obiekt typu MyService? Nie wiadomo.
3. Czy da się napisać testy dla czegoś takiego? Da. Ale w strasznie niewygodny sposób.
4. Czy utworzę dwa obiekty klasy MyService korzystające z dwóch różnych mailerów? Jak się odpowiednio wygimnastykuję to może tak. Zamiast prostego:

[PHP] pobierz, plaintext 
$realService = new MyService($mailer, $repo);
$dummyService = new MyService(new DummyMailer(), $repo);
[PHP] pobierz, plaintext 

Muszę robić jakieś cuda w postaci klonowania całego kontenera, ustawiania mu jednego innego serwisu i przekazywania dopiero tego do MyService. A i tak cholera wie czy to w ogóle dalej będzie działało.

To tak na szybko...

W podobny sposób używa się większości kontenerów IoC... przynajmniej w framewrokach PHP.

Raczej zawsze odradza się bezpośrednie korzystanie z kontenera. Jedynie wewnątrz kontrolerów jest to nagminnie "promowane". Ale jest to jeszcze raczej akceptowane ze względu na to, że kontrolery są pod tym względem na tyle specyficzne, że często jest to zwyczajnie w świecie bardziej opłacalne. Pamiętajmy, że ostatecznie te techniki mają nam pomóc, a nie być sztuką dla sztuki. (IMG:style_emoticons/default/wink.gif)