使用结构体、数组与方法的C++程序无有效输出问题排查求助

我帮你梳理了代码里的几个核心问题，这些逻辑错误和细节疏漏导致了程序输出异常，下面是具体的问题分析和修改方案：

1. findOwner函数的致命逻辑错误 #

你在这个函数里犯了两个关键错误：

• 未初始化变量：int chosen;没有赋初始值，如果遍历完所有宠物都没找到"Uyasolu"，访问pet[chosen]会触发未定义行为；
• 赋值方向搞反：你写的i = chosen;是把未定义的chosen值赋给循环变量i，完全和你要“记录目标宠物索引”的逻辑相反，应该改成chosen = i;。

修改后要增加“未找到目标”的判断，避免空输出。

2. 头文件错误 #

你引入的<stream>是无效头文件，C++处理文件流需要包含<fstream>，同时全局的stringstream可以放到循环内部，避免全局状态带来的潜在问题。

3. 缺失文件打开的错误检查 #

如果petSold.txt不存在、路径错误或者没有读取权限，后续的所有读取操作都会失败，导致pet数组全是空值。必须在打开文件后添加检查逻辑，确保文件正常打开。

4. petPrice函数的输出对象错误 #

你现在的代码是把符合条件的内容输出到控制台（cout），但函数目标是写入到catSold.txt文件，所以要把cout改成outfile，同时可以增加判断宠物类型为"cat"的条件，更符合函数命名和文件名的意图。

修改后的完整代码 #

#include<iostream>
#include<fstream>
#include<string>
#include<sstream>
#include<cstdlib> // 用于exit函数处理错误
using namespace std;

struct PetLabel {
    string petname;
    string pettype;
    int petprice;
    string petowner;
    string phoneno;
};

PetLabel pet[10];

void DataPet();
void petPrice();
void findOwner();

int main(){
    DataPet();
    findOwner();
    petPrice();
    return 0;
}

void DataPet(){
    ifstream infile("petSold.txt");
    // 检查文件是否成功打开
    if (!infile.is_open()) {
        cerr << "错误：无法打开petSold.txt文件，请检查文件路径或权限" << endl;
        exit(EXIT_FAILURE);
    }

    for(int i = 0; i < 10; i++){
        // 将stringstream放在循环内部，避免全局状态干扰
        stringstream ss;
        string cnvt;

        getline(infile, pet[i].petname, ',');
        getline(infile, pet[i].pettype, ',');
        getline(infile, cnvt, ',');
        ss << cnvt;
        ss >> pet[i].petprice;
        getline(infile, pet[i].petowner, ',');
        getline(infile, pet[i].phoneno);

        // 如果文件提前读完，直接跳出循环，避免读取无效内容
        if (infile.eof()) break;
    }
    infile.close();
}

void findOwner(){
    int chosen = -1; // 初始化-1表示未找到目标
    for (int i = 0; i < 10; i++){
        if (pet[i].petname == "Uyasolu"){
            chosen = i;
            break; // 找到目标后直接跳出循环，提升效率
        }
    }

    if (chosen != -1) {
        cout << "Owner name : " << pet[chosen].petowner << " Phone no : " << pet[chosen].phoneno << endl;
    } else {
        cout << "未找到名为Uyasolu的宠物记录" << endl;
    }
}

void petPrice(){
    ofstream outfile("catSold.txt");
    if (!outfile.is_open()) {
        cerr << "错误：无法创建或写入catSold.txt文件" << endl;
        return;
    }

    outfile << "The cat sold with price greater than RM1000" << endl;
    for (int i = 0; i < 10; i++){
        // 增加宠物类型判断，符合函数和文件名的意图
        if (pet[i].pettype == "cat" && pet[i].petprice > 1000){
            outfile << pet[i].petname << "," << pet[i].pettype << "," << pet[i].petprice << "," << pet[i].petowner << "," << pet[i].phoneno << endl;
        }
    }
    outfile.close();
}

额外说明 #

• 我添加了文件读取到末尾时的跳出逻辑，避免数组越界或读取无效内容；
• 所有文件操作都增加了错误检查，方便你快速排查文件相关问题；
• 在petPrice里增加了宠物类型为"cat"的判断，让函数功能更贴合命名。

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