Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Ocena kodu OOP - rozwiazania, Prosze o uwagi do implementacji koszyka wg zadanego interfejsu
Lechus
post
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
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
viking
post
Post #2





Grupa: Zarejestrowani
Postów: 6 381
Pomógł: 1116
Dołączył: 30.08.2006

Ostrzeżenie: (0%)
-----


Zdaję sobie sprawę że przydałby mi się urlop i moja zdolność tłumaczenia innym kodu drastycznie ostatnimi czasy spadła, liczyłem jednak że mnie zrozumiecie. Po pierwsze, PHP samo z siebie nie odczyta w Twoim kodzie zależności tylko potrzebujesz dołączyć kod jakiejś biblioteki DI (stąd mówię o implementacjach i akademickich teoriach, to samo tyczy artykułu Fowlera na który się powołujesz). Może jeszcze dodam że wszystkie porównania w pierwszym poście robiłem do Springa. Po drugie, czy widzisz różnicę między swoim kodem:

  1. /**
  2.   * @ var LoggerInterface
  3.   */
  4. private $logger;
  5.  
  6. public function setLogger(LoggerInterface $logger)
  7. {
  8. $this->logger = $logger;
  9. }


a

  1. /**
  2.   * @Inject
  3.   * @var LoggerInterface
  4.   */
  5. protected $logger;
  6.  
  7. /**
  8.   * @Inject
  9.   */
  10. public function setLogger(LoggerInterface $logger)
  11. {
  12. }


Czy teraz rozumiesz co miałem na myśli pisząc o "zwykłym rzutowaniu typów"? Po co był cały mój wywód o komentarzach i braku adnotacji w PHP jako języku (cały czas mając na uwadze IoC Springa, konfigurację beansów, @Autowired)? I nie próbujcie mi wmawiać że mieszam tutaj DI z SM/SL bo nie robię tego tylko pokazałem użycie implementacji DI jako antywzorca. Jeśli nie rozumiecie różnicy to nawet udało się znaleźć linka: http://php-di.org/doc/getting-started.html (sekcja Get objects from the container). I tutaj Dejmien_85 bardzo dobrze zauważył że jak programista jest słaby to stosowanie IoC narobi bardzo dużo szkody. I na koniec. Kto używa ZF2 i przypomni w ramach dyskusji dlaczego powstaje ZF3 i co było źle? ;-) (w szczególności to, że programiści "źle zrozumieli koncepcję")
Go to the top of the page
+Quote Post

Posty w temacie
- 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
- - 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


Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 12.03.2026 - 13:30