Vettore C++ con puntatore

Question

Sono bloccato su un compito a casa. Devo leggere il testo da un file, assegnare ogni parola alla memoria, quindi utilizzare un puntatore per inviarlo a vector<string*>. Il mio programma continua a sovrascrivere il vettore con la nuova parola dal file invece di aggiungerlo. Non riesco a capire perché questo sta accadendo.

#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

utilizzare un vector<string> o allocare la nuova stringa sul mucchio:

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

Answer 2

Hai spingendo lo stesso elemento in vettore che è l'indirizzo della parola. Mi massaggio un po 'sul vostro codice

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

la stringa word ha sempre lo stesso indirizzo di memoria, quindi nel ciclo si sta modificando il valore della stringa, ma poi si chiama WordFunctions passando a lui sempre lo stesso indirizzo .

Se si tratta di un vincolo di utilizzare vector<string*> invece di vector<string>, è probabile che sia necessario allocare memoria per le nuove stringhe in loop, c'è copiare la tua parola e quindi passare il nuovo riferimento per WordFunctions

char *wordPtr 

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

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

    WordFunctions(wordPtr, a); 
}