Fonction non coopérante (toUpperCase)

Bonjour,

J'ai un problème avec une de mes fonctions...

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

function auteurRedressement() { var nomAuteur=document.getElementById("nomAuteur").value; var nomAuteurRevu = nomAuteur[0].toUpperCase(); // nomAuteur[0].toUpperCase() = nomAuteurRevu; var i=0; for (i=0; i<nomAuteur.length; i++) { if (nomAuteur[i]=="," || "-" || "") { nomAuteurRevu = nomAuteur[i+1].toUpperCase(); // nomAuteur[i+1].toUpperCase() = nomAuteurRevu; } } }

---

Elle est censée remettre en forme une saisie de type
nom,prenom en Nom,Prénom
ou nom, prenom1-prenom2 en Nom, Prénom1-Prénom2


Ma console m'affiche cette erreur :

Citation:

---

TypeError: nomAuteur[(i + 1)] is undefined

---

J'ai défini toutes mes variables et pense - peut-être à tort - que JS accepte, comme Python, les i+1...

Alors quel est le problème??

Merci pour votre aide !

Bonsoir,

tu as fait une alert de nomAuteur.length pour voir ?

Pour moi ça:

Code:

---

if (nomAuteur[i]=="," || "-" || "")

---

, ce n'est pas bon.

Natso

Citation:

---

Envoyé par mlebeguec

tu as fait une alert de nomAuteur.length pour voir ?

---

Bonsoir Natso.
nomAuteur.length me retourne 16 quand je saisis "Atwood, Margaret"
Donc ça, ça fonctionne :)

J'ai encore chipoté à ma fonction...
Qui est devenue entre temps :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

function auteurRedressement() {   var nomAuteur=document.getElementById("nomAuteur").value; alert(nomAuteur.length);   var nomAuteurRevu = nomAuteur[0].toUpperCase()+ nomAuteur.substr(1,nomAuteur.length).toLowerCase();   var i=0; for (i=0; i<nomAuteur.length; i++) { if (nomAuteur[i]=="," || "-" || "") { nomAuteurRevu = nomAuteur[i+1].toUpperCase(); document.getElementById("nomAuteur").value = nomAuteurRevu; } } }

---

Mais euh... il ne me retourne rien quand je tape "atwood, margaret"
Et "B" quand je tape "henry-levy, bernard"

Je ne comprends pas très bien quelle bêtise je fais :lol:

Citation:

---

Pour moi ça:<code>if (nomAuteur[i]=="," || "-" || "")</code> , ce n'est pas bon.

---

Ce sont les "ou" qui posent problème, d'après toi?
parce que la "," seule, mon script l'acceptait, au départ...

Merci pour la réponse rapide.

Pour moi ce serait:

Code:

---

1 2	nomAuteur[i]=="," || nomAuteur[i]=="-" || nomAuteur[i]==""

---

nomAuteurRevu est écrasé dans ta boucle

Je vais regarder plus en détail ton problème

Merci beaucoup.

Citation:

---

Code:

---

nomAuteur[i]=="," || nomAuteur[i]=="-" || nomAuteur[i]==""

---

---

J'ai essayé ta solution, il semblerait qu'elle génère la même chose que la mienne...

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

function auteurRedressement() {   var nomAuteur=document.getElementById("nomAuteur").value;   var nomAuteurRevu = nomAuteur[0].toUpperCase()+ nomAuteur.substr(1,nomAuteur.length).toLowerCase();   var i; for (i=0; i<nomAuteurRevu.length; i++) { //if (nomAuteur[i]==("," || "-" || "")) { if (nomAuteurRevu[i]=="," || nomAuteurRevu[i]=="-" || nomAuteurRevu[i]=="") { nomAuteurRevu = nomAuteurRevu[i+1].toUpperCase()+ nomAuteurRevu.substr(i+2,nomAuteurRevu.length).toLowerCase(); document.getElementById("nomAuteur").value = nomAuteurRevu; } } }

---

J'ai encore changé ma fonction...
Voici désormais où j'en suis...
si je tape henry-levy, bernard, il me retourne " bernard" (avec un espace devant)

je me suis dit que ma ligne 11 écrasait p-e ma chaine, en effet...

...et en même temps, si je tape "atwood, margaret" (stockée dans la variable nomAuteur),
que ma ligne 5 transforme celle-ci en "Atwood, margaret", nvlle valeur affectée à la variable nomAuteurRevu,
Que ma variable nomAuteurRevu est elle-même reparcourue avec un for, et modifiée (avec des toUpperCase et des toLowerCase) seulement à partir du moment où mes caractères ",", "-" et " " sont trouvés,
je ne comprends pas bien comment il peut y avoir un écrasement de ma chaîne?

Je suppose que ma logique est pourrie puisque le résultat de mon script en témoigne, mais je ne parviens pas à comprendre où mon raisonnement bat de l'aile...

Les premiers posts étaient pour régler les problèmes de syntaxe.

Normal tu fais ça: nomAuteurRevu[i+1]

Donc pour moi déjà, supprime les espaces avant et après ta valeur nomAuteur. La personne qui écrit pourrait mettre un espace au début.

Il y a plusieurs façons de faire ce n'est qu'un exemple

Code:

---

1 2 3 4 5 6 7 8 9

var nomAuteurRevu = nomAuteur[0].toUpperCase(); for (var i=1; i<=nomAuteur.length; i++) { if (nomAuteur[i-1]=="," || nomAuteur[i-1]=="-" || nomAuteur[i-1]=="") { nomAuteurRevu += nomAuteur[i].toUpperCase; else { nomAuteurRevu += nomAuteur[i]; } }

---

Un truc comme ça. Je suis désolé je n'ai pas testé. Un peu fatigué.
Natso

Je teste ça demain (je fatigue aussi ;) ) !
Mille mercis de t'être penché sur mon problème :ccool:

Il y a des outils surpuissants pour les chaines de caractères, utilisons les

Code:

---

1 2 3 4 5 6 7 8 9

function upperCaseFirstLetter(str) { return str.replace(/(^|[\-,])(\w)(\w*)/g,function(a,b,c,d) { return b + c.toUpperCase() + d; });   }   console.log(upperCaseFirstLetter('machin-bidule-truc,pouet'));

---

Merci,Arnogues.
J'ai pas très bien compris ta fonction, mais je vais enquêter pour la clarifier.

Bon dimanche.