Default constructor

[Igors]

Добрый день

Справочник открывал, но на сей раз там так много и мутно... лучше спрошу у знающих людей

Есть масса простых структур, напр

Код

C++ (Qt)
struct CDrawInst {
// data
  CMaterial * m_material;
  int m_count;
...
};
 

Единственное что мне нужно в конструкторе - прописать все такие POD члены нулями, и все. CDrawInst() = default это делает? Или это уже и так делается с новыми стандартами? Или по старинке расписывать? Это конечно нетрудно, но надо же как-то у культуре приобщаться

Спасибо

[kambala]

Чтобы default сработал, надо присвоить значения в объявлении

[Igors]

Чтобы default сработал, надо присвоить значения в объявлении

Я это всю жизнь и деаю

Код

C++ (Qt)
struct CDrawInst {
  CDrawInst( void ) : m_material(0), m_count(0) 
  {
  }
 
// data
  CMaterial * m_material;
  int m_count;
...
};

Вопрос был можно ли как-то занулять "автоматом", как делают template для указателей, int и др

[ViTech]

CDrawInst() = default это делает? Или это уже и так делается с новыми стандартами?

Нет.

Вопрос был можно ли как-то занулять "автоматом", как делают template для указателей, int и др

Совсем автоматом нельзя, надо скобочки руками писать (Member initialization.):

Код

C++ (Qt)
struct CDrawInst {
// data
  CMaterial * m_material{};
  int m_count{};
...
};

[Igors]

Совсем автоматом нельзя, надо скобочки руками писать (Member initialization.):

Понял, спасибо. Заметим что по приведенной ссылке конкретно приведенного Вами примера нет (как-то не очень хорошо документируют)

[sergek]

На мой взгляд, лучше делать это явно:

Код

C++ (Qt)
struct CDrawInst {
// data
  CMaterial * m_material = nullptr;
  int m_count = 0;
...
};
 

[ViTech]

Заметим что по приведенной ссылке конкретно приведенного Вами примера нет (как-то не очень хорошо документируют)

По приведённой ссылке примеры, что поля класса в принципе можно инициализировать (начиная с С++11). А способов инициализации в С++ до фига, и в них чёрт ногу сломит . Например, агрегат  можно целиком "занулить" так (без явной инициализации полей класса):

Код

C++ (Qt)
struct CDrawInst {
// data
  CMaterial * m_material;
  int m_count;
...
};
 
void test()
{
  CDrawInst inst{};
  assert(inst.m_material == nullptr);
  assert(inst.m_count == 0);
}

Но если CDrawInst перестанет быть агрегатом:

Код

C++ (Qt)
struct CDrawInst {
  CDrawInst() {};
// data
  CMaterial * m_material;
  int m_count;
...
};

то "зануление" inst в такой форме записи (CDrawInst inst{};) может и не происходить.

[ViTech]

На мой взгляд, лучше делать это явно:

Код

C++ (Qt)
struct CDrawInst {
// data
  CMaterial * m_material = nullptr;
  int m_count = 0;
...
};
 

Можно, конечно, перестраховаться, но Value initialization:

This is the initialization performed when a variable is constructed with an empty initializer.
...
The effects of value initialization are:
1) if T is a class type with no default constructor or with a user-provided or deleted default constructor, the object is default-initialized;
2) if T is a class type with a default constructor that is neither user-provided nor deleted (that is, it may be a class with an implicitly-defined or defaulted default constructor), the object is zero-initialized and then it is default-initialized if it has a non-trivial default constructor;
3) if T is an array type, each element of the array is value-initialized;
4) otherwise, the object is zero-initialized.

Так что int m_count{} должно в 0 инициализироваться.

[Azazello]

Добрый день

Справочник открывал, но на сей раз там так много и мутно... лучше спрошу у знающих людей

Есть масса простых структур, напр

Код

C++ (Qt)
struct CDrawInst {
// data
  CMaterial * m_material;
  int m_count;
...
};
 

Единственное что мне нужно в конструкторе - прописать все такие POD члены нулями, и все. CDrawInst() = default это делает? Или это уже и так делается с новыми стандартами? Или по старинке расписывать? Это конечно нетрудно, но надо же как-то у культуре приобщаться

Спасибо

Могу ещё посоветовать обнулить в конструкторе в стиле С: memset.
В большинстве случаев оно того не стОит, но это же работа с графикой, а мемсет очень хорошо оптимизирован под SSE.

[Racheengel]

На мой взгляд, лучше делать это явно:

Код

C++ (Qt)
struct CDrawInst {
// data
  CMaterial * m_material = nullptr;
  int m_count = 0;
...
};
 

+1. Я тоже так делаю. Скобки малочитаемы.

[Авварон]

На мой взгляд, лучше делать это явно:

Код

C++ (Qt)
struct CDrawInst {
// data
  CMaterial * m_material = nullptr;
  int m_count = 0;
...
};
 

Вот только такая инициализация не всегда работает - например QMutex m_mutex = QMutex::Recursive вроде не компилируется, а с {} компилируется.

+1. Я тоже так делаю. Скобки малочитаемы.