c - Return 语句被忽略

Question

对于 future 的读者

不要编码疲劳。它会让你变得有点笨。

我的问题是我有两个独立的输出，它们似乎相互冲突。其中一个超出了 return 语句，因此令人震惊的是代码正在执行。但是，我没有想到使用 Visual Studio 的搜索功能来检查所有其他文件，以查看是否有其他原因导致我获得输出。据我当时所知，只有一次对 LoadGLSLFromFile 函数的调用，而实际上有两次。

结果让我感到极度沮丧，因为我确信没有对该函数的其他调用，而且一定会对我的代码进行优化，这要么会打乱函数的顺序函数，或者 return 语句没有发挥应有的作用(无论出于何种原因)。

如果您要发布问题，我强烈建议您重新评估被视为“相关”的代码。毕竟，您来到这里是因为您看不到其他人可以看到的东西。

如果你读过这篇文章的评论，你会发现 StoryTeller 试图让我明白，我假设发生的事情确实是不可能的，而且我的调试方法是不正确的。讲故事的人，如果你正在读这篇文章，我真诚地道歉。

=================================

编辑2:请注意，我遇到的问题是，当我告诉它时，该函数没有返回。它在返回之后继续执行，在同一函数内，直​​到遇到第二个。

所以，我真的不知道为什么会发生这种情况，但这是我的代码。因此，通常我所做的是检查严重故障，如果存在严重问题，我会返回一个已知的“错误”值。但是，返回在多个地方被忽略，并且代码以错误的顺序执行。 (顺序错误见第二张图)

编辑:

我的问题是代码的执行超出了 return 语句。

#include "Loader.h"
#include <stdio.h>
#include <stdlib.h>
#include <direct.h>
#include <string.h>
#include <GL/glew.h>
#include <GL/GL.h>
#include <GL/GLU.h>
#include <GLFW/glfw3.h>

static const GLchar * VSource[] = {
    "#version 450 core\n"
    "layout (location = 0) in vec4 offset;\n"
    "layout (location = 1) in vec4 color;\n"
    "out VS_OUT {\n"
    "   vec4 color;"
    "} vs_out;\n"
    "void main(void)\n"
    "{\n"
    "   const vec4 vertices[3] = vec4[3](vec4(0.25, -0.25, 0.5, 1.0),\n"
    "                                    vec4(-0.25, -0.25, 0.5, 1.0),\n"
    "                                    vec4(0.25, 0.25, 0.5, 1.0));\n"
    "   gl_Position = vertices[gl_VertexID] + offset;\n"
    "   vs_out.color = color;"
    "}\n"
};

static const GLchar* FSource[] = {
    "#version 450 core\n"
    "in VS_OUT {\n"
    "   vec4 color;\n"
    "} fs_in;\n"
    "out vec4 color;\n"
    "void main(void)\n"
    "{\n"
    "   color = fs_in.color;\n"
    "}\n"
};

static const GLchar* TControlSource[] = {
    "#version 450 core\n"
    "layout (vertices = 3) out;\n"
    "void main(void) {\n"
    "   if(gl_InvocationID == 0) {\n"
    "       gl_TessLevelInner[0] = 5.0;\n"
    "       gl_TessLevelOuter[0] = 5.0;\n"
    "       gl_TessLevelOuter[1] = 5.0;\n"
    "       gl_TessLevelOuter[2] = 5.0;\n"
    "   }\n"
    "   gl_out[gl_InvocationID].gl_Position = gl_in[gl_InvocationID].gl_Position;\n"
    "}"

};

GLuint LoadAllShaders() {

    LoadGLSLFromFile("glsl", GL_VERTEX_SHADER);


    GLuint VShader = glCreateShader(GL_VERTEX_SHADER);
    glShaderSource(VShader, 1, VSource, NULL);
    glCompileShader(VShader);
    LogCompileStatus(VShader, "VShader");

    GLuint FShader = glCreateShader(GL_FRAGMENT_SHADER);
    glShaderSource(FShader, 1, FSource, NULL);
    glCompileShader(FShader);
    LogCompileStatus(FShader, "FShader");

    GLuint TShader = glCreateShader(GL_TESS_CONTROL_SHADER);
    glShaderSource(TShader, 1, TControlSource, NULL);
    glCompileShader(TShader);
    LogCompileStatus(TShader, "Tessellation Shader");

    GLuint Program = glCreateProgram();

    glAttachShader(Program, VShader);
    glAttachShader(Program, FShader);
    glAttachShader(Program, TShader);
    glLinkProgram(Program);
    glDeleteShader(VShader);
    glDeleteShader(FShader);
    glDeleteShader(TShader);
    return Program;
}

void LogCompileStatus(GLuint Shader, char* ShaderName) {
    // Checking compile status of VShader
    if (ShaderName == NULL || sizeof(ShaderName) == 0)
        ShaderName = ("Unnamed Shader with ID: %i" + (char)&Shader);
    GLuint ShaderSuccess = GL_FALSE;
    glGetShaderiv(Shader, GL_COMPILE_STATUS, &ShaderSuccess);

    if (ShaderSuccess == GL_TRUE)
        printf("%s successfully compiled\n", ShaderName);
    else {
        GLint LogLength;
        glGetShaderiv(Shader, GL_INFO_LOG_LENGTH, &LogLength);

        char* buffer = (char*)malloc(LogLength);
        glGetShaderInfoLog(Shader, LogLength, NULL, buffer);

        printf("%s failed to compile.\n%s\n", ShaderName, buffer);
        free(buffer);
    }
}


GLuint LoadGLSLFromFile(char* location, GLenum ShaderType) {
    if (sizeof(location) < 6 || strstr(location, ".glsl") == NULL) {
        fprintf(stderr, "Attempted to load invalid file.\n");
        return 0;
    }

    switch (ShaderType) {
    case(GL_COMPUTE_SHADER):
    case(GL_VERTEX_SHADER):
    case(GL_TESS_CONTROL_SHADER):
    case(GL_TESS_EVALUATION_SHADER):
    case(GL_GEOMETRY_SHADER):
    case(GL_FRAGMENT_SHADER):
        break;
    default:
        fprintf(stderr, "Invalid Shadertype\n");
        break;
    }

    FILE* shaderFile = fopen(location, "r");
    if (shaderFile == NULL) {
        fprintf(stderr, "Wurbulgurb\n");
        return 0;
    }
    size_t bufferSize = fseek(shaderFile, 0, SEEK_END);
    printf("File Buffer size: %i\n", (int)bufferSize);


    fclose(shaderFile);
    /*
        glShaderSource(shd, 1, VSource, NULL);
        glCompileShader(shd);
        LogCompileStatus(shd, "VShader");
    */
    

    return 1;
}

主文件:

#include <stdio.h>
#include <stdlib.h>
#include <GL\glew.h>
#include <GL\GL.h>
#include <GL\GLU.h>
#include <GL\wglew.h>
#include <GLFW\glfw3.h>
#include "Loader.h"

#define CL_BUFFER (GLfloat[4]) { 0.3f, 0.3f, 0.3f, 1.0f }

int main(void);
void err(int error, const char * msg);
void keypress(GLFWwindow *window, int key, int scancode, int action, int mods);

static GLfloat attrib[] = { 0.0f, 0.0f, 0.5f, 1.0f };

int main(void) {
    if (!glfwInit())
        return -1;


    GLFWwindow *window = glfwCreateWindow(1024, 768, "Fididdler", NULL, NULL);
    glfwWindowHint(GLFW_VERSION_MAJOR, 4);
    glfwWindowHint(GLFW_VERSION_MINOR, 5);
    glfwSetInputMode(window, GLFW_CURSOR, GLFW_CURSOR_DISABLED);
    if (window == NULL)
        return -1;

    LoadGLSLFromFile("./Condoms.glsl", GL_VERTEX_SHADER);

    glfwSetErrorCallback(err);
    glfwSetKeyCallback(window, keypress);

    glfwMakeContextCurrent(window);
    if (glewInit() != GLEW_OK)
        return -1;


    printf("GL Version String: %s \n", glGetString(GL_VERSION));
    GLuint RProg = LoadAllShaders();
    GLuint VAO;
    glCreateVertexArrays(1, &VAO);
    glBindVertexArray(VAO);
    printf("Loading complete");

    


    while (!glfwWindowShouldClose(window)) {
        glClearBufferfv(GL_COLOR, 0, CL_BUFFER);
        glUseProgram(RProg);
        glVertexAttrib4fv(0, attrib);
        glDrawArrays(GL_TRIANGLES, 0, 3);
        
        /* End drawing logic*/

        glfwSwapBuffers(window);
        glfwPollEvents();
    }


    glDeleteVertexArrays(1, &VAO);
    glDeleteProgram(RProg);
    glfwTerminate();
    return 1;
}

void err(int error, const char * msg) {
    printf("Error: %s", msg);
}

void keypress(GLFWwindow *window, int key, int scancode, int action, int mods) {
    if (key == GLFW_KEY_ESCAPE && action == GLFW_PRESS) {
        glfwSetWindowShouldClose(window, GLFW_TRUE);
    }

    switch (key) {
        case(GLFW_KEY_W): 
            attrib[1] += 0.01f;
            printf("W pressed");
            break;
        case(GLFW_KEY_S):
            attrib[1] -= 0.01f;
            printf("S pressed");
            break;
    }

    switch (key) {
    case(GLFW_KEY_A):
        attrib[0] -= 0.01f;
        printf("A pressed");
        break;
    case(GLFW_KEY_D):
        attrib[0] += 0.01f;
        printf("D pressed");
        break;
    }
}

这是 LoadGLSLFromFile("TessControl.glsl", GL_VERTEX_SHADER); 的控制台输出。注意:第 3 行不应该发生，因为它之前有一个 return 语句。

这是 LoadGLSLFromFile("glsl", GL_VERTEX_SHADER); 的控制台输出注意:第 3 行不应该发生，因为它之前有一个 return 语句。

有一个 return 语句专门阻止进一步执行，但它仍然执行。我对这种行为感到非常困惑，因为据我所知，返回意味着“立即返回给您打电话的线路”。

任何和所有的帮助将不胜感激。

Answer 1

你说返回值被忽略了。这正是您的代码正在做的事情:

GLuint LoadAllShaders() {

    // return value is ignored
    LoadGLSLFromFile("glsl", GL_VERTEX_SHADER);


    GLuint VShader = glCreateShader(GL_VERTEX_SHADER);

因为您不检查返回值，所以没有什么可以阻止您的代码继续运行。

也许您将 return 与 exit() 混淆了。 return 语句从函数返回一个值，以便调用函数可以决定如何继续。 exit 函数导致程序终止，传入的值是程序的返回值。

如果您希望程序退出，您应该使用exit。如果没有，那么您需要检查上面代码中的返回值并采取相应的操作。

编辑:

这就是为什么 MCVE是如此重要。

发布主代码后，问题是您调用了 LoadGLSLFromFile 两次。您首先直接从 main 调用它。然后 main 调用 LoadAllShaders，后者再次调用 LoadGLSLFromFile。

因此，“Wurbulgurb”会在第一次调用时打印，而“尝试加载无效文件”或“文件缓冲区大小”会在第二次调用时打印。