1. ホーム
  2. c++

[解決済み] クラス内部でfstreamのgetline()関数を使用する

2022-03-01 16:16:35

質問

辞書の単語を含むテキストファイルの行を、配列オブジェクトに読み込もうとしています。配列には、"a" で始まるすべての単語を格納し、別の配列には "b" ... アルファベットのすべての文字を格納したいのですが、どうすればよいですか?

これが、私が書いたarrayオブジェクトのクラスです。

    #include <iostream>
    #include <string>
    #include <fstream>

    using namespace std;

    class ArrayObj
    {
    private:

        string *list;
        int size; 

    public:


        ~ArrayObj(){ delete list;}

        void loadArray(string fileName, string letter)
        {
            ifstream myFile;
            string str = "";
            myFile.open(fileName);

            size = 0;

            while(!myFile.eof())
            {
                myFile.getline(str, 100);

                if (str.at(0) == letter.at(0))
                    size++;
            }
            size -= 1; 

            list = new string[size];

            int i = 0;
            while(!myFile.eof())
            {
                myFile.getline(str, 100);

                if(str.at(0) == letter.at(0))
                {
                    list[i] = str;
                    i++;
                }
            }

            myFile.close();
        }


    };

というエラーが出るんだけど。

2   IntelliSense: no instance of overloaded function     "std::basic_ifstream<_Elem, _Traits>::getline [with _Elem=char, _Traits=std::char_traits<char>]" matches the argument list d:\champlain\spring 2012\algorithms and data structures\weeks 8-10\map2\arrayobj.h  39

getline関数をオーバーロードする必要があるのでしょうが、どうすればいいのか、なぜそれが必要なのか、よくわかりません。

何かアドバイスがあればお願いします。

解決方法は?

std::stringを扱うストリーム用の関数は、istreamのメンバ関数ではなく、フリー関数で、このように使用します。(メンバ関数版ではchar*を扱います)。

std::string str;
std::ifstream file("file.dat");
std::getline(file, str);

このように、あなたがやろうとしていることを行うには、より安全な方法があることは知っておく価値があります。

#include <fstream>
#include <string>
#include <vector>

//typedeffing is optional, I would give it a better name
//like vector_str or something more descriptive than ArrayObj
typedef std::vector<std::string> > ArrayObj

ArrayObj load_array(const std::string file_name, char letter)
{
    std::ifstream file(file_name);
    ArrayObj lines;
    std::string str;

    while(std::getline(file, str)){
        if(str.at(0)==letter){
            lines.push_back(str);
        }
    }
    return lines;
}


int main(){
    //loads lines from a file
    ArrayObj awords=load_array("file.dat", 'a');
    ArrayObj bwords=load_array("file.dat", 'b');
    //ao.at(0); //access elements
}

ベクターは標準的なものであり、多くの時間と苦痛を軽減してくれます。

最終的には using namespace std 代わりに std::cout や std::string のように std オブジェクトに std:: というプレフィックスをつけてください。

http://en.cppreference.com/w/cpp/container/vector http://en.cppreference.com/w/cpp/string/basic_string/getline http://en.cppreference.com/w/cpp/string