尝试在函数内动态分配父指针给子指针会导致分段错误

Question

我一直在努力理解多态性和指针，但这个概念让我有些不知所措。幸运的是，我有 stackoverflow 和 google，所以我的大部分问题都可以轻松解决。但是，我已经尝试修复这个分段错误几个小时了，但没有运气，我不确定是什么问题。我已经剥离了所有内容以尝试了解出了什么问题，但我相信我缺少动态分配的核心概念。

我将一个字符指针传递给我的函数初始化，我不能在函数内动态分配它吗？我的代码可以编译，但在为（预期的）开关提供输入后，出现分段错误和核心转储。

但是，如果我排除函数初始化并仅在我的主函数中运行代码，则不会出现任何中断，并且 zed 会正确初始化为野蛮人类型。

#include "character.h"//abstract class
#include "barbarian.h"//inherits character



void initialize(character *object){//gets the values for the object
                              //and lets user deem which type of object
    int x=0;
    cout<< "1. Barbarian."<<endl;
    cout<< "2. Gollum."<<endl;
    cout<< "3. Baba Yaga."<<endl;
    cout<< "4. Blue Men Group."<<endl;
    cout<< "5. Unicorn."<<endl;
    while(!(cin>>x)){
        cout<<"Input not valid."<<endl;
        cin.clear();
        cin.ignore(10000,'\n');
    }
    cout<<"Input is: "<<x<<endl;               //debug line
    object = new barbarian("conan");           //seg fault here
    //down here is a switch that was excluded for this question
 }

void die(character *kill){//is now kill
    delete kill;
}               //this should delete the character after it is used

int main(){
    srand(time(0));//used for roll functions in the parent class
    cout<<(-time(0))<<"seconds until 1970"<<endl;//debug makes sure srand is
                                            //doing what I want it to

    character* zed;  //zed = new barbarian("conan");//In main this works
    initialize(zed);
    //zed = new barbarian("conan"); This works
    (*zed).atkRoll();
    (*zed).defRoll();
    die(zed);
}

我希望能够传递一个字符指针，然后将它作为用户选择的类分配到堆上，并在 main 调用的其他函数中使用它。我认为可以只传递指针并将其作为函数中的参数。然后本地指针将指向与 main 中的指针相同的位置。当函数作用域关闭并且“对象”被删除时，main 中的指针应该保持指向野蛮人的内存位置。这无效吗？如果是这样，我能做些什么作为替代方案？ 不告诉指针就不能在main中改变指针的类型吗？

我很困惑>.

Answer 1

我认为这应该是这样的：

void initialize(character **object){
    /* ... */
    *object = new barbarian("conan");
}

否则，您不会修改实际指针，而是修改它的临时副本。您需要将指针传递给指针。然后，在main()：

character* zed;
initialize(&zed);

对上一段中问题的解释：

您在main() 中有character* zed。这意味着，一个包含地址的指针变量（还没有有意义的地址，只是垃圾）：

zed = 0x12345678

现在您拨打initialize(zed)。在该函数中，有一个类型为“指向野蛮人的指针”的参数，称为object。它被初始化为zed的副本，即zed的值：

zed = 0x12345678
object = 0x12345678

现在您拨打object = new barbarian()。内存被分配，地址保存在object:

zed = 0x12345678
object = 0xABCDEF00

现在initialize() 退出。 object 是临时的并被销毁。剩下的：

zed = 0x12345678 (garbage)
object doesn't exist

修复在我的回答开头。

段错误应该发生在main()，而不是initialize()，我认为：

(*zed).atkRoll();
(*zed).defRoll();

因为您在未初始化的指向对象的指针上调用方法，这些对象（假定）尝试访问对象的字段。

还有：

void die(character **kill){
    delete *kill; *kill == nullptr;
}

尽管这种方法的想法对我来说看起来很糟糕。