validering af input

<%
If Right(Request("navn"),3) = "jpg" then
response.write "Godkendt"
Else
response.write "Fejl"
End if

If Right(Request("navn"),4) = "jpeg" then
response.write "Godkendt"
Else
response.write "Fejl"
End if
%>

Kan du bruge det....?

ikke rigtigt - det er javascript-validering jeg ønsker

Her:
http://www.friserverplads.dk/phoenixv/file_vali.html

phoenixv > den melder nu ikke fejl ved alle fil-typer. Prøv f.eks. at vælg en HTML-fil.

2 sek.

Jo, det gør da! Både ved .html og .htm!
Man må bare ikke selv ændre på filens endelse manuelt.

Indsæt dette ved din input file:

<input type="file" name="fil" readonly>

Så kan man ikke selv ændre på teksten i boksen; man skal bruge Gennemse-knappen!

ja ja - det er ikke problemet. Prøv dit script på http://www.friserverplads.dk/phoenixv/file_vali.html og forsøg at upload f.eks. en HTML-fil. Så får man ikke nogen fejlmeddelelse.

ah - det kan være at det er fordi hvis man først klikker og så prøver igen...

Næ, du har ret; det virker ikke med .htm-filer...
Jeg prøver igen! :)

ja - prøv at vælg f.eks. en html-fil og klik. Så kommer der fejl - klik så igen - ingen fejl.

jo - den virker med htm - men ikke hvis man klikker flere gange. mærkeligt.

Nej, nu har jeg fundet ud af min fejl. Jeg havde bare genbrugt et funktionsnavn som et variabelnavn. **lyden af et tog, der suser forbi**
Jeg uploader lige det færdige resultat...

Nu er det uploadet på den samme url.

Har lige 2 hurtige follow-ups:

1) For nu at blive HELT picky - så burde den vel først melde fejl HVIS feltet ikke er tomt. Det kunne jo være brugeren skulle udfylde andre ting i formen men IKKE billedet.

2) Min form har allerede en onSubmit="return false" - hvis "return valider(this);" skal ind - bliver den samlede streng så:

onSubmit="return false; return valider(this);"

?

Mht. det første, så retter jeg det lige.

Man bør ikke sætte return false ind i onSubmit, da det betyder, at formularen slet ikke sendes!
Du kunne evt. lige smide et link!

1'eren er rettet nu.

mht (2) - så jo for jeg validerer en masse på om andre felter er udfyldt/ikke er tomme ?!

Derfor bør man stadig ikke skrive return false i onSubmit-handleren!
Det kunne godt lyde, som om det er lidt dårligt kodet. Jeg vil gerne se på den og optimere den, hvis du synes!

Jamen et eksempel:

<form name="bruger" action="noget.asp" method=post onSubmit="return false">
<input type="text" name="brugernavn">
<input type="submit" value="Tilføj bruger" onClick=validate()>

Har en henvisning til scriptet (<script language="javascript" src="scripts/bruger.js"></script>)

der lyder sådan:

function validate(){
if (document.bruger.brugernavn.value=="") {alert("Angiv venligst et brugernavn!"); document.bruger.brugernavn.focus(); return}
else document.bruger.submit()
}

virker fint

Det virker, men det er programmeringsmæssigt forkert! :)
Alt halløjet burde lægges ind i én valideringsfunktion, som så blev kaldet i formens onSubmit:

<form onSubmit="return valider();">

Valideringsfunktionen skal så returnere enten true eller false, alt efter om formen skal sendes eller ej!

Hvad mere er, hvis brugeren skriver noget i tekstfeltet og så trykker enter (hvilket vedkommende er vant til fra fx Google), så sker der ikke noget!

Altså med dit script! :)

aha... kunne godt være jeg skulle ændre det, så...

Husk, at det optimale er kun at bruge én funktion til validering, som kaldes ved onSubmit og returnerer en værdi.
Du skal derfor bruge <input type="submit" value="Send"> istedet for <input type="button" value="Send" onClick="valider();">.
Det er også en god idé at sende en reference til formularen med i scriptet; det gør kodningen meget nemmere:

<form onSubmit="return valider(this);">

Så kan du fange referencen i funktionen:

function valider(frm)

... og derefter bruge den fx sådan her:

frm.brugernavn.value = "blabla";

Det sparer dig for at skrive document.formNavn hver gang.

Men jeg smutter for i dag! Håber du finder ud af det! :)

ok - tak for hjælpen...

Men der skal vel stå noget "return false" et eller andet sted i scriptet - har lige prøvet at ændre det. Fejlmeddelelsen kommer godt nok men klikke jeg ok - så submitter den formen.

ah ja - selvfølgelig. F*ck det skal rettes mange steder :-)

Nå hej igen

Jeg går ud fra det er sådan:

if (document.bruger.brugernavn.value=="") {alert("Angiv venligst et brugernavn!"); document.bruger.brugernavn.focus(); return false}
if (document.bruger.password.value=="") {alert("Angiv venligst et password!"); document.bruger.password.focus(); return false}
osv...

Ja, men husk det der med at referere til formen. Det er god programmeringsskik!
document.bruger bør ikke blive brugt.

Selvom det nok strider mod din kode-disciplin kan du så lave koden om til at den passer med:

<form name="bruger" action="noget.asp" method=post onSubmit="return false">
<input type="text" name="brugernavn">
<input type="submit" value="Tilføj bruger" onClick=validate()>

- Der er simpelthen for meget der ellers skal rettes for mig.

Så meget kan det vel umuligt være. Desuden mister brugeren jo muligheden for at trykke enter, hvilket kun kan være dårligt.
Men ok, så skal min kode være således:

function valider(frm) {
val = frm.fil.value;
if (val == "")
frm.submit();
end = val.substring(val.lastIndexOf("."), val.length).toLowerCase();
f = false;
for (i = 0; i < ft.length; i++)
if (ft[i] == end) {
f = true;
break;
}
if (!f)
alert("Forkert filtype!");
else
frm.submit();
}

Validér-funktionen skal så kaldes således:

<input type="button" value="Tilføj bruger" onClick="valider(this.form);">

1) Enter virker altså fint nok.

2) Jeg har prøvet at rettet scriptet i stil med:
if (document.bruger.brugernavn.value=="") {alert("Angiv venligst et brugernavn!"); document.bruger.brugernavn.focus(); return false}
if (document.bruger.password.value=="") {alert("Angiv venligst et password!"); document.bruger.password.focus(); return false}

men så submitter den - og jeg får f.eks. en ASP fejl at brugernavnet er taget selvom det ikke er tilfældet. Derfor bliver det nu som det er :-)

3) Jeg kan ikke få dit script til at virke - jeg skal ha' det ind mellem:

function validate(){
if (document.bruger.password.value=="") {alert("Angiv venligst et password!"); document.bruger.password.focus(); return}

if (document.bruger.email.value=="") {alert("Angiv venligst en e-mail!"); document.bruger.email.focus(); return}
if (!checkmail(document.bruger.email.value)) {alert("Indtast venligst en rigtig e-mail!"), document.bruger.email.focus(); return}

else document.bruger.submit()
}
function checkmail(Email){
sted=Email.indexOf("@") , punkt=Email.indexOf(".") , tal=punkt-sted
if (Email=="" || sted<=0 || punkt==-1 || tal<2) return false
return true
}

I HTML'en er koden:
<form name="bruger" action="processing.asp" method=post onSubmit="return false">
...
<input type="submit" value="Tilføj bruger" onClick=validate()>

Prøv dette:

function validate(){
if (document.bruger.password.value=="") {alert("Angiv venligst et password!"); document.bruger.password.focus(); return}

if (document.bruger.email.value=="") {alert("Angiv venligst en e-mail!"); document.bruger.email.focus(); return}
if (!checkmail(document.bruger.email.value)) {alert("Indtast venligst en rigtig e-mail!"), document.bruger.email.focus(); return}

else document.bruger.submit()

frm = document.bruger;
val = frm.fil.value;
if (val == "")
frm.submit();
end = val.substring(val.lastIndexOf("."), val.length).toLowerCase();
f = false;
for (i = 0; i < ft.length; i++)
if (ft[i] == end) {
f = true;
break;
}
if (!f)
alert("Forkert filtype!");
else
frm.submit();
}

Nu er det blevet noget af en rodekode, men du er selv ude om det! :)

Well - det virker nu stadig ikke... :-)

"Et objekt var ventet"

Fejl-linie:
<input type="submit" value="Ret profil" onClick=validate()>

Smid et link.