Question

我阅读了一个文本文件，内容类似于“sasdfsdf”，代码如下：

char* o = new char[size];
c = 0;
for (i = 0; i < n; i++)
{
    fseek(pFile, i, SEEK_SET);
    b = fgetc(pFile);
    if (b == '\r') {
        o[c] = b;
        c++;
        o[c] = '\n';
    }
    else {
        o[c] = b;
    }
    c++;
}
fclose(pFile);

SetWindowTextA(TextBox1.hWnd, (n > 0) ? o : NULL);

delete[] o;

首先我想知道这段代码是否干净。我认为这不是因为我是 C/Cpp 的新手，并且有时在理解分配内容方面存在一些问题。我想使用 C 样式（FILE*、fopen、fseek、fgetc）来获取文件的内容。问题是 char* o 总是添加一些东西。我有一个例子：它不是“sasdfsdf”（文本文件内容），而是将“sasdfsdf¨‰»3”写入编辑控件。我发现当 for-loop-scope 被留下时添加了“¨‰»3”。我认为这类似于内存泄漏。我不知道这些字符应该来自哪里。

Answer 1

正如评论所示，您是用 C 编写的，但已将您的问题标记为 C++。在这里，C++ 提供了一个更短的解决方案，不需要您进行任何动态分配（即使是您的 C 风格代码，也不需要，只需使用 constexpr size_t nelem = size; char o[nelem] = ""; 即可创建所需长度的简单数组）。

您上面缺少的部分是 fgetc() 每次从文件流中读取一个字符时都会使 flle-position 前进，因此不需要 fseek(pFile, i, SEEK_SET);完全是多余的。

在 C++ 中，您最好从 fstream 中读取而不是使用 C FILE*。使用 std::string 比使用 char* 好得多，因为 std::string 消除了写入超出数组末尾的可能性（您无法在循环中检查）。在使用 getline() 从文件中读取一行后，您可以简单地使用 std::replace() 将所有 '\r' 替换为 '\n'。完成后，所有内存都会自动释放。

您将编写类似于以下内容的例程：

#include <fstream>
#include <string>
#include <algorigthm>

void somefunc (std::fstream& stream)
{
    std::string line{}, windowtext{};
    
    while (getline (stream, line)) {
        std::replace (line.begin(), line.end(), '\r', '\n');
        windowtext += line;
    }
    
    SetWindowTextA (TextBox1.hWnd, windowtext.length() > 0 ? windowtext : nullptr);
    
    /* close file back in caller */
}

（注意：在 C++ 中，当文件流对象超出范围时，打开的文件流将被关闭，因此您无需手动关闭流）

另请注意，正如@RemyLebeau 在评论中指出的那样，在 Windows 上，getline() 将删除构成 DOS 行结尾的 CR 和 LF。如果您需要手动 '\n' 来创建换行符，那么您将需要 windowtext += line + '\n'; 而不是 std::replace 来注入 '\n'。

检查一下，如果您有问题，请告诉我。

Answer 2

您必须通过添加终止空字符 '\0' 来终止字符串。

                    char* o = new char[size];
                    c = 0;
                    for (i = 0; i < n; i++)
                    {
                        fseek(pFile, i, SEEK_SET);
                        b = fgetc(pFile);
                        if (b == '\r') {
                            o[c] = b;
                            c++;
                            o[c] = '\n';
                        }
                        else {
                            o[c] = b;
                        }
                        c++;
                    }
                    o[c] = '\0'; // add this to terminate the string
                    fclose(pFile);

                    SetWindowTextA(TextBox1.hWnd, (n > 0) ? o : NULL);

                    delete[] o;

内存泄漏与字符*？

2 个答案: