 Ocena kodu OOP - rozwiazania, Prosze o uwagi do implementacji koszyka wg zadanego interfejsu |
| Ocena kodu OOP - rozwiazania, Prosze o uwagi do implementacji koszyka wg zadanego interfejsu |
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 |
|
|
|
Lechus Ocena kodu OOP - rozwiazania 17.04.2014, 08:09:44 
matix No witaj,
Na szybko, co zauważyłem (wieczorem dok... 18.04.2014, 11:06:55 
Dejmien_85 Cytat(skowron-line @ 18.04.2014, 12... 18.04.2014, 23:05:46 viking Cytat(matix @ 18.04.2014, 12:06:55 ) ... 19.04.2014, 06:47:53 Dejmien_85 Cytat(viking @ 19.04.2014, 07:47:53 )... 19.04.2014, 09:27:24 destroyerr @viking ale namieszałeś. Odnośnie przejmowania wię... 19.04.2014, 09:17:13 viking @destroyerr To co pokazałeś to zwykłe rzutowanie t... 19.04.2014, 16:37:43 destroyerr To właśnie mój kawałek kodu pokazuje czym jest DI.... 19.04.2014, 19:06:42 Crozin @viking: Trochę Cię zmartwię. To o czym piszesz to... 19.04.2014, 19:11:32 viking To o czym piszę jest dokładnie implementacja DI we... 19.04.2014, 21:41:12 destroyerr Ale Ty piszesz o implementacji kontenera/menedżera... 19.04.2014, 22:41:00 viking Zdaję sobie sprawę że przydałby mi się urlop i moj... 20.04.2014, 06:24:34 Crozin @viking: Jeszcze raz, wyjaśnijmy sobie proszę bard... 20.04.2014, 09:00:07 viking W porządku. Jeśli potraktować sprawę w ten sposób ... 20.04.2014, 09:40:36 Crozin Cytat[...] Jednak idei nie wprowadzimy w aplikacji... 20.04.2014, 10:06:56 JacekJagiello Corzin, piszesz, że zapis typu:
[PHP] pobierz, pla... 21.04.2014, 18:53:33 Crozin Trochę większy przykład:[PHP] pobierz, plaintext c... 22.04.2014, 14:23:23 JacekJagiello Właśnie, takie użycie kontenera rozpatrywałem w ko... 23.04.2014, 22:24:06 Ormin Cytat(JacekJagiello @ 23.04.2014, 23:24... 25.04.2014, 15:20:56  |
|
Aktualny czas: 14.10.2025 - 22:53 |
