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?

[PHP] pobierz, plaintext 
 
 class ImageParameter // kąt oraz pliki zdjęć
  {
    protected $angleImage;
    protected $fileImage;
 
 
	  public function downloandParam(OperationDir $operationDir, AngleImage $angleImage )
	  {
	    $this ->fileImage = $operationDir ->getArrayDir();
	    $this ->angleImage = $angleImage ->getAngleImage();
	  }
 
	  public function getAngleImage()
	  {
	    return $this ->angleImage;
	  }
	   public function getFileImage()
	  {
	    return $this ->fileImage;
	  }  
   }
 
 
   class SessionParameter
   {
    protected $instance;
 
     function takeInstance (ImageParameter $imageParameter)
	 {
	    $this ->instance = $imageParameter;
	 }
     function sessionAngleImage()
	 {
	   return $_SESSION['angleImage'] = $this ->instance ->getAngleImage();
	 }
	 function sessionFileImage()
	 {
	   return $_SESSION['fileImage'] = $this ->instance ->getFileImage();
	 }
 
[PHP] pobierz, plaintext 

To moje zdanie, a w OOP nie jestem aż tak dobry.

Jedna klasa powinna być od Sesji.
Druga powinna w konstruktorze dostawać wstrzykniętą klasę sesji. Wtedy operacje zapisu powinny być w klasie Image.
Przykład:

[PHP] pobierz, plaintext 
class ImageParameter 
{
    protected $session;
 
    public function __construct(Session $session){
        $this->session = $session;
    }
 
    public function saveToSession(){
        $this->session->set('angleImage', $this->getAngleImage());
        $this->session->set('fileImage', $this->getFileImage());
        return $this;
    }
 
}
[PHP] pobierz, plaintext 

Najlepiej żebyś ktoś mądrzejszy się wypowiedział, a jak mój tok myślenia będzie dobry to będę szczęśliwy.

Mi się podoba

Możecie mnie przy okazji nakierować co nam daje wstrzykiwanie? Dependency Injection.

@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:

[PHP] pobierz, plaintext 
interface StorageInterface {
  public function get($key);
  public function getAll();
  public function set($key, $value);
  public function has($key);
 
  ..
}
 
interface SessionInterface extends StorageInterface {
  public function getId();
  ...
  public function getFlash($key);
  public function setFlash($key, $value);
  public function hasFlash($key);
}
 
class ImageParameter {
  ...
 
  public function __construct(StorageInterface $storage) {
    ..
  }
}
[PHP] pobierz, plaintext 

@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).

Evinek, dał mi rozwiązanie mojego dylematu. Crozin, natomiast Twój kod dał mi sporo do myślenia, aktualnie mój skrypt nie wymaga owego zastosowania, ale na pewno mi się przyda w najbliższej przyszłości.