![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 10 Pomógł: 0 Dołączył: 4.02.2014 Ostrzeżenie: (0%) ![]() ![]() |
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 |
|
|
![]() |
![]()
Post
#2
|
|
Grupa: Zarejestrowani Postów: 4 340 Pomógł: 542 Dołączył: 15.01.2006 Skąd: Olsztyn/Warszawa Ostrzeżenie: (0%) ![]() ![]() |
- 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 |
|
|
![]()
Post
#3
|
|
Grupa: Zarejestrowani Postów: 251 Pomógł: 23 Dołączył: 23.04.2013 Ostrzeżenie: (0%) ![]() ![]() |
- 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. ; ) |
|
|
![]() ![]() |
![]() |
Aktualny czas: 15.10.2025 - 07:28 |