Problem z zachowaniem hermetyzacji. Opcje

[daniel1302]

Witam mam takie klasy(klasy są pisane dla przykładu z głowy).

[PHP] pobierz, plaintext 
<?php
abstract class Punkt
{
	private $x;
	private $y;
	private $type;
 
	public function setX($x) {.......}
	public function getX() {.......}
 
	public function setY($y) {.......}
	public function getY() {.......}
 
	.
	.
	.
}
?>
[PHP] pobierz, plaintext 

I to jest klasa główna, wszystkie punkty będą miały te cechy($x,$y, $type)
Będą punkty które posiadają tylko te cechy np:

[PHP] pobierz, plaintext 
<?php
class StandardowyPunkt extends Punkt
{
}
?>
[PHP] pobierz, plaintext 

i tutaj nie ma problemu.

Ale będą punkty które będą posiadały dodatkowe cechy np wysokośc, kolor, obrazek, zależnie od użytego grafu(mam różne typy grafów).

[PHP] pobierz, plaintext 
<?php
class RozszezonyPunkt extends Punkt
{
	private $wysokosc;
	private $color;
	private $obrazek;
 
	public function set*() { .... }
	public function get*() { .... }
}
?>
[PHP] pobierz, plaintext 

I podczas użycia odpowiedniego grafu wybieram odpowiedni opis punktu np

[PHP] pobierz, plaintext 
<?php
class GrafGalileusza extends Graf
{
	public function __construct($file, $classPunkt)
	{
		//Prasuj plik $file.XML do zmiennej $graf
 
		if (!class_exists($classPunkt))
			throw new Exception(....);
 
		//Sprawdź czy klasa dziedziczy po Punkcie
		$Test = new $classPunkt ();
		if (($Test instanceof 'Punkt') === false)
			throw new Exception(...);	
 
		foreach($graf as $coords => $v)
		{
			//$coords jest typu(X:Y)
			list($x, $y) = explode(':', $coords);
			$this -> graf[$x][$y] => new $classPunkt();
			$this -> graf[$x][$y] -> setWysokosc();
                        $this -> graf[$x][$y] -> setColor();
                        $this -> graf[$x][$y] -> set*();
			...
		}
	}
}
?>
[PHP] pobierz, plaintext 

Pytanie mam właśnie do tego momętu:
Gdy ktoś poda jako $classPunkt klasę StandardowyPunkt a klasa GrafGalileusza musi używać klasy RozszezonyPunkt, to wystąpi błąd.

[PHP] pobierz, plaintext 
Co zrobić w takiej sytuacji? 
[PHP] pobierz, plaintext 

Sam się strasznie gmatwam z tym.

Myślałem, żeby zrobić jedną klasę Punkt i do niej dodawać wszystkie cechy punktu które będe używał, lub ktoś sobie doda gdy będzie potrzebował innej cechy.

[PHP] pobierz, plaintext 
<?php
final class Punkt
{
	private $x;
	private $y;
	private $type;
 
	private $wysokosc;
	private $color;
	private $obrazek;
 
	.
	.
	.
}
?>
[PHP] pobierz, plaintext 

Tylko czy tutaj użytkownik nie ma za wielkiej swobody?
Co możecie mi polecić?

Chyba, że na stałe ustalać w klasie GrafGalileusza jaka klasa punktu ma być używana i np zrobic PunktGalileusza

[PHP] pobierz, plaintext 
<?php
class GrafStandardowy extends Graf
{
	private $opisPunktu = 'StandardowyPunkt';
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
class GrafGalileusza extends Graf
{
	private $opisPunktu = 'RozszeżonyPunkt';
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
class GrafOli extends Graf
{
	private $opisPunktu = 'PunktOli';
}
?>
[PHP] pobierz, plaintext 

Pytania pogrubiłem.

Proszę o odpowiedź i dziękuje.

Ten post edytował daniel1302 26.12.2011, 13:08:10

[mortus]

Ciężko jednoznacznie odpowiedzieć na Twoje pytania, bo wszystko zależy od założeń tego projektu. Zauważ tylko, że jeśli GrafGalileusza składa się tylko z punktów rozszerzonych, to ani użytkownik, ani jakaś klasa/metoda pośrednia nie powinny mieć możliwości podania z jakiej klasy punktu korzystamy tworząc ten graf. Przecież to musi być właśnie RozszerzonyPunkt i koniec. W tym przypadku trzeba tylko zweryfikować, czy dane zawarte w pliku xml, służącym do wygenerowania grafu, rzeczywiście odpowiadają klasie RozszerzonyPunkt. Można ewentualnie odpowiednią informację umieścić w tym pliku xml. Wszystko jednak zależy od Twoich potrzeb.

Przykład jest troszkę niedopracowany, więc ciężko tutaj dywagować na jakiekolwiek temat.

Ten post edytował mortus 26.12.2011, 14:17:25

[Crozin]

PHP nie wspiera żadnego mechanizmu w stylu typów generyczych z Javy czy szablonów z C++. Ciężko więc w czasie wykonywania o zarządzanie typami.

1. Jakiś taki dziwny ten graf się wydaje... taki nie grafowaty (IMG:style_emoticons/default/wink.gif) Ani tu wierzchołków tylko punkty, ani tu krawędzi tylko mapa. Zmień nazwę na mapę czy jakiś układ punktów, bo trochę mylące to jest w tej chwili.
2. Konstruktor obiektu mapy (tu: grafu) raczej nie powinien zajmować się pobieraniem danych z pliku i uzupełnianiem siebie. Bo co jak zmienisz sobie źródło albo format danych? Dane wprowadzaj z zewnątrz.
3. Do sprawdzenia czy dana klasa dziedziczy po innej możesz skorzystać z mechanizmu refleksji (Reflection API) albo funkcji is_subclass_of. Tworzenie obiektu tylko po to by sprawdzić czy jego typ jest odpowiedni to zły pomysł - szczególnie dlatego, że trzeba by uwzględnić argumenty konstruktora oraz fakt, że tworzenie tego typu obiektu może być kosztowne.
4. Mapa oraz Punkt powinny być interfejsami.
5. Wiem, że to przykład, ale nigdy nie powinieneś rzucać wyjątkami klasy Exception.

Co do samego problemu. W pełni statycznego typowania po prostu w PHP w tym przypadku nie osiągniesz. Mógłbyś próbować robić jakieś protezy typu __construct(RozszerzonyPunkt $punktBazowy) i później klonować ten obiekt w momencie gdy potrzebna Ci nowa instancja, ale to strasznie kulawe i podatne na błędy rozwiązanie. Przekaż odpowiednią klasę (przez mechanizm refleksji) i w czasie wykonywania sprawdzaj:

[PHP] pobierz, plaintext 
public function __construct(ReflectionClass $class) {
  if (!$class->implementsInterface('RozszerzonyPunkt')) { // czy tam sprawdzanie dziedziczenia po konkretnej klasie
    throw new InvalidArgumentException(sprintf('Argument $class powinien być RozszerzonymPunktem (refleksja), podano: %s', $class->getName()));
  }
}
 
// .....
 
$map = new Map(new ReflectionClass('Rozszerzony punkty'));
[PHP] pobierz, plaintext 

PHP nie ma możliwości osiągnięcia tutaj statycznego, silnego typowania - język na to nie pozwala.

Ten post edytował Crozin 26.12.2011, 14:34:09