Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Dziedziczenie, czy niezależne komponenty?
q3trm
post
Post #1





Grupa: Zarejestrowani
Postów: 83
Pomógł: 1
Dołączył: 26.02.2013

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


Cześć.


Mam dwie klasy, Pierwsza która pobiera wyniki wykonania odrębnych klasa. Druga klasa Pobierającą wyniki pierwszej klasy i zamieniającą je na dane sesyjne.

Moje pytanie, czy zgodne z zasadami OOP byłoby połączenie, a właściwie wykonanie tych operacji w jednej klasie?


  1.  
  2. class ImageParameter // kąt oraz pliki zdjęć
  3. {
  4. protected $angleImage;
  5. protected $fileImage;
  6.  
  7.  
  8. public function downloandParam(OperationDir $operationDir, AngleImage $angleImage )
  9. {
  10. $this ->fileImage = $operationDir ->getArrayDir();
  11. $this ->angleImage = $angleImage ->getAngleImage();
  12. }
  13.  
  14. public function getAngleImage()
  15. {
  16. return $this ->angleImage;
  17. }
  18. public function getFileImage()
  19. {
  20. return $this ->fileImage;
  21. }
  22. }
  23.  
  24.  
  25. class SessionParameter
  26. {
  27. protected $instance;
  28.  
  29. function takeInstance (ImageParameter $imageParameter)
  30. {
  31. $this ->instance = $imageParameter;
  32. }
  33. function sessionAngleImage()
  34. {
  35. return $_SESSION['angleImage'] = $this ->instance ->getAngleImage();
  36. }
  37. function sessionFileImage()
  38. {
  39. return $_SESSION['fileImage'] = $this ->instance ->getFileImage();
  40. }
  41.  
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
Crozin
post
Post #2





Grupa: Zarejestrowani
Postów: 6 476
Pomógł: 1306
Dołączył: 6.08.2006
Skąd: Kraków

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


@q3trm: Mógłbyś proszę przepisać swój pierwotny post - czytam go kolejny raz i nadal nie jestem pewien czy aby na pewno dobrze rozumiem Twoje pytanie. Nie mniej jednak to co zasugerował @Evinek jest najprawdopodobniej poprawnym rozwiązaniem. Jedyne co można by w nim poprawić to zamiast jawnie oczekiwać konkretnej klasy (tutaj: Session) skorzystać z interfejsu (jakiś tam SessionInterface) czy nawet jeszcze lepiej, z ogólnego interfejsu jakiegoś "magazynu" (np. StorageInterface, którego rozwinięciem mógłby być interfejs sesji), bo po co ograniczać swój kod do współpracy z tylko jednym magazynem danych? Oczywiście, jeżeli w Twoim przypadku byłoby to przesadą nie musisz "uciekać" do interfejsów - jednak jest to bardzo dobra praktyka:
  1. interface StorageInterface {
  2. public function get($key);
  3. public function getAll();
  4. public function set($key, $value);
  5. public function has($key);
  6.  
  7. ..
  8. }
  9.  
  10. interface SessionInterface extends StorageInterface {
  11. public function getId();
  12. ...
  13. public function getFlash($key);
  14. public function setFlash($key, $value);
  15. public function hasFlash($key);
  16. }
  17.  
  18. class ImageParameter {
  19. ...
  20.  
  21. public function __construct(StorageInterface $storage) {
  22. ..
  23. }
  24. }


@Szymciosek: Było to już poruszane wielokrotnie. Google: https://www.google.com/search?q=dependency+...me&ie=UTF-8 czy bardziej ogólnie https://www.google.com/search?q=ioc&aq=...600&bih=775 (IoC).
Go to the top of the page
+Quote Post

Posty w temacie


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: 14.10.2025 - 02:15