Przekazywanie parametrów funkcji w klasie Opcje

[Eldiane]

Mam pewien problem, mam oto takie dwie funkcje w klasie:

[PHP] pobierz, plaintext 
public $przyciski = array('TWORZENIE_STRONY_OBIEKTY.php', 'Button2', 'Button3', 'Button4');
 
	public function SprURL($przyciski)
	{
		for($i=0; count($przyciski) >= $i ; $i++)
		{
			if(strstr($_SERVER['PHP_SELF'], $przyciski[$i]))
			{
				$Przycisk_on = $przyciski[$i];
				return $Przycisk_on;
			}
		}
	}
 
	public function lewe_menu($przyciski)
	{
	?>
		<table valign="top" border="1" width="100%">
				<?php
				for($i=0; count($this->przyciski) > $i ; $i++)
				{
					?>
					<tr>
						<td align="center">
						<?php
							if($this->SprURL($przyciski) == $this->przyciski[$i])
								echo $przycisk_on;
							else
								echo "<a href=\"http://localhost\">".$this->przyciski[$i]."</a>";
						?>
						</td>
					</tr>
					<?php
				}
				?>
		</table>
	<?php
	}
[PHP] pobierz, plaintext 

Funkcja pierwsza SprURL($przyciski) powinna zwrócić jeden parametr (return $Przycisk_on). Dlaczego gdy wywołam ją w innej funkcji (chodzi tutaj o konstrukcje if, $this->SprURL($przyciski) == $this->przyciski[$i]) nie wysyła tego parametru, już późno być może mózg wysiada ;P ale nie wiem dlaczego tak jest, gdy funkcję zadeklaruje bez klasy wszystko jest OK, zostaje zwrócona wartość, dlaczego natomiast w klasie jest zupełnie inaczej?

Z góry dziękuje
Eldiane

[mortus]

Zwróć uwagę na linie 26 i 27. W tej chwili skrypt nie wie, czym jest zmienna $przycisk_on i domyślnie nadaje jej wartość null. Po prostu nie przypisujesz zmiennej $przycisk_on wartości, jaką ma zwrócić funkcja SprURL(), co można zrobić w linii 26 w warunku:

[PHP] pobierz, plaintext 
if(($przycisk_on = $this->SprURL($przyciski)) == $this->przyciski[$i])
[PHP] pobierz, plaintext 

Wszystko działa jak należy.

W wykryciu błędu mogło pomóc włączenie wyświetlania wszystkich błędów zgodnie z zaleceniami w tym temacie.

[Eldiane]

[PHP] pobierz, plaintext 
<?php
$przycisk_on = $this->SprURL($przyciski);
	if($przycisk_on == $this->przyciski[$i])
		echo $przycisk_on;
	else
	        echo "<a href=\"http://localhost\">".$this->przyciski[$i]."</a>";
?>
[PHP] pobierz, plaintext 

Zmieniając składnie na taką jaką podałeś nadal występuje błąd, zawsze przechodzi do warunku else... Tak jakby zmienna $przycisk_on była nadal wartością NULL (IMG:style_emoticons/default/thumbsdownsmileyanim.gif)

[mortus]

Po wywołaniu tego:

[PHP] pobierz, plaintext 
$przycisk_on = $this->SprURL($przyciski);
[PHP] pobierz, plaintext 

sprawdź, czym jest zmienna $przycisk_on:

[PHP] pobierz, plaintext 
var_dump($przycisk_on);
[PHP] pobierz, plaintext 

Jeśli się okaże, że jej wartość to null, to przeanalizuj funkcję SprURL() sprawdzając przy tym zawartość $_SERVER['PHP_SELF'].
Swoją drogą funkcja SprURL() jest źle napisana, bo nie zwraca jakiejś domyślnej wartości, która niekoniecznie musi się znaleźć w tablicy $przyciski.

W ogóle całość jest do poprawy, bo logiki w tym nie ma żadnej. Na przykład obie funkcje korzystają z tej samej tablicy przycisków, więc po co je przekazywać, jako parametr.

[bastard13]

Twoje przyciski to:

[PHP] pobierz, plaintext 
public $przyciski = array('TWORZENIE_STRONY_OBIEKTY.php', 'Button2', 'Button3', 'Button4');
[PHP] pobierz, plaintext 

W metodzie SprUrl() masz warunek:

[PHP] pobierz, plaintext 
if(strstr($_SERVER['PHP_SELF'], $przyciski[$i]))
[PHP] pobierz, plaintext 

A ze strony http://www.php.net/manual/en/function.strstr.php masz to co zwraca funkcja:

Returns the portion of string, or FALSE if needle is not found.

Teraz już powinno wszystko być jasne.

A poza tematem to metoda SprUrl() powinna przyjmować i sprawdzać tylko jeden przycisk, ponieważ teraz w metodzie lewe_menu() wielokrotnie przechodzisz przez tablice $przyciski.
Dodatkowo, jeżeli zostawisz to tak, jak jest teraz to SprUrl() zawsze zwróci (o ile zwróci, bo w tym momencie jest to void) tą samą wartość.

[Eldiane]

Dzięki wszystkim za pomoc, jeżeli wystąpią jeszcze jakieś problemy to się zgłoszę =]

//Edit użyłem funkcji strpos, ponieważ przeczytałem że działa ona troszeczkę szybciej od strstr, zawsze lepiej nauczyć się dobrych nawyków, teraz wszystko (chyba) działa już poprawnie, chodź wskazówki byłyby mile widziane (IMG:style_emoticons/default/smile.gif)

//Edit
Kolejna poprawka, nie chce robić nowych postów więc staram się wciskać wszystkie prace jakie dokonam w jednym temacie, dopóki nie zrobię tego dobrze. Teraz kolejne pytanie - jak nadać wartość zmiennym $przyciski public $przyciski_rek?

$Obiekt = new Strona();
$Obiekt->przyciski = array('Button1', 'Button2')
$Obiekt->przyciski_rek = array('Reklama1', 'Reklama2')

Nie daje pożądanego rezultatu, gdy napisze tak jak wyżej wartość zmiennej $tablica w funkcji URL jest wartością NULL (IMG:style_emoticons/default/thumbsdownsmileyanim.gif)

A oto sam kod:

[PHP] pobierz, plaintext 
class Strona(){
		public $przyciski = array('Button1', 'Button2');
		public $przyciski_rek = array('Reklama1', 'Reklama2');
 
	private function URL($tablica)
	{		
		for($i=0; count($tablica) > $i ; $i++)
		{
		?>
			<tr>
				<td align="center">
					<?php
						$link_true = strpos($_SERVER['PHP_SELF'],$tablica[$i]);
							if(!($link_true === false))
								echo $tablica[$i];
							else
								echo "<a href=\"http://localhost/Tworzenie_strony_obiekty/".$tablica[$i].".php\">".$tablica[$i]."</a>";
					?>
				</td>
			</tr>
		<?php
		}
	}
 
	public function lewe_menu($przyciski)
	{
	?>
		<table valign="top" border="1" width="100%">
				<?php
					$przyciski = $this->przyciski;
					$this->URL($przyciski);
				?>
		</table>
	<?php
	}
 
	public function prawe_menu($przyciski_rek)
	{
	?>
		<table valign="top" border="1" width="100%">
				<?php
					$przyciski_rek = $this->przyciski_rek;
					$this->URL($przyciski_rek)
				?>
		</table>
	<?php
	}
}
[PHP] pobierz, plaintext 

Z góry dziękuje.
Eldiane

Ten post edytował Eldiane 16.08.2011, 16:41:44