 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: 22 Pomógł: 0 Dołączył: 21.11.2012 Ostrzeżenie: (0%)
|
Właśnie, takie użycie kontenera rozpatrywałem w kontekście kontrolera, stąd się zastanawiałem dlaczego to jest źle. Rzeczywiście, w przypadku jakiegoś modułu wpychanie całego kontenera do klasy nie ma sensu, dobrze, że na to zwróciłeś uwagę. Dzięki (IMG:style_emoticons/default/smile.gif)
|
|
|
|
|
Post
#3
|
|
|
Grupa: Zarejestrowani Postów: 64 Pomógł: 0 Dołączył: 3.02.2009 Ostrzeżenie: (0%)
|
Cytat(JacekJagiello @ 23.04.2014, 23:24:06 )  Właśnie, takie użycie kontenera rozpatrywałem w kontekście kontrolera, stąd się zastanawiałem dlaczego to jest źle. Rzeczywiście, w przypadku jakiegoś modułu wpychanie całego kontenera do klasy nie ma sensu, dobrze, że na to zwróciłeś uwagę. Dzięki (IMG:style_emoticons/default/smile.gif) To jest oczywiste dla każdego, kto myśli (IMG:style_emoticons/default/haha.gif) To tak jakby powiedzieć ,,do działania klasy X potrzebuję kontenera DI". Guzik z pętęlką, wcale nie, Twoja klasa X potrzebuje innej klasy Y. Co do oceny, dodałbym jeszcze: nie używaj dziedziczenia, zapoznaj się z kompozycją ponad dziedziczenie. |
|
|
|
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 skowron-line - echo w metodzie
- addItem zmien na add i niech z... 18.04.2014, 11:54:15 
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  |
|
Aktualny czas: 15.10.2025 - 07:03 |
