【无标题】

bug源码：

#include <stdio.h>
#include <iostream>
#include <string.h>

using namespace std;

/*
    使用fgets和fputs来完成两个文件的拷贝
*/

void testcp(){

    //定义指向文件的指针
    FILE* srcfp=NULL;   //源文件 
    FILE* destfp=NULL;  //目标文件

    //1.打开文件
    if((srcfp=fopen("./file.txt","r"))==NULL){
        perror("fopen error");
        return ;
    }

    // if((srcfp=fopen("./aa.txt","w"))==NULL){  bug代码
    if((destfp=fopen("./aa.txt","w"))==NULL){
        perror("fopen error");
        return ;
    }

    //2.开始拷贝文件
    char buf[1024]="";  //定义缓冲数组接受数据
    while(true){
        //将数据从源文件中读取出来
        if(fgets(buf,sizeof(buf),srcfp)==NULL){
            break;  //如果读取完就结束循环
        }

        //将从源文件读取的数据写入到目标文件中
        fputs(buf,destfp);
    }

    //3.关闭所有打开的文件，防止文件数据泄露
    fclose(srcfp);
    fclose(destfp);
    cout<<"拷贝成功,你真厉害啊"<<endl;


}

int main(int argc , const char* argv[]){

    testcp();

    return 0;
}

bug分析：

一、错误分析

1. 核心错误：文件指针赋值错误（导致段错误的直接原因）

• 错误点：打开目标文件时，错误地将文件指针赋值给了 srcfp，而非 destfp。代码中：if((srcfp=fopen("./aa.txt","w"))==NULL)
• 错误原因：
  • 第一个 fopen 已经将源文件指针存入 srcfp，但第二个 fopen 错误覆盖了 srcfp，导致源文件指针丢失（后续读取源文件时，实际操作的是目标文件的指针）。
  • 目标文件指针 destfp 始终为 NULL（未被正确赋值），后续调用 fputs(buf, destfp) 时，对 NULL 指针进行操作，触发 段错误（Segmentation fault）。

2. 资源泄露：文件打开失败时未关闭已打开的文件

• 错误点：当第二个 fopen（打开目标文件）失败时，直接 return 未关闭已打开的源文件 srcfp。
• 错误原因：文件打开后会占用系统资源（文件描述符），若不关闭会导致资源泄露，尤其在循环或多次调用场景下可能引发程序异常。

3. 循环条件的兼容性问题（非致命但不规范）

• 错误点：使用 while(true) 作为循环条件。
• 错误原因：true 是 C++ 关键字，依赖 iostream 等头文件，在纯 C 环境中不兼容。虽然代码包含 using namespace std; 可以编译，但用 while(1) 更通用（C/C++ 均支持）。

二、修复后的代码及改动说明

cpp

运行

#include <stdio.h>
#include <iostream>
#include <string.h>

using namespace std;

/*
    使用fgets和fputs来完成两个文件的拷贝
*/

void testcp(){

    //定义指向文件的指针
    FILE* srcfp=NULL;   //源文件 
    FILE* destfp=NULL;  //目标文件

    //1.打开源文件（只读模式）
    if((srcfp=fopen("./file.txt","r"))==NULL){
        perror("打开源文件失败");  //修改提示信息，更清晰
        return ;
    }

    //2.打开目标文件（只写模式）
    //【改动1】将目标文件指针赋值给destfp，而非srcfp（修复核心错误）
    if((destfp=fopen("./aa.txt","w"))==NULL){
        perror("打开目标文件失败");
        //【改动2】若目标文件打开失败，先关闭已打开的源文件，避免资源泄露
        fclose(srcfp);
        srcfp = NULL;  //避免重复关闭野指针
        return ;
    }

    //3.开始拷贝文件
    char buf[1024]="";  //定义缓冲数组接受数据
    //【改动3】将while(true)改为while(1)，兼容C/C++环境
    while(1){
        //将数据从源文件中读取出来
        //此时srcfp是正确的源文件指针（未被覆盖）
        if(fgets(buf,sizeof(buf),srcfp)==NULL){
            break;  //读取完毕或出错时结束循环
        }

        //将数据写入目标文件（destfp已正确赋值，不再是NULL）
        fputs(buf,destfp);
    }

    //4.关闭所有打开的文件（先判断指针是否为NULL，避免对空指针操作）
    //【改动4】关闭文件前检查指针有效性，更健壮
    if(srcfp != NULL){
        fclose(srcfp);
        srcfp = NULL;
    }
    if(destfp != NULL){
        fclose(destfp);
        destfp = NULL;
    }

    cout<<"拷贝成功，你真厉害啊"<<endl;
}

int main(int argc , const char* argv[]){
    testcp();
    return 0;
}

三、关键改动总结

1. 修复文件指针赋值错误：将目标文件的 fopen 结果赋值给 destfp，确保 srcfp（源文件）和 destfp（目标文件）指针正确分离。
2. 避免资源泄露：当目标文件打开失败时，先关闭已打开的源文件，再退出函数。
3. 增强兼容性：用 while(1) 替代 while(true)，兼容更多编译环境。
4. 健壮性优化：关闭文件前检查指针是否为 NULL，避免对空指针调用 fclose（虽然此处场景不会触发，但养成良好习惯）。

修复后，程序可正常实现文件拷贝，且不会出现段错误和资源泄露问题。