时间戳

首页 > 解决方案 > 如何解决 C6386 警告？

c++ - 如何解决 C6386 警告？

问题描述

我正在编写一个简单的代码来从 .txt 文件中读取系统化数据，并收到警告“C6386：写入‘点’时缓冲区溢出：可写大小为‘num*8’字节，但可能写入‘16’字节”。在我的情况下如何解决它？附上代码。

struct point {
    int x, y;
};

void main()
{
    fstream file;
    point* points;
    int num, 
        i = 0;

    file.open("C:\\Users\\Den\\Desktop\\file.txt", fstream::in);
    if (!file.is_open()) {
        cout << "No file found\n";
        exit(1);
    }
    else {
        file >> num;
        points = new point[num];
    }

    while (file >> num) {
        points[i].x = num;   // <- here
        file >> num;
        points[i].y = num;
        i++;
    }

    file.close();
}

标签： c++visual-studiowarnings

解决方案

这只是一个警告，但它提供了很好的建议。如果文件包含多个num项目怎么办？警告告诉你，应该确保你写的内容不会超过数组的末尾。具体来说：

此警告表明指定缓冲区的可写范围可能小于用于写入它的索引。这可能会导致缓冲区溢出。 [msdn]

此代码不会产生警告（VS2019）：

int x, y;
while (i < num && (file >> x >> y)) {
    points[i].x = x;
    points[i].y = y;
    i++;
}

还有更多错误检查要添加。如果file >> num;失败了怎么办？如果num是负数怎么办？如果points = new point[num];失败（返回nullptr）怎么办？

更新了完整的错误检查：

struct point {
    int x, y;
};

void main()
{
    ifstream file("C:\\Users\\Den\\Desktop\\file.txt");
    if (!file) {
        cerr << "No file found\n";
        exit(-1);
    }

    int num;
    if (!(file >> num) || num <= 0) {
        cerr << "invalid num\n";
        exit(-1);
    }
    point *points = new point[num];
    if (!points) {
        cerr << "new failed\n";
        exit(-1);
    }
    int num_items = 0;
    while (num_items < num && file >> points[num_items].x >> points[num_items].y) {
        num_items++;
    }
    // Do some work here
    delete [] points;
}

将来，考虑使用std::vectorover raw 数组。

推荐阅读