ポインタ付きC++ベクタ

Question

私は宿題に固執しています。私はファイルからテキストを読んで、各単語をメモリに割り当ててから、ポインタをvector<string*>に送るためのポインタを割り当てなければなりません。私のプログラムは、ファイルを単に追加するのではなく、新しい単語でベクトルを上書きし続けます。なぜこれが起こっているのか分かりません。

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


void WordFunctions(string *pstr, vector<string*> &words) 
{ 
    words.push_back(pstr); 
} 
int main(){ 
    ifstream file; 
    vector<string*> a; 
    string word; 
    int w =0; 
    file.open("word.txt"); 
    while (!file.eof()) 
    { 
     w++; 
     file >> word; 

     WordFunctions(&word, a); 
    } 
    file.close(); 

    for (int i=0;i<10;i++){ 
     cout<<(*a[i])<<" "; 
     delete a[i]; 
    } 

    system ("pause"); 
} 

Answer 1

はvector<string>を使用するか、ヒープ上に新しい文字列を割り当てる次のいずれか

void WordFunctions(string *pstr, vector<string*> &words) 
{ 
    words.push_back(new string(*pstr)); 
} 

Answer 2

をあなたはワードのアドレスであるベクターに同じ要素を推進しています。私はあなたのコードにビットをマッサージ

// pass reference to eliminate copy 
void WordFunctions(string &str, vector<string> &words) 
{ 
    words.push_back(str); 
} 
int main(){ 
    ifstream file; 
    vector<string> a; // you want to store string not the address of the string 
    string word; 
    int w =0; 
    file.open("words.txt"); 
    while (!file.eof()) 
    { 
     w++; 
     word.clear(); // clear the content before store something into it 
     file >> word; 
     WordFunctions(word, a); 
    } 
    file.close(); 

    for (size_t i=0;i<a.size();i++){ // use size instead of hard code magic number 
     cout<<(a.at(i))<<" "; // use at function instead of [] 
    } 

    system ("pause"); 
} 

Answer 3

ループ内では、文字列の値を変更しているが、その後、あなたはいつも彼に同じアドレスを渡すWordFunctionsを呼び出すように、あなたのword文字列は、常にメモリ内の同じアドレスを持っています。それは代わりにvector<string>のvector<string*>を使用するための制約があります場合は

は、あなたはおそらく、あなたの言葉をそこにコピーしてWordFunctions

char *wordPtr 

while (!file.eof()) 
{ 
    w++; 
    file >> word; 

    wordPtr = (char *)malloc((strlen(word)+1)*sizeof(char)); 
    strcpy(wordPtr, *word); 

    WordFunctions(wordPtr, a); 
}