0

我对以下代码有指示的问题，但我不知道它可能是什么原因造成的。我在发布问题之前进行了搜索，我了解到它可能超出了范围，例如对已释放内存位置的引用，但我自己找不到它。感谢你们对我的帮助。

#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;
}
4

5 回答 5

4

问题是您分配了指针数组，但您从未为指针本身分配任何东西。

这为您提供了指针数组：

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];
    }
};

不要忘记你也需要一个析构函数。

于 2011-11-27T19:57:13.460 回答
3

您的方法getUser返回一个未初始化的指针（的构造函数A创建一个未初始化的指针数组）。您看到的错误是取消引用该方法返回的未初始化指针的结果。

于 2011-11-27T19:56:23.717 回答
1

您初始化了一个 s 数组USR*，但尚未初始化单个USR*对象。

于 2011-11-27T20:00:22.743 回答
1

您只是在创建一个新USR*数组而不是一个USR对象数组。访问指针

USR* u = test.getUser();

会给你一个未初始化的指针。打电话

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

因此会出现段错误。

于 2011-11-27T19:57:34.247 回答
0

由于您已经声明了 USR 的 2D 数组，所以我认为您需要用户名数组的数组，尽管对我来说这看起来很奇怪，为什么您不能简单地使用 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];
    }
};
于 2015-07-09T08:08:06.780 回答