Bonjour à tous,

Je ne suis pas très bonne en C et je gère encore très mal les structures. Mon projet est ici de récupérer des données via un capteur et de mettre ces données au format CSV. 

Voici ma structure et mon main

struct cvp { 
    float current;        
    float power;
    float voltage;
};

int main(int argc, char **argv)
{ 
    struct i2c_slave s; 
    struct cvp c;
   /* … */
   Measurements (s.fd, s.s_addr, &c);
    return 0;
}

La partie commentée concerne la communication i2C et n'est pas intéressante pour ma question.

Et mes fonctions : 

void Measurements (int fd, unsigned int s_addr, struct cvp *cvp)
{
    Read_I_P_V(fd, s_addr, cvp.&current, cvp.&power, cvp.&voltage);
   LTC2947_Export_CSV(cvp.current, cvp.power, cvp.voltage);
}
void Read_I_P_V(int fd, unsigned int s_addr, float *I, float *V, float *P){...}
void Export_CSV(float current, float power, float voltage){...};

J'ai pas mal d'erreur et je ne vois pas ce qui bloque, je fais des mauvais appels de structure je pense ... 

Pouvez vous m'aider s'il vous plait ?

-
Edité par Kakooo 10 août 2017 à 13:18:58

DiscorverMynewlife a écrit:

Hello, et quelles sont ces erreurs ?

error : conflicting types for 'Measurements'
Measurements(int fd, unsigned int s_addr, struct cvp *cvp)

error : expected identifier before '&' token 
Read_I_P_V(fd,s_addr,cvp.&current,cvp.&power,cvp&voltage);

error : request for member 'current' in something not a structure or union printf("%f,%f,%f\n", cvp.current, cvp.power, cvp.voltage);


error : request for member 'power' in something not a structure or union printf("%f,%f,%f\n", cvp.current, cvp.power, cvp.voltage);


error : request for member 'voltage' in something not a structure or union printf("%f,%f,%f\n", cvp.current, cvp.power, cvp.voltage);

As-tu déclaré la fonction Measurements avant de l'appeler ?

Measurements(int fd, unsigned int s_addr, struct cvp *cvp);

L'address-of operator se place à l'identificateur et non aux champs de celui-ci :

Read_I_P_V(fd, &s_addr, cvp.current, &cvp.power, &cvp.voltage);

Et concernant les trois dernières erreurs. Tu t'es trompé d'identifiant, ce n'est pas cvp, mais c.

printf("%f,%f,%f\n", c.current, c.power, c.voltage);

-
Edité par DiscorverMyfuckinglife 10 août 2017 à 15:50:10

Hello,

En complément, cvp est, dans Measurements(), un pointeur. Donc, lignes 3 et 4:

Read_I_P_V(fd, s_addr, cvp->current, cvp->power, cvp->voltage);
LTC2947_Export_CSV(cvp->current, cvp->power, cvp->voltage);

-
Edité par edgarjacobs 10 août 2017 à 15:47:05

C'est bizarre ces fonctions, on travail avec des structures et on envoie en paramètre les éléments un par un de cette structure... Autant leur balancer uniquement un pointeur vers cette structure. Enfin ça limite le nombre de paramètres ça me parait plus simple, mais je peux me tromper...

-
Edité par LChalam 10 août 2017 à 16:28:49

Merci à tous pour vos réponse !!! 

DiscorverMyfuckinglife a écrit:

As-tu déclaré la fonction Measurements avant de l'appeler ?

Measurements(int fd, unsigned int s_addr, struct cvp *cvp);

Oui j'ai bien déclaré la fonction, cependant l'erreur subsiste. J'ai également testé avec ta réponse aux dernières erreurs mais ça ne marche pas.

edgarjacobs :

void Measurements (int fd, unsigned int s_addr, struct cvp *cvp)
{
        Read_I_P_V(fd, s_addr, cvp->current, cvp->power, cvp->voltage);
        Export_CSV(cvp->current, cvp->power, cvp->voltage);

}

Voici mon code actuel et j'ai toujours les erreurs et warnings suivants (mon main n'a pas bougé) :

warning : 'struct cvp' declared inside parameter list void Measurements(int fd, unsigned int s_addr, struct cvp *cvp)

warning : passing argument 3 of 'Measurements' from incompatible pointer type Measurements(s.fd, s.s_addr, &c);

error: conflicting types for 'Measurements' void Measurements(int fd, unsigned int s_addr, struc cvp * cvp)

error : incompatible type for argument 3 of 'Read_I_P_V' Read_I_P_V(fd, s_addr, cvp->current, cvp->power, cvp-> voltage)

la dernière erreur pour argument 3, 4, 5 également.

-
Edité par Kakooo 14 août 2017 à 11:33:39

Quelqu'un s'il vous plait ?

Salut.

Ta structure est déclarée à un endroit, tu as un include juste au dessus ? Comment se termine le fichier inclus ?

Déjà, quand je lis ça :

void Export_CSV(float current, float power, float voltage){...};

Il n'y a pas besoin du ; (mais ça ne devrait pas être ça le soucis).

Il est possible qu'au contraire, tu n'ais pas fermé un truc dans le fichier inclus du dessus, et donc il considère que tu n'as pas fermé un truc avant de déclarer ta structure.

Fvirtman a écrit:

Salut.

Ta structure est déclarée à un endroit, tu as un include juste au dessus ? Comment se termine le fichier inclus ?

 Alors effectivement j'ai un fichier .h avec toutes mes déclarations de fonctions, et j'inclus ce fichier dans mon .c.

J'ai bien vérifié que chaque fonction et prototype de fonction correspondaient. Mon fichier h termine par ma dernière déclaration de fonction à savoir 

void Measurements (
	int fd, 
	unsigned int s_addr, 
	struct cvp *cvp
);

Je ne trouve rien de pas fermé également Je n'arrive pas à trouver l'origine de l'erreur.

j'ai une note en + de l'erreur qui me dit : 

test.h:64:6:note : previous declaration of 'Measurements' was here void Measurements (int fd, unsigned int s_addr, struct cvp *cvp);

sauf que les deux prototypes sont bien identiques...

-
Edité par Kakooo 14 août 2017 à 14:52:47

J'ai écris une grosse bêtise !

-
Edité par edgarjacobs 14 août 2017 à 15:36:16

edgarjacobs a écrit:

J'ai écris une grosse bêtise !

-
Edité par edgarjacobs 14 août 2017 à 15:36:16

Avec les -> ?

rouloude a écrit:

Est-ce que lors de la déclaration de Measurements, la structure struct cvp est connu ?

struct cvp {
    float current;       
    float power;
    float voltage;
};

La déclaration de Measurements se fait dans le .h.

En déplaçant l'implémentation de la structure dans le .h je n'ai effectivement plus l'erreur de "conflicting type" pour Measurement.  Parcontre j'ai toujours :

error incompatible type for argument 3, 4, 5 of Read_I_P_V

-
Edité par Kakooo 16 août 2017 à 11:03:30