Discussion :

[pegase.90]

Bonjour,

Je veux effectuer la chose suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void fct1(struct TATA *pTata){
...
...
 fct2(pTATA);  //erreur ici
..
..
 
}
 
char fct2(struct TATA pTata){
  return (char)(pTata.x);
}

Moi je pensais que c'etait possible mais vu que le compilo me jette avec "Error 2 : <function argument>: implicit cast of pointer to non-equal pointer", je ne dois pas faire comme il faut.
S'il quelqu'un peut m'expliquer cela serait super. Merci d'avance.

Pegase.90

[stephl]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void fct1(struct TATA *pTata){
...
...
 fct2(*pTATA);  //erreur ici
..
..

}

void fct2(struct TATA pTata){
  return (pTata.x);
}

EDIT: Un petit problème, non?

Code :

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

1
2
3
void fct2(struct TATA pTata){
  return (pTata.x);
}

[pegase.90]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void fct1(struct TATA *pTata){
...
...
 fct2(*pTATA);  //erreur ici
..
..
 
}
 
void fct2(struct TATA pTata){
  return (pTata.x);
}

Oui j'ai également pensé à ca mais la j'obtient le message d'erreur suivant :"Error 2 : <function argument>: illegal cast of 'struct' to pointer".

Pegase.90

[stephl]

Oui j'ai également pensé à ca mais la j'obtient le message d'erreur suivant :"Error 2 : <function argument>: illegal cast of 'struct' to pointer".

Le problème ne vient pas du code posté. Faites voir votre appel à fct1().

[souviron34]

C'est normal que ça te jette :

Dans fct1 pTata est un POINTEUR sur TATA
dans fct2 tu attends la structure....

Pourquoi faire compliqué quand on peut faire simple ???

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
void fct1(struct TATA *pTata){
char c ;
...
...
 c = fct2(pTATA);
..
 
}
 
char fct2(struct TATA *pTata){
  return (char)(pTata->x);
}

[pegase.90]

Le problème ne vient pas du code posté. Faites voir votre appel à fct1().

Voici comment j'appelle ma fonction fct1.

Code :

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

1
2
3
4
5
6
7
8
 
int main(void){
struct TATA pTata;
...
 fct1(&pTata);
..
return(1);
}

La stucture est défini dans un *.h que j'appele dans le fichier main.c.

[stephl]

L'appel est correct. Pourriez-vous donner la ligne indiquée par le compilateur lorsqu'il signale: "Error 2 : <function argument>: illegal cast of 'struct' to pointer"?

[pegase.90]

L'appel est correct. Pourriez-vous donner la ligne indiquée par le compilateur lorsqu'il signale: "Error 2 : <function argument>: illegal cast of 'struct' to pointer"?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void fct1(struct TATA *pTata){
...
...
 fct2(*pTATA);  //<--- Le compilateur indique l'erreur ici
..
..
 
}
 
char fct2(struct TATA pTata){
  return (pTata.x);
}

[stephl]

Pourtant, je ne vois aucune erreur. Pourriez-vous poster l'intégralité de votre code, ou du moins une version compilable (hormis cette erreur)?

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
6
7
8
9
10
 
void fct1(struct TATA *pTata){
...
 fct2(*pTATA);  //<--- Le compilateur indique l'erreur ici
 
}
 
char fct2(struct TATA pTata){
  return (pTata.x);
}

• Est-ce fct2() est bien déclarée avant fct1() ?
• Pourquoi dans fct2(), le paramètre s'appelle pTata alors que ce n'est pas un pointeur ?
• Pourquoi n'est-ce pas un pointeur ?
• Souviron34 a donné un solution. Pourquoi n'est-elle pas appliquée ?

[pegase.90]

Alors je ne suis pas ua boulot donc je vais essayer de répodnre de mémoire.

[LIST][*]Est-ce fct2() est bien déclarée avant fct1() ?

Pas sur, est-ce important?

[*]Pourquoi dans fct2(), le paramètre s'appelle pTata alors que ce n'est pas un pointeur ?

Je l'avais appelé autrement mais ca ne change rien au problème.

[*]Pourquoi n'est-ce pas un pointeur ?

Je n'ai pas précisé mais c'est du C embarqué et donc les pointeurs ne sont pas recommandés. J'essaye donc d'en mettre le minimum.

[*]Souviron34 a donné un solution. Pourquoi n'est-elle pas appliquée ?

Cette solution provoque également une erreur de compilation.

Merci
Pegase.90

[Emmanuel Delahaye]

Pas sur, est-ce important?
Ben oui....

Essaye ça

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
char fct2(struct TATA *pTata){
  return (char)(pTata->x);
}
 
void fct1(struct TATA *pTata){
char c ;
...
...
 c = fct2(pTATA);
..
 
}

[Desboys]

Bonjour,
permettez-moi de réagir à ceci :

Je n'ai pas précisé mais c'est du C embarqué et donc les pointeurs ne sont pas recommandés. J'essaye donc d'en mettre le minimum.

Je suis surpris de cette remarque. L'avantage de passer un pointeur plutôt qu'une structure complète me paraît pourtant intéressant. Lorsqu'on passe une structure en paramètre d'une fonction, on empile l'intégralité de la structure. Si votre structure fait 32 octets par exemple, comparé à un pointeur de, disons, 4 octets, pour de l'embarqué, je choisi le pointeur, car sur plusieurs appels de suite utilisant cette structure, l'utilisation de la pile est bien plus faible.
De plus, si ta fonction ne fait qu'un accès à un élément de ta structure, autant y accéder directement. Ca évite le surcoût ( overhead dans la littérature anglaise ) lié à l'appel de fonction.

Cordialement,
S.Desbois

[fveysseire]

Bonjour,
permettez-moi de réagir à ceci :

Je suis surpris de cette remarque. L'avantage de passer un pointeur plutôt qu'une structure complète me paraît pourtant intéressant. Lorsqu'on passe une structure en paramètre d'une fonction, on empile l'intégralité de la structure. Si votre structure fait 32 octets par exemple, comparé à un pointeur de, disons, 4 octets, pour de l'embarqué, je choisi le pointeur, car sur plusieurs appels de suite utilisant cette structure, l'utilisation de la pile est bien plus faible.
De plus, si ta fonction ne fait qu'un accès à un élément de ta structure, autant y accéder directement. Ca évite le surcoût ( overhead dans la littérature anglaise ) lié à l'appel de fonction.

Cordialement,
S.Desbois

Moi aussi je suis surpris, je programme en C pour de l'embarqué et la consigne est toujours plus de pointeurs.

[Nil_ct]

Bonjour,

Pour l'embarqué les pointeurs permettent effectivement d'alléger les appels de fonctions.
Par contre, il me semble que la création dynamique d'objet (avec des malloc et des free) est déconseillée car cela peut engendrer/favoriser une fragmentation de la mémoire (et qui dit création dynamique, dit pointeur, non ?).
(bien sûr, les pointeurs ne servent pas qu'à ça)
Je parle d'un nombre de créations/destructions important (et de tailles variables) relativement à la durée de vie de l'exécution (je ne suis pas anti-pointeur ).

Est-ce faux ?

Nil

[Médinoc]

Non, ce n'est pas faux. Les allocations dynamiques sont lentes et on tendance à fragmenter la mémoire. Aussi, il peut être intéressant de privilégier les buffers statiques quand la taille est connue.

[reptils]

pourquoi creer une fonction pour récupérer un membre de ta structure?