obj model loading texture coords and vertex position not loaded correctly

Question

im loading a 3d .obj model to my game, however this is what it looks like now:3d model

which definitely is not what it suppose to look like. im thinking maybe is my way of connecting the indices are wrong, but I just can't find the problem

this is my code:

 char lineHeader[128];
FILE * file = fopen(fileName, "r");

if (file == NULL) {
    printf("Impossible to open the file !\n");
}

vector <vec3> positionVec;
vector <vec2> texCoordVec;
vector <vec3> normalVec;
vector <GLuint> vertexIndices;
vector <GLuint> uvIndices;
vector <GLuint> normalIndices;

while (true) {
    char lineHeader[128];
    int res = fscanf(file, "%s", lineHeader);

    if (res == EOF) {
        break;
    }

        if (strcmp(lineHeader, "v") == 0) {
            glm::vec3 vertex;
            fscanf(file, "%f %f %f\n", &vertex.x, &vertex.y, &vertex.z);
            positionVec.push_back(vertex);
        }
        else if (strcmp(lineHeader, "vt") == 0) {
            glm::vec2 uv;
            fscanf(file, "%f %f\n", &uv.x, &uv.y);
            texCoordVec.push_back(uv);
        }
        else if (strcmp(lineHeader, "vn") == 0) {
            glm::vec3 normal;
            fscanf(file, "%f %f %f\n", &normal.x, &normal.y, &normal.z);
            normalVec.push_back(normal);
        }
        else if (strcmp(lineHeader, "f") == 0) {
            int vertexIndex1, vertexIndex2, vertexIndex3, uvIndex1, uvIndex2, uvIndex3, normalIndex1, normalIndex2, normalIndex3;
            fscanf(file, "%d/%d/%d %d/%d/%d %d/%d/%d\n", &vertexIndex1, &uvIndex1, &normalIndex1, &vertexIndex2, &uvIndex2, &normalIndex2, &vertexIndex3, &uvIndex3, &normalIndex3);
            vertexIndices.push_back(vertexIndex1);
            vertexIndices.push_back(vertexIndex2);
            vertexIndices.push_back(vertexIndex3);
            uvIndices.push_back(uvIndex1);
            uvIndices.push_back(uvIndex2);
            uvIndices.push_back(uvIndex3);
            normalIndices.push_back(normalIndex1);
            normalIndices.push_back(normalIndex2);
            normalIndices.push_back(normalIndex3);
        }
    }

vector <vec3> vertexPositions;
vector <vec2> textureCoords;
vector <vec3> normals;

for (unsigned int i = 0; i < vertexIndices.size(); i++) {
    vertexIndices[i] = vertexIndices[i] - 1;
}

for (unsigned int i = 0; i < uvIndices.size(); i++) {
    unsigned int index = uvIndices[i];
    vec2 uv = texCoordVec[index - 1];
    textureCoords.push_back(uv);
} 

for (unsigned int i = 0; i < normalIndices.size(); i++) {
    unsigned int index = normalIndices[i];
    vec3 normal = normalVec[index - 1];
    normals.push_back(normal);
} 

    GLuint modelVertecesBuffer;
    glGenBuffers(1, &modelVertecesBuffer);
    glBindBuffer(GL_ARRAY_BUFFER, modelVertecesBuffer);
    glBufferData(GL_ARRAY_BUFFER, positionVec.size() * sizeof(glm::vec3), &positionVec[0], GL_STATIC_DRAW);
    glEnableVertexAttribArray(0);
    glVertexAttribPointer(0, 3, GL_FLOAT, GL_FALSE, 0, 0);

    GLuint modelTextureCoordBuffer;
    glGenBuffers(1, &modelTextureCoordBuffer);
    glBindBuffer(GL_ARRAY_BUFFER, modelTextureCoordBuffer);
    glBufferData(GL_ARRAY_BUFFER, texCoordVec.size() * sizeof(vec2), &texCoordVec[0], GL_STATIC_DRAW);
    glEnableVertexAttribArray(2);
    glVertexAttribPointer(2, 2, GL_FLOAT, GL_FALSE, 0, 0);

    GLuint modelNormalBuffer;
    glGenBuffers(1, &modelNormalBuffer);
    glBindBuffer(GL_ARRAY_BUFFER, modelNormalBuffer);
    glBufferData(GL_ARRAY_BUFFER, normals.size() * sizeof(vec3), &normals[0], GL_STATIC_DRAW);
    glEnableVertexAttribArray(1);
    glVertexAttribPointer(1, 3, GL_FLOAT, GL_FALSE, 0, 0);

    GLuint positionIndicesBuffer;
    glGenBuffers(1, &positionIndicesBuffer);
    glBindBuffer(GL_ELEMENT_ARRAY_BUFFER, positionIndicesBuffer);
    glBufferData(GL_ELEMENT_ARRAY_BUFFER, vertexIndices.size() * sizeof(int), &vertexIndices[0], GL_STATIC_DRAW);

Edit in a [mcve]. Feel free to use [this](http://stackoverflow.com/a/14887071/44729) as a base. — genpfault, Oct 25 '16 at 12:02

Nico Schertler · Accepted Answer · 2016-10-25T13:22:21.730

0

There are a few things:

Indices in the OBJ model are 1-based. You consider this when filling the vectors vertexPositions etc. But you don't use these vectors after that. Using the indices from the OBJ directly as an index buffer won't work. You have to subtract one. As genpfault mentioned in the comments, indices can even be negative for relative referencing.

Face vertices in the OBJ may reference different position and normal ids (and texture coordinate ids). This is something that you cannot represent in OpenGL and you would need to duplicate some data. So this approach will only work if the referenced ids are all the same.

edited Oct 25 '16 at 13:22

answered Oct 25 '16 at 13:04

Nico Schertler

32,049
4
39
70

1

OBJ indices can also be negative. – genpfault Oct 25 '16 at 13:19
@genpfault Thanks, added to the answer. – Nico Schertler Oct 25 '16 at 13:22
@NicoSchertler thank you for your help, I fixed the position indices so the model looks correct now, but its texture is still messed up, do you have any ideas about what's still going wrong? (i updated the code) – lucasyyy Oct 25 '16 at 14:02
Still, the OBJ might use different indices for positions and texture coordinates. You cannot do anything about that except duplicating some vertices. – Nico Schertler Oct 25 '16 at 15:11

obj model loading texture coords and vertex position not loaded correctly

1 Answers1