Wszyscy wiemy, że IE7/8 to badziewie jakich mało. Dotychczas myślałem, że te przeglądarkopodobne produkty wtrącają się tylko do HTML i CSS. Nic bardziej błędnego. Okazuje się, że potrafią namieszać również w kodzie PHP.

Mam poniższy kod:

[PHP] pobierz, plaintext 
	public function deleteQuestion ($id)
	{
 
		if ( in_array($id, array_keys($this->_questions)) )
		{
			unset($this->_questions[$id]);	#problematyczna linia
 
			#usuniecie pytania z bazy
			$this->dbm->deleteQuestion($id);
 
			return true;
		}
		return false;
	}
[PHP] pobierz, plaintext 

We wszystkich przeglądarkach działa poprawnie. Natomiast w IE7/8 po wejściu do tej funkcji a następnie do bloku warunkowego, zaznaczona przeze mnie linia powoduje wyjście z funkcji bez zgłoszenia błędu czy wyjątku. Jest to tym bardziej dziwne, że istnieje, a jeszcze bardziej ponieważ kolejna linia usuwająca pytanie z bazy jeszcze się wykonuje! Ale return true już najwyraźniej nie bo funkcja nie zwraca true!

Gdy zmienię kod na:

[PHP] pobierz, plaintext 
if ( $this->questions[$id] )
{
	unset($this->_questions[$id]); #problematyczna linia
}
[PHP] pobierz, plaintext 

sytuacja nie ulega zmianie.

Zaznaczę, że do testów używam emulatorów IE7 na XP (ale różnych i we wszystkich jest tak samo), natomiast IE8 w pełnej wersji również na ten system.

IE nie ma nic do php.

[PHP] pobierz, plaintext 
public function deleteQuestion ($id)
	{
 
var_dump($id);
var_dump($this->_questions);
		if ( in_array($id, array_keys($this->_questions)) )
		{
			unset($this->_questions[$id]);	#problematyczna linia
 
			#usuniecie pytania z bazy
			$this->dbm->deleteQuestion($id);
 
			return true;
		}
		return false;
	}
[PHP] pobierz, plaintext 

I nagle wszystko staje się jasne. Czemu nie robicie głupiego var_dump()? To takie trudne??

Ee raczej się mylisz z tym wtrącaniem się w kod php, raczej chodzi o cache. Odpal debugger i wywołaj gdzieś
$temp = $myObj->deleteQuestion(123);
$TEST= 123;
(ustaw breakpointa dokładnie dla linii $TEST) sprawdź wartość zmiennej $temp, oczywiście poustawiaj odpowiednie wartości zmiennych.

I jeszcze
Temat: Jak poprawnie zada pytanie
Wyświetlanie wszystkich błędów, analiza zmiennych i kupa innych. - To podstawy.

Nie no, aż tak naiwny nie jestem. Wszystko sprawdziłem zanim napisałem.

To, że stanie się jasne na końcu nie ulega wątpliwości ale na razie nie jestem bliżej.

Zastosowałem var_dump w zalecanym miejscu w trochę zmodyfikowanej formie, aby nie zaciemniać obrazu:

[PHP] pobierz, plaintext 
	var_dump($id);
	var_dump(array_keys($this->_questions));
[PHP] pobierz, plaintext 

Dane wejściowe identyczne:

-trzy pytania w obiekcie i w bazie (2423, 2424, 2425)
-usuwane: 2425

Rezultat w IE


Rezultat w FF

No i już wiesz czemu nie działa unset - bo nie masz takiego elementu na liscie, który chcesz usunać. Już wiesz, że błedem nie jest unset a Twoja lista - czyż nie jest od razu jaśniej? Bo moim zdaniem jest, już nie krążysz przy unset, które nie jest niczemu winne


ps: zastosowałeś się do porad z linku co ci podałem? Wyswietlanie wszystkich błędów i itp?

Widać nie sprawdzałeś. Jakbyś sprawdził to byś wiedział, że to nie unset.

Nospor, sprawa nie jest taka prosta jak się na pierwszy rzut oka wydaje. Zwróć uwagę, że napisałem w pierwszym poście


P.S. Oczywiście, że tu musi chodzić o jakieś cache/sesje czy cookies. Ja nie sugeruję, że przeglądarka wchodzi mi na serwer i zmienia tam kod

SKoro nie istnieje na liscie twoje ID, to kod nie ma prawa wejść do tego IFa.

Prosty test:

[PHP] pobierz, plaintext 
public function deleteQuestion ($id)
	{
 
var_dump($id);
var_dump($this->_questions);
		if ( in_array($id, array_keys($this->_questions)) )
		{
die('JUPI, jestem!');
			unset($this->_questions[$id]);	#problematyczna linia
 
			#usuniecie pytania z bazy
			$this->dbm->deleteQuestion($id);
 
			return true;
		}
		return false;
	}
[PHP] pobierz, plaintext 

Co się wyświetla?

Wyświetla się

Dla IE?
To się zdecyduj, co ci wyświetla, bo teraz napisałeś:
array(3) { [0]=> int(2423) [1]=> int(2424) [2]=> int(2425) }
a wcześniej pisałeś:
array(2) { [0]=> int(2423) [1]=> int(2424) }
Więc która wkońcu wersja jest prawdziwa?

Chciałbym się zdecydować, ale to niestety nie ode mnie zależy. Też mnie to dziwi ale takie są właśnie dane na wyjściu.

A jak skasujesz DIE to znowu wyświetlają się tylko 2 elementy?
Znaczy, że Twój kod wykonuje się dwa razy - za pierwszy razem jest ok, a za drugim nie ok

Ok, już sprawdzam...

Na razie nie znalazłem na to dowodów:

W kodzie mam:

[PHP] pobierz, plaintext 
						if ( !($testObject->deleteQuestion($id)) )
						{
							throw new ExtendedException('Nie posiadasz uprawnień do modyfikowania tego pytania.');
						}
[PHP] pobierz, plaintext 

Zmieniłem na:

[PHP] pobierz, plaintext 
						if ( !($testObject->deleteQuestion($id)) )
						{
							throw new ExtendedException('Nie posiadasz uprawnień do modyfikowania tego pytania.');
						}else
						{
							throw new ExtendedException('Wykonane przynajmniej raz dobrze');
						}
[PHP] pobierz, plaintext 

Dane wejściowe te same i w obu przypadkach wynik

zapytam ponownie:
czy bez DIE dla IE wyświetlają się dwa elementy na liście czy 3?
Czy z DIE znowu zaczynają się wyświetlać 3?

Bez die: 2 elementy w tablicy
z die: 3 elementy

Oba przypadki odnoszą się do IE

[PHP] pobierz, plaintext 
var_dump(array_keys($this->_questions));
[PHP] pobierz, plaintext 

=>


a:

[PHP] pobierz, plaintext 
var_dump($this->_questions);
[PHP] pobierz, plaintext 

=>




Jak się chcecie we 2 dogadać, to najpierw (do autora) zacznij robić tak, jak pisze nospor, bo inaczej się pokręcicie zaraz...

Czyli kod wykonuje ci się wiecej niż jeden raz - to oczywiste i stwierdzone naukowo. Teraz zastanów się czemu.
@drPayton bo on var_dump robi na array_keys(). Przecież pisał o tym.

@drPayton dał jednak dobry trop:

O dziwo jeśli zrobię var_dump($this->_questions) to wszystko działa poprawnie. Sam fakt dodania tej linii do kodu! Wtedy zawsze pokazuje, że są 3 elementy w tablicy.

Ale jak usunę tę linię

[PHP] pobierz, plaintext 
var_dump($this->_questions);
[PHP] pobierz, plaintext 

to znowu to samo.

To musi mieć jakiś związek z wyjściem. Tak samo z tym die. Jeśli jest coś więcej na wyjściu kodu to wszystko działa.

Kod nie wykonuje się dwa razy. Mam w kodzie tylko jedno użycie deleteQuestion i jeśli test, który zrobiłem parę postów wyżej nic nie wykazał to nie ma takiej możliwości.

Control question:

$this->dbm != $this?

Czyli na pewno nie zapętlasz działania tego skryptu? (identyczne nazwy metod, mały błąd przy definicji $this->dbm i kaput...)

@drPayton - o zapętlaniu nie ma mowy, choć jeszcze sprawdzę ten wątek.

Dlaczego podejrzewam funkcję unset? Ponieważ to die(), które @nospor zaproponował wykona się jeśli jest przed nią. Umieszczone po tej linii gdzie jest unset() już się nie wykona.

Edit: sprawdziłem zmieniając nazwy OBU deleteQuestion na różne. Bez zmian. Zresztą o zapętlaniu nie mogło być mowy. $dbm to obiekt kontaktujący się z bazą i operujący bezpośrednio na niej.

No i teraz wkońcu podałeś konkretny powód.

Włączyłeś wyświetlanie wszystkich błędów jak cię prosiłem już o to kilka razy? Przejrzałeś logi?

Ustawienie raportowania na E_ALL wyrzuciło parę błędów.

Jednak więcej w FF niż w IE. W obu przypadkach szereg "Undefined index" i "Undefined variable".

Nie wydają się mieć związku z tematem i raczej nikomu poza mną nic nie powiedzą ale wklejam dla większej przejrzystości:



Natomiast zastanawia mnie jedno. Dlaczego te błędy pojawiają się tylko właśnie przy usuwaniu pytań. Wysłałem parę innych formularzy. Część z nich polegająca na usuwaniu innych obiektów z bazy i tych błędów nie było.

Edit: już wiem czemu tylko przy usuwaniu pytań się pojawia, bo w złym miejscu dałem

[PHP] pobierz, plaintext 
error_reporting(E_ALL);
[PHP] pobierz, plaintext 

Poprawiłem i teraz błędy są już wszędzie.

-----------------------------------------------------------------------------
Podsumujmy zatem naszą arcyciekawą (dzięki IE) sytuację:

Aplikacja wchodzi w tejże "przeglądarce" (7/8) prawidłowo do funkcji deleteQuestion i do bloku warunkowego if. Gdy już wejdzie do bloku if to w poniższym kodzie die() przerwie działanie.

[PHP] pobierz, plaintext 
public function deleteQuestion ($id)
	{
		if ( in_array($id, array_keys($this->_questions)) )
		{
die('JUPI, jestem!');
			unset($this->_questions[$id]);	#problematyczna linia
 
			#usuniecie pytania z bazy
			$this->dbm->deleteQuestion($id);
 
			return true;
		}
		return false;
	}
[PHP] pobierz, plaintext 

Natomiast tu już nie przerwie:

[PHP] pobierz, plaintext 
public function deleteQuestion ($id)
	{
		if ( in_array($id, array_keys($this->_questions)) )
		{
 
			unset($this->_questions[$id]);	#problematyczna linia
die('JUPI, jestem!');
			#usuniecie pytania z bazy
			$this->dbm->deleteQuestion($id);
 
			return true;
		}
		return false;
	}
[PHP] pobierz, plaintext 

To die() w tym drugim przypadku wykonuje się ale jakby tylko na poziomie funkcji deleteQuestion(). Nic po tej instrukcji już się nie wykona, czyli:

[PHP] pobierz, plaintext 
			#usuniecie pytania z bazy
			$this->dbm->deleteQuestion($id);
 
			return true;
[PHP] pobierz, plaintext 

nie zostanie wykonane. Natomiast poza funkcją deleteQuestion kod okazuje się dalej wykonywać jakby tego die() nie było w ogóle.


Funkcja deleteQuestion na 100% wykonuje się raz!

Raportowanie błędów na poziomie E_ALL nie wykazało istotnych błędów.

edit-----------------------------------------------------
Problem zostawiam nierozwiązany bo odkryłem, że problem występuje tylko lokalnie - u mojego providera nie. Obsługę IE7 wyłączam 1 stycznia 2012 a IE8 1 stycznia 2013 - a do tego czasu pozostanę najprawdopodobniej u tego providera.

Ale włos się się jeży na myśl co to microsoftowskie g**no potrafi.

Za przeproszeniem: Pieprzysz farmazony że to aż strach czytać.


1. Popraw błędy które się Tobie wyświetlają.
2. Pokaż więcej kody szczególnie tych funkcji które wykonujesz.
3. Przeglądarka do kodu na serwerze ma tyle co piernik do wiatraka.
4. Opisz normalnie (logicznie) problem.

1. Nie za bardzo mi się w tej chwili chce, choć kiedyś do tego wrócę i na pewno poprawię. Choć nie mają one związku z opisywaną sytuacją to mają z pewnością (marginalny ale jednak) potencjał problemotwórczy.

2. Pokazałem to co istotne. Cała aplikacja ma przynajmniej kilka tysięcy linii kodu, nikt tego raczej nie będzie analizował.
3. Temu ile ma wspólnego poświęciłem już jedno zdanie i nie będę się powtarzał.
4. Nie wiem co rozumiesz pod pojęciem normalnie (logicznie). Coś tu można jeszcze jaśniej napisać niż ja to zrobiłem? Nie sądzę.

A jak ma się wykonać skoro przerywasz skrypt?


Jeżeli die się nie wykonuje to znaczy że warunek nie został spełniony.

No nie do końca.

Tak jak pisałem. Die() się wykonuje, ale na poziomie funkcji. Co przez to rozumiem?

- die() przerywa funkcję niczym break pętlę,
- funkcja zostaje przerwana ale kod skryptu wykonuje się dalej
- nie wyświetla się komunikat z die() - wynika to poniekąd z tego, że skrypt wykonuje się dalej