[javascript]form validation werkt niet

[Verwijderd]

Hallo,
Ik ben een beetje een noob met javascript en kom er echt niet meer uit. Ik heb de volgende code om te kijken of de extensie goed is:

code:

<script type="text/javascript">
function validate()
{
toegestaan = new Array("mid","jpg","gif","bmp")

bestandsnaam = document.upload.upload.value
extensie = bestandsnaam.substr(bestandsnaam.length-3,3)
validate = 0
for (i = 0; i < toegestaan.length; i++){
    if(extensie == toegestaan[i]){
        validate = 1
    }
}

if (validate == 0)
    alert("Deze extensie(" + extensie + ") is niet bruikbaar voor mobiele telefoons.")
    return false
}
</script>

Deze wordt aangeroepen door onsubmit="return validate()" in de <form>. Nu werkt het script voor 1 klik. Klik je daarna nog een keer op de submit knop dan slikt ie alles....

Iemand een idee wat ik fout doe?

[Bosmonster]

Ooit gedacht aan extenties van 4 letters zoals jpeg?

Betere manier om extenties te achterhalen is splitten op . (punt) en dan de laatste uit de verzameling te nemen,

Overigens gaat het fout door je validate variabele.. deze blijft op 1 en wordt nergens gereset.... (neem aan dat je naar een ander frame/window post ofzo dat het uberhaupt zo werkt )

En Javascript hoort in Webdesign & Graphics uiteraard

[ Voor 22% gewijzigd door Bosmonster op 08-12-2004 22:30 ]

[NMe]

Bosmonster schreef op woensdag 08 december 2004 @ 22:29:
Ooit gedacht aan extenties van 4 letters zoals jpeg?

Betere manier om extenties te achterhalen is splitten op . (punt) en dan de laatste uit de verzameling te nemen,

Overigens gaat het fout door je validate variabele.. deze blijft op 1 en wordt nergens gereset.... (neem aan dat je naar een ander frame/window post ofzo dat het uberhaupt zo werkt )

En regel 8 dan?

[Verwijderd]

code:

if (validate == 0)
    alert("Deze extensie(" + extensie + ") is niet bruikbaar voor mobiele telefoons.")
    return false

Ik zie geen accolades om dit if statement staan, dus deze functie geeft volgens mij altijd false terug

[ Voor 7% gewijzigd door Verwijderd op 08-12-2004 22:35 ]

[crisp]

--> Webdesign & Graphics

en warhead5 is right

statements afsluiten met een ;, en je variabelen declareren met het 'var' keyword is trouwens ook good practice. dot-notatie daarentegen is evil...
Het is trouwens ook netjes om als je eenmaal de extensie in je array gevonden hebt uit je loop te springen mbv break, of om meteen return true te doen.
En verder wat Bosmonster zei natuurlijk

[ Voor 92% gewijzigd door crisp op 08-12-2004 22:39 ]

[Verwijderd]

volgens mij gaat het fout omdat je een functie en een variabele met dezelfde naam hebt (namelijk validate), dat mag niet

[crisp]

Verwijderd schreef op woensdag 08 december 2004 @ 22:39:
volgens mij gaat het fout omdat je een functie en een variabele met dezelfde naam hebt (namelijk validate), dat mag niet

Dat kan prima, maar niet als je die variabele global maakt. Door het niet netjes lokaal declareren wordt hij inderdaad global en overschrijft het functie-object. Dat is dus inderdaad het 2e grote probleem met deze functie buiten dat hij altijd false retourneert. Scherp!

[Verwijderd]

bij mij returnt ie niet altijd false:

code:

<script type="text/javascript">
function f_validate(bestandsnaam)
{
toegestaan = new Array("mid","jpg","gif","bmp")

//bestandsnaam = document.upload.upload.value
extensie = bestandsnaam.substr(bestandsnaam.length-3,3)
validate = 0
for (i = 0; i < toegestaan.length; i++){
    if(extensie == toegestaan[i]){
        validate = 1
    }
}

if (validate == 0)
    alert("Deze extensie(" + extensie + ") is niet bruikbaar voor mobiele telefoons.")
    return false
}
</script>

<script>
f_validate('aaa');
f_validate('jpg');
f_validate('aaa');
f_validate('jpg');
</script>

[crisp]

woe: hoe maak je die returnvalue dan zichtbaar?
Probeer het zo eens:

JavaScript:

alert(f_validate('aaa'));
alert(f_validate('jpg'));
alert(f_validate('aaa'));
alert(f_validate('jpg'));

ik denk dat je dan toch wel een aantal keer 'false' ge-alert krijgt

[Verwijderd]

je hoeft geen alert te programmeren, dat doet de functie zelf al.
Je krijgt in mijn voorbeeld 2x een alert dat de extentie niet is toegestaan, de 2x jpg wordt dus wel toegestaan

[Wolfboy]

crisp schreef op woensdag 08 december 2004 @ 22:46:
woe: hoe maak je die returnvalue dan zichtbaar?

Je maakt een leesfoutje, die return value hoort bij de functie, niet bij de if.
De functie geeft idd altijd false terug, maar dat maakt niet uit.
Het doel van de functie is om een alert te geven als het niet goed is en anders gewoon niets te doen.

offtopic:
in plaats van: bestandsnaam.substr(bestandsnaam.length-3,3)
kan je ook: bestandsnaam.substr(-3);
gebruiken

JavaScript:

<script type="text/javascript">
function f_validate(bestandsnaam){
  toegestaan = new Array("mid","jpg","gif","bmp")

  //bestandsnaam = document.upload.upload.value
  extensie = bestandsnaam.substr(bestandsnaam.length-3,3)
  validate = 0
  for (i = 0; i < toegestaan.length; i++){
    if(extensie == toegestaan[i]){
      validate = 1
    }
  }

  if (validate == 0){
    alert("Deze extensie(" + extensie + ") is niet bruikbaar voor mobiele telefoons.")
  }

  return false
}
</script>

Zo duidelijker?

[ Voor 40% gewijzigd door Wolfboy op 08-12-2004 22:59 ]

[crisp]

Wolfboy schreef op woensdag 08 december 2004 @ 22:57:
[...]
Je maakt een leesfoutje, die return value hoort bij de functie, niet bij de if.
De functie geeft idd altijd false terug, maar dat maakt niet uit.
Het doel van de functie is om een alert te geven als het niet goed is en anders gewoon niets te doen.

Het maakt wel degelijk uit als je iets doet met de returnvalue, een actie cancellen bijvoorbeeld. In dit geval zou een bepaalde actie (bijvoorbeeld het posten van een formulier) dan dus altijd gecancelled worden ongeacht of hij door de check heenkomt.
Als je niets doet met een returnvalue, zet dan ueberhaupt geen return in je functie.

offtopic:
in plaats van: bestandsnaam.substr(bestandsnaam.length-3,3)
kan je ook: bestandsnaam.substr(-3);
gebruiken

Probeer dit eens in IE:

JavaScript:

alert('blaat.gif'.substr(-3));

[Verwijderd]

Thx voor de tips nu werkt het script

Weten jullie nog een site met een goede functie overview van javascript? Zoals bij php www.php.net is?

Dat het script split bij de punt is idd practischer bij bijv jpeg files.

edit: dat substr(-3) werkt niet echt..
Wat crisp ook zegt

[ Voor 13% gewijzigd door Verwijderd op 08-12-2004 23:18 ]

[crisp]

Hier nog 2 alternatieven verpakt in een werkend voorbeeld:

JavaScript:

function f_validate(bestandsnaam)
{
    var re = /\.(mid|jpg|jpeg|gif|bmp)$/;
    if (re.test(bestandsnaam)) return true;

    var extension = bestandsnaam.substring(bestandsnaam.lastIndexOf('.')+1);
    alert('Deze extensie (' + extension + ') is niet bruikbaar voor mobiele telefoons.');
    return false;
}

[Wolfboy]

crisp schreef op woensdag 08 december 2004 @ 23:07:
Probeer dit eens in IE:

JavaScript:

alert('blaat.gif'.substr(-3));

Oeps, alleen in FF getest

En ik had idd niet gedacht aan hoe het formpje zou reageren op return false