Discussion :

[ludo894]

Bonjour,

J'ai une structure avec des champs de differentes tailles. J'aimerais déclarer des 'defines' définissant l'offset de chacun des champs. Concrètement :

Code :

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

1
2
3
4
5
6
7
8
9
10
typedef struct
  {
    tModbusFrame   RequestBuf;
    bool           FrameReceived;
    tModbusStatus  ModbusStatus;
    byte           NbReceiveErrors;
} tModbusPort;
 
#define MODBUS_STATUS_OFFSET      sizeof( tModbusFrame ) + sizeof( bool )
#define NB_ERRORS_OFFSET          MODBUS_STATUS_OFFSET + sizeof( tModbusStatus)

Le but des sizeof est de rendre les offsets independant de la cible (exemple, bool = 8 bits sur µcontrolleur, 16 bits sur pc).

Quelqu'un aurait-il quelque chose de plus simple pour faire mes define (car ma vraie structure a une trentaine de champs...) ?

Merci à tous.

Ludo

[hegros]

Par curiosité ca te sert a quoi ? Parce que pour accéder un membre on fait MaGrosseStruct.champs1

[ludo894]

En fait, la variable de type cette structure est locale à un fichier toto. J'accède aux champs depuis un fichier titi via des fonctions Read ou Write, prenant comme paramètre l'offset d'ecriture/lecture. Est-ce clair ?

[Thierry Chappuis]

Voici comment je pratique:

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
#include <stdio.h>
#include <stdlib.h>
 
#define OFFSET_OF(type, membre) ((size_t) &((type *)0)->membre)
 
struct Essai {
	int x;
	int y;
	int z;
};
 
int main(void)
{
	size_t offset = OFFSET_OF(struct Essai, y);
 
	printf("L'offset de z est %u bytes!", offset);
 
	return EXIT_SUCCESS;
}

Ton implantation de MODBUS_STATUS_OFFSET par exemple n'est pas portable, car ne tient pas compte des éventuelles contraintes d'alignement. Ex:

Code :

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

1
2
3
4
5
struct Essai {
    int x;
    char c;
    int y;
};

Sur ma machine, l'offset de y n'est pas de 5 bytes, mais de 8 bytes à cause de l'alignegnement des variables de type int sur des adresses multiples de 4.

Thierry

[JeitEmgie]

vérifiez sur votre plate-forme si stddef.h définit offsetof(s,m) et/ou __offsetof(s,m)

notez qu'en pratiquant de la sorte, si vous rendez votre code source compatible multi-environnement , les fichiers générés eux ne le seront pas…

[Thierry Chappuis]

En fait, la variable de type cette structure est locale à un fichier toto. J'accède aux champs depuis un fichier titi via des fonctions Read ou Write, prenant comme paramètre l'offset d'ecriture/lecture. Est-ce clair ?

La sauvegarde d'une structure dans un fichier binaire est un problème délicat. As-tu un pouvoir de décision sur le format du fichier à utiliser pour stocker ta structure? Si tel est le cas, cet article est pour toi http://emmanuel-delahaye.developpez....m#enreg_struct

Personnellement, j'aurais tendance à enregistrer une structure dans un fichier texte au format CSV (comma separated value), voir XML..

Thierry

[gege2061]

Bonjour,

G_STRUCT_OFFSET

Code :

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

1
2
#define G_STRUCT_OFFSET(struct_type, member)	\
    ((glong) ((guint8*) &((struct_type*) 0)->member))

[ludo894]

Ta solution est interressante Thierry

Code :

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

#define OFFSET_OF(type, membre) ((size_t) &((type *)0)->membre)

Le problème est que le calcul de l'offset se fait dynamiquement : j'aurais préféré un 'calcul' statique...

Bien vu pour le probleme d'alignement : je viens justement d'avoir le bug !
Ma structure est forcemment déclarée dans un .h.

[Thierry Chappuis]

Ta solution est interressante Thierry

Code :

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

#define OFFSET_OF(type, membre) ((size_t) &((type *)0)->membre)

Le problème est que le calcul de l'offset se fait dynamiquement : j'aurais préféré un 'calcul' statique...

Bien vu pour le probleme d'alignement : je viens justement d'avoir le bug !
Ma structure est forcemment déclarée dans un .h.

C'est pas dynamique, l'offset est calculé à la compilation. La solution de la glib proposée par gege est probablement encore plus portable. Je persiste à dire que ce n'est pas une bonne idée de vouloir sauver ta structure dans un fichier comme tu le fais.

Thierry

[Jean-Marc.Bourguet]

Pourquoi chercher a definir une macro qui existe deja dans stddef.h et qui s'appelle alors offsetof?

Sinon, les IO binaires se basant sur l'interpretation d'une structure C, on sait ce que ca donne: des bugs quand on change de flags de compilation, de version de compilateur, d'architecture ou d'OS.

[ludo894]

Oups, j'y suis allé un peu vite : effectivement c'est un 'calcul statique'...
Pourquoi est-ce une si mauvaise idée de sauver ma structure dans un fichier ?

[Thierry Chappuis]

Pourquoi chercher a definir une macro qui existe deja dans stdlib.h et qui s'appelle alors offsetof?

Simplement parce que je l'ignorais! Merci pour le renseignement

EDIT: c'est standard? C99?

Thierry

[ludo894]

Pourquoi chercher a definir une macro qui existe deja dans stdlib.h et qui s'appelle alors offsetof?

Perso cette macro n'est pas définie dans mon stdlib.h !

[Thierry Chappuis]

Oups, j'y suis allé un peu vite : effectivement c'est un 'calcul statique'...
Pourquoi est-ce une si mauvaise idée de sauver ma structure dans un fichier ?

En vrac:
1) La représentation interne des données en C n'est pas fixée par la norme et peut varier d'une architecture à l'autre. La norme se contente de spécifier les valeurs extrêmes que doivent être capables de stocker chacuns des types.
2) Les contraintes d'alignement
3) L'endianesse (ça se dit ça). Sur certaines architectures, les données sont codées avec le byte de poids faible à une adresse inférieur à celle du byte de poids fort (little endian), et sur d'autres, c'est le contraire (big endian).

Thierry

[Jean-Marc.Bourguet]

Perso cette macro n'est pas définie dans mon stdlib.h !

Desole, c'est dans stddef.h. (je corrige le message initial).

[Jean-Marc.Bourguet]

EDIT: c'est standard? C99?

C90, mais dans stddef.h et pas stdlib.h.

[Thierry Chappuis]

Desole, c'est dans stddef.h. (je corrige le message initial).

Verifié! Et en c'était disponible depuis C90...

Il faut dire que je ne sers pas souvant de cette macro...

Thierry

[Emmanuel Delahaye]

J'ai une structure avec des champs de differentes tailles. J'aimerais déclarer des 'defines' définissant l'offset de chacun des champs. Concrètement :

C'est pas possible. Il faut utiliser une structure 'en dur', c'est à dire un tableau de unsigned char. C'est la seule façon d'être portable (indépendant de l'architecture). Les structures, c'est pour les données internes. Pas pour les interfaces physiques (flux).

On peut automatiser ça à coup de macros. Faut maitriser...

[Emmanuel Delahaye]

Ta solution est interressante Thierry

Code :

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

#define OFFSET_OF(type, membre) ((size_t) &((type *)0)->membre)

Le problème est que le calcul de l'offset se fait dynamiquement : j'aurais préféré un 'calcul' statique...

Bien vu pour le probleme d'alignement : je viens justement d'avoir le bug !
Ma structure est forcemment déclarée dans un .h.

C'est du statique. Et c'est en fait la macro offsetof() qui est dans <stddef.h>

Mais ça ne résout rien, ton code n'est pas portable.

Je t'ai déjà donné une solution (enfin, le début)...

[Thierry Chappuis]

C'est pas possible. Il faut utiliser une structure 'en dur', c'est à dire un tableau de unsigned char. C'est la seule façon d'être portable (indépendant de l'architecture). Les structures, c'est pour les données internes. Pas pour les interfaces physiques (flux).

On peut automatiser ça à coup de macros. Faut maitriser...

La solution binaire (TLV ou XDR: http://emmanuel-delahaye.developpez....m#enreg_struct ) est-elle de manière générale à préférer à un fichier texte (du type CSV ou autre)? Le problème de l'encodage (ASCII) est-il une limitation importante d'un point de vue de la portabilité?

Salutations

Thierry