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

[skowron-line]

- echo w metodzie
- addItem zmien na add i niech zwraca $this; wtedy bedziesz mogl zrobic ->add()->add()
- nie formatuj ceny w metodzie w widok jest tym miejscem.
- getLengthOfString powinno byc w osobnej klasie S z SOLID
- phpdoc (wogole nie wiadomo co mial autor na mysli) np. priceAndDiscounts

[Dejmien_85]

- addItem zmien na add i niech zwraca $this; wtedy bedziesz mogl zrobic ->add()->add()

Nie polecam. O ile kodu jest mało i wiemy o co chodzi, to jednak trzeba myśleć przyszłościowo. Nie wyobrażam sobie tak mętnej nazwy metody jak "add" w jakiejś większej aplikacji. Zbyt duża abstrakcja i zamieszanie, a addItem mówi samo za siebie. Na rzecz czytleności zostawiłbym addItem - przeczytałem za dużo książek z tematyki "Clean Code", aby coś takiego poczynić. Nie utrudniajmy życia innym. (IMG:style_emoticons/default/biggrin.gif)

PS Tak, wiem, to nic wielkiego, ale wszystko zawsze zaczyna się od małego, niewinnego piwa. ; )