传递给着色器的颜色不起作用

Question

我对OpenGL很陌生，并且有一个我目前正在使用的简单程序。

我遇到的问题是当我传递一个数组来表示点的颜色时，颜色最终变成了黑色。 如果我仅在片段着色器中明确定义颜色，并且在VAO中切换数据的索引，三角形将移动到作为颜色的点，则效果很好。

我尝试使用API​​跟踪并获得以下信息：

6872: message: major api error 1282: GL_INVALID_OPERATION error generated. <program> is not a program object, or <shader> is not a shader object.
6872 @0 glAttachShader(program = 1, shader = 1)
6872: warning: glGetError(glAttachShader) = GL_INVALID_OPERATION
6876: message: major api error 1282: GL_INVALID_OPERATION error generated. Handle does not refer to the expected type of object (GL_SHADER_OBJECT_ARB).
6876 @0 glDeleteShader(shader = 1)
6876: warning: glGetError(glDeleteShader) = GL_INVALID_OPERATION
6878: message: major api error 1282: GL_INVALID_OPERATION error generated. Handle does not refer to the expected type of object (GL_SHADER_OBJECT_ARB).
6878 @0 glDeleteShader(shader = 1)
6878: warning: glGetError(glDeleteShader) = GL_INVALID_OPERATION
Rendered 507 frames in 8.52598 secs, average of 59.4653 fps

我所有的代码都在这里 ，但正在进行中。 以下图形段可能会引起某种类型的问题：

    // TODO: Use this code once per mesh
    // Make a VBO to hold points
    GLuint vbo = 0;
    glGenBuffers(1, &vbo);
    glBindBuffer(GL_ARRAY_BUFFER, vbo);
    glBufferData(GL_ARRAY_BUFFER, points.size() * sizeof(GLfloat), &points[0], drawType);

    // VBO TO hold colors
    GLuint colorVbo = 0;
    glGenBuffers(1, &colorVbo);
    glBindBuffer(GL_ARRAY_BUFFER, colorVbo);

    float colors[] = {
            0.5, 0.0, 0.0,
            0.0, 0.5, 0.0,
            0.0, 0.0, 0.5
    };

    glBufferData(GL_ARRAY_BUFFER, 9 * sizeof(float), colors, GL_STATIC_DRAW);

    // Make a VAO for the points VBO
    GLuint vao = 0;
    glGenVertexArrays(1, &vao);
    glBindVertexArray(vao);

    // Points
    glBindBuffer(GL_ARRAY_BUFFER, vbo);
    glVertexAttribPointer(0, 3, GL_FLOAT, GL_FALSE, 0, NULL);

    // Colors
    glBindBuffer(GL_ARRAY_BUFFER, colorVbo);
    glVertexAttribPointer(1, 3, GL_FLOAT, GL_FALSE, 0, NULL);

    GLint colorpos = glGetAttribLocation(Program::getCurrentProgram(), "vertColor");

    glEnableVertexAttribArray(0);   // Points
    glEnableVertexAttribArray(1);   // Colors

    glLinkProgram(Program::getCurrentProgram());

    // Draw Mesh
    glDrawArrays(drawShape, 0, points.size()/2);
}

有些你可能需要知道的事情是， points是一个Vector<float> ， Program::getCurrentProgram()是一个静态函数，返回当前正在使用的程序， drawShape在这种情况下是GL_TRIANGLES 。

顶点着色器：

#version 400 core

layout(location=0) in vec3 vert;
layout(location=1) in vec3 vertColor;

out vec3 color;

void main()
{
    color = vertColor;
    gl_Position = vec4(vert, 1);
}

片段着色器：

#version 400 core

in vec3 color;

out vec4 finalColor;

void main()
{
    finalColor = vec4(color, 1.0);
}

如果这是一个需要仔细考虑的问题，我深表歉意。 在过去的一天左右的时间里，我一直在环顾四周，并尝试了几种不同的方法，但所有方法均无效。 如果需要其他任何信息，以使某人不必对我的所有代码进行排序，请告诉我。

我从apitrace得到的信息似乎表明我可能对着色器ID做错了，尽管如果我将颜色编码到片段着色器中，然后该颜色可以工作，则该错误仍然会发生。

我正在使用GLFW创建我的OpenGL上下文。 我设置了一个错误回调，但从中没有得到任何结果，给我的印象是它也应该通过OpenGL错误，尽管我在他们的FAQ上没有看到任何明确的说法。

我还在编译着色器和链接程序时检查错误，那里也没有任何反应。

另外，我想知道一旦对象超出范围并调用delete时，使用C ++是否会冒失去OpenGL状态的风险。

编辑：这是我没有提到的几件事：

着色器初始化：

// Create OpenGL Shader
shaderID = glCreateShader(shaderType);

glShaderSource(shaderID, 1, &cShaderString, NULL);

// Compile
glCompileShader(shaderID);

着色器析构glDeleteShader(shaderID);仅调用glDeleteShader(shaderID);

这是很标准的。 这在shader类的构造函数中， shaderID是Shader的成员变量。 shaderType是一个GLenum ，在这种情况下为GL_VERTEX_SHADER或GL_FRAGMENT_SHADER 。

程序初始化：//创建程序programID = glCreateProgram（）;

// Iterate through Shaders
for(std::vector<Shader>::iterator shader = shaders.begin(); shader != shaders.end(); shader++)
{
    // Attach Shader
    glAttachShader(programID, shader->getShaderID());
}

// Link program
glLinkProgram(programID);

程序析构函数：

// Iterate through attached shaders
for(std::vector<Shader>::iterator shader = shaders.begin(); shader != shaders.end(); shader++)
{
    // Detach Shader
    glDetachShader(programID, shader->getShaderID());
}

// Delete program
glDeleteProgram(programID);

再次，这似乎很标准，并且在Program类的构造函数中。

编辑2：在弄乱了一些代码之后，我发现了一件相当奇怪的事情。 如果我移动用于创建和绑定缓冲区的代码，并将三角形绘制到我用作类来表示三角形的构造函数上，则将位置作为颜色而不是位置。 当我在数据上切换索引时（位置为1，颜色为0），这也是发生的事情。

编辑3：实际上，经过多看之后，似乎绘制的三角形与颜色完全不同。 为Triangle类创建vao成员并在平局上绑定它似乎可以解决此问题。

编辑4：看来glValidateProgram()确实产生了成功的结果，这使我怀疑其着色器存在问题。

Answer 1

另外，我想知道一旦对象超出范围并调用delete时，使用C ++是否会冒失去OpenGL状态的风险。

就在这里。 我相信这就是这里正在发生的事情。 这并不意味着您不能在C ++类中包装OpenGL对象，但是必须非常小心。

您遇到的问题是，您在C ++对象的析构函数中删除了包装的OpenGL对象。 因此，只要这些对象之一超出范围或由于任何其他原因而被销毁，它就会携带相应的OpenGL对象。

如果复制对象，这将导致特别难看的问题。 让我们看一个人工的例子：

Shader a(...);
{
    Shader b = a;
}

在此之后，你在构造函数中创建OpenGL的对象a已经被删除。 当创建b作为副本a ，默认的拷贝构造函数将复制存储在着色器ID a到b 。 然后，当b超出范围时，其析构函数将删除着色器ID。 它仍然是存储在a ，但现在无效。

当然，您不会完全像上面那样编写代码。 但是，如果按值将对象传递给函数/方法，则可能会遇到非常类似的情况，这部分发生在代码中。

如果将这些对象存储在像std::vector这样的容器中，将会发生更危险的事情，因为难以识别。 当添加新元素会使vector超过其当前容量时， vector重新分配其内存。 在重新分配期间，它将创建vector中所有现有元素的副本（调用复制构造函数），然后销毁原始元素。 再次与上面的示例非常相似，在该示例中，您最终得到引用已删除的OpenGL对象的对象。

这正是您的情况。 你推Shader对象为vector ，而且已经在对象的OpenGL的IDS vector作为一部分被删除vector ，当你在多个元素推被重新分配。

核心问题是无法安全地复制这些C ++对象。 默认的复制行为（成员分配）不起作用。 通常，对于默认实现不足的类，您需要在C ++中实现复制构造函数和赋值运算符。 但是，如果您将OpenGL对象的ID作为成员，则实际上是没有好的方法，除非您实施更复杂的方案，其中可能涉及共享子对象，引用计数等。

我总是建议的一件事是为无法正确复制的类提供私有副本构造函数和赋值运算符。 如果您不小心复制了对象而不是神秘的运行时行为，这将确保您得到编译时错误。

然后，要将对象存储在容器中，最简单的方法是将指向对象的指针存储在容器中，而不是对象本身。 您可以在执行此操作时使用智能指针（例如smart_ptr ）来简化内存管理。