Logika w PHP Opcje

[seaquest]

Otóż mam sobie metodę klasy Member:

[PHP] pobierz, plaintext 
<?php
/**
 * Finds a row by specified key
 *
 * @param string $key
 * @return Zend_Db_Table_Row|null
 * @author Michal Plachta
 **/
public function findByKey( string $key )
{
	return $this->fetchRow( array( 'key = ?' => $key ) );
}
?>
[PHP] pobierz, plaintext 

Wywołuję ją w sposób następujący:

[PHP] pobierz, plaintext 
<?php
$member = new Member;
$current = $member->findByKey( $key );
?>
[PHP] pobierz, plaintext 

I co dostaję?

Argument 1 passed to Member::findByKey() must be an instance of string, string given ...

I gdzie tu logika? Dodam tylko, że analogiczna sytuacja dla typu array działa.
Daję na Hydepark, bo nie jest to dla mnie problem nie do rozwiązania. Traktuję to raczej jako ciekawostkę. Jak ktoś uzna, że trzeba przenieść, niech przeniesie.

Ten post edytował seaquest 20.09.2007, 23:49:02

[Cysiaczek]

Stricte dyskusj to ja nie widziałem - powtarzam tylko słowa developerów, którzy przy okazji komentowania próśb o wymuszanie typów argumentów powiedzieli, że jest to niezgodne z ideą php.
Co do typów wartości zwracanych - nie zgadzam się, że nie ma sensu przy braku takiego samego mechanizmu przy argumentach. Wywołujący funkcję (użytkownik kodu) musi zadbac o poprawne argumenty, ale już to, co otrzyma niekoniecznie musi mu odpowiadać. Dam taki przykład:

[PHP] pobierz, plaintext 
<?php
class someData
{
	function getData($bool=false)
	{
		if($bool)
		{
			return new dataObject();
		}
		else
		{
			return array("no", "data");
		}
 
	}
}
$obj=new someData()
$data=$obj->getData();
// i teraz nastepują testy
	if(is_array($data)){}
	  elseif(is_object($data)){}
?>
[PHP] pobierz, plaintext 

Gdyby metoda getData() miała ustalony typ wartości zwracanej, to testy niebyłyby potrzebne. Odpowiedzialnośc spadłaby na metodę getData(), która musiałaby zwracać np. tablicę.

[PHP] pobierz, plaintext 
<?php
function Array getData($bool=false)
	{
		if($bool)
		{
			$obj=new dataObject();
			return $obj->getAsAray(); //albo jak kto tam chce
		}
		else
		{
			return array("no", "data");
		}
 
	}
?>
[PHP] pobierz, plaintext 

Niweluje to trochę zamętu w kodzie. Ja czasami piszę tak, jakby oba obiekty były umówione co do typu, ale to utrudnia wysledzenie błędu, bo musze go szukać w dwóch obiektach. Omijam zatem testowanie, co może się odbić czkawką przy późniejszyxch zmianach. Gdyby dali mi możliwość wymuszenia typu - od razu miałbym błąd i nie musiałbym odtwarzać procesu, który do niego doprowadził. Tak się dzieje najczęsciej, gdy zwrócona wartość nie wpływa bezpośrednio na kod użytkownika, ale jest jedynie zapamiętana i wykorzystana w jakimś odległym miejscu, albo (o zgrozo!) zapisana.

Pozdrawiam.