Servlet controllo dati form

[mauc]

ho una servlet con una form

out.println("<form onSubmit=return (checkrequired(this)) action=AggiuntaFile method=POST  name=modulo enctype=multipart/form-data>"
+ "Nome documento: input name=nomeDoc type=text>"
+ "<input type=submit value=Aggiungi></input>"
+ "</form>"

la funzione è cosi fatta

        out.println("<script type=text/javascript language=javascript>"
                + "function checkrequired(which){"
                + "var nomeD = document.modulo.nomeDoc.value;"
                + "if (nomeD== "" )"  <---- cosi scritta mi da errore
                + " {alert"
                + "(Il campo nome è obbligatorio);"
                + "document.modulo.nameD.focus();"
                + "return false"
                + "}"
                + "else return true");

la domanda è, come fare il controllo (l'if) senza che mi dia l'errore?

[oronze]

è un problema di virgolette... o usi il carattere \ prima del doppio apice, oppure usi il singolo apice

[mauc]

facendo cosi

+ "if (nomeD== ' ')"

il codice non da problemi ma non mi esce l'alert

se faccio

+ "if (nomeD==  \" ")"

mi da errore

[andbin]

Originariamente inviato da mauc

        out.println("<script type=text/javascript language=javascript>"
                + "function checkrequired(which){"
                + "var nomeD = document.modulo.nomeDoc.value;"
                + "if (nomeD== "" )"  <---- cosi scritta mi da errore
                + " {alert"
                + "(Il campo nome è obbligatorio);"
                + "document.modulo.nameD.focus();"
                + "return false"
                + "}"
                + "else return true");

A prima vista vedo 2 errori:

La quarta riga dovrebbe essere
+ "if (nomeD== \"\")"
oppure
+ "if (nomeD== '')"

(o escape dei apici doppi o solo apici singoli)

La sesta riga dovrebbe essere
+ "(\"Il campo nome è obbligatorio\");"
oppure
+ "('Il campo nome è obbligatorio');"

Idem come sopra o escape di apici doppi o apici singoli

Ah no aspetta. Anche la function checkrequired deve essere terminata da un } che non vedo.

[LeleFT]

Ovviamente devi quotare TUTTE le virgolette che fanno parte del literal:

"if (nomeD== \"\" )"

Edit: preceduto...

Ciao.

[mauc]

ci sono riuscito,

ho modificato la chiamata della funzione da

onSubmit=return(checkrequired(this))

a

onSubmit=\"return(checkrequired(this))\"

un ultima cosa

nel form c'è anche

<label>Documento:</label><input type=file name=documento>

come posso verificare se ho selezionato un file dal pc?