problème au niveau de fonction

Salut à toutes et à tous ;)
mon problème est le suivant :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

#include <stdio.h> #include <conio.h> #include <string.h>   void main() { char* inverse(char s[30]) { char ch[30]; int i; for(i=0; i<=strlen(s)-1; i++) ch[i] = ch[strlen(s)-i-1]; return ch; } char s[30]; printf("Entrer un entier entre 0 et 99999 : "); gets(s); if(!(strcmp(inverse(s),s))) puts("palindrome"); else puts("n'est pas palindrome"); getch(); }

---

quand j'exécute ce petit bout de code, il m'affiche comme erreur :

error C2601: 'inverse' : définitions de fonctions locales non conformes


Merci de m'aider

1) Penser à utiliser l'icone CODE (le #) pour insérer du code.

2) Bien mettre des tabulations afin de mieux lire le code des prototypes de fonctions:

3) Ne pas déclarer de fonctions à l'intérieur de fonctions

Code:

---

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

#include <stdio.h> #include <conio.h> #include <string.h>     char* inverse(char s[30]) { char ch[30]; int i; for(i=0; i<=strlen(s)-1; i++) ch[i] = ch[strlen(s)-i-1]; return ch; }   void main(void) { char s[30]; printf("Entrer un entier entre 0 et 99999 : "); gets(s); if(!(strcmp(inverse(s),s))) puts("palindrome"); else puts("n'est pas palindrome"); getch(); }

---

4) Ce warning n'est pas un warning mais bel est bien une erreur dans votre cas:

Citation:

---

return ch;
main.c|13|warning: function returns address of local variable|

---

Vous déclarez une variable sur la pile... et en suite vous la mettez à disposition.
Je vous encourage à déclarer ch en dehors de la fonction.
En variable globale (avec un static devant) ce serait bien.

5) Ne déclarez pas de variables avec une ou 2 lettres, la lisibilité n'est pas une option.

Bon courage et week-end.

Citation:

---

Envoyé par marguerite99

quand j'exécute ce petit bout de code, il m'affiche comme erreur :

error C2601: 'inverse' : définitions de fonctions locales non conformes

---

Le message d'erreur me semble pourtant assez explicite.

En C, il n'existe pas de fonction locale. On ne peut donc pas définir de fonction à l'intérieur d'une autre fonction, il faut donc sortir la définition de la fonction inverse du main, ce qui donne:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

#include <stdio.h> #include <conio.h> #include <string.h>   char* inverse(char s[30]) { char ch[30]; int i; for(i=0; i<=strlen(s)-1; i++) ch[i] = ch[strlen(s)-i-1]; return ch; }   void main() { char s[30]; printf("Entrer un entier entre 0 et 99999 : "); gets(s); if(!(strcmp(inverse(s),s))) puts("palindrome"); else puts("n'est pas palindrome"); getch(); }

---

Ceci, il existe d'autres problèmes dans ce code :

• La fonction gets() ne vérifiant pas la taille de la saisie peut conduire à des débordement. IL ne faut pas l'utiliser et lui préférer fgets().
• Dans la fonction inverse, tu retournes l'adresse d'une variable locale (ch), ce qui n'est pas correcte.
• conio.h et getch() ne sont pas standard.
• La norme prévoit deux formes pour main : int main(void) et int main(int argc, char* argv[]), void min() n'est pas une forme normalisée et n'est donc pas portable.
• La fonction inverse est probablement fausse, puisque tu copies un élément de ch non initialisé dans ch, ne serait-ce pas plutôt ch[i] = s[strlen(s)-i-1];
• En outre, d'un point de vue des performance, tu ne devrait appeler strlen() qu'une seule et unique fois et conserver le résultat.

Merci Bayard
:oops:

Merci gl
;)