Newsgroup: it.comp.lang.javascript
Titolo: Correzione javascript

Salve, c'è qualcuno che può aiutarmi a verificare se il mio script è
corretto? Funziona, ma non so se è scritto bene. Ho bisogno dello
script corretto per un esame che devo sostenere domani.Vi prego datemi
una mano!!
Ecco lo script:
<html>
<head>
<title> esempio 9 </title>
<SCRIPT language=JavaScript>
<!--
defaultStatus = "Ciao a tutti!! Questo è il mio primo script!Spero vi
piaccia!"
// calcoliamo la data
  oggi = new Date()
  document.write("L'ora attuale è:
",oggi.getHours(),":",oggi.getMinutes(),".  ")
  document.write("<br>La data di oggi è: ",
oggi.getDate(),"/",oggi.getMonth() + 1,"/",oggi.getYear());
// introdotto lo statement "if-else"
function check() {  // inizio della definizione della funzione
while (document.questionario.username.value == "nome") {
	alert("Bisogna inserire il proprio nome");
	document.questionario.username.value = prompt("Inserisca il suo
nome","nome")
	}
 
	var hello ="Benvenuto, ";
        hello += document.questionario.username.value;
	hello += ". \n Ecco come giudica il mio lavoro";
	alert(hello);
 
        var risposta = "Lei giudica il mio lavoro:";
	if (document.questionario.Ottimo.checked) {
		risposta += "\n ottimo!..Uow!! No pensavo di essere diventata così
brava!!"
		}
	if (document.questionario.Buono.checked) {
		risposta += "\n Buono!Queste sì che sono soddisfazioni!"
		}
	if (document.questionario.Sufficiente.checked) {
		risposta += "\n Sufficiente...mi accontento, potrò sempre
migliorare.."
		}
        if (document.questionario.Insufficiente.checked) {
		risposta += "\n Insufficiente...abbi pietà..sono solo una povera
principiante..."
		}
        if (risposta == "Lei giudica il mio lavoro:") {
		risposta += "\n ...Perchè non risponde?Ci terrei a conoscere il suo
giudizio!"
		}
 
	alert(risposta);
        var risposta= "Il colore della pagina.."
        if (document.questionario.positivo.checked) {
		risposta += "\n le piace: mi fa piacere!!"
		}
        if (document.questionario.Negativo.checked) {
		risposta += "\n Non le piace: nessun problema!!In fondo alla pagina
potrà scegliere il colore che preferisce!"
		}
        if (risposta=="Il colore della pagina.."){
		risposta += "\n sembra proprio non interessarla!"
		}
        alert(risposta);
 
 
}  // fine della definizione della funzione
//--!>
</script>
</head>
<h1> forms interattivi </h1>
<hr>
<form name="questionario" >
<p> <b> Domanda 1: </b> come ti chiami? </p>
<input type="text" name="username" value="nome">
<hr>
<p> <b> Domanda 2: </b> Come giudica il mio lavoro? </p>
<input type="checkbox" name="Ottimo"> Ottimo;
<input type="checkbox" name="Buono"> Buono;
<input type="checkbox" name="Sufficiente"> Sufficiente;
<input type="checkbox" name="Insufficiente"> Insufficiente;
<p> <b> Domanda 3: </b> Le piace il colore dello sfondo? </p>
<input type="checkbox" name="positivo"> Si;
<input type="checkbox" name="Negativo"> No;
<hr>
<p> clicca qui per effettuare la verifica dei dati immessi </p>
<input type="button" name="pcontrollo" value="pulsante di controllo"
onclick="check()">
<input type="reset">
</form>
<BODY bgcolor="violet">
<CENTER>
<br><br><br>
<H2>Scelga il colore dello sfondo che più le piace!</H2></CENTER>
<CENTER>
<TABLE>
 
  <TR heigth="40">
    <TD><A onmouseover="document.bgColor='green'" 
      href="
http://www.unite.it/
">Green</A> </TD>
    <TD><A onmouseover="document.bgColor='seagreen'" 
      href="
http://www.unite.it/
">Sea Green</A> </TD>
    <TD><A onmouseover="document.bgColor='red'" 
      href="
http://www.unite.it/
">Red</A> </TD>
    <TD><A onmouseover="document.bgColor='magenta'" 
      href="
http://www.unite.it/
">Magenta</A> </TD></TR>
  <TR heigth="40">
    <TD><A onmouseover="document.bgColor='fuchsia'" 
      href="
http://www.unite.it/
">Fuchsia</A> </TD>
    <TD><A onmouseover="document.bgColor='purple'" 
      href="
http://www.unite.it/
">Purple</A> </TD>
    <TD><A onmouseover="document.bgColor='navy'" 
      href="
http://www.unite.it/
">Navy</A> </TD>
    <TD><A onmouseover="document.bgColor='blue'" 
      href="
http://www.unite.it/
">Blue</A> </TD></TR>
  <TR heigth="40">
    <TD><A onmouseover="document.bgColor='royalblue'" 
      href="
http://www.unite.it/
">Royal Blue</A> </TD>
    <TD><A onmouseover="document.bgColor='brown'" 
      href="
http://www.unite.it/
">Brown</A> </TD>
    <TD><A onmouseover="document.bgColor='almond'" 
      href="
http://www.unite.it/
">Almond</A> </TD>
    <TD><A onmouseover="document.bgColor='black'" 
      href="
http://www.unite.it/
">Black</A> </TD></TR>
  <TR heigth="40">
    <TD><A onmouseover="document.bgColor='greem'" 
      href="
http://www.unite.it/
">Bright Green</A> </TD>
    <TD><A onmouseover="document.bgColor='pink'" 
      href="
http://www.unite.it/
">Pink</A> </TD>
    <TD><A onmouseover="document.bgColor='Skyblue'" 
      href="
http://www.unite.it/
">Sky Blue</A> </TD>
    <TD><A onmouseover="document.bgColor='yellow'" 
      href="
http://www.unite.it/
">Yellow</A> </TD></TR>
  <TR heigth="40">
    <TD><A onmouseover="document.bgColor='white'" 
      href="
http://www.unite.it/
">White</A> </TD>
    <TD><A onmouseover="document.bgColor='coral'" 
      href="
http://www.unite.it/
">Coral</A> </TD>
    <TD><A onmouseover="document.bgColor='orange'" 
      href="
http://www.unite.it/
">Orange</A> </TD>
    <TD><A onmouseover="document.bgColor='olivedrab'" 
      href="
http://www.unite.it/
">Olive Drab</A>
</TD></TR></TABLE></CENTER>
<br><br>
<H3>Questo script e' molto semplice..ma carino. Non crede?
</H3></BODY></HTML>
</html>

 oggi.getDate(),"/",oggi.getMonth() + 1,"/",oggi.getYear());
getYear() non va più usato (mantenuto solo per compatibilità); se noti,
infatti, anziché 2004 ti scrive 104. Sostituisci con getFullYear()
A parte questo di altri errori non ne vedo; domanda: perché hai usato dei
radio? In questo modo uno può dirti che sei stata bravissima, brava, così
così ma comunque pessima! In questi casi (dove si può scegliere solo una
opzione) esistono i radio.
Ti ho messo al posto del voto con checkbox dei radio, vedi se è meglio (se
è meglio lascio a te la modifica per indicare se piace o meno il colore
dello sfondo).
http://www.criosoftware.it/valeria.htm
A parte questo lo script va bene; si potrebbe ottimizzare usando if...else
anziché tutti if.
-- 
"Affacciati affacciati benedici, guardaci
 guardaci...guardaci!
 Tanto sono quasi duemila anni che stai a guardare!"
									    E.B.
--

 č meglio lascio a te la modifica per indicare se piace o meno il colore


 Grazie mille, seguirò i tuoi consigli!!Ciao
Sarà difficile, visto che hai l'esame oggi! (in bocca al lupo => crepi!)
-- 
"That until there is no longer first class
And second class citizens of any nation
Until the colour of a man's skin
Is of no more significance than the colour of his eyes
Me say war"
B.M.
--