web-dev-qa-db-fra.com

Bonne façon d'initialiser une structure dans un constructeur de classe

Je veux donc ajouter une structure à partir d'un fichier d'en-tête c en tant que membre de classe à une classe c ++. Mais j'obtiens une erreur de compilation pour le fichier cpp: bar was not declared inn this scope. Voici ce que j'ai:

//    myClass.hpp
#include fileWithStruct.h

class myClass
{
    public:
        struct foo bar;
};


//myClass.cpp

#include "myClass.hpp"

//Initialize structure in Constrcutor
myClass::myClass(  )
{
    bar = {1, 0, "someString", 0x4};
}
c++c++11struct
27
tzippy

Style C++ 03

#include "fileWithStruct.h"
/* say the contents were
struct foo
{
   int foo1;
   float foo2;
};
*/

class myClass
{
    public:
        int val;
        foo bar;
        // since foo is a POD-struct (a.k.a C struct), no constructor would be present
        // however bar() will zero-initialize everything in the struct
        myClass() : val(), bar()
        {
        }
};

Les les parenthèses après bar comptent . Reportez-vous valeur et initialisation zéro pour comprendre pourquoi cela fonctionne. Il est à noter qu'en ajoutant un constructeur à myClass, nous en avons fait un type non POD. Pour contourner ce problème, on peut conserver myClass comme agrégat et écrire:

class myClass
{
    public:
        int val;
        foo bar;
};

int main()
{
   myClass zeroed_obj = { };
   myClass inited_obj = { 2, {0, 1.0f} };
   myClass partially_inited_obj = { 2 };    // equivalent to {2, {}}; which would zero all of myClass::bar
   myClass garbage_obj;    // warning: when left uninitialized, every member without a constructor will end up with garbage value
}

Style C++ 11

class myClass
{
public:
   // default member initializations
   int val = { };         // zero-initialization
   foo bar = { 0, 0.0f }; // aggregate-initializing foo here, just giving { } will zero all of myClass::bar

   // should you want to receive an element from the constructor, this can be done too
   // aggregate initializing a struct in constructor initialization list is allowed from C++11 onwards
   // in C++03, we would've resorted to just setting the member of bar inside the constructor body
   myClass(int _foo1) : bar{_foo1, 0.f}, val{}
   {
   }

   // since we've a non-default constructor, we've to re-introduce the default constructor
   // if we need the above in-class initialization to work
   myClass() = default;
};

Ici, nous utilisons C++ 11 syntaxe d'initialisation uniforme . Cependant, en faisant cela, myClass devient un type non POD; l'initialisation des membres s'apparente à l'ajout d'un constructeur à la classe, rendant ainsi myClass une classe non triviale mais de mise en page standard. Selon C++ 11 pour qu'une classe soit POD, elle devrait être à la fois triviale et standard. Au lieu de cela

#include "fileWithStruct.h"
#include <type_traits>
#include <iostream>

class myClass
{
public:
   int val;
   foo bar;
};

int main()
{
    myClass obj { }; // initializes val, bar.foo1 and bar.foo2 to 0
    myClass m { 0, {1, 2.0f} }; // initilizes each member separately
    std::cout << std::is_pod<myClass>::value << std::endl; // will return 1
}

conservera myClass en tant que POD.

Reportez-vous à cet excellent article pour en savoir plus sur les agrégats et les POD.

32
legends2k

Ce que vous faites là-bas est une affectation, pas une initialisation. L'initialisation se produit dans la liste d'initialisation d'un constructeur, avant le corps du constructeur, ou en C++ 11 dans un initialiseur juste après la déclaration de la variable membre:

myClass.hpp, cas général:

/** you might want to do this if you are linking 
 * against the C lib or object file of that header:
 */
extern "C" { 
  #include fileWithStruct.h
}

class myClass
{
public:
  foo bar; //no need for "struct" in C++ here
};

C++ 11:

myClass.cpp

#include "myClass.hpp"

//Initialize structure in Constrcutor
myClass::myClass(  )
  : bar{1, 0, "someString", 0x4}
{}

Une autre option consiste à fournir la valeur initiale de foo avec un accolade-ou-égal-initialiseur à la déclaration de la variable membre:

myClass.hpp

extern "C" { 
  #include fileWithStruct.h
}

class myClass
{
public:
  foo bar{1, 0, "someString", 0x4};
};

Dans ce cas, vous n'avez pas besoin de définir un constructeur, car il est généré implicitement par le compilateur (si nécessaire), initialisant correctement bar.

C++ 03:

Ici, l'initialisation agrégée dans les listes d'init n'est pas disponible, vous devez donc utiliser des solutions de contournement, par exemple:

myClass.cpp

#include "myClass.hpp"

//Initialize structure in Constrcutor
myClass::myClass(  )
  : bar() //initialization with 0
{
  const static foo barInit = {1, 0, "someString", 0x4}; //assignment
  bar = barInit;
}

Ou:

#include "myClass.hpp"
namespace {
  foo const& initFoo() {
    const static foo f = {1, 0, "someString", 0x4};
    return f;
  }
}

//Initialize structure in Constrcutor
myClass::myClass(  )
  : bar(initFoo()) //initialization
{ }
9
Arne Mertz

L'initialisation doit être effectuée de cette façon (C++ 11):

myClass::myClass(  )
: bar{1, 0, "someString", 0x4}
{

}

N'oubliez pas non plus de déclarer votre constructeur dans la définition de votre classe.

7
Matthieu Rouget

Vous devez spécifier que la structure foo doit avoir "C-linkage". Voici un exemple complet.

// fileWithStruct.h
#ifdef __cplusplus
extern "C" { // Declare as extern "C" if used from C++
#endif

typedef struct _foo
{
  int a;
  int b;
  const char* c;
  int d;
} foo;


#ifdef __cplusplus
}
#endif

Le fichier d'en-tête myClass:

// myClass.hpp
#include "fileWithStruct.h"

class myClass
{
public:
  myClass();

  foo bar;
};

L'implémentation C + 11 de myClass qui utilise des listes d'initialisation étendues:

// myClass.cpp
#include "myClass.hpp"

myClass::myClass(  )
  : bar({1, 0, "someString", 0x4})
{
}

... et la version C++ 03 si vous n'avez pas encore migré vers C++ 11:

#include "myClass.hpp"

myClass::myClass(  )
  : bar()
{
  bar.a = 1;
  bar.b = 0;
  bar.c = "someString";
  bar.d = 0x4;
}
1
tarjeik