c++ - 从方法非确定性行为返回 std::string？

Question

我有一个读取文件并将其内容作为 std::string 返回的方法。我使用返回的 std::string 来编译 OpenGL 程序。链接失败(编译的顶点着色器已损坏。)在某些情况下，由于一个或两个着色器部分(读取文件内容)为 NULL。

• 如果我逐步调试我的代码，一切都很好。
• 如果我打印文件内容，链接失败的频率似乎会降低。

为什么它的行为不同，我的错误在哪里？

末尾的 cout 总是打印正确的文件内容:

std::string read_file(const char* filePath) {
    std::string content;
    std::ifstream stream(filePath, std::ios::in);
    if (stream.is_open()) {
        std::string line = "";
        while(getline(stream, line)) {
            content += "\n" + line;
        }
    }
    std::cout << "read_file " << content << "end read_file" << std::endl; // always prints the correct content

    return content; // makes a copy, don't like that, unless RVO?..
}

fragmentSource 和/或 vertextSource 有时是空的:

    GLuint fragmentShaderID = glCreateShader(GL_FRAGMENT_SHADER);
    GLuint vertexShaderID = glCreateShader(GL_VERTEX_SHADER);

    const char* fragmentSource = tools::read_file(_fragmentShaderPath).c_str();
    std::cout << "vertext shader source: " << std::endl << fragmentSource; // empty sometimes
    glShaderSource(fragmentShaderID, 1, &fragmentSource, NULL);
    glCompileShader(fragmentShaderID);
    CheckCompilation(fragmentShaderID);

    const char* vertexSource = tools::read_file(_vertexShaderPath).c_str();
    std::cout << "vertext shader source: " << std::endl << vertexSource; // empty sometimes
    glShaderSource(vertexShaderID, 1, &vertexSource, NULL);
    glCompileShader(vertexShaderID);
    CheckCompilation(vertexShaderID);

    GLuint programId = glCreateProgram();
    glAttachShader(programId, fragmentShaderID);
    glAttachShader(programId, vertexShaderID);
    glLinkProgram(programId);

    GLint result = GL_FALSE;
    GLint infoLogLength;
    glGetProgramiv(programId, GL_LINK_STATUS, &result);
    glGetProgramiv(programId, GL_INFO_LOG_LENGTH, &infoLogLength);
    if (infoLogLength > 0) { // this fails sometimes, corresponding to either vertexSource or fragmentSource being empty
        std::vector<char> infoLog(infoLogLength + 1);
        glGetProgramInfoLog(programId, infoLogLength, NULL, &infoLog[0]);
        std::cout << &infoLog[0] << std::endl;
    }

Answer 1

替换

const char* fragmentSource = tools::read_file(_fragmentShaderPath).c_str();

与

std::string fragmentSourceStr = tools::read_file(_fragmentShaderPath);
const char* fragmentSource = fragmentSourceStr.c_str();

对于 vertexSource 也是一样。

问题是您返回的字符串是临时的，因此在您使用指向其数据的指针初始化 fragmentSource 后它会被销毁，这使得 fragmentSource 指向一个已经破坏的存储。