 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: 6 381 Pomógł: 1116 Dołączył: 30.08.2006 Ostrzeżenie: (0%)
|
Cytat(matix @ 18.04.2014, 12:06:55 )  Zyjemy w czasach, gdzie podejście DependencyInjection to norma. Dlaczego z tego nie skorzystać. Co wcale nie oznacza że w PHP to dobry pomysł. Wręcz bym powiedział że kopiowanie wszystkiego jak leci choćby z Javy już bokiem temu językowi wychodzi. A spróbuj przejąć po kimś większy projekt, jeszcze w dodatku bez komentarzy. Tu jakieś wstrzykiwanie zależności, po drodze zmiana nazw klas, brak komentarzy - brak podpowiadania składni (albo bardzo słabe podpowiadanie) i musisz się wspomagać wyszukiwaniem po plikach całego projektu, jeszcze po drodze jakieś eventy robiące 1000 innych rzeczy. Jest zabawa. Szkoda że taka bezsensowna. |
|
|
|
|
Post
#3
|
|
|
Grupa: Zarejestrowani Postów: 251 Pomógł: 23 Dołączył: 23.04.2013 Ostrzeżenie: (0%)
|
Cytat(viking @ 19.04.2014, 07:47:53 ) Co wcale nie oznacza że w PHP to dobry pomysł. Wręcz bym powiedział że kopiowanie wszystkiego jak leci choćby z Javy już bokiem temu językowi wychodzi. A spróbuj przejąć po kimś większy projekt, jeszcze w dodatku bez komentarzy. Tu jakieś wstrzykiwanie zależności, po drodze zmiana nazw klas, brak komentarzy - brak podpowiadania składni (albo bardzo słabe podpowiadanie) i musisz się wspomagać wyszukiwaniem po plikach całego projektu, jeszcze po drodze jakieś eventy robiące 1000 innych rzeczy. Jest zabawa. Szkoda że taka bezsensowna. PHP jest językiem popularnym (według W3Techs 81% serwerów działa na PeHaPie) i prostym, także mamy w swoim gronie masę amatorów i wanna-be programistów. Jesteśmy skazani na przeróżne "ciekawe" rozwiązania - wystarczy poczytać sobie to forum. (IMG:style_emoticons/default/closedeyes.gif) Ale mogę Cię pocieszyć jednym, nie tylko w PHP może zabraknąć komentarzy (brrr, nie lubię). Kiepski kod i "fractal design" to problem tyczący się każdego języka i wszelkiej maści programistów. Jednak trzeba przyznać, że w PHP z racji popularności tego języka, częściej można spotkać się ze słabym kodem. Jednak pamiętać należy, że mamy w swoim gronie doświadczonych developerów, którzy wybrali ten język jako jedno ze swoich narzędzi. Obyśmy z ich kodem obcowali i uczyli się od nich. Ejmen. (IMG:style_emoticons/default/businesssmiley.png) PS Chcesz zmienić świat na lepsze? Zacznij od siebie! (IMG:style_emoticons/default/wink.gif) PS DI jest okay, stoją za tym mocne walory. Ten post edytował Dejmien_85 19.04.2014, 09:31:08 |
|
|
|
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 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: 15.10.2025 - 06:26 |
