Discussion :

[Francky44003]

bonsoir a tous je fais appelle a vous car je rencontre des difficultés sur ma fonction qui est "censés" compter le nombre de lettre d'un mot parmi une liste (situées dans une matrice).

J'ai tenté de creer une fonction qui me permettra de compter ces lettres mais dès que je compile mon programme plante et je n'arrive pas a comprendre pourquoi. Du moins, je sais que c'est a cause de ma fonction que mon programme plante mais je n'arrive pas a comprendre pourquoi celle -ci fait planter. c'est pour cela que je fais appelle a vous.

voici le code en question :

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 compterNombreDeLettre (char ListeMots[5][10], int MotAtrouver)
    {
        int x = MotAtrouver, y = 0;
 
        while (ListeMots[x][y] != '0')
        {
            y++;
        }
 
        return y;
    }
 
    srand(time(NULL));
    int PositionMot = rand()%5;
    int NombreLettres = 0;
 
 
    char MotAtrouver[5][10];
 
 
         MotAtrouver[1][7] = "chiens";
         MotAtrouver[2][6] = "chats";
         MotAtrouver[3][7] = "avions";
         MotAtrouver[4][9] = "voitures";
         MotAtrouver[5][8] = "oiseaux";
 
 
 
 
    NombreLettres = compterNombreDeLettre (MotAtrouver[5][10], PositionMot);
    printf("%d", NombreLettres);

PS : je vous remercie d'avance pour l'aide que vous m'apporterez

[Lucien63]

Code :

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

1
2
3
4
5
         MotAtrouver[1][7] = "chiens";
         MotAtrouver[2][6] = "chats";
         MotAtrouver[3][7] = "avions";
         MotAtrouver[4][9] = "voitures";
         MotAtrouver[5][8] = "oiseaux";

Alors déjà ici, ce n'est pas bon du tout.
Soit tu initialise ton tableau lors de sa déclaration.
Soit tu utilise les fonctions strcpy ou mieux strncpy pour copier les chaine dans le tableau.

[Francky44003]

ok je vais donc initialiser le tableau dès sa declaration, mais par contre peux tu m'aider en ce qui concerne la fonction car je ne sais pas comment m'y prendre

[Lucien63]

Code :

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

while (ListeMots[x][y] != '0')

dans le while tu testes le caractère 0 alors que je supose que tu veux tester le caractère nul, ce n'est pas du tout la même chose.

[Sve@r]

dans le while tu testes le caractère 0 alors que je supose que tu veux tester le caractère nul, ce n'est pas du tout la même chose.

Bien vu. Ne pas confondre '0' (valeur ascii 48) et '\0' (valeur ascii 0)

bonsoir a tous je fais appelle a vous car je rencontre des difficultés sur ma fonction qui est "censés" compter le nombre de lettre d'un mot parmi une liste (situées dans une matrice).

J'ai tenté de creer une fonction qui me permettra de compter ces lettres mais dès que je compile mon programme plante et je n'arrive pas a comprendre pourquoi. Du moins, je sais que c'est a cause de ma fonction que mon programme plante mais je n'arrive pas a comprendre pourquoi celle -ci fait planter. c'est pour cela que je fais appelle a vous.

voici le code en question :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 int compterNombreDeLettre (char ListeMots[5][10], int MotAtrouver)
    {
        int x = MotAtrouver, y = 0;
 
        while (ListeMots[x][y] != '0')
        {
            y++;
        }
 
        return y;
    }

Je ne comprends pas la logique de cette fonction.
Elle reçoit en paramètre l'ensemble de tous les mots et la position du mot à compter (accessoirement ton nom de variable est à revoir)

Pourquoi ne pas la faire recevoir juste le mot à compter ???

PS: une initialisation (y=0); un test (ListeMots[x][y] != '\0'); un incrément (y++). Tu ne reconnais pas là tous les éléments d'un "for" ???

[DeathMixer]

Code :

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

1
2
3
4
5
         MotAtrouver[1][7] = "chiens";
         MotAtrouver[2][6] = "chats";
         MotAtrouver[3][7] = "avions";
         MotAtrouver[4][9] = "voitures";
         MotAtrouver[5][8] = "oiseaux";

Alors déjà ici, ce n'est pas bon du tout.
Soit tu initialise ton tableau lors de sa déclaration.
Soit tu utilise les fonctions strcpy ou mieux strncpy pour copier les chaine dans le tableau.

Le problème n'est pas vraiment la façon dont il fait la déclaration. L'erreur la plus flagrante est sur le type des données.

Code :

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

1
2
MotAtrouver[1][7] = "chiens";
...

MotAtrouver est censé être un 'char **'... Donc 'MotAtrouver[1][7]' doit être un caractère, et non une chaîne de caractères...

[Lucien63]

MotAtrouver est censé être un 'char **'... Donc 'MotAtrouver[1][7]' doit être un caractère, et non une chaîne de caractères...

Tu ne veux quand même pas lui faire remplir le tableau caractère par caractère, malgré que cela soit possible.
Pour moi la façon dont il rempli le tableau c'est n'importe quoi !
Il faut qu'il regarde son cours sur les tableaux et les chaine de caractères avant d'aller plus loin.

PS : MotAtrouver est un char[][10] pas un char**
tu va l'embrouiller.

[Mat.M]

Salut Momo-Ben je te déconseille de déclarer des tableaux avec des tailles fixes c'est ingérable.
Il vaut mieux utiliser strcpy comme le dit lucien63 ou bien faire un mallloc à chaque fois.
Pour des performance de recherche accrues il vaut mieux envisager de faire des listes chainées

[Sve@r]

Salut Momo-Ben je te déconseille de déclarer des tableaux avec des tailles fixes c'est ingérable.
Il vaut mieux utiliser strcpy comme le dit lucien63 ou bien faire un mallloc à chaque fois.
Pour des performance de recherche accrues il vaut mieux envisager de faire des listes chainées

Non mais attend !!! Tu vois bien que c'est un exo de débutant qu'on lui donne à faire. Ne va pas lui parler de malloc et de listes chainées ; surtout qu'une liste chainée n'offre aucune performance de recherche mais juste des performances au niveau de l'insertion d'un nouvel élément...

Code C :

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

1
2
3
4
5
6
7
char *MotAtrouver[]={
    "chiens",
    "chats",
    "voitures",
    "avions",
    "oiseaux",
};

[Francky44003]

merci a tous, je vais mettre vos conseils en pratique

[Francky44003]

re bonjour a tous, j'ai resolu mon probleme en mettant les mots dans un fichier.txt, l'implementation des fonctions qui travaillent sur le tableau de caractere devient beaucoup plus simple

PS : merci a tous encore une fois pour vos conseils

[Invité]

Bonjour,
Je vais juste ajouter un mot : une matrice est quelque-chose de bien particulier.
Quant on lit le titre, on détecte tout de suite le contexte. Vous remarquerez d'ailleurs que le terme "matrice" n'a été employé par aucun de ceux qui vous ont répondu. Il s'agit de tableau. Naturellement tout le monde avait compris qu'il s'agissait de tableau, mais surtout tout le monde avait compris que vous ne saviez pas ce qu'était une matrice.