javascript sprawdzanie wyboru pola <SELECT> i IE Opcje

[mpps]

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"? (IMG:http://forum.php.pl/style_emoticons/default/mad.gif)

pozdrawiam
marcin

[wipo]

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
}

[mpps]

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... (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
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 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Dzięki za poradę!
Może masz (inni oczywiście też (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) ) jeszcze jakieś pomysły, bo mi się wyczerpały...

pozdrawiam
marcin

[Zajec]

W pełni poprawna składnia to:

Kod

document.forms['formzgl'].elements['prodagd'].value

chociaż to nie rozwiąże problemu. Błąd musi chybależeć gdzieś indziej.

Próbowałeś przed linijką z if() zrobić coś takiego:

Kod

alert(document.forms['formzgl'].elements['prodagd'].value);

(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) 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.

[wipo]

a napewno dobre wartosci badasz z pola select?

spróbuj może

Kod

alert(document.forms['formzgl'].elements['prodagd'].text);

[mpps]

@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... (IMG:http://forum.php.pl/style_emoticons/default/mad.gif)

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ć (IMG:http://forum.php.pl/style_emoticons/default/guitar.gif)
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 

[wipo]

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

[mpps]

@wipo jesteś geniuszem! (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)
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) (IMG:http://forum.php.pl/style_emoticons/default/party.gif)

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

[..]
text podaje Ci opis pola a value jego wartość

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

[wipo]

Nie do końca

Kod

<option value="wartość"> Opis </option>
            ^^^^^^    ^^^^^^
              value          text

Chyba będzie to czytalne

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

[mpps]

(IMG:http://forum.php.pl/style_emoticons/default/withstupidsmiley.gif)
oczywiście masz rację...
przecież value to inaczej wartość...

[Zajec]

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

[mpps]

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... (IMG:http://forum.php.pl/style_emoticons/default/blink.gif)

[Zajec]

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... (IMG:http://forum.php.pl/style_emoticons/default/blink.gif)

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