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

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.

[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

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.

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:


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ść.

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

//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

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