std :: string :: assign中的分段错误(std :: string const&)

时间:2011-11-27 19:54:50

标签: c++ string segmentation-fault assign

我有以下代码的指示问题,我不知道它可能导致它。我在发布问题之前进行了搜索,并且我了解到它可能会超出范围,就像对已释放的内存位置的引用一样,但我无法自己找到它。 感谢你们对我的帮助。

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

5 个答案:

答案 0 :(得分:4)

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

这为您提供了指针数组:

a = new USR*[size];

但是你从未为每个指针分配任何东西。

因此,它在这里崩溃:

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

因为*u未初始化。


有两种方法可以解决这个问题:

  1. 为每个USR指针分配(并初始化)一些东西。
  2. 不要使用双指针。只需使用一个简单的USR对象数组。
  3. 我更喜欢后者,因为你所拥有的东西可能比它需要的更复杂。

    这样的事情可能会做你想做的事情:

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

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

答案 1 :(得分:3)

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

答案 2 :(得分:1)

您只是创建一个新的USR*数组而不是USR个数组。访问

中的指针
USR* u = test.getUser();

会给你一个单位指针。调用

(*u).setName("test");
因此,

将是段错误。

答案 3 :(得分:1)

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

答案 4 :(得分: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];
    }
};