[PHP] Instrukcja if / elseif w funkcji, no kurde, mądrzejszego tytułu nie wymyśliłem ;d Opcje

[red9skull]

Ta funkcja działa:

[PHP] pobierz, plaintext 
function resize() 
   {
	$x = $this->getWidth();
	$y = $this->getHeight();
 
	if($this->rodzaj == "pionowy")
	{
		$width = 90;
		$radio = $x / $width;
 
		if($radio < 0)
		{
			$height = $y * $radio;
			$height = (int)$height;
		}
		elseif($radio > 0)
		{
			$height = $y / $radio;
			$height = (int)$height;
		}
 
		$new_image = imagecreatetruecolor($width, $height);
		imagecopyresampled($new_image, $this->image, 0, 0, 0, 0, $width, $height, $this->getWidth(), $this->getHeight());
		$this->image = $new_image;   
	}
	elseif($this->rodzaj == "poziomy")
	{
		$height = 90;
		$radio = $y / $height;
 
		if($radio < 0)
		{
			$width = $x * $radio;
			$width = (int)$width;
		}
		elseif($radio > 0)
		{
			$width = $x / $radio;
			$width = (int)$width;
		}
 
		$new_image = imagecreatetruecolor($width, $height);
		imagecopyresampled($new_image, $this->image, 0, 0, 0, 0, $width, $height, $this->getWidth(), $this->getHeight());
		$this->image = $new_image;   
	}	
   }
[PHP] pobierz, plaintext 

Natomiast ta nie:

[PHP] pobierz, plaintext 
function resize() 
   {
	$x = $this->getWidth();
	$y = $this->getHeight();
 
	if($this->rodzaj == "pionowy")
	{
		$width = 90;
		$radio = $x / $width;
 
		if($radio < 0)
		{
			$height = $y * $radio;
			$height = (int)$height;
		}
		elseif($radio > 0)
		{
			$height = $y / $radio;
			$height = (int)$height;
		}
	}
	elseif($this->rodzaj == "poziomy")
	{
		$height = 90;
		$radio = $y / $height;
 
		if($radio < 0)
		{
			$width = $x * $radio;
			$width = (int)$width;
		}
		elseif($radio > 0)
		{
			$width = $x / $radio;
			$width = (int)$width;
		} 
	}
 
	$new_image = imagecreatetruecolor($width, $height);
	imagecopyresampled($new_image, $this->image, 0, 0, 0, 0, $width, $height, $this->getWidth(), $this->getHeight());
	$this->image = $new_image;
   }
[PHP] pobierz, plaintext 

Kod jest mój i stąd pytanie: czemu muszę dodawać 2 razy ten fragment:

[PHP] pobierz, plaintext 
$new_image = imagecreatetruecolor($width, $height);
imagecopyresampled($new_image, $this->image, 0, 0, 0, 0, $width, $height, $this->getWidth(), $this->getHeight());
$this->image = $new_image;
[PHP] pobierz, plaintext 

A nie jeden tak jak w drugim przykładzie?

Ten post edytował red9skull 20.06.2010, 16:35:18

[Zyx]

Miło byłoby, gdybyś jeszcze napisał, czym objawia się "niedziałanie". Obstawiam, że pole "rodzaj" ma inną wartość, niż "pionowy" oraz "poziomy", przez co w drugim wariancie funkcji "wspólny" kod próbuje korzystać ze zmiennych, które nie powstały.

Ponadto masz następujące błędy i niedoróbki:

1. Nie lepiej rodzaj oznaczać liczbami i dozwolone stany opisać stałymi? Tekst w takim wypadku to bardzo niewydajna i podatna na błędy forma definiowania.
2. Nie uwzględniasz sytuacji, gdy ratio (bo radio to do słuchania jest) wynosi równe 0. Wprawdzie w obliczeniach zmiennoprzecinkowych ciężko jest tak precyzyjnie trafić, ale nie możesz tego nigdy wykluczyć, bo jeśli się trafi, Twoja funkcja skalująca nie stworzy zmiennej $height. Polecam Ci po prostu zastąpić elseif przez else.

[red9skull]

Miło byłoby, gdybyś jeszcze napisał, czym objawia się "niedziałanie".

Zmienne $width, $height nie istnieją.

ratio (bo radio to do słuchania jest)

Ups (IMG:style_emoticons/default/smile.gif)

Btw:

Kod

function proporcje()
   {
       $x = $this->getWidth();
    $y = $this->getHeight();
    
    if($x - $y < 0)
    {
        $this->rodzaj = "pionowy";
    }
    elseif($x - $y > 0)
    {
        $this->rodzaj = "poziomy";
    }
   }

Rozwiązałem problem: zdjęcia były KWADRATOWE (IMG:style_emoticons/default/biggrin.gif)

Ten post edytował red9skull 20.06.2010, 16:46:29