Validering af filnavn

Noget i denne stil må kunne bruges:
tegn="abcdefghijklmnopqrstuvwxyz1234567890"
filnavn="abcdefg"
for i=1 to len(filnavn)
  if instr(tegn,mid(filnavn,i,1)=0 then
    'fejl skal håndteres
    exit for
  end if
next

ups: jeg har gjort det igen. Jeg så ikke det skulle være i JavaScript.
Mit forslag er i VBscript.

eller noget regular expressions

function ValidNumber(item) {
regex = /^\a+$/;
return (item.match(regex));
}

picture = String(fil.foto.value);
    pic = picture.substr(0,picture.lastIndexOf("."))
    pic = pic.replace(/\346/gi, "ae");
    pic = pic.replace(/\370/gi, "oe");
    pic = pic.replace(/\345/gi, "aa");
    pic = pic.replace(/[^a-zA-Z0-9]+/gi, "");

- for 200 point vil jeg gerne have en færdig validering der passer ind i den jeg bruger i forvejen.

/Goose

lklingenbergs idé er klart den bedste, men jeg kender ikke den der \a, men ville bruge \w som betyder tal, bogstaver (a-z) og _, som vel også er det som må være lovligt, så:

function validFilename(txt){
regExp = /^\w+$/i;
return txt.match(regExp);
}

ups - det var en 'typo' med det \a. roenving, du kan sikkert give ham en færdig valideringspakke som inkluderer regex-funktionen.

<script type="text/javascript">
// formular tjek start
function tjek_fil(){
var tjek = document.fil;

if (fil.foto.value == ""){
    alert("Du skal vælge en fil");
    fil.foto.focus();
    return (false);
}
// checker filtype
    if (fil.foto.value.length>0){
            FileType = fil.foto.value.substring(fil.foto.value.length-4)
            FileType = FileType.toLowerCase()
            file = fil.foto.value.substring(fil.foto.value.lastIndexOf("\"),fil.foto.value.lastIndexOf("."))
            if (FileType!=".jpg" && FileType!=".gif" || !validFileName(file)){
                alert("Fejl i filnavn.\nKun tal og/eller bogstaver i filnavn, eller typefejl.\nDer kan kun bruges filer af typen .jpg og .gif !");
                fil.foto.focus();
                return (false);
            }


}
// slut for filcheck
return (true);
}
function validFilename(txt){
regExp = /^\w+$/i;
return txt.match(regExp);
}
</script>

<script type="text/javascript">
function tjek_fil(){
    if(fil.foto.value == ""){
        alert("Du skal vælge en fil");
        fil.foto.focus();
        return (false);
        }
    if(fil.foto.value.length>0){
        FileType = fil.foto.value.substring(fil.foto.value.length-4)
        FileType = FileType.toLowerCase()
        if(FileType!=".jpg" && FileType!=".gif"){
            alert("Der kan kun bruges filer af typen .jpg og .gif !");
            fil.foto.focus();
            return (false);
            }
        pic = String(fil.foto.value);
        pic = pic.substr(0,pic.lastIndexOf("."))
        if(pic.match(/[^a-zA-Z0-9]+/gi)){
            alert("må kun indeholde a-z og 0-9")
            return (false);
            }
        }
    // slut for filcheck
    return (true);
    }
</script>

Ups, denne linje skal lige have en +1 med:

            file = fil.foto.value.substring(fil.foto.value.lastIndexOf("\")+1,fil.foto.value.lastIndexOf("."))

og her en rettelse til min :o)

denne linie:
  pic = pic.substr(0,pic.lastIndexOf("."))
skal selvfølgelig være:
  pic = pic.substring(pic.lastIndexOf("\134")+1,pic.lastIndexOf("."))

mm12010 >> jeg har nu prøvet dit script, det submitter uanset hvad, også selvom der slet ikke er valgt en fil.

roenving >> jeg er ikke helt sikker på jeg forstår det med regExp og jeg kan heller ikke rigtigt få dit script til at virke, jeg vil gerne have at den laver en alert hvis der er ulovlige karakterer i navnet.

/Goose

Der var en fejl mht. \ -- den kører helt på min egen nu !-)

<script type="text/javascript">
// formular tjek start
function tjek_fil(){
var tjek = document.fil;

if (fil.foto.value == ""){
    alert("Du skal vælge en fil");
    fil.foto.focus();
    return (false);
}
// checker filtype
    if (fil.foto.value.length>0){
            FileType = fil.foto.value.substring(fil.foto.value.length-4)
            FileType = FileType.toLowerCase()
            file = fil.foto.value.substring(fil.foto.value.lastIndexOf("\\")+1,fil.foto.value.lastIndexOf("."))
            if (FileType!=".jpg" && FileType!=".gif" || !validFilename(file)){
                alert("Fejl i filnavn.\nKun tal og/eller bogstaver i filnavn, eller typefejl.\nDer kan kun bruges filer af typen .jpg og .gif !");
                fil.foto.focus();
                return (false);
            }


}
// slut for filcheck
return (true);
}
function validFilename(txt){
regExp = /^\w+$/i;
return txt.match(regExp);
}
</script>

<html>
<head>
<title>404206</title>
<script type="text/javascript">
function tjek_fil(){
    if(fil.foto.value == ""){
        alert("Du skal vælge en fil");
        fil.foto.focus();
        return (false);
        }
    if(fil.foto.value.length>0){
        FileType = fil.foto.value.substring(fil.foto.value.length-4)
        FileType = FileType.toLowerCase()
        if(FileType!=".jpg" && FileType!=".gif"){
            alert("Der kan kun bruges filer af typen .jpg og .gif !");
            fil.foto.focus();
            return (false);
            }
        pic = String(fil.foto.value);
        pic = pic.substring(pic.lastIndexOf("\134")+1,pic.lastIndexOf("."))
        if(pic.match(/[^a-zA-Z0-9]+/gi)){
            alert("må kun indeholde a-z og 0-9");
            return (false);
            }
        }
    // slut for filcheck
    return (true);
    }
</script>
</head>
<body>
<form action="?submit=true" method="post" enctype="multipart/form-data" name="fil" onSubmit="return tjek_fil();">
  <input type="file" name="foto">
  <br>
  <input type="submit" name="Submit" value="Submit">
</form>
</body>
</html>

Tak for det, så virker det, smider du også lige et svar roenving så deler du med mm12010 ?

/Goose

Velbekomme '-)

-- og tak for points ;~}

- lige et tillægsspørgsmål hvis i lytter med endnu, jeg bruger nu koden fra mm12010 men hvad nu hvis jeg vil tillade brugen af underscore og minus udover bogstaver og tal?

/Goose

if(pic.match(/[^\w-]+/gi)){

Takker :o)

/Goose