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

Segmentation fault in std::string::assign(std::string const&)

本文关键字:std string 错误 分段 const assign      更新时间:2023-10-16

我有以下代码所指示的问题,我不知道它可能导致什么。在发布问题之前我搜索了一下,我了解到它可能是超出范围的东西,比如对释放内存位置的引用,但我自己找不到它。谢谢你的帮助。

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

问题是您分配了指针数组,但您从未为指针本身分配任何内容。

指针数组:

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

别忘了你还需要一个析构函数

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

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

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

将给你一个初始化的指针。调用

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

将因此分段错误。

初始化了一个USR*对象数组,但还没有初始化单个USR*对象。

因为你已经声明了一个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];
    }
};