Чтение данных из файла в векторе

Question

Моя задача - прочитать эти данные из файла в вектор:

21000 Landhau Nolte brown
19000 Modern_fit Hoeffner magnolie
14700 Pure_Style Wellmann black

Это моя попытка, но откат не работает. Я уже посмотрел некоторые примеры здесь, на переполнение стека, но почему-то это не работает.

functions.h

#pragma once
#include <iostream>
#include <string>
#include <vector>
#include <fstream>
using namespace std;

struct Kitchen {
    double price;
    string name;
    string manufacturer;
    string color;
};

main.cpp:

#include "functions.h"

int main(){

    vector<Kitchen> Kitchens;

    fstream myFile;
    myFile.open("kitchen.txt", ios::in);
    if (myFile.is_open()) {
        while (!myFile.eof()) {
            double price;
            string name;
            string manufacturer;
            string color;
            myFile >> price >> name >> manufacturer >> color;
            Kitchens.push_back(price, name, manufacturer, color);

        }

        myFile.close();
    }
    else cout << "not opened." << endl;

    system("PAUSE");
    return EXIT_SUCCESS;
}

Что я делаю не так?

Answer 1

Давайте сделаем это правильно.

Начиная с заголовка, не включайте больше, чем необходимо для определений в этом заголовке, и не импортируйте все идентификаторы из std в глобальное пространство имен.

#include <string>

struct Kitchen {
    double price;
    std::string name;
    std::string manufacturer;
    std::string color;

    Kitchen(double price, std::string name,
            std::string manufacturer, std::string color)
        : price{price}, name{name}, manufacturer{manufacturer}, color{color}
    {}
};

Я добавил простой конструктор, так как он понадобится нам для emplace_back позже.

Теперь реализуем main().Для воспроизводимого примера лучше читать из потока строк, чем возиться с файлами:

#include <vector>
#include <sstream>
#include <iostream>

int main()
{
    std::vector<Kitchen> kitchens;

    std::istringstream file("21000 Landhau Nolte brown\n"
                            "19000 Modern_fit Hoeffner magnolie\n"
                            "14700 Pure_Style Wellmann black\n");

    {
            double price;
            std::string name;
            std::string manufacturer;
            std::string color;
            while (file >> price >> name >> manufacturer >> color) {
                kitchens.emplace_back(price, name, manufacturer, color);
            }
    }

    std::clog << "Read " << kitchens.size() << " kitchens from input\n";
}

Обратите внимание, что !eof() не гарантирует, что чтение будет успешным.Вместо этого мы предпринимаем попытку чтения, а затем проверяем, находится ли входной поток в состоянии сбоя.После цикла мы могли бы (если бы захотели) фактически проверить, что мы достигли конца файла, а не какого-либо условия сбоя - я пропустил это для этой простой программы.

Answer 2

структура является агрегатным типом, но для помещения объекта структуры в вектор структуры вам необходимо создать объект, даже если он может быть временным:

#include <iostream>
#include <vector>
using namespace std;
struct Kitchen {
    double price;
    string name;
    string manufacturer;
    string color;
};
int main() {
  std::vector<Kitchen> kt;
  kt.push_back(Kitchen{21000,"Landhau","Nolte","brown"});

 return 0;
}

Также с незначительной модификацией и наличием параметризованного конструктора в вашей структуре Kitchen вы можете избежать внутренней операции копирования / перемещения push_back и напрямую использовать emplace_back.

#include <iostream>
#include <fstream>
#include <vector>
using namespace std;
struct Kitchen {
    double price;
    string name;
    string manufacturer;
    string color;
    Kitchen(double p,
            const string& n,
            const string &m,
            const string &c):price(p),name(n),manufacturer(m),color(c) {}
};
int main(){

    vector<Kitchen> Kitchens;

    fstream myFile;
    myFile.open("kitchen.txt", ios::in);
    if (myFile.is_open()) {
        while (!myFile.eof()) {
            double price;
            string name;
            string manufacturer;
            string color;
            myFile >> price >> name >> manufacturer >> color;
            Kitchens.emplace_back(price, name, manufacturer, color);

        }

        myFile.close();
    }
    else cout << "not opened." << endl;

    system("PAUSE");
    return EXIT_SUCCESS;
}

Answer 3

push_back занимает Kitchen. Код предоставляет кусочки Kitchen. Дайте emplace_back попробовать или создать временный Kitchen для перехода в push_back.

Answer 4

То, что вы делаете неправильно, пытается передать 4 случайные переменные в push_back, который на самом деле принимает только одну, а эта имеет тип значения вектора.

Kitchen k; 
myFile >> k.price >> k.name >> k.manufacturer >> k.color;
Kitchens.push_back(k);