クラス、getline、およびストリングストリームでの解析でのC++演習

Question

getlineとstringstreamを一緒に使用してファイルを解析する方法を理解するのは本当に苦労しています。誰かが説明するのを助けることができたら、私は非常に感謝します。非常に。

私は実際にReadFileという関数をビルドしようとしています。これは、50の名前と給与の「名前、給与」形式のテキストで構成されるファイル名を読み込む関数です。 names [] salary []という名前の配列に分割したいと思います。

これまで私が教えてくれたのは、プロンプトに従ってクラス構造が正しいと思いますが、ファイルを別々の配列に正しく読み込むのを助けることができましたか？ありがとうございました！

class Players { 

    public: 
     //Initialize 
     float salaries[50]; 
     string names[50]; 

     Players(){ 
      for (int i=0; i<50; i++) { 
      float salaries[i]={-1.0}; 
      string names[i]= {""} ; } 
     } 

     void ReadFile(string filename) { 
      ifstream file; 
      file.open(filename); 

      string line; 
      int index=0; 
      while (getline(file, line)) { 
       stringstream ss(line); 
       getline(ss, names[index], ','); 
       getline(ss, salaries[index]); 

       index ++ 
       string names; 
       float salaries; 

       ss >> names >> salaries >> ',';   
      } 
     } 

    float MaxSalary() { 

    } 

    string MaxSalaryName() { 

    } 
}; 

Answer 1

このコードはコンパイルしないでください。将来的には、コード例を変更せずにコンパイルし、すべての#includeを含めてください。

あなたReadFile機能についていくつかのこと：

• あなたは、変数namesとsalariesを再定義しています。あなたはおそらくそれを望んでいません。
• index ++は、index++;である必要があります。
• ifstream::openはC文字列を受け取りますので、filename.c_str()を渡す必要があります。
• 最初のgetlineは問題ありませんが、2番目の方法は文字列をフロートに直接コピーしようとしています。明示的に型変換を行う必要があります。 1つの方法は、一時的な文字列変数を作成し、それをgetlineに渡し、std::stofを使用してその文字列をfloatに変換することです。これにはC++ 11が必要です。 C++ 11を使用できない場合は、strtofを<cstdilib>から使用できます。
• ss >> ...が達成しようとしていることがわかりませんが、おそらくそれは欲しがらないでしょう。あなたは、変数名の前に種類が含まれているため
  • あなたはあなたのメンバ変数を再定義している：あなたはあなたの結果を印刷したい場合は、cout << names[index] << ": " << salaries[index] << "\n";

  ような何かごPlayers()コンストラクタについてのいくつかのことをしたいです。タイプを削除します。

• デフォルトの文字列コンストラクタは空の文字列を作成するため、空の文字列で初期化する必要はありません。
• {}を初期値の周りに安全にドロップできます。

これらの問題のほとんどは、コンパイラのエラーを調べ、適切なC++ドキュメントを参照することで検出されている可能性があります。つまり、私はC++が始めるのが最も簡単な言語ではなく、早期にポインタを持てるようにするのに役立ちます。それを保つとあなたはそれを得るでしょう！