Discussion :

[kojima]

salut
voila j'ai un probleme dans ce code

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
#include<stdio.h>
#include<string.h>
int main(){
int n,i,x;
char t[50][50];
int *p;
printf("entrer la dimension du tableau : ");
scanf("%d",&n);
for(i=0;i<n;i++){
        printf("\n entrer t[%d] :",i);
        getchar();
       // scanf("%s",t[i]);
       gets(t[i]);
    }
    printf("\n");
 for(i=0;i<n;i++){
x=strlen(t[i]);
 printf("la longueur du mot %d est %d\n",i+1,x);
 }
 
    printf("\n");
 
 for(i=0;i<n;i++){
 
        printf("\n le mot %d est %s",i+1,t[i]);
    }
}

le problème ici c'est pour le strlen dans le code pour i=0 ca marche
mais pouri=1,2,3,4,5
strlen(forum) donne 4 et non 5
dans l'affichage a la place forum le code affiche orum
je ne voix pas d'où vient le problème
merci

[diogene]

C'est à cause du getchar() qui avale le premier caractère de la ligne suivante.
Si tu l'as mis pour supprimer le '\n' restant en attente dans le buffer après la lecture de n, il faut le sortir de la boucle :

Code :

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

1
2
3
4
5
6
7
8
....
scanf("%d",&n);
getchar();
for(i=0;i<n;i++){
   printf("\n entrer t[%d] :",i);
   gets(t[i]);
}
....

[SofEvans]

Bonjour,

Le probleme provient du getchar().

Lorsque tu fais ta saisie, le premier caractere est pris par getchar(), donc, c'est normal que tu est un caractere de moins.

Je suppose que le getchar est la pour eviter le '\n' de la precedente saisie.
C'est l'idée, mais ce n'est pas l'application. Va faire un tour du coté de la FAQ, il y a un fonction qui permet de vider le buffer clavier.

Sinon, des remarque au passage :

**
Tu as fait l'effort d'indenter, mais essai d'etre plus rigoureux. Il y a la place pour l'amelioration.


**
fais attention a gets, il ne permet pas de savoir si tu as taper plus de 50 caractere ! Tant qu'a securiser un minimum, autant le faire. Regarde du cote de fgets. c'est comme gets, mais en plus securiser (et c'est important).


**
Attention, tu declare un pointeur p et tu ne l'initialise pas.
Je comprend que tu est du mal a voir ce qui va se passer, mais tu peux faire planter tes programme a cause de ca.
Il faut toujours initialiser ses pointeur, soit a la bonne adresse si tu la connais, soit a NULL pour eviter une betise.

**
Main retourne un int. Il faut rajouter return EXIT_SUCCESS; si ce n'est pas deja fait

[kojima]

merci pour les reponses

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
#include<stdio.h>
#include<string.h>
int main(){
int n,i,x;
char t[50][50];
int *p;
printf("entrer la dimension du tableau : ");
scanf("%d",&n);getchar();
for(i=0;i<n;i++){
        printf("\n entrer t[%d] :",i);
 
       // scanf("%s",t[i]);
       gets(t[i]);
    }
    printf("\n");
 for(i=0;i<n;i++){
x=strlen(t[i]);
 printf("la longueur du mot %d est %d\n",i+1,x);
 }
 
    printf("\n");
 
 for(i=0;i<n;i++){
 
        printf("\n le mot %d est %s",i+1,t[i]);
    }
}

ca marche bien
le fget n'est pas reconnu
je dois encore copier les mot terminant par oir dans un autre tableau
je poserais le code après
merci

[SofEvans]

c'est fgets

Tu as tout le temps pour voir ces notions, ne t'inquiète pas outre mesure (mais intéresse toi y si tu veux continuer le C)

[kojima]

bonsoir
et merci àSofEvans et diogene pour l'aide
j'ai modifié le code
les mots finissant par oir je dois les copier dans un autre tableau
voila mon code

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
#include<stdio.h>
#include<string.h>
#include<conio.h>
int main(){
int n,i,x,j;
char t[50][50],m[50][50];
char *p;
printf("entrer la dimension du tableau : ");
scanf("%d",&n);getchar();
for(i=0;i<n;i++){
        printf("\n entrer t[%d] :",i+1);
       scanf("%s",t[i]);
    }
    printf("\n");
    j=0;
 for(i=0;i<n;i++){
x=strlen(t[i]);
p=t[i]+x-3;
if (strcmp(p,"oir")==0)){
            m[j]=t[i];
            j++;
            }
 
 }
 //printf("\n");
 for(i=0;i<n;i++){
        printf("\n le mot %d termine par oir est %s",i+1,m[i]);
    }
    return 0;
}

merci d'avance

[diogene]

1-

Code :

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

m[j]=t[i];

est incorrect : m[j] est un tableau et il est impossible d'écrire "tableau = quelque chose"

2- Si tu veux copier la chaine, tu dois utiliser à la place strcpy()

Code :

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

1
2
3
char m[50][50];
....
strcpy(m[j],t[i]);

3- Si tu veux effectivement ne copier que le pointeur sur la chaine, alors m doit être un tableau de pointeurs, pas un tableau de tableau :

Code :

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

1
2
3
char  *m[50];
....
m[j]=t[i]

4- Le code suivant est faux si x<3 (p pointe alors AVANT le début de la chaîne, donc sur la fin du tableau précédent si il existe)

Code :

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

1
2
3
4
x=strlen(t[i]);
if(x>2 && strcmp(t[i]+x-3,"oir")==0){
....
}

5- La boucle d'affichage doit se terminer lorsqu'on arrive à la fin des valeurs stockées dans m. Il y en a j :

Code :

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

1
2
3
 for(i=0;i<j;i++){
        printf("\n le mot %d termine par oir est %s",i+1,m[i]);
    }

[kojima]

merci pour les réponses
mais même en modifiant le code ça ne marche pas
merci

[nsanabi]

une parenthese en plus ici :

Code :

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

if (strcmp(p,"oir")==0))

[kojima]

voila le nouveau code

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
#include<stdio.h>
#include<string.h>
#include<conio.h>
int main(){
int n,i,x,j;
char t[50][50],m[50][50];
char *p;
printf("entrer la dimension du tableau : ");
scanf("%d",&n);getchar();
for(i=0;i<n;i++){
        printf("\n entrer t[%d] :",i+1);
       scanf("%s",t[i]);
    }
    printf("\n");
    j=0;
 for(i=0;i<n;i++){
x=strlen(t[i]);
if(x>3){
p=t[i]+x-3;
if (strcmp(p,"oir")==0){
            strcpy(m[j],t[i]);
            j++;
            }
 }}
 for(i=0;i<j;i++){
        printf("\n le mot %d termine par oir est %s",i+1,m[i]);
    }
    return 0;
}

le code affiche tous les mots commençant par les mots finissant par oir puis les mots se trouvant dans la mémoire
je ne trouve pas l'erreur
merci

[Pouet_forever]

Ton code à l'air correct.
Tu arrives à te relire là ?

[gl]

le code affiche tous les mots commençant par les mots finissant par oir puis les mots se trouvant dans la mémoire
je ne trouve pas l'erreur

Es-tu bien certain d'avoir bien recompiler le programme et d'exécuter le bon programme ?
Parce que là la trace d'exécution ne correspond pas du tout au code fourni.

[kojima]

merci le code fonctionne bien