std :: string :: assign（std :: string const＆）中的分段錯誤

Question

我有以下代碼所示的問題，我不知道這可能是什么原因。 我在發布問題之前進行了搜索，然后得知它可能超出了范圍，例如對釋放的內存位置的引用，但我自己找不到。 感謝你們對我的幫助。

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

class USR{
private:
    string name;
public:
    void setName(string name){  
        this->name = name;
    }
    string getName(){
        return name;
    }

};

class A{
private:
    USR* * a;
public:
    A(int size){
        a = new USR*[size];
    }   
    USR* getUser(){
        return a[0];
    }
};

int main(){
    A test = A(5);
    USR* u = test.getUser(); 
    (*u).setName("test");
    USR* u2 = test.getUser(); 
    cout << (*u2).getName() << endl;
    cout << (*u).getName() << endl;
}

Answer 1

問題是您分配了指針數組，但從未為指針本身分配任何東西。

這為您提供了指針數組：

a = new USR*[size];

但是您從來沒有為每個指針分配任何東西。

因此，它在這里崩潰：

(*u).setName("test");

因為*u未初始化。

---

有兩種方法可以解決此問題：

1. 為每個USR指針分配（並初始化）某些內容。
2. 不要使用雙指針。 只需使用簡單的USR對象數組即可。

我更喜歡后者，因為您所擁有的可能比需要的更為復雜。

這樣的事情可能會滿足您的要求：

class A{
private:
    USR *a;
public:
    A(int size){
        a = new USR[size];
    }   
    USR* getUser(){
        return &a[0];
    }
};

不要忘記，您也將需要一個析構函數。

Answer 2

您的方法getUser返回一個未初始化的指針（ A的構造函數創建一個未初始化的指針數組）。 您看到的錯誤是取消引用該方法返回的未初始化指針的結果。

Answer 3

您僅創建一個新的USR*數組，而不是USR對象的數組。 訪問指針

USR* u = test.getUser();

將為您提供一個統一的指針。 呼喚

(*u).setName("test");

因此將出現段錯誤。

Answer 4

您初始化了USR*的數組，但尚未初始化單個USR*對象。

Answer 5

由於您已經聲明了USR的2D數組，所以我認為您對用戶名數組必須具有Array的要求，盡管這對我來說很奇怪，為什么您不能簡單地使用USR * userArray;

無論如何，如果要使用戶名數組的數組起作用，則需要按如下所示修改類A：

class A{
private:
    USR* * a;
public:
    A(int size){
        a = new USR*[size];
        int iter = 0;
        for(; iter < size; iter++)
        {
           a[iter] = new USR[size_of_user_names_for_each_user_array];
        }
    }   
    USR* getUser(){
        return a[0];
    }
};