Dziedziczenie, czy niezależne komponenty? Opcje

[q3trm]

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 

[Evinek]

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. (IMG:style_emoticons/default/biggrin.gif)