semplice funzione.. non dà errore ma non funziona :(

[dops]

ciao, devo semplicemente controllare che due campi non siano vuoti.. quindi:

function controlla(f){
var titolo=document.f.titolo.value;
var testo=document.f.testo.value;

if(titolo==""){
alert("Attenzione inserire il titolo");
document.form1.titolo.focus();
return false;
}

if(testo==""){
alert("Attenzione inserire il testo");
document.form1.testo.focus();
return false;
}

return true;
}

e nel form

<form name="form1" method="post" action="gestnews.asp" style="margin:0px" onSubmit="return controlla(this);">

Non mi riporta errori, ma se il form viene inviato anche se i campi sono vuoti

[Mich_]

Ci sono errori nell'uso del DOM.

Alla funzione passi un oggetto, il form. Da tale oggetto devi partire per tutte le elaborazioni.

Nella funzione document.f non e` definito.
document.form1 invece e` definito e corretto, ma perche` partire dal document se abbiamo gia` a disposizione un oggettopiu` specifico (che non dipende dai nomi che abbiamo dato al form).

L'uso corretto degli oggetti del DOM e`:

function controlla(f){
  var titolo = f.titolo.value;
  var testo = f.testo.value;

  if(titolo==""){
    alert("Attenzione inserire il titolo");
    f.titolo.focus();
    return false;
  }

  if(testo==""){
    alert("Attenzione inserire il testo");
    f.testo.focus();
    return false;
  }

  return true;
}

[dops]

il problema praticamente è che richiamavo l'oggetto passato tramite document quando invece non era necessario? mhh. capito

[Mich_]

Originariamente inviato da dops
il problema praticamente è che richiamavo l'oggetto passato tramite document quando invece non era necessario? mhh. capito

No, il problema era che a livello dell'oggetto document non esiste nessun oggetto f.
f e` un oggetto definito all'interno della funzione (e` una variable locale), ma se chiami l'oggetto document poi non puoi far riferimento ad una variabile locale.

JS e` un linguaggio OO (Object Oriented = orientato agli oggetti): significa che gli oggetti sono elementi che il linguaggio riconosce e sa usare.
ogni oggetto possiede delle proprieta` e dei metodi.
Un metodo e` in pratica una funzione che agisce su quell'oggetto, una proprieta` e` un sotto-oggetto definito al suo interno.

[dops]

Originariamente inviato da Mich_
No, il problema era che a livello dell'oggetto document non esiste nessun oggetto f.
f e` un oggetto definito all'interno della funzione (e` una variable locale), ma se chiami l'oggetto document poi non puoi far riferimento ad una variabile locale.

JS e` un linguaggio OO (Object Oriented = orientato agli oggetti): significa che gli oggetti sono elementi che il linguaggio riconosce e sa usare.
ogni oggetto possiede delle proprieta` e dei metodi.
Un metodo e` in pratica una funzione che agisce su quell'oggetto, una proprieta` e` un sotto-oggetto definito al suo interno.

hehe.. grazie della lezione ma sapevo cosa signifiva object oriented
Cmq hai detto + o - meno quello che dicevo io.. in pratica richiamavo il nome del form tramite "l'oggetto" document..