Validering af radioknapper

if (feltnavn.cheked)
mener at det er nok.. ellers.. if(feltnavn.checked == true)

hovsa (manglede et "c"): if (feltnavn.checked)

/theSurfer

PS: Det er det samme, for checkboxe..

/theSurfer

Har nu prøvet med:

if(tjek.f01.checked == true) {
alert('Du skal vælge en gruppe');
tjek.f01.focus();
return false;
}

- og det virker ikke, det tager ikke højde for om der er valgt en radioknap eller ej.

/Goose

Dvs, du vil bruge 1 funktion, til at validere hele formen?

/theSurfer

Hvordan ser din form ud? Indsæt koden her..

/theSurfer

Hvis jeg forstår dig korrekt, så ja.

Jeg har en hel masse felter af forskellig art (tekstfelter, radioknapper m.v.) som jeg skal have valideret, men hvad skulle det betyde ?

/Goose

Jeg vil ikke indsætte min kode her, den er meeeeget lang.

Jeg kan godt få mine øvrige felter til at tage imod valideringen, det er kun de radioknapper den inorerer.

/Goose

Der er forskel på validering på textbox, radioknapper osv.. de har forskellige egenskaber..

Så må du bare lave et check:

if (feltnavn.type == "radio") {
  // det er en radioknap
  if (feltnavn.checked) {
    // radioknappen er markeret
  } else {
    // radiokanppen er IKKE markeret
  }
} else {
  // det er IKKE en radioknap
}

Normalt bruger man ikke direkte navn på objekter.. man laver en loop.. og så er det godt at bruge denne metode..

Men loops bruger man kun, hvis man har mange elementer i formen.

Hvis du har mulighed for det, skriv lige et link til siden, så vi kan test/finde løsninger..

/theSurfer

Nej, jeg har ikke noget link, men jeg har lige snippet lidt kode:

<FORM NAME="oplysning" action="resultat.asp" method="post" onSubmit="return tjek_form_oplysning();">

...

<td width="20"><INPUT TYPE="radio" VALUE="Ja" NAME="f01"></td>
<td width="40"><INPUT TYPE="radio" VALUE="Nej" NAME="f01"></td>

...

- og så den kode jeg startede med at poste

Jeg er ikke interesseret i at loope idet ikke alle felter skal valideres, ligesom jeg gerne vil kunne "smide" brugeren tilbage til et felt som "fejler".

/Goose

Dvs, du vil faktisk skrive IF-sætninger til textboxe, og IF-sætninger til radioknapper?

Koden kan forkortes lidt, ved at tilføje "this" i din onsubmiit-kode:

<FORM NAME="oplysning" action="resultat.asp" method="post" onSubmit="return tjek_form_oplysning(this);">


<script type="text/JavaScript">

// nu kan vi henvise til formen, ved at bruge "f" (eller andet, hvis man ønsker det):

funcion tjek_form_oplysning(f)
{
if (!f.f01.checked) {
  // brugeren hverken valgt "ja" eller "nej", og sendes derfor tilbage:
  alert("Du skal vælge ja eller nej");
  return false;
}

return true; // mener faktisk at man skal have "true" med, for at det er korrekt kode..
}


Når man putter "!" foran, betyder det "false".. dvs, linien svarer til:

if (f.f01.checked == false)

/theSurfer

Okay, men jeg vil være rimelig ked af, at lave hele min validering om.

Hvis jeg nu sætter ! foran min oprindelige validering, så fortæller den mig godt nok, at der ikke er valgt noget, men den submitter alligevel når jeg trykker "OK":

if(!tjek.f01.checked) {
alert('Spørgsmål nr. 1 er ikke besvaret');
tjek.f01.focus();
return false;
}

Kan det ikke løses uden at skulle ændre hele min kode ?

/Goose

Hele din kode? Mener du de der 3-5 linier..? :-)

Det her virker fint:

<script type="text/JavaScript">
function tjek(f)
{
if (!f.testradio[0].checked && !f.testradio[1].checked) {
  alert("Du skal vælge enten Ja eller Nej");
    return false;
}

if (f.testtext.value == "") {
  alert("Du skal skrive noget");
    f.testtext.focus();
    return false;
}

return true;

}
</script>

<form onsubmit="return tjek(this)">
<input type="radio" name="testradio" value="ja">Ja
<br>
<input type="radio" name="testradio" value="nej">Nej
<br>
<input type="textbox" name="testtext">(tekstfelt)
<br>
<input type="submit" value="Send">
</form>

Hvis du undrer dig over, at der står [0] og [1], så er det fordi radioknapper er arrays (der står samme navn: name="DetSammeBeggeSteder").. og det skal det også være..

/theSurfer

Jeg finder på noget.
Nu er det mere end 3-5 linier der skal rettes, der er over 500 felter i formularen :o)

Smid et svar

/Goose

Svar smidt :-)

Jeg lavede engang et script, hvor man kunne skrive hvilke felter der skulle valideres.. eller hvilke felter der *ikke* skulle valideres.. husker det ikke lige.. :-)

Men det blev gjort via en array, og en loop.. så sparer man en masse kode..

/theSurfer

Det er simpelthen utroligt, nu har jeg cut & pastet det eksempel du skriver skulle virke og det gør det bare ikke, den submitter uanset hvad ! :o(

/Goose

Tegnet ';' var ventet ??

/Goose

Så mangler du et ";" tegn, et eller andet sted, i din kode :-)  (ikke min kode)

Jeg har lige testet koden (fra 26/05-2005 18:34:48).. koden virker både i Internet Explorer og Opera..

Kig på linien, lige før den linie, med fejlen.. der skulle gerne være noget galt der..

/theSurfer

Ja den havde jeg ligesom selv regnet ud, men hvor ?
Jeg skrotter sg* det hele og starter forfra.

Jeg kan iøvrigt ikke helt se hvor det er den mangler ud fra det du har skrevet:

<script language="javascript">
funcion tjek_form_oplysning(v)
{
if (!v.f01[0].checked && !v.f01[1].checked) {
  alert("Du skal vælge enten Ja eller Nej");
    return false;
}
if (v.navn.value == "") {
  alert("Du skal skrive noget");
    v.navn.focus();
    return false;
}
return true;
}
</SCRIPT>

/Goose

Du mangler et "t" i function: funcion tjek_form_oplysning(v)

Den tror jo, at "funcion" (uden "t") er en funktion, som den skal kalde på :-)

/theSurfer

Forresten.. Så hedder det <script type="text/JavaScript"> nu.. :-)

/theSurfer

..og, takker for points :-)

/theSurfer

Ja det har du ret i, der manglede et t og nu får jeg også alerten, men den submitter sg* alligevel :o(

/Goose

woot?.. forstår jeg ikke lige..

Prøv at rydde cachen, og genstarte computeren.. kan være at det bliver fixet..

Ellers må du smide siden op et eller andet sted, så vi kan kigge på den..

/theSurfer

Okay, så lykkedes det vist.
Problemet/fejlen var, at der på samme side var en anden formular som gik ind og konfliktede med den nye.

/Goose