Инициализация массива заголовка C++ в конструкторе

Question

Я работаю с OpenGL, и я пытаюсь создать простой наземный класс, который рисует прямоугольник.

У меня есть класс с именем Vertex:

class Vertex 
{ 
    public: 
     Vertex(
      glm::vec3 _position, 
      glm::vec3 _color = glm::vec3(0.0, 0.0, 0.0), 
      glm::vec3 _normal = glm::vec3(0.0, 1.0, 0.0), 
      glm::vec3 _texture = glm::vec3(0.0, 0.0, 0.0)); 

     void setNormal(glm::vec3 _normal); 
     void setTexture(glm::vec3 _texture); 
     virtual ~Vertex(); 
    protected: 
    private: 
     glm::vec3 position; 
     glm::vec3 color; 
     glm::vec3 normal; 
     glm::vec3 texture; 
}; 

И это мой Ground класс:

class Ground 
{ 
    private: 
    double widht; 
    double length; 
    double y; 
    int* indexes; 
    Vertex* vertices; 
    public: 
    Ground(double _width, double_length, double y); 
} 

И вот что я хочу сделать в конструкторе Ground:

this->indexes = {0, 3, 1, 1, 3, 2}; 
this->vertices = { 
     Vertex(glm::vec3(0 - width/2, y, 0-length/2)), 
     Vertex(glm::vec3(0 - width/2, y, 0+length/2)), 
     Vertex(glm::vec3(0 + width/2, y, 0+length/2)), 
     Vertex(glm::vec3(0 + width/2, y, 0-length/2)) 
    } 

Я получаю эту ошибку при компиляции:

error: cannot convert ‘<brace-enclosed initializer list>’ to ‘Vertex*’ 

Теперь я знаю, что я мог бы жестко запрограммировать все это, но я хочу решение для общего определения заголовка класса ClassX * и инициализации конструктора.

Answer 1

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

для первого пути что-то вроде этого:

const int NUM_VERTS = 4; 

class Ground{ 
. 
. 
. 
Vertex vertices[NUM_VERTS] 
. 
. 
. 
}; 

Или, если вы хотите несколько различных наземных объектов с переменным числом вершин, которые можно использовать шаблоны, но это вызовет большое разрастание кода и размер исполняемого файл. Не очень хорошая идея, но она является возможно

template<size n> 
class Ground{ 
. 
. 
. 
Vertex vertices[n]; 
. 
. 
. 

} 

иначе, конечно

vertices = new Vertex[4]; 
vertices[0] = Vertex(glm::vec3(0 - width/2, y, 0-length/2)); 
//etc 
. 
. 
. 

Answer 2

Изменение indexes (индексы) и vertices быть std::vector<int> и std::vector<Vertex> соответственно. Тогда вы можете написать:

Ground::Ground(double _width, double _length, double y) 
    : indexes{0, 3, 1, 1, 3, 2} 
    , vertices{ 
     Vertex(glm::vec3(0 - width/2, y, 0-length/2)), 
     Vertex(glm::vec3(0 - width/2, y, 0+length/2)), 
     Vertex(glm::vec3(0 + width/2, y, 0+length/2)), 
     Vertex(glm::vec3(0 + width/2, y, 0-length/2)) 
    } 
{ 
}