Forum PHP.pl

Witam, napotkałem na pewien problem. Mam utworzony obiekt 'router' który ma za zadanie zwracać odpowiednie zmienne. Deklaruję taką metodę:

[PHP] pobierz, plaintext 
<?php
public function postVar($name) {
		if (isset($this->postArray[$name])) return $this->postArray[$name];
		else return null;
	}
?>
[PHP] pobierz, plaintext 

I teraz chcę sprawdzić, czy zwracana wartość nie jest pusta:

[PHP] pobierz, plaintext 
<?php
if (empty($router->postVar('submit'))) {
	echo 'pusta';
}
else {
	echo 'jest';
}
?>
[PHP] pobierz, plaintext 

Powyższy kod nie wygeneruje żadnego błędu, ale spowoduje wyświetlenie się pustej strony. Będzie ona pusta nawet wtedy, gdy przed całym warunkiem dodam jakieś echo.
Poniższy kod działa:

[PHP] pobierz, plaintext 
<?php
$submit = $router->postVar('submit');
 
if (empty($submit)) {
	echo 'pusta';
}
else {
	echo 'jest';
}
?>
[PHP] pobierz, plaintext 

Nie mam pojęcia dlaczego tak się dzieje, dla mnie to trochę nielogiczne jest. Pewnie po prostu czegoś nie rozumiem, ale nie mogę samodzielnie tego uzasadnić...
Dodaję log do wincachegrind, jeżeli pomoże on w rozwiązaniu problemu: http://www.sendspace.pl/file/LGfYFlka/

Proszę o pomoc i pozdrawiam

Ten post edytował l0ud 30.04.2008, 15:28:22

sprawdz czy isset($this->postArray[$name]))
czyli odpowiednia wlasciwosc obiektu nie jest wczesniej zainicjowana/stworzona mimo, ze pusta

wtedy isset nie daje null tylko return
a empty widzi, ze empty i daje "pusta"

WersjaOpisPHP 5As of PHP 5, objects with no properties are no longer considered empty.

PHP 4As of PHP 4, The string value "0" is considered empty.



prawdopodobnie tu sęk

kszychu, wszystkie błędy powinny być wyświetlane. Na samym początku całości jest:

[PHP] pobierz, plaintext 
<?php
error_reporting(E_ERROR | E_WARNING | E_PARSE | E_NOTICE | E_STRICT);
?>
[PHP] pobierz, plaintext 

I dotychczas to działało. Jeżeli dostawię gdzieś jakąś literkę to wyskakuje parse error, więc wszystko ok...

cbagov, problem w tym, że warunek z empty zupełnie nic nie zwraca. Po prostu kod przerywa się nie dając żadnego wyniku.

[edit]

Przyjrzałem się jednak temu error_reporting. Teraz wygląda tak

[PHP] pobierz, plaintext 
<?php
error_reporting(E_ALL | E_ERROR | E_WARNING | E_PARSE | E_NOTICE | E_STRICT);
?>
[PHP] pobierz, plaintext 

No i coś się zmieniło - mam błąd


Teraz tylko pozostaje zrozumieć, dlaczego on występuje...

Ok, wyjaśnione


Ten post edytował l0ud 30.04.2008, 15:44:23

Dokładnie, problem rozwiązany

Ostatecznie zrobiłem na ten sposób:

[PHP] pobierz, plaintext 
<?php
if ($router->postVar('submit') === null) {
	echo 'puste';
}
else {
	echo 'jest';
}
?>
[PHP] pobierz, plaintext 

Ja polece zrobienie
$router->postIsset('submit');

[PHP] pobierz, plaintext 
<?php
function postIsset($sName)
{
return isset($sName);
}
?>
[PHP] pobierz, plaintext 

Wygodnie się wtedy pisze

Ten post edytował bim2 30.04.2008, 15:59:07

W manualu jest wszystko ładnie opisane: empty" title="Zobacz w manualu PHP" target="_manual


(sekcja: Parameters)

@bim2 Obawiam sie, ze tak 'sprawdzalbys' stan slowa ['submit']
a l0ud trzyma wartosci w tablicy



[quote name='bim2' date='30.04.2008, 16:58:52 ' post='466812']
Ja polece zrobienie
$router->postIsset('submit');

function postIsset($sName)
{return isset($sName);}

Ten post edytował cbagov 30.04.2008, 16:29:45

To jest masło maślane. Co innego empty() którego parametr musi być zmienną. (chyba że źle pamiętam, bo teraz nie sprawdzę)
Do tego jednak lepiej spr. jak napisał l0ud, czyli porównanie wartości zwróconej.
Bo w odróżnieniu od zmiennej, ma ona konkretną wartość.