C语言收银程序文件操作问题：仅最后商品库存修改生效+文件删改失败

问题1：仅最后选购商品库存扣减生效的原因 #

你的代码在循环中每次尝试基于当前库存文件修改单个商品，再替换原文件，但由于文件替换逻辑（remove+rename）未生效，导致每次循环实际都在读取初始库存文件进行修改。前几次修改的临时文件无法替换原文件，只有最后一次循环的修改可能成功覆盖（或误读取了最后一次生成的临时文件），最终仅最后一个商品的库存变化被保留。

问题2：删除、重命名逻辑未生效的原因 #

1. 平台兼容性限制：Windows系统下rename函数无法覆盖已存在的文件，如果remove操作失败（比如文件被占用、权限不足），原文件依然存在，rename会直接失败，临时文件无法替换原文件。
2. 未检查操作返回值：fopen、remove、rename都可能执行失败，但你未检查它们的返回值，无法及时发现错误并处理。
3. 文件操作潜在异常：如果fopen失败导致文件指针为NULL，后续fclose(NULL)会触发未定义行为，可能干扰后续文件操作。

代码中的其他逻辑缺陷 #

1. 循环内重复文件操作的效率与可靠性问题：每次循环都重新读取整个文件、生成临时文件，不仅效率低下，还容易因文件替换失败导致数据不一致。
2. fgets与fscanf的同步风险：依赖item_list.txt和item_stock.txt的行数严格一致，且fgets读取的行一定包含换行符。如果某行没有换行符（比如最后一行），buffer_char[strlen(buffer_char)-1] = '\0'会访问越界内存，破坏数据甚至导致程序崩溃。

修复建议 #

1. 优先采用内存缓存方案：一次性将所有商品信息读取到内存数组中，批量修改库存后再一次性写入文件，这是更高效、更可靠的方式，避免多次文件替换的问题。
2. 检查所有文件操作的返回值：添加错误处理逻辑，示例如下：

   flist = fopen("item_list.txt", "r");
   if (flist == NULL) {
       perror("Failed to open item_list.txt");
       // 处理错误，比如退出循环或程序
   }
   
   if (remove("item_list.txt") != 0) {
       perror("Failed to remove item_list.txt");
   }
   if (rename("temp_list.txt", "item_list.txt") != 0) {
       perror("Failed to rename temp_list.txt");
   }
   

3. 改进文件替换逻辑：使用更安全的备份方式，避免原文件无法删除导致的失败：

   // 先将原文件重命名为备份
   rename("item_list.txt", "item_list.bak");
   // 再将临时文件重命名为原文件
   rename("temp_list.txt", "item_list.txt");
   // 最后删除备份（可选，也可保留用于恢复）
   remove("item_list.bak");
   

4. 修复换行符处理逻辑：避免越界访问：

   size_t buf_len = strlen(buffer_char);
   if (buf_len > 0 && buffer_char[buf_len - 1] == '\n') {
       buffer_char[buf_len - 1] = '\0';
   }
   

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