Just nu i M3-nätverket
Jump to content

Form validation


Johan Abelson

Recommended Posts

Johan Abelson

Hur kan man skriva istället för att skriva såhär... det blir så lång kod, och varför funkar inte "return false;"???

 

<script language="javascript">

<!--

 

function validate()

{

 

if ((document.addlink.url.value.indexOf("http://") == -1) ||

(document.addlink.url.value.indexOf(".com") == -1) ||

(document.addlink.url.value.indexOf(".se") == -1) ||

(document.addlink.url.value.indexOf(".nu") == -1) ||

(document.addlink.url.value.indexOf(".net") == -1) ||

(document.addlink.url.value.indexOf(".org") == -1) ||

(document.addlink.url.value.indexOf(".dk") == -1))

{

alert("assafsdas");

return false;

}

 

}

 

 

//-->

</script>

 

 

Funktionen aktiveras av en OnSubmit i en form tagg (OnSubmit="validate()")

 

 

Tack på förhand

 

[inlägget ändrat 2001-12-01 22:10:43 av Ablo]

Link to comment
Share on other sites

Stoppa in dem i en array och loopa istället, glöm inte att det numera ochså finns .info och .biz

 

Returnera false i validate men lägg ochså in detta i OnSubmit

 

OnSubmit="return validate()"

 

om du vill att formuläret INTE ska skickas vid fel.

 

[inlägget ändrat 2001-12-01 22:29:24 av Nibbler]

Link to comment
Share on other sites

jag hade gjort nåt sånt här, känns lite onödigt att göra en array, och du kodar ju bara en gång:

 

 

function validate()

{

var pre = document.addlink.url.value.indexOf;

if ((pre("http://") == -1) ||

(pre(".com") == -1) ||

(pre(".se") == -1) ||

(pre(".nu") == -1) ||

(pre(".net") == -1) ||

(pre(".org") == -1) ||

(pre(".dk") == -1))

{

alert("assafsdas");

return false;

}

 

}

 

Sen undrar jag om du inte vill kolla ifall ingen av de här finns, så ska du returnera false?

Som det nu är så måste ju addlink.url innehålla alla, annars blir det false......

Du kanske vill ha && istället för ||, men vad vet jag........

 

[inlägget ändrat 2001-12-06 02:03:40 av Steve]

[inlägget ändrat 2001-12-06 02:08:34 av Steve]

Link to comment
Share on other sites

Archived

This topic is now archived and is closed to further replies.



×
×
  • Create New...