调用fopen触发SIGSEGV错误，分页联系人函数随机崩溃求助

看起来你的listarContactos函数里有几个明确的问题会导致SIGSEGV（段错误）或者随机崩溃，我来逐个拆解并给出修复方案：

1. 空指针调用fclose导致直接崩溃 #

当fopen失败返回NULL时，你直接调用了fclose(c)——对空指针执行fclose是未定义行为，这绝对会触发段错误。这应该是你调试时看到fopen后崩溃的直接原因之一（其实是后续的fclose导致的，不是fopen本身）。

2. 变长数组越界访问 #

你声明了int vecPag[qContacts/pageLimit+aux];，但如果没有任何有效联系人（qContacts=0），这个数组的长度会是0+0=0（因为0%3==0，所以aux=0）。这时候你尝试给vecPag[0]赋值，属于数组越界，会破坏栈内存，引发随机崩溃。

3. 分页切换时的越界判断错误 #

在切换下一页的逻辑中，你判断if(nPag < qPages)，但nPag是从0开始的索引，总页数qPages是实际的页数，比如总页数是3，索引范围是0-2。当nPag已经是2时，nPag < qPages（2<3）为真，执行nPag++后变成3，后续访问vecPag[nPag]就会越界。

4. 输入缓冲区溢出风险 #

char choice[4];配合sys::getline(choice,3)，如果用户输入超过3个字符，会超出缓冲区长度，破坏栈上的其他数据，导致随机崩溃或奇怪的行为。

修复后的完整代码 #

#include <iostream>
#include <cstdio>
#include <cstdlib>
#include <cstring>
#include <stdbool.h>

// 假设tContact和相关辅助函数的定义如下（请根据实际情况调整）
struct tContact {
    bool Active;
    // 你的联系人其他字段...
};

void pNULL(int code) {
    std::cerr << "Error opening file, code: " << code << std::endl;
}

void showContacto(const tContact& contact) {
    // 显示联系人信息的逻辑
    std::cout << "Active contact details here..." << std::endl;
}

namespace sys {
    void getline(char* buf, int size) {
        fgets(buf, size, stdin);
        // 自动去掉输入末尾的换行符
        buf[strcspn(buf, "\n")] = '\0';
    }
}

void listarContactos(){ 
    struct tContact contact; 
    FILE* c = fopen("Contact.dat","rb"); 
    if(c == NULL){ 
        pNULL(4); 
        // 空指针不需要close，直接退出
        exit(1); 
    } 

    int pageLimit = 3; 
    int qContacts = 0; 

    // 先统计有效联系人总数
    while(fread(&contact, sizeof(tContact), 1, c)){ 
        if(contact.Active){ 
            qContacts++; 
        } 
    } 

    // 处理无有效联系人的边界情况
    if(qContacts == 0) {
        std::cout << "No active contacts found." << std::endl;
        fclose(c);
        return;
    }

    int aux = (qContacts % pageLimit != 0) ? 1 : 0; 
    int qPages = qContacts / pageLimit + aux; 
    // 用动态数组替代变长数组，避免栈越界问题（也可以用std::vector更省心）
    int* vecPag = new int[qPages]; 
    if(vecPag == NULL) {
        std::cerr << "Memory allocation failed." << std::endl;
        fclose(c);
        exit(1);
    }

    fseek(c, 0, SEEK_SET); 
    vecPag[0] = ftell(c); 
    qContacts = 0; 
    int nPag = 0; 

    // 记录每页的起始文件位置
    while(fread(&contact, sizeof(tContact), 1, c)){ 
        if(contact.Active){ 
            qContacts++; 
        } 
        // 仅在未到最后一页时记录下一页起始位置
        if(qContacts % pageLimit == 0 && nPag < qPages - 1){ 
            nPag++; 
            vecPag[nPag] = ftell(c); 
        } 
    } 

    nPag = 0; 
    char choice[10]; // 扩大缓冲区，避免输入溢出
    bool exit = false; 

    while(!exit){ 
        system("cls"); 
        std::cout << "CONTACT LIST" << std::endl; 
        qContacts = 0; 
        fseek(c, vecPag[nPag], SEEK_SET); 

        // 显示当前页的联系人
        while(qContacts < pageLimit){ 
            if(fread(&contact, sizeof(tContact), 1, c)){ 
                if(contact.Active){ 
                    showContacto(contact); 
                    qContacts++; 
                } 
            } else {
                // 文件读取完毕，退出循环
                break;
            }
        } 

        std::cout << "Page number: " << nPag + 1 << "/" << qPages << std::endl; 
        std::cout << "4 = PREVIOUS PAGE" << std::endl; 
        std::cout << "6 = NEXT PAGE" << std::endl; 
        std::cout << "0 = EXIT" << std::endl; 
        std::cout << "Option: "; 
        sys::getline(choice, sizeof(choice)-1); 

        switch(choice[0]){ 
            case '4':{ 
                if(nPag > 0){ 
                    nPag--; 
                } 
                break;
            } 
            case '6':{ 
                // 修正判断条件，避免索引越界
                if(nPag < qPages - 1){ 
                    nPag++; 
                } 
                break;
            } 
            case '0':{ 
                exit = true; 
                break;
            } 
            default:{ 
                std::cout << "Wrong option!" << std::endl; 
                // 等待用户确认，避免提示一闪而过
                std::cout << "Press enter to continue...";
                getchar();
                break;
            } 
        } 
    } 

    delete[] vecPag; // 释放动态分配的内存
    fclose(c); 
    return; 
}

关键修复说明 #

• 移除空指针的fclose调用：当fopen失败时，直接退出即可，不需要关闭空指针。
• 处理无有效联系人的边界情况：避免后续数组越界和无效操作。
• 用动态数组替代变长数组：变长数组在C++中是非标准扩展，且容易出现栈越界问题，用new分配内存更安全（推荐用std::vector<int>，无需手动管理内存）。
• 修正分页切换的判断条件：确保nPag不会超过最大索引qPages-1，避免数组越界。
• 扩大输入缓冲区：避免用户输入过长导致缓冲区溢出，同时在默认选项中添加等待，让用户看到错误提示。
• 添加文件读取结束的判断：在显示联系人的循环中，当fread失败时退出循环，避免无限读取。

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