Wektor w C++ ze wskaźnikiem

Question

Utknąłem w zadaniach domowych. Muszę odczytać tekst z pliku, przydzielić każde słowo do pamięci, a następnie wskazać użytkownikowi wskaźnik, aby wysłać go do vector<string*>. Mój program nadpisuje wektor nowym słowem z pliku zamiast go tylko dodawać. Nie rozumiem, dlaczego tak się dzieje.

#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

Albo użyć vector<string> lub przydzielić nowy ciąg na stercie:

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

Answer 2

Ty pchają ten sam element do wektora, który jest adres słowa. I masować trochę na kodzie

// 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

ciąg word ma zawsze ten sam adres w pamięci, więc w pętli zmieniasz wartość ciągu, ale potem zadzwonić WordFunctions przechodząc do niego zawsze ten sam adres .

Jeśli jest to ograniczenie korzystania vector<string*> zamiast vector<string>, będzie prawdopodobnie trzeba przydzielić pamięci dla nowych ciągów w pętli, skopiuj tam swoje słowo, a następnie zdać nowy odnośnik do WordFunctions

char *wordPtr 

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

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

    WordFunctions(wordPtr, a); 
}