onchange visible

[Ely88M]

Ciao
Premetto che di javascript non ne so niente.....
Ho un combobox con un elenco di valori presi da un db.
dovrei associare un javascript che mi controlli se il combo ha un certo valore la form1 diventa visibile altrimenti la form3 diventa visibile....
ho provato così... ma non funziona
Questo è lo script e di seguito l'intestazione del combo

<script>
<!--
function validate(this) {
if (document.selects.value=="") {alert("Selezionare una tabella");return false;}
else{if (document.selects.value=="2") {form1.style.visibility= "visible"}
else {form3.style.visibility= "visible"}
}}} }
//-->
</script>

<select id="selects" name="selects" style="width: 150px;" onchange="validate(this);">

chi mi aiuta a scoprire perchè non va??

Grazie

[homezappa]

Inizia da qui (non sono entrato nel merito della funzione):
Se tu avessi scritto il codice con un minimo di indentazione,
ti saresti accorto subito che in fondo alla funzione
c'erano due graffe chiuse di troppo!

<script>
<!--
function validate(this) {
	if (document.selects.value=="") {
		alert("Selezionare una tabella");return false;
	}
	else {
		if (document.selects.value=="2") {
			form1.style.visibility= "visible"
		}
		else {
			form3.style.visibility= "visible"
		}
	}
}
//-->
</script>

Poi, usa Firefox, con l'estensione FireBug, per 'debuggare' i javascript, te lo consiglio!

HTH
Zappa

[Ely88M]

Grazie per il suggerimento proverò firefox

anche con le tue correzioni non funziona comunque...
hai altri suggerimenti?

scusa se ti rompo... ma non ne capisco niente

[Goonie]

<script>
<!--
function validate(this) {

var MioForm = window.document.NomeForm;
var IdValSelezionato = MioForm.selects.selectedIndex;

if(MioForm.selects[IdValSelezionato].value==''){
alert("Selezionare una tabella");
return false;
}else{
if(MioForm.selects[IdValSelezionato].value==2){
MioForm.style.visibility='visible';
}else{
MioForm.style.visibility='hidden';
}
}

}
//-->
</script>

ps non so se dico una baggianata comunque selects come id/name non so se possa dare problemi, potrebbe essere uno di quei nomi "riservati" da non usare sui eventuali oggetti del form

[deleted_110]

questo va

la parola this e' riservata e non puoi usarla nella definizione di una funzione

function validate(selectObj) {
  if (selectObj.value=="") 
  {
    alert("Selezionare una tabella");return false;
  }
  else
  {
    if (selectObj.value=="2") 
    {
      document.form1.style.visibility= "visible"
    }
    else 
    {
      document.form3.style.visibility= "visible"
    }
  }
}

[Goonie]

vedo anche dal codice che hai scritto che passando il this nella funzione sul onchange passi direttamente l'oggetto "selects" alla funzione

quindi potresti scrivere anche così:

<script>
<!--
function validate(LaSelect) {

var IdValSelezionato = LaSelect.selectedIndex;

if(LaSelect[IdValSelezionato].value==''){
alert("Selezionare una tabella");
return false;
}else{
if(LaSelect[IdValSelezionato].value==2){
LaSelect.style.visibility='visible';
}else{
LaSelect.style.visibility='hidden';
}
}

}
//-->
</script>

[homezappa]

Originariamente inviato da Ely88M
anche con le tue correzioni non funziona comunque...

Avevo detto che non ero entrato nel merito!

raven74 ha scritto quello che avrei scritto io (mi ha battuto sul tempo!)

Zappa

[Ely88M]

Grazie 1000 a tutti!!!!
ho sempre detto che non vado d'accordo con javascript... bisogna che prenda ripetizioni

Ciao