C++实现节点结构触发Segmentation fault及未初始化值报错如何解决

报错原因分析 #

• 野指针访问：你最初写的Child1 *child1;仅声明了一个指向Child1类型对象的指针，没有给它分配有效的内存空间，也没有指向任何已存在的合法对象，属于野指针。直接通过野指针调用setName成员方法时，会尝试访问随机的非法内存地址，直接触发段错误。你后续补充的new Child1()操作，就是在堆上申请了一块大小符合Child1类的内存，完成了对象的初始化，让指针指向合法的内存区域，因此报错消失。
• 额外隐藏问题：你当前代码还存在其他容易触发崩溃的隐患：
  1. 变量名遮蔽：Graph构造函数里重新定义了局部变量Node *root和string name，会覆盖类的同名成员变量，你在构造函数里操作的都是局部变量，类的root和name成员实际没有被正确赋值。
  2. 空指针风险：Node类的getParent方法直接对parent指针解引用，如果parent是nullptr的话，调用这个方法会直接崩溃。
  3. 内存泄漏：所有用new申请的内存都没有对应的delete释放，程序退出后会残留内存泄漏。

规范写法建议 #

1. 指针初始化规则：所有指针声明时要么指向合法的已有对象，要么初始化为nullptr，要么直接分配内存，禁止出现未初始化的指针。
2. 类成员初始化优先用初始化列表：比如Node的构造函数可以改写成：

Node(string nodeName, Node *nodeParent) : name(std::move(nodeName)), parent(nodeParent) {}

比在构造函数里调用set方法效率更高，也能避免初始化阶段的异常风险。
3. 避免变量名遮蔽：局部变量不要和类成员变量同名，或者给成员变量统一加前缀（比如m_name、m_parent）区分。
4. 指针访问前做空校验：比如getParent可以改写成返回指针，避免空指针解引用：

Node* getParent() {
    return parent;
}

5. 内存管理：如果不需要手动管理生命周期，可以优先用智能指针std::unique_ptr或者std::shared_ptr代替裸指针，自动回收内存，避免泄漏。

重复root节点的优化方法 #

你现在在main和Graph构造里各建了一个root是因为变量遮蔽的问题，完全可以删掉构造函数里的局部root定义，直接把main里传入的root作为Graph的根节点即可，不需要重复新建。

内容的提问来源于stack exchange，提问作者Jess Ocean