Discussion :

[laurentSc]

Bonsoir,

j'essaie de faire une fonction qui fait ce qui est dans le titre.
J'ai fait ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
<script>
function nbChar(chaine,lettre) {
var nb = 0;
foreach (ch in chaine)
{
if (ch == lettre) nb++;
}
return nb;
}
chaine='ceci est une chaine';
document.write('chaine:'+chaine);
document.write('nb de c ds '+chaine+':'+nbChar(chaine,"c"));
</script>

mais ça ne marche pas ; qu'est-ce qui cloche ?

[Willpower]

Code js :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
function nbChar(chaine,lettre) {
  return chaine.match(new RegExp(lettre,"gi")).length;
}

nb: remplace "gi" par "g" pour avoir une recherche sensible à la case (majuscule != minuscule)


edit:
autrement ton code ne fonctionnait pas pour la simple raison que "foreach" n'existe pas. il existe "for each" sous firefox... mais comme tu ne codes certainement pas exclusivement sous firefox, ce qui s'en rapproche le plus aurait été "for(i in chaine) ch=chaine[i]" et encore, ça n'aurait pas fonctionné sous internet explorer car les string ne sont pas considérés comme des tableaux(donc pas de parcours), si je ne m'abuse.

ce qui donnerait ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
function nbChar(chaine,lettre) {
  var nb = 0;
  chaine = chaine.split(""); // cette ligne n'est pas nécessaire pour firefox ni chrome
  for(var i in chaine)
    nb += chaine[i] == lettre;
  return nb;
}

[SpaceFrog]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
function nbChar(chaine,lettre) {
  var nb = 0;
  chaine = chaine.split("");
  return chaine.length-1;
}

[laurentSc]

Merci,

mais avec ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
function nbChar(chaine,lettre) {
  var nb = 0;
  chaine = chaine.split(lettre);
  return chaine.length-1;
}

c'est encore mieux...

[SpaceFrog]

oui desolé c'est en fait ce que je voulais ecrire
d'ou le length -1

[Willpower]

Merci,

mais avec ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
function nbChar(chaine,lettre) {
  var nb = 0;
  chaine = chaine.split(lettre);
  return chaine.length-1;
}

c'est encore mieux...

J'avais aussi pensé au split mais la fonction match me semblait plus appropriée.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
function nbChar(chaine,lettre) {
  return chaine.split(lettre).length-1;
}