Discussion :

[rikudou]

salut j'ai finis de créer ce programme mais j'aii un peu de doute pouvez vous m'aider a preciser les point faibles de ce programme.
voila le programme :

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
32
33
34
35
36
37
38
39
40
#include<stdio.h>
#define Max 200
 
int longeur(char tab[Max]){
int i=0;
int cmpt=-1;
while(tab[i++]!='\0')  {  cmpt++;      }
 
return cmpt;
}
 
void lire_chaine(char tab[Max]){
int i=0;
char c;
c=getchar();
while((c!='\n')&& (i<Max))
{
tab[i++]=c;
c=getchar();
}
tab[i]='\0';
}
 
int pal(char tab[Max])  {
int ln =longeur(tab);
int i=0;
while(i<=ln/2){
if(tab[i]!=tab[(ln-1)-i])return 0;
 
}
   return 1;
}
int main(){
char tab[Max];
    lire_chaine(tab);
    if(pal(tab))
printf("la longeur est Pal");
else printf("la longeur est Pal");
getchar();
}

[Trademark]

Salut,

Je vais faire fonction par fonction.

1) int longueur(char tab[Max])

Tu pourrais n'utiliser qu'une seule variable, ça serait plus élégant.

2) void lire_chaine(char tab[Max])

Il y a un problème de borne, si tu mets Max à 1 caractère, tu comprendras mieux ce qui se passe. Le caractère de fin '\0' est stocké en dehors de la zone allouée.

3) int pal(char tab[Max])

i n'est jamais incrémenté... as-tu testé ton programme ?
De plus, je pense que le mieux, c'est que lire_chaine renvoie un entier pour dire le nombre de caractère lu. Tu pourrais ainsi faire :

Code :

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

1
2
3
int est_un_palindrome(char *tab, int size);
 
printf("%d", est_un_palindrome(tab, lire_chaine(tab));

4) Le main : tes deux printf ne veulent pas dire grand chose.

[rikudou]

merci bcp .
Pouvez-vous corriger mon programme, s'il vous plaît

[Trademark]

Non, et puis quoi encore, tu as tout ce qu'il te faut pour le faire toi même.

[rikudou]

Ca y est j'ai trouve
merci bcp