C4047 : Incompréhensible

**chental** · 04/08/2006, 09h47

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 :

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

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;
}

**fearyourself** · 04/08/2006, 09h49

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 :

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

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 ?

**chental** · 04/08/2006, 09h53

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

**Foobar1329** · 04/08/2006, 09h54

Hello,

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+

**fearyourself** · 04/08/2006, 09h54

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 ?

**fearyourself** · 04/08/2006, 09h55

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

**chental** · 04/08/2006, 09h59

Code :

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

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.

**fearyourself** · 04/08/2006, 10h05

Envoyé par fearyourself

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

EEEEEEETTTTTTTTTTTTTTTT :

Envoyé par chental

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

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

**chental** · 04/08/2006, 10h09

Envoyé par fearyourself

EEEEEEETTTTTTTTTTTTTTTT :

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.

**Foobar1329** · 04/08/2006, 10h09

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 :

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

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

A+

**fearyourself** · 04/08/2006, 10h11

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 :

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

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

**Médinoc** · 04/08/2006, 10h13

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.

**chental** · 04/08/2006, 10h16

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 :

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

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

**Foobar1329** · 04/08/2006, 10h23

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 :

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

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

A+

**chental** · 04/08/2006, 10h30

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 :

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

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

A+

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

C4047 : Incompréhensible

C

Discussions similaires

Partager

Partager