C4047 : Incompréhensible

Version imprimable

Salut à tous,

Je ne comprends pas l'erreur C4047 que me renvoie mon compilateur avec cette fonction sur la ligne de comparaison en gras
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 int Comparaison(char* Nom,char* Code) { int i,j; printf("\n%s %c\n",Nom,Code); for (i=0;i<nb_max;i++ ) { if (strcmp(Pelm[i].nom,Nom)==0 && strcmp(Pelm[i].code,Code)==0) { return i; } } return -1; }

04/08/2006, 09h49
fearyourself

Citation:

Envoyé par chental

Salut à tous,
Je ne comprends pas l'erreur C4047 que me renvoie mon compilateur avec cette fonction sur la ligne de comparaison en gras

Alors je cherche l'erreur C4047... Ah oui, c'est en rapport avec la remarque C231...

[EDIT]
D'ailleurs :

Code:

printf("\n%s %c\n",Nom,Code);

le %c avec un type char* n'est pas vraiment recommandé...

et si on avait la déclaration de Pelm (donc la structure...) ce serait aussi un +
[/EDIT]

Jc

PS : Le message d'erreur en complet serait plus sympa non ?
04/08/2006, 09h53
chental

erreur complète

L'erreur complète est : C4047 : 'function ' : 'const char *' differs in levels of indirection from 'char'
04/08/2006, 09h54
Foobar1329

Hello,

Citation:

Envoyé par chental

Salut à tous,

Je ne comprends pas l'erreur C4047 que me renvoie mon compilateur avec cette fonction sur la ligne de comparaison en gras

int Comparaison(char* Nom,char* Code)
{
int i,j;
printf("\n%s %c\n",Nom,Code);
for (i=0;i<nb_max;i++ )
{
if (strcmp(Pelm[i].nom,Nom)==0 && strcmp(Pelm[i].code,Code)==0)
{
return i;
}
}
return -1;
}

Ce n'est pas incompréhensible, cela veut dire d'après MSDN que tu as des niveaux d'indirection incompatibles [Edit, Remove : (pas le même type, mais ne génère qu'un warning car il s'agit de types pointeurs)], e.g. tu donnes un char** à quelquechose qui attend un char *.

[Edit : Je mettais mis dans la tête que ça devait générer une erreur...Ben non. A part ça, cela veut dire que tu passes un char à strcmp() qui n'attend que des const char*. Regardes bien le type des variables que tu passes à strcmp()].

A+
04/08/2006, 09h54
fearyourself

Citation:

Envoyé par chental

L'erreur complète est : C4047 : 'function ' : 'const char *' differs in levels of indirection from 'char'

Et ta déclaration de ta structure ressemble à quoi ?
04/08/2006, 09h55
fearyourself

Citation:

Envoyé par Foobar1329

Hello,

Ce n'est pas incompréhensible, cela veut dire d'après MSDN que tu as des niveaux d'indirection incompatibles (pas le même type, mais ne génère qu'un warning car il s'agit de types pointeurs), e.g. tu donnes un char** à quelquechose qui attend un char *.

A+

Je crois que c'est l'inverse ici... Un des champs Nom ou Code est sûrement de type char...

Jc
04/08/2006, 09h59
chental

Voici ma structure
Code:

1 2 3 4 5 6 7 8 typedef struct /* Déclaration de notre structure */ { char nom[30]; char code; int cotes[5]; }Element; Element Pelm;
La partie printf() fonctionne, c'est la partie strcmp qui pose problème.
Effectivement lors de la compil., j' ai un warning donc ça passe mais à l'exécution , gros bugggggg.
04/08/2006, 10h05
fearyourself

Citation:

Envoyé par fearyourself

Je crois que c'est l'inverse ici... Un des champs Nom ou Code est sûrement de type char...

EEEEEEETTTTTTTTTTTTTTTT :

Citation:

Envoyé par chental

typedef struct /* Déclaration de notre structure */
{
char nom[30];
char code;
int cotes[5];
}Element;

:mouarf:

Aller, regarde le prototype de la fonction strcmp, regarde le type du paramètre Code, regarde le type de ton élément code et réfléchit...

Jc
04/08/2006, 10h09
chental

Citation:

Envoyé par fearyourself

EEEEEEETTTTTTTTTTTTTTTT :

:mouarf:

Aller, regarde le prototype de la fonction strcmp, regarde le type du paramètre Code, regarde le type de ton élément code et réfléchit...

Jc

Ils sont tous du même type, et je ne vois aucune difference.
Sauf que ma fonction fait appel à un pointeur d'ailleurs de m^me type.
04/08/2006, 10h09
Foobar1329
Citation:

Envoyé par chental

typedef struct /* Déclaration de notre structure */
{
char nom[30];
char code;
int cotes[5];
}Element;

Element Pelm;

La partie printf() fonctionne, c'est la partie strcmp qui pose problème.
Effectivement lors de la compil., j' ai un warning donc ça passe mais à l'exécution , gros bugggggg.

Hé bien, Pelm.code est un char. Si tu veux comparer 2 char, ce n'est pas strcmp, c'est opérateur d'égalité direct qu'il faut utiliser. En supposant que Code est aussi un char, ça doit donner
Code:

1 2 3 if (!strcmp(Pelm[i].nom,Nom) && Pelm[i].code == Code) { /*...*/ }
A+
04/08/2006, 10h11
fearyourself
Citation:
Envoyé par Foobar1329

Hé bien, Pelm.code est un char. Si tu veux comparer 2 char, ce n'est pas strcmp, c'est opérateur d'égalité direct qu'il faut utiliser. En supposant que Code est aussi un char, ça doit donner

Code:

1 2 3 if (!strcmp(Pelm[i].nom,Nom) && Pelm[i].code == Code) { /*...*/ }

A+
Pas exactement, le paramètre Code est aussi un char *... Mais je pense que si on regardait comment l'appel est fait, on remarquerait vite que le 2ème paramètre est aussi un char...

Il faut faire un peut attention à ce qu'on fait...
Jc
04/08/2006, 10h13
Médinoc

Citation:

Envoyé par chental

Ils sont tous du même type, et je ne vois aucune difference.
Sauf que ma fonction fait appel à un pointeur d'ailleurs de m^me type.

C'est faux : nom est un char[30], code est un char.
Tu ne vois pas une petite différence ?

char[30] est implicitement convertible en const char *.
Ce n'est pas le cas de char, donc "niveaux d'indirection différents" : strcmp() demande un pointeur, tu lui passes un char.
04/08/2006, 10h16
chental
Citation:
Envoyé par Foobar1329

Hé bien, Pelm.code est un char. Si tu veux comparer 2 char, ce n'est pas strcmp, c'est opérateur d'égalité direct qu'il faut utiliser. En supposant que Code est aussi un char, ça doit donner

Code:

1 2 3 if (!strcmp(Pelm[i].nom,Nom) && Pelm[i].code == Code) { /*...*/ }

A+
En effet, tu as raison par contre je devrais passer Code en valeur et non en réference.
Là ça marche mais ça me fait perdre la portabilité.
04/08/2006, 10h23
Foobar1329
Citation:

Envoyé par chental

En effet, tu as raison par contre je devrais passer Code en valeur et non en réference.
Là ça marche mais ça me fait perdre la portabilité.

??? C'est quoi cette perte de portabilité ? Tu mest ton paramètre Code de ta fonction Comparaison() en char et le tour est jour. Si éventuellement Code dans Comparaison() est utilisé ailleurs, tu peux le laisser en char * en paramètre : tu vérifies que Code != NULL puis tu fais
Code:

1 2 3 if (!strcmp(Pelm[i].nom,Nom) && Pelm[i].code == *Code) { /*...*/ }
A+
04/08/2006, 10h30
chental
Citation:
Envoyé par Foobar1329

??? C'est quoi cette perte de portabilité ? Tu mest ton paramètre Code de ta fonction Comparaison() en char et le tour est jour. Si éventuellement Code dans Comparaison() est utilisé ailleurs, tu peux le laisser en char * en paramètre : tu vérifies que Code != NULL puis tu fais

Code:

1 2 3 if (!strcmp(Pelm[i].nom,Nom) && Pelm[i].code == *Code) { /*...*/ }

A+
C'est vrai t'as raison, testé et Ok.