Discussion :

[dev0712]

bonjour

j'ai utilisé cette fonction pour calculer le nombre occurrences d'un caractère dans une chaine mais ce programme m'a affiche toujours 0.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
int nboccurences(char tab[],char x)
    {
     int nb=0,i;     
 
         for(i=0;i<strlen(tab);i++)
         {
          if(tab[i]=='x')
            nb++;
         }
     return nb;
    }
 
 
int main()  
    {
 
char ch[300];
char k;
int n;
 
printf("\necrire une caractere \n");
scanf("%c",&k);
printf("\ndonner une mot SVP\n");
scanf("%s",&ch);
n=nboccurences(ch,k);
printf("\nle nombre d'occurences du %c egale a %d\n",k,n);
system("pause");    
    }

[diogene]

Code :

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

          if(tab[i]== x)

x sans les ' '. Sinon, il cherche le caractère 'x'

[droggo]

Jei,

De plus, il faut faire

Code :

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

scanf("%s",ch);

et non

Code :

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

scanf("%s",&ch);

et conseil gratuit: éviter de faire

Code :

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

1
2
3
4
5
         for(i=0;i<strlen(tab);i++)
         {
          if(tab[i]=='x')
            nb++;
         }

car strlen(tab) est évalué à chaque tour de boucle.

Préférer (tant que la longueur n'est pas modifiée dans la boucle, bien entendu )

Code :

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

1
2
3
4
5
6
         int len = strlen(tab);
         for(i=0;i<len;i++)
         {
          if(tab[i]=='x')
            nb++;
         }

[Emmanuel Delahaye]

j'ai utilisé cette fonction pour calculer le nombre occurrences d'un caractère dans une chaine mais ce programme m'a affiche toujours 0.

en dehors des défauts signalés, il n'est pas utile de parcourir la chaine 2 fois. Une fois suffit, il suffit de tester la fin de chaine...

D'autre part, éviter les tests mettant en œuvre un opérateur humain, mais privilégier au contraire une série de tests automatiques...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
 
int nboccurences (char const tab[], char x)
{
   int nb = 0, i;
 
   for (i = 0; tab[i] != 0; i++)
   {
      if (tab[i] == x)
      {
         nb++;
      }
   }
   return nb;
}
 
#if TEST
#include<stdio.h>
#include <assert.h>
 
int main (void)
{
   assert (nboccurences ("a", 'a') == 1);
   assert (nboccurences ("b", 'a') == 0);
   assert (nboccurences ("ab", 'a') == 1);
   assert (nboccurences ("aba", 'a') == 2);
   assert (nboccurences ("aba", 'b') == 1);
   assert (nboccurences ("aba", 'c') == 0);
   puts ("P A S S E D");
   return 0;
}
#endif