[PHP]Niedziałający return. Opcje

[krzywy36]

[PHP] pobierz, plaintext 
<?php
function reSize($width, $height)
{
	if($width > 640 OR $height > 480)
	{
		if($width > $height AND $width > 640)
		{
			$height = ($height * 640)/$width;
			$width = 640;
		}
		elseif($width > $height AND $height > 480)
		{
			$width = ($width * 480)/$height;
			$height = 480;
		}
		reSize($width, $height);
	}
	else
	{
		return $width;
	}
}
 
$zmienna = reSize(720, 576);
 
echo $zmienna;
?>
[PHP] pobierz, plaintext 

funkcja nic nie zwraca. Jeśli nad return wpisze echo $width; to wyświetla prawidłowo, więc skrypt przechodzi do tego else ale return coś nie chce działać, ktoś widzi gdzie jest bląd?

[pedro84]

Usuń else, bo przecież chcesz, żeby zawsze zwracało Ci $width, prawda?

Kod powinien wyglądac tak:

[PHP] pobierz, plaintext 
<?php
function reSize($width, $height)
{
	if($width > 640 OR $height > 480)
	{
		if($width > $height AND $width > 640)
		{
			$height = ($height * 640)/$width;
			$width = 640;
		}
		elseif($width > $height AND $height > 480)
		{
			$width = ($width * 480)/$height;
			$height = 480;
		}
		reSize($width, $height);
	}
	return $width;
}
 
$zmienna = reSize(720, 576);
 
echo $zmienna;
?>
 
[PHP] pobierz, plaintext 

[krzywy36]

tylko, że w tym kodzie $width zwróci się nawet jeśli będzie większe od 640 a ma się zwrócić dopiero jeśli warunki będą spełnione.
Generalnie ma tam zwracać tablice składającą się z $wdth i $height ale dla uproszczenia kodu niech narazie zwraca samo $width.
Chodzi o to że w kodzie który podałeś jest zwracane $width = 640; a ma być zwracane $width = 600 czyli dopiero po kolejnym wywołaniu funckji.
Jakieś pomysły?

Edit: dam jeszcze przykład:

[PHP] pobierz, plaintext 
<?php
function reSize($width, $height)
{
	if($width > 640 OR $height > 480)
	{
		if($width > $height AND $width > 640)
		{
			$height = ($height * 640)/$width;
			$width = 640;
		}
		elseif($width > $height AND $height > 480)
		{
			$width = ($width * 480)/$height;
			$height = 480;
		}
		reSize($width, $height);
	}
	else
	{
		$size = array($width, $height);
		print_r($size);
		return $width;
	}
}
 
$zmienna = reSize(720, 576);
 
echo $zmienna;
?>
[PHP] pobierz, plaintext 

ten kod drukuje tablice z prawidłowymi danymi (600, 480) ale nie zwraca nic ;/

Ten post edytował krzywy36 25.04.2010, 13:55:21

[pedro84]

Teraz łapię.

Zobacz, zwracasz $width dopiero po else, widzisz? Nie zwracasz zmiennej wcześniej, czyli Twój drugi przykład powinien wyglądać tak:

[PHP] pobierz, plaintext 
<?php
function reSize($width, $height)
{
	if($width > 640 OR $height > 480)
	{
		if($width > $height AND $width > 640)
		{
			$height = ($height * 640)/$width;
			$width = 640;
		}
		elseif($width > $height AND $height > 480)
		{
			$width = ($width * 480)/$height;
			$height = 480;
		}
		reSize($width, $height);
                return $width;
	}
	else
	{
		$size = array($width, $height);
		print_r($size);
		return $width;
	}
}
 
$zmienna = reSize(720, 576);
 
echo $zmienna;
?>
 
[PHP] pobierz, plaintext 

[krzywy36]

fakt, zwraca ale ciągle 640 ;/ ten else jest właśnie po to, żeby nie zwracało dopóki nie będzie mniej niz 640, czyli ciągle nie działa :<