Dziwne argumenty metod, prośba o wyjaśnienie Opcje

[Jarod]

Przykład z książki:

[PHP] pobierz, plaintext 
<?php
interface Band {
	public function getName();
	public function addMusician (Musician $musician);
	...
}
 
interface Musician {
	public function addInstrument (Instrument $instrument);
	public function assignToBand(Band $band);
	...
}
 
interface Instrument {
	public function getName();
	public function getCategory();
	...
}
 
 
class Guitarist implements Musician {
 
	...
	private $last;   
	private $instruments;   
	...
 
	public function addInstrument(Instrument $instrument) {
		array_push($this->instruments, $instrument)
	}
 
	...
}
?>
[PHP] pobierz, plaintext 

I teraz mam prosbę o wyjaśnienie

1. Wiadomo, że można tworzyć obiekt gdy definicja klasy występuje w kodzie po utworzeniu obiektu (nie ma konieczności napisania definicji funkcji przed jej wywołaniem). Czy nie wyraźniej (bardziej zrozumiale dla osoby, która przejmuje np. jakiś projekt) byłoby napisać te interfejsy w kolejności: Instrument, Musician, Band ?

2. W definicji interfejsów i klasy (które podałem wyżej) nie rozumię pewnej kwestii i prosiłbym osoby, które dobrze znają OOp o wytłumaczenie:

Dlaczego w interfejsie Musician jest

public function addInstrument (Instrument $instrument);
a nie
public function addInstrument ($instrument); ?

Podobnie jest w klasie Guitarist - dlaczego zamiast
public function addInstrument(Instrument $instrument)
nie jest
public function addInstrument(Instrument $instrument) ?

[mike]

1. Przyjęta kolejnośc to tylko i wyłącznie decyzja autorów. Ja też uważam że powinno być Instrument, Musician, Band
2. Taki zapis oznacza wymuszenie typu. public function addInstrument (Instrument $instrument); oznacza że parametr $instrument musi byc instancją klasy Instrument. Więcej: Podręcznik php :: Classes and Objects (php 5) :: Type Hinting

[bela]

2. Taki zapis oznacza wymuszenie typu. public function addInstrument (Instrument $instrument); oznacza że parametr $instrument musi byc instancją klasy Instrument.

Albo, co chyba najważniejsze, jej potomnych.

[Jarod]

2. Taki zapis oznacza wymuszenie typu. public function addInstrument (Instrument $instrument); oznacza że parametr $instrument musi byc instancją klasy Instrument.

Musi być instancją klasy intrument czy klas, które implementują interfejs instrument?

[mike]

Musi być instancją klasy intrument czy klas, które implementują interfejs instrument?

Hmm, nieprecyzyjnie napisałem.
Musi być instancją klas implementujących interfejs Instrument, lub jak ~bela zauważył klas pochodnych do tych klas.


P.S.
Nie można utworzyć instancji interfejsu.

[Jarod]

Hmm, nieprecyzyjnie napisałem.
Musi być instancją klas implementujących interfejs Instrument, lub jak ~bela zauważył klas pochodnych do tych klas.
P.S.
Nie można utworzyć instancji interfejsu.

Czyli do metody public function addInstrument (Instrument $instrument); można przekazać wszystkie obiekty, które implementują interfejs Instrument?

[PHP] pobierz, plaintext 
<?php
class Klasa1 implements Instruments {
 
	public function addInstrument(Instrument $instrument) {
		array_push($this->instruments, $instrument)
	} 
	...
}
 
class Klasa2 implements Instruments {
 
	public function addInstrument(Instrument $instrument) {
		array_push($this->instruments, $instrument)
	} 
	...
}
 
class Klasa3 implements Musician {
	...
	...
}
 
 
$myClass1 = new Klasa1;
$myClass2 = new Klasa2;
$myClass3 = new Klasa3;
 
$myClass1->addInstrument(Instrument $instrument); //Dobrze?
$myClass2->addInstrument(Instrument $instrument); //Dobrze?
$myClass3->addInstrument(Instrument $instrument); // Źle?
?>
[PHP] pobierz, plaintext 

[mariuszn3]

Wymuszenie typów możesz zastosować tylko w definicji metod, kiedy wywołujesz metode nie podpowiadasz już jakiego typu jest przesłana przez Ciebie zmienna a więc tak:

[PHP] pobierz, plaintext 
<?php
$myClass1->addInstrument($instrument);
?>
[PHP] pobierz, plaintext 

Zakładam, że gdzieś wcześniej określiłeś zmienną $instrument i jest to instancja klasy implementującej Instrument.

Ten post edytował mariuszn3 6.08.2006, 13:34:04

[Jarod]

[PHP] pobierz, plaintext 
<?php
$myClass1->addInstrument($instrument); //Dobrze?
$myClass2->addInstrument($instrument); //Dobrze?
$myClass3->addInstrument($instrument); // Źle?
?>
[PHP] pobierz, plaintext 

Czy to jest poprawne rozumowanie?

[mariuszn3]

Nie wiem rozumowanie jakiej kwestii masz na myśli.. ale zapis jest jak najbardziej prawidłowy..

[Jarod]

Nie wiem rozumowanie jakiej kwestii masz na myśli.. ale zapis jest jak najbardziej prawidłowy..

Chodzi mi o to czy dobrze rozumiem, że

[PHP] pobierz, plaintext 
<?php
$myClass3->addInstrument($instrument);
?>
[PHP] pobierz, plaintext 

jest błędne

[mariuszn3]

Jest błędne tylko dlatego, że Klasa3 nie ma zdefiniowanej metody addInstrument()

[Jarod]

Jest błędne tylko dlatego, że Klasa3 nie ma zdefiniowanej metody addInstrument()

A nawet gdyby miała to i tak byłoby błędnie. Przecież nie implementuje interfejsu Instrument.. :/

[mariuszn3]

To, że nie implementuje nie oznacza, że jej metody nie mogą przyjmować instancji obiektów które implementują instrument.

[Jarod]

To, że nie implementuje nie oznacza, że jej metody nie mogą przyjmować instancji obiektów które implementują instrument.

Zgłupiałem już:

Hmm, nieprecyzyjnie napisałem.
Musi być instancją klas implementujących interfejs Instrument, lub jak ~bela zauważył klas pochodnych do tych klas.

[mariuszn3]

No dobrze.. ale co musi być instancją tych klas? Klasa do której metody się odwołujesz.. czy obiekt, który przekazujesz do metody? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ..przyjrzyj się temu uważnie.

[Jarod]

No dobrze.. ale co musi być instancją tych klas? Klasa do której metody się odwołujesz.. czy obiekt, który przekazujesz do metody? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ..przyjrzyj się temu uważnie.

Nie wiem już o czym piszesz. Możesz to jakoś jaśniej wytłumaczyć?

[mariuszn3]

Przyjrzyj się uważnie podanemu przez siebie przykładowi z książki i temu co Ty później napisałeś. Kluczową różnicą jest to, że w przykładzie z książki klasa Guitarist nie implementuje interfejsu Instrument.. nie mniej jej metoda addInstrument() przyjmuje obiekty, które implementują instrument..
To co później napisałeś to coś zupełnie innego - stwierdzenie, że jeśli klasa sama w sobie nie implementuje danego interfejsu to jej metody nie mogą przyjmować obiektów, które implementują dany interfejs co jest z zasady bzdurnym założeniem.

[envp]

Stary, nie zagłębiaj się, aż tak, bo po pierwsze w tej książce w kodach są błędy, po drugie, autorzy napisali ją tak jakby to było dla nich karą a po trzecie do OOP trzeba dorosnąć i i tego nei nauczysz się tak o .. pstryk i juz (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) OOP to idea programowania. Przeczytaj gdzies tak do narzedzi uzytecznych czy jak to tam bylo i od tamtad zacznij dopiero psiac (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) . Pozatym uzupelniaj sie googlem, bo naprawde ta ksiazka jest 'zakrecowna'