Конструкция объекта C++ в многомерном динамическом массиве

Question

Я пытаюсь создать массив объектов nxn, но я не знаю, где назвать их конструкторы. Вот мой код:

class obj { 
    private: int x; 
    public: obj(int _x) { x = _x; } 
}; 

int main(int argc, const char* argv[]) { 

    int n = 3; //matrix size  

    obj** matrix = new obj*[n]; 
    for(int i = 0; i < n; i++) 
    matrix[i] = new obj[n]; 

    return 0; 
} 

Answer 1

Если требуется только вызов конструктора по умолчанию, ваш код уже вызывает его.

Для не-конструктор по умолчанию добавить вложенный цикл, как это:

for(int i = 0; i < n; i++) { 
    matrix[i] = new obj[n]; 
    for (int j = 0 ; j != n ; j++) { 
     matrix[i][j] = obj(arg1, arg2, arg3); // Non-default constructor 
    } 
} 

Лучший подход заключается в использовании std::vector из obj если не требуется полиморфное поведение, или смарт-указатели на obj если вам нужно полиморфное поведение.

Answer 2

Если вы действительно хотите отключить управление памятью низкого уровня, то вы не можете - форма массива new не позволяет передавать аргументы конструкторам объектов. Лучшее, что вы могли бы сделать, это позволить им создавать по умолчанию (добавив конструктор в ваш класс, если это необходимо), а затем выполнить цикл и переназначить их. Это утомительно и подвержено ошибкам.

Я предлагаю использовать классы более высокого уровня; для динамических массивов, использовать вектор:

std::vector<std::vector<obj>> matrix(n); 
for (auto & row : matrix) { 
    row.reserve(n); 
    for (size_t i = 0; i < n; ++n) { 
     row.emplace_back(/* constructor argument(s) here */); 
    } 
} 

В качестве бонуса, RAII означает, что вам не нужно, чтобы удалить все Распределения самостоятельно, и это не будет утечка памяти, если все это не удается.

Answer 3

dasblinkenlight уже дал неплохой ответ, однако есть второй способ сделать это, что более эффективно. Потому что, если вы делаете (код взят из его ответа)

matrix[i] = new obj[n]; 
for (int j = 0 ; j != n ; j++) { 
    matrix[i][j] = obj(arg1, arg2, arg3); // Non-default constructor 
} 

произойдет следующее:

1. памяти для n объектов выделяется.

2. Все объекты n по-умолчанию построены.

3. Для каждого объекта в стек создается пользовательский объект.

4. Этот объект в стеке копируется в выделенный объект.

5. Объект в стеке разрушен.

Очевидно, что гораздо больше, чем было бы необходимо (хотя и совершенно правильно).

Вы можете обойти эту проблему с помощью размещения нового синтаксиса:

matrix[i] = (obj*)new char[n*sizeof obj]; //Allocate only memory, do not construct! 
for (int j = 0 ; j != n ; j++) { 
    new (&matrix[i][j]) obj(arg1, arg2, arg3); // Non-default constructor called directly on the object in the matrix. 
}