C++二进制文件修改函数额外写入全0条目问题求助

看起来你在二进制文件的结构体修改操作上踩了个常见的坑——多余写入空条目。结合你描述的现象（加modOk == false前多写多个，加了后还剩一个），我大概能猜到你的代码逻辑问题出在哪，给你捋清楚并修复：

问题根源分析 #

你应该是在循环读取文件中的proyecto结构体时，循环结束后额外执行了一次fwrite(&datos, ...)操作。因为datos是局部变量，默认初始化全为0，所以就会在文件末尾多写一个空条目。之前没加modOk判断时，每次循环都会触发这次写入，所以会有多个空条目；加了判断后只在没找到目标ID时写一次，但实际上不管找没找到，都不该在循环外做这次写入。

修复后的完整代码逻辑 #

我把你的函数补全并修正了关键逻辑，你可以参考：

#include <iostream>
#include <cstdio>
#include <cstdlib>
using namespace std;

// 假设你的proyecto结构体是这样的（根据实际情况调整）
struct proyecto {
    int id;
    char name[50];
    // 其他字段...
};

void option12(){
    proyecto datos;
    int id;
    bool modOk = false;
    system("cls");
    cout << "Type project ID: ";
    cin >> id;
    
    // 用rb+模式打开：读写二进制文件，不截断原有内容
    FILE *p = fopen("proyectos.dat", "rb+"); 
    if (!p) {
        cout << "Error opening file!" << endl;
        return;
    }

    // 循环读取每个结构体，直到文件末尾
    while (fread(&datos, sizeof(proyecto), 1, p) == 1) {
        if (datos.id == id) { // 匹配到目标ID
            // 这里添加你需要修改的字段逻辑，比如修改项目名称
            cout << "Enter new project name: ";
            cin >> datos.name;
            
            // 关键！把文件指针移回当前条目的起始位置
            // 因为fread已经把指针推进了一个结构体的大小
            fseek(p, -sizeof(proyecto), SEEK_CUR);
            
            // 写入修改后的结构体
            fwrite(&datos, sizeof(proyecto), 1, p);
            modOk = true;
            cout << "Project updated successfully!" << endl;
            break; // 找到目标后直接退出循环，不用继续读取
        }
    }

    if (!modOk) {
        cout << "No project found with ID: " << id << endl;
    }

    fclose(p); // 务必关闭文件，避免资源泄漏
}

几个关键注意事项 #

• 文件打开模式：一定要用rb+，而不是wb或者ab——wb会清空文件，ab只能追加写入，都不适合修改已有条目。
• 指针定位：修改条目时必须用fseek回退指针，不然fwrite会覆盖下一个条目或者写到文件末尾。
• 循环终止逻辑：找到目标后直接break退出循环，没必要继续读取后面的内容，提升效率。
• 避免多余写入：只在找到匹配条目时执行写入操作，循环结束后不要额外写空结构体。

内容的提问来源于stack exchange，提问作者Paulo