.txt к вектору <string>, ошибка типа некласса

Question

Привет, я только начал изучать C++ на этой неделе, и мне нужна помощь. В основном то, что я пытаюсь сделать, читается из .txt-файла и «конвертирует» его в векторную строку, а затем отображает его. моя ошибка в этой строке: text.readFile («script.txt»), он говорит: «запрос для члена« readFile »в« text », который относится к классу типа« Конверсия() » , что делает это значит? , а также мой метод getLines() не может быть разрешен.

main.cpp

#include <iostream> 
#include <string> 
#include <fstream> 
#include <sstream> 
#include <vector> 
#include "Conversion.h" 

using namespace std; 

int main() 
{ 
    vector<string> lines; 

    Conversion text(); 

    if(text.readFile("scenario.txt") == true) 
     lines = text.getLines(); 

    for(int i = 0; i < lines.size(); ++i) 
     cout << lines[i] << endl; 

    return 0; 
} 

Conversion.cpp

#include <string> 
#include <iostream> 
#include <fstream> 
#include <sstream> 
#include <vector> 
#include "Conversion.h" 

using namespace std; 
vector<string> lines; 

Conversion::Conversion(std::vector<std::string> lines) { 
    lines.clear(); 
} 
Conversion::Conversion() { 
} 

Conversion::~Conversion() { 
} 

bool Conversion::readFile(string filename) { 
    ifstream file; 
    string line; 

    file.open(filename.c_str()); 

    if(!file.is_open()) 
     return false; 

    while(getline(file, line)) 
     lines.push_back(line); 

    return true; 
} 

vector<string> Conversion::getLines(){ 
    return lines; 
} 

Conversion.h

#ifndef CONVERSION_H_ 
#define CONVERSION_H_ 

#include <string> 
#include <iostream> 
#include <fstream> 
#include <sstream> 
#include <vector> 

class Conversion { 
public: 
    Conversion(); 
    Conversion(std::vector <std::string>); 
    ~Conversion(); 
    std::vector<std::string> getLines(); 
    bool readFile(std::string); 

private: 
    std::vector<std::string> lines; 
    std::string line; 
    std::ifstream file; 
}; 

#endif 

Answer 1

Еще раз Conversion text(); это объявление функции, а не класс конкретизации. Для вызова конструктора по умолчанию, измените его на Conversion text;

Вы, вероятно, хотели, чтобы скопировать пропущенные строки в конструкторе:

Conversion::Conversion(std::vector<std::string> const& lines) : lines(lines) { } 

Ваш код должен работать сейчас, но можно сделать некоторые улучшения. Для того, чтобы избежать копирования, getLines должен вернуться к референтной к-ПОСТОЯННЫЕ:

std::vector<std::string> const& getLines(); 

// you don't have to create lines in main, you can print like this: 

for(auto const& x : text.getLines()) 
    cout << x << endl; 

, и я хотел бы использовать его даже здесь:

bool readFile(std::string const&); 

Я надеюсь, что это последняя вещь - std::ifstream конструктор и open функция также принимают std::string:

file.open(filename); 

Answer 2

Вы должны удалить скобки при создании экземпляра Conversion объекта:

Conversion text; 

Смотрите этот вопрос для детальных ответов: Is no parentheses on a constructor with no arguments a language standard?