Discussion :

[Ikes72000]

Bonjour,

J'ai une variable de type struct définie comme suit dans le fichier d'entête de la classe où je souhaite l'utiliser :

Code :

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

1
2
3
4
5
struct device_t
{
    uint8_t id;
    uint8_t status;
};

Ma classe est la suivante:

Code :

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

1
2
3
4
5
6
7
8
9
class HomeDevice
{
private:
    device_t _device;
 
public:
    HomeDevice();
    HomeDevice(uint8_t deviceId);
};

Dans le constructeur par défaut, si j’essaie d'initialiser l'id du device dans la liste d'initialisation, j'ai une erreur.

Code :

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

1
2
3
4
HomeDevice::HomeDevice() : _device.id(0)
{
 
}

Est-ce la syntaxe qui n'est pas la bonne, ou n'est il tout simplement pas possible d'initialiser une variable de type struct de cette façon?

Merci.

[foetus]

Tu as plusieurs choix

Le constructeur dans la structure (ajout du C++)
En C++, la différence entre 1 structure et 1 classe, c'est l'encapsulation : public vs private.

Avec 1 valeur par défaut, pas besoin de l'appeler dans ta classe HomeDevice.

Code :

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

1
2
3
4
5
6
7
8
9
10
struct device_t {
public :
 
    device_t(uint8_t init_id=0) : id(init_id) {}
 
public:
 
    uint8_t id;
    uint8_t status;
};

Tu as aussi la syntaxe à la C

• HomeDevice() : _device{0} {}
• HomeDevice() : _device{0, 0} {}
• HomeDevice() : _device{id: 0} {} - pas sûr que ce soit standard

[Bousk]

HomeDevice() : _device{id: 0} {} - pas sûr que ce soit standard

Non standard mais C++20 ajoute cette syntaxe : _device{.id= 0} avec les designated initializers
https://en.cppreference.com/w/cpp/la...initialization

Sinon le plus clair je trouve c'est d'initializer les membres directement à leur définition, depuis C++11 je crois

Code :

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

1
2
3
4
5
struct device_t
{
    uint8_t id{0};
    uint8_t status{0};
};

[foetus]

Après recherche , effectivement les "designated initializers" existe depuis le C99 et la syntaxe _device{id: 0} est obsolète au profit de _device{.id = 0}.

Designated Initializers, gcc documentation en anglais
"In ISO C99 you can give the elements in any order, specifying the array indices or structure field names they apply to"

[Ikes72000]

Merci pour vos réponses et les liens pour les explications.
Effectivement, il y a quelques solutions, mais parmi celles-ci, y en a t elles qui sont plus dans les règles de l'art ou standards?

On va dire que je 'bricole' en C++, essentiellement du code pour Arduino et consorts, mais j'essaie de faire du code propre et de me documenter un maximum.

[foetus]

Effectivement, il y a quelques solutions, mais parmi celles-ci, y en a t elles qui sont plus dans les règles de l'art ou standards?

je dirais le constructeur de struct ... voire à la limite remplacer les structures par des classes.

Sinon utilise 1 structure si :

• tu n'as aucune méthode, qu' 1 constructeur (c'est 1 POD, "Pain Old Data")
• tu dois faire le lien avec du code C.