C++ vector với con trỏ

Question

Tôi bị kẹt trên một bài tập về nhà. Tôi phải đọc văn bản từ một tập tin, phân bổ mỗi từ vào bộ nhớ, sau đó người dùng một con trỏ để gửi nó đến một vector<string*>. Chương trình của tôi tiếp tục ghi đè vectơ với từ mới từ tệp thay vì chỉ thêm nó. Tôi không thể hiểu tại sao điều này lại xảy ra.

#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

Hoặc sử dụng một vector<string> hoặc phân bổ chuỗi mới trên heap:

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

Answer 2

Bạn đang đẩy mạnh các yếu tố tương tự vào vector đó là địa chỉ của từ. Tôi xoa bóp một chút về code của bạn

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

chuỗi word của bạn luôn luôn có cùng một địa chỉ trong bộ nhớ, vì vậy trong vòng lặp bạn đang thay đổi giá trị của chuỗi, nhưng sau đó bạn gọi WordFunctions đi với anh luôn cùng một địa chỉ .

Nếu đó là một hạn chế sử dụng vector<string*> thay vì vector<string>, bạn có thể sẽ cần phải phân bổ bộ nhớ cho chuỗi mới trong vòng lặp, sao chép có văn bản của bạn và sau đó vượt qua tham chiếu mới để WordFunctions

char *wordPtr 

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

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

    WordFunctions(wordPtr, a); 
}