Forum PHP.pl

witam,
może ktoś z Was mi powie, dlaczego to poniżej działa w Operze a w IE nie?

[HTML] pobierz, plaintext 
if(document.formzgl.prodagd.value=='' || document.formzgl.prodagd.value=='----------' || document.formzgl.prodagd.value=='--| naprawa gwarancyjna' || document.formzgl.prodagd.value=='--| naprawa pogwarancyjna'){
	alert('Proszę wybrać producenta AGD.');
	document.formzgl.prodagd.focus();
	return false;
}
[HTML] pobierz, plaintext 

instrukcja ma zadanie sprawdzać, co zostało wybrane w polu <SELECT>.
jeśli któryś z wpisów, to ma wyrzucać alert.
czy spowodowane to jest złą składnią czy tym, że IE obsługuje JS "na swój sposób"?

pozdrawiam
marcin

Spróbuj pozamykać warunku IF w oddzielne nawiasy. Może to pomoże.
Tak pozatym to troche kiepska składnia (FF nie wyrzuca Ci ostrzeżeń - to znaczy Warningów)?

moim znamien powinieneś dać
if ((document.forms['formzgl'].prodagd.value=='coś')||(document.forms['formzgl'].prodagd.value=='coś')){
//cos tam
}

Probowałem też w ten sposób (zamykając IF-y w osobne nawiasy) - dla Opery jest to bez różnicy.
Wiem, że Opera mocno wspomaga niedokształconych w pisaniu skryptów i przymyka Oko na drobne błędy...
Nawet, jeśli włączę w Operze opcję otwierania konsoli w momencie wystąpienia błędu w JS, to nic mi nie wyrzuca, więc (teoretycznie) ze skryptem jest wszystko w porządku...
Nie wiem, szczerze mówiąc, jaka powinna być prawidłowa składnia.
Spotkałem się właśnie z taką składnią.
Co do zastosowania

[HTML] pobierz, plaintext 
forms['formzgl']
[HTML] pobierz, plaintext 

, to dla Opery też to nie robi różnicy...
Nawet, jeśli zostawię jeden warunek:

[HTML] pobierz, plaintext 
if(document.formzgl.prodagd.value=='')
[HTML] pobierz, plaintext 

to w IE nie działa.
Czyściłem oczywiście pamięć podręczną przeglądarki po każdej zmianie wprowadzanej w pliku ze skryptem, żeby nie było, że ładuje stary skrypt

Dzięki za poradę!
Może masz (inni oczywiście też ) jeszcze jakieś pomysły, bo mi się wyczerpały...

pozdrawiam
marcin

W pełni poprawna składnia to:chociaż to nie rozwiąże problemu. Błąd musi chybależeć gdzieś indziej.

Próbowałeś przed linijką z if() zrobić coś takiego: Dostajesz w IE alert z wartością pola formularza?

Jak coś, to podaj kod HTML formularza oraz sposób wywołania funkcji i jej pełny kod.

a napewno dobre wartosci badasz z pola select?

spróbuj może

@wipo

[HTML] pobierz, plaintext 
alert(document.forms['formzgl'].elements['prodagd'].text);
[HTML] pobierz, plaintext 

wyrzuca alert undefined.

@Zajec

[HTML] pobierz, plaintext 
alert(document.forms['formzgl'].elements['prodagd'].value);
[HTML] pobierz, plaintext 

wyrzuca pusty alert w IE, i alert z prawidłową wartością w Operze...

mój zapis

[HTML] pobierz, plaintext 
alert(document.formzgl.prodagd.value);
[HTML] pobierz, plaintext 

działa tak samo jak Twój, a więc chyba można też w ten sposób pisać
ale my nie o tym...

różnica jest tylko taka, że IE nieprawidłowo przepuszcza dalej, natomoast Opera prawidłowo zatrzymuje wykonywanie skryptu na tym polu (po wyrzuceniu alertu ustawiam na tym polu fokus).

podaję kod formularza:

[HTML] pobierz, plaintext 
<SELECT NAME="prodagd">
	<OPTION>----------</OPTION>
</SELECT>
[HTML] pobierz, plaintext 

i podpięty skrypt:

[HTML] pobierz, plaintext 
alert(document.formzgl.prodagd.value);
	if(document.formzgl.prodagd.value=='----------'){
		alert('Proszę wybrać producenta AGD.');
		document.formzgl.prodagd.focus();
		return false;
	}
[HTML] pobierz, plaintext 

Zamień document.....value na text albo daj w polu
<option value="jakas wartosc"> text </option>
i wtedy badaj value
text podaje Ci opis pola a value jego wartość

@wipo jesteś geniuszem!
podsunąłeś mi pomysł, który okazał się kluczem...

powinno być:

[HTML] pobierz, plaintext 
<SELECT NAME="prodagd">
	<OPTION value="----------">----------</OPTION>
</SELECT>
[HTML] pobierz, plaintext 

i sprawdzenie:

[HTML] pobierz, plaintext 
if(document.formzgl.prodagd.value=='----------'){
	alert('Proszę wybrać producenta AGD.');
	document.formzgl.prodagd.focus();
	return false;
}
[HTML] pobierz, plaintext 

a co było błędem?
sprawdzałem value, którego nie określiłem w option (w ogóle go nie dałem)

a tak na marginesie, to chyba powinno być odwrotnie:


a nie tak?
text podaje Ci wartość pola a value jego opis

Nie do końca

Chyba będzie to czytalne

Ten post edytował wipo 1.10.2006, 08:56:39

oczywiście masz rację...
przecież value to inaczej wartość...

Sprawdzę to zachowanie Opery, jeśli faktycznie dzieje się tak w dokumencie strict to zgłoszę jako bug.

A gdyby ktoś chciał sprawdzać text, to należy użyć konstrukcji

.elements['pole'].innerText

no właśnie...
skąd opera wiedziała, jaki był wybór, jeśli nie miała na podstawie czego sprawdzać?!
tzn. sprawdzałem value, a przecież nie była ona określona dla <option>, bo jej w ogóle nie wstawiłem?
brała pod uwagę text?

ale rokendrol...

Tak właśnie robiła. Niestety to są efekty prób bycia kompatybilnym z IE. Czasami aż przerasta IE.