删除函数定义中的重复代码

Question

这是我的第一个问题，所以我可能会错过“正确的结构”。

无论如何，我有一个头文件，有一个函数。此函数 (void readFile()) 在 cpp 文件中定义。在这个定义中，我有很多重复的代码。

如果它在 main 中，我只需声明一个新函数，在其中定义可重复的，然后每次调用该函数。但由于它位于非主 cpp 文件中，因此我在此过程中遇到问题。

基本上，我的函数所做的是逐个字符地读取文件，并根据文本将数据保存到不同的对象中。

我的代码如下所示：

source.open("bookings.txt", std::ios::in);
char c;
source.get(c); 
            while (c != '|'){
                CurrentID.push_back(c);
                source.get(c);
            }
object.setID(CurrentID)

此代码重复自身，仅替换“object.setID”行。我尝试声明函数“search(std::ifstream x, char y, std::string z);”有定义

void Search(std::ifstream x, char y, std::string z){
    x.get(y); // next after |
    while (y != '|'){
        z.push_back(y);
        x.get(y);
    }
} 

但是，如果我尝试在“void readFile()”定义中调用此函数，如下所示：

// First block as a repeatable

source.get(c); 
            while (c != '|'){
                CurrentID.push_back(c);
                source.get(c);
            }
object->setID(CurrentID)
CurrentID.clear();

// second block as a function, with repeatable code commented out

void Search(std::ifstream quelle, char c, std::string &CurrentID);

            /* source.get(c); 
            while (c != '|'){
                CurrentID.push_back(c);
                source.get(c);
            }*/
            object->setPrice(stof (CurrentID));
            CurrentID.clear();

它从第一个块中的“CurrentID.clear()”直接跳转到第二个块中的“object->setPrice”，忽略了 void Search 函数的存在。 有什么建议如何使函数工作，或者以其他方式删除重复的代码？

Answer 1

我不知道这是否能准确回答你的问题。如果没有，请发布您的整个代码，尤其是 readFile 函数。

假设您想要一个 readFile 函数：

• 解析输入流，然后
• 填写object结构列表的字段ID（字符串）和价格（浮点数），
• 流中的值由| 字符分隔，并且，
• 对重复代码使用第二个函数readToken（即，从输入流中读取，直到找到分隔符并返回一个字符串）。

下面的代码就是这样做的。注意：

• 您将readFile 和readToken 定义为单独的函数，并且
• 两者都改变​​了输入流的状态。

[Demo]

#include <iostream>
#include <sstream>
#include <string>  // stof
#include <vector>

struct object {
    std::string id;
    float price;
};

std::string readToken(std::istringstream& iss) {
    std::string ret{};
    char c{};
    while (true) {
        iss >> c;
        if (not iss.eof() and c != '|') {
            ret.push_back(c);
        } else {
            return ret;
        }
    }
}

std::vector<object> readFile(std::istringstream& iss) {
    std::vector<object> ret{};
    while (not iss.eof()) {
        auto id = readToken(iss);
        auto price = std::stof(readToken(iss));
        ret.emplace_back(id, price);
    }
    return ret;
}

int main() {
    std::istringstream input{"ID1|25.5|ID2|3.14"};
    auto objects = readFile(input);
    for (const auto& o : objects) {
        std::cout << o.id << ", " << o.price << "
";
    }
}


// Outputs:
//
//   ID1, 25.5
//   ID2, 3.14

---

更简单的方法是让std::getline 读取令牌：

[Demo]

std::vector<object> readFile(std::istringstream& iss) {
    std::vector<object> ret{};
    while (not iss.eof()) {
        std::string id{};
        std::getline(iss, id, '|');

        std::string price_str{};
        std::getline(iss, price_str, '|');
        auto price = std::stof(price_str);

        ret.emplace_back(id, price);
    }
    return ret;
}