Где объекты, созданные при передаче параметров, должны быть удалены?

Question

У меня есть что-то вроде этого:

class GLSLShader { 
    public: 
     GLSLShader(GLenum); 
     ~GLSLShader(); 
}; 

class GLSLProgram { 
    public: 
     GLSLProgram(GLSLShader *, GLSLShader *); 
     ~GLSLProgram(); 

    private: 
     GLSLShader *m_VertexShader; 
     GLSLShader *m_FragmentShader; 
}; 

GLSLProgram конструктор собирается назвать так:

GLSLProgram program(new GLSLShader(GL_VERTEX_SHADER), new GLSLShader(GL_FRAGMENT_SHADER)); 

Мой вопрос, где я должен удалить выделенные объекты Shader. Должен ли я удалить его в деструкторе GLSLProgram или мне нужно управлять им по-другому с помощью кода ниже?

GLSLShader *vertex = new GLSLShader(GL_VERTEX_SHADER); 
GLSLShader *fragment = new GLSLShader(GL_FRAGMENT_SHADER); 
GLSLProgram program(vertex, fragment); 
delete vertex; 
delete fragment; 

Answer 1

У вас есть две основные проблемы.

Первой проблемой является неправильное использование объектов RAII. Точка объектов RAII предназначена для управления временем жизни объекта с использованием автоматических переменных, а не с использованием new и delete явно. Используя new на GLSLShader, не обертывая указатель на контейнер RAII, вы эффективно смягчаете работу, которую делают ваши деструкторы.

Ваш конструктор GLSLProgram не должен принимать объекты указателем. Он должен взять их на const&. Это позволяет абоненту решить, как долго эти объекты будут длиться.

Вторая проблема логична. Нет причин для того, чтобы GLSLProgram отвечал за уничтожение объектов шейдера, которые ему даны. Это должно быть до вызывающего кода, поскольку объекты шейдера GLSL могут быть повторно использованы.

Таким образом, GLSLProgram не должен пытаться уничтожить эти объекты. Но после успешного создания программы конструктор должен отсоединить шейдеры GLSL от программы GLSL с помощью glDetachShader.

В целом, ваш код должен выглядеть следующим образом:

GLSLShader vertex(GL_VERTEX_SHADER); 
GLSLShader fragment(GL_FRAGMENT_SHADER); 
//Fill `vertex` and `fragment` with actual shader code. 
GLSLProgram program(vertex, fragment); 
//Destructors for `vertex` and `fragment` will take care of themselves. 

Кроме того, вы должны знать о некоторых других вещах:

1. Программа может быть создана с помощью более 2 объектов шейдерных , Поэтому у вас должны быть альтернативные конструкторы, которые могут принимать много таких объектов.

2. Separable programs может быть создан непосредственно из строк, без использования шейдерных объектов.

Answer 2

Должен ли я удалить его деструктор GLSLProgram или я должен управлять им по-разному

Это зависит от многого.

Нужно ли кому-либо кроме GLSLProgram получить доступ к этим динамическим объектам? Могут ли такие объекты переживать экземпляр GLSLProgram, который указывает на них? Если да для обоих, то память не может управляться только program.

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

С другой стороны, если GLSLProgram может управлять памятью исключительно, то, безусловно, также следует выделить их. Кроме того, по-видимому, нет причин использовать динамическое распределение вообще. Вместо этого я рекомендую использовать объекты-члены, если только не существует конкретной причины.

Учитывая попытки в этом вопросе, я предлагаю следующее:

class GLSLProgram { 
    public: 
     GLSLProgram(): 
      m_VertexShader(GL_VERTEX_SHADER), 
      m_FragmentShader(GL_FRAGMENT_SHADER) {} 

    private: 
     GLSLShader m_VertexShader; 
     GLSLShader m_FragmentShader; 
};