Kiki2sirom -> le .length c'est loin d'être une erreur, ça rajoute en lisibilité au code et la différence est quasiment nulle.
kiki2sirom
Messages postés153Date d'inscriptionmardi 17 août 2004StatutMembreDernière intervention23 décembre 2010 24 nov. 2007 à 19:32
petite précision :
ici je commente et note la syntaxe, et non le code en lui-même, inutile pour moi.
Ma note s'explique donc qu'il y a 2 erreurs grossières, pour moi, sur 15 lignes de code.
Persévère.
kiki
kiki2sirom
Messages postés153Date d'inscriptionmardi 17 août 2004StatutMembreDernière intervention23 décembre 2010 24 nov. 2007 à 19:31
salut
bon :
1) déjà vu 100 et 1000 fois, sur le site ou autre part
2)
for (var j=0;j<a.length;j++){
devient:
var taille=a.length;
for (var j=0;j<taille;j++){
pourquoi ? ne pas faire le calcul de la longueur de la chaîne à chaque tour de boucle, mais 1 seule fois : IMPORTANT
3)
if(valide.indexOf(a.charAt(j)) != -1){
// rien
};
else{
alert('Le caractère '+a.charAt(j)+'est interdit');
devient :
if(valide.indexOf(a.charAt(j)) == -1)
alert('Le caractère '+a.charAt(j)+'est interdit');
non ? si !!!!
4) pas du tout convaincu par l'optimisation du code (vérifier que chaque lettre est autorisée)
25 nov. 2007 à 13:10
Je vais la supprimer cete mini source .
Mais avant pouvez vous me fournir une fonction qui teste la faille d'un fichier en upload
( )
Le js je l'ai jamais pratiqué , je fais juste copier coller... celui là est mon 1er essaie , je ne connais pas
la syntaxe ( ^[a-zA-Z]+$/ .... )
24 nov. 2007 à 22:18
24 nov. 2007 à 22:13
return /^[a-zA-Z]+$/.test(document.form_inscription.txt_login.value);
Kiki2sirom -> le .length c'est loin d'être une erreur, ça rajoute en lisibilité au code et la différence est quasiment nulle.
24 nov. 2007 à 19:32
ici je commente et note la syntaxe, et non le code en lui-même, inutile pour moi.
Ma note s'explique donc qu'il y a 2 erreurs grossières, pour moi, sur 15 lignes de code.
Persévère.
kiki
24 nov. 2007 à 19:31
bon :
1) déjà vu 100 et 1000 fois, sur le site ou autre part
2)
for (var j=0;j<a.length;j++){
devient:
var taille=a.length;
for (var j=0;j<taille;j++){
pourquoi ? ne pas faire le calcul de la longueur de la chaîne à chaque tour de boucle, mais 1 seule fois : IMPORTANT
3)
if(valide.indexOf(a.charAt(j)) != -1){
// rien
};
else{
alert('Le caractère '+a.charAt(j)+'est interdit');
devient :
if(valide.indexOf(a.charAt(j)) == -1)
alert('Le caractère '+a.charAt(j)+'est interdit');
non ? si !!!!
4) pas du tout convaincu par l'optimisation du code (vérifier que chaque lettre est autorisée)
merci passez votre chemin
kiki