使用重载 >> 从文件创建对象正在创建空白对象（C++）

Question

所以我有一个 >> 重载设置，可以从文件中提取标签和文本（图像）块：

friend std::istream& operator>>(std::istream &is, Image data) {
    std::string line;
    std::string full_image;

    getline(is, line);
    data.SetLabel(line);

    for (size_t i = 0; i < data.GetImageSize(); ++i) { // Stores each full image as a single string
        getline(is, line);
        full_image += '\n' + line;
    }
    data.SetImage(full_image);
    
    return is;
}

我已经能够验证这是否正常工作，因为如果我在结尾处为data.GetLabel() 和data.GetImage() 粘贴一个打印语句，则会输出正确的字符串。

在另一个类中，我有这个函数，它使用重载的 >> 创建一个 Image 对象列表（每个标签/图像集一个）：

void NaiveBayesTrainer::ReadTrainingData(std::string filepath) {
    Image data(image_size_);
    std::ifstream input_file(filepath);

    if (input_file.is_open()) {
        while (input_file >> data) { // Using overloaded extraction operator
            training_data_.push_back(data);
        }
    } else {
        throw std::invalid_argument("File not found");
    }
}

但无论出于何种原因，从文件（数据）中提取的图像对象都是空的。如在training_data_.push_back(data) 之前调用data.GetLabel() 或data.GetImage() 返回一个空白字符串。

运算符重载是否正确设置对象的属性，但这些更新没有传递到我用来创建列表的对象中？

Answer 1

参数Image data 是副本传递的内容，对其进行修改不会影响调用者中传递的内容。

您应该在参数中的Image 之后添加&（使其变为Image& data）以使其传递引用并在NaiveBayesTrainer::ReadTrainingData 函数中修改data。