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]

[...] Jednak idei nie wprowadzimy w aplikacji dlatego zacząłem pisać o implementacji i zarazem kontenerach.

Właśnie cały ten nasz wywód był w celu odróżnienia implementacji DI od DIC-ów. Przykład @destroyerra:

[PHP] pobierz, plaintext 
class SomeService
{
  /**
   * @ var LoggerInterface 
   */
  private $logger;
 
  public function setLogger(LoggerInterface $logger)
  {
    $this->logger = $logger;
  }
 
  public function execute()
  {
    $this->logger->debug('hello');
  }
}
[PHP] pobierz, plaintext 

Jest już kompletną*, nieakademicką implementacją IoC w formie DI (konkretnie zrealizowaną przez setter injection). I tego typu konstrukcje spotkamy w większości bibliotek i aplikacji - tych napisanych z zachowaniem minimum standardów jakości. Nie potrzebujemy tutaj żadnych kontenerów zależności by z tego kodu korzystać z pełnym wykorzystaniem zalet obiektówki i DI.

* tak tylko w ramach formalności, należałoby tutaj zauważyć, że logger powinien zostać przekazany w konstruktorze, albo w metodzie execute musielibyśmy sprawdzić czy aby na pewno pod $this->logger jest jakiś obiekt, a nie NULL. Ale to już w tym przykładzie jest nieistotne. (IMG:style_emoticons/default/wink.gif)