---

Hej NG.


Jeg bruger et script (som jeg har tyv-stjålet et sted) til at validere
form-input med. Det ser sådan her ud:

function Checkform(thisform) {
strError = 'Hold nu op! Du skal:\n\n';
intError = 0;

if (thisform.Navn.value == "")
{
strError = strError + '- Skrive dit navn!\n'
intError = 1;
thisform.Navn.focus();
}

if (thisform.Ord.value == "")
{
strError = strError + '- Skrive en lille beskrivelse!\n'
intError = 1;
thisform.Ord.focus();
}

if (thisform.Fil.value == "")
{
strError = strError + '- Markere en fil til upload!\n'
intError = 1;
thisform.Fil.focus();
}

if (intError == 1) {
alert(strError)
return false;
}
}

Nu er det sådan, at jeg i et textarea (thisform.Ord) kun vil have 150 tegn
som et maksimum. Er der noget, der kan hjælpe med at modificere ovenstående
script sådan, at hvis feltet "Ord" i formen indeholder over 150 tegn, så
popper en advarsel op (i strError)?



Mvh. Voller.

---

"Thomas Voller" <voller@voller.dk> writes:

....
> Nu er det sådan, at jeg i et textarea (thisform.Ord) kun vil have 150 tegn
> som et maksimum. Er der noget, der kan hjælpe med at modificere ovenstående
> script sådan, at hvis feltet "Ord" i formen indeholder over 150 tegn, så
> popper en advarsel op (i strError)?

Tilføj

if (thisform.Ord.value.length > 150)
{
strError = strError + '- Skrive en *lille* beskrivelse!\n'
intError = 1;
thisform.Ord.focus();
}

Det ville nok være pænere at bruge en boolean i stedet for en integer
til at huske om der har været en fejl, altså skift
intError = 1
ud med
boolError = true
og så videre. Det med at bruge 0 som falsk og 1 som sandt, er en uskik
C-programmører har fra et sprog der ikke har "true" og "false".

/L
--
Lasse Reichstein Nielsen - lrn@hotpop.com
Art D'HTML: <URL:http://www.infimum.dk/HTML/randomArtSplit.html>
'Faith without judgement merely degrades the spirit divine.'

---

"Lasse Reichstein Nielsen" <lrn@hotpop.com> skrev i en meddelelse op (i
strError)?
>
> Tilføj
>
> if (thisform.Ord.value.length > 150)
> {
> strError = strError + '- Skrive en *lille* beskrivelse!\n'
> intError = 1;
> thisform.Ord.focus();
> }

Ville det ikke være bedre bare, at sætte en Maxlength="150" på tekstboxen ??

MVH
RoJo

---

RoJo <rjonassen@hotmail.com> har skrevet:
>
> Ville det ikke være bedre bare, at sætte en Maxlength="150" på
> tekstboxen ??
>
Det hjælper ikke hvis det er et textarea

--
Niels

---

> Tilføj
> ...

Det virker jo fint, tak.


> Det ville nok være pænere at bruge en boolean i stedet for en integer
> til at huske om der har været en fejl, altså skift
> intError = 1
> ud med
> boolError = true
> og så videre. Det med at bruge 0 som falsk og 1 som sandt, er en uskik
> C-programmører har fra et sprog der ikke har "true" og "false".

Jeg har ikke så meget overblik over JavaScript, men jeg værdsætter altid god
kodepraksis.
Tak for rådet.



Voller.