Discussion :

[marguerite99]

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

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
#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

[Bayard]

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 :

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
 
#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:

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.

[gl]

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 :

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
#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.

[marguerite99]

Merci Bayard

[marguerite99]

Merci gl