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

[viking]

@destroyerr To co pokazałeś to zwykłe rzutowanie typów i jakaś metoda statyczna więc nie wiem co tam jest lepsze gorsze. Ja mam na myśli odnoszenie się do kodu poprzez stringi czyli np

[PHP] pobierz, plaintext 
$di->get('Db\Adapter\Oracle');
[PHP] pobierz, plaintext 

Odpada ci cała zaleta podpowiadania składni bo nazwę musisz znać i jeszcze ją całą ręcznie wklepać. Ktoś teraz załóżmy miał wizję i zrobił coś takiego:

[PHP] pobierz, plaintext 
$im = $di->instanceManager();
$im->addAlias('trolololo', 'Db\Adapter\Oracle');
[PHP] pobierz, plaintext 

I skąd ty do siedmiu piekieł masz wiedzieć że jakieś trolololo używane wszędzie w kodzie to adapter Oracle? Albo znasz super aplikację (która załóżmy była wcześniej rozwijana przez kogoś dwa lata), albo masz mega dokumentację. Jak żadnego to praktycznie leżysz i sobie humor psujesz. Częste używanie słów na K słabo działa na ogólne samopoczucie. Druga sprawa że jeśli ktoś był leniwy i nie chciało mu się dodawać:

[PHP] pobierz, plaintext 
/* @var $db Db\Adapter\Oracle */
[PHP] pobierz, plaintext 

to masz jeszcze bardziej przechlapane.
Kolejną sprawą jest że praktycznie nie ma dobrego sposobu na dumpa takich kontenerów bo dostajesz całą masą rekurencji. DI jest też zazwyczaj potwornie wolne w działaniu.

PHP jako językowi brakuje podstaw do takich zabaw. To nie Java. Nie ma choćby adnotacji i od dłuższego czasu wszyscy się głowią czy je wprowadzić czy dalej udawać że te śmieszne, parsowane dodatkowo komentarze (co daje kolejny narzut i jeszcze wolniejsze wykonywanie) zostawić jak są czy w końcu coś z tym zrobić. IoC w takiej postaci jak teraz to zwyczajne "widzimisię". Inni mają to my też się pobawimy.