使用类实现链表时触发Segmentation fault，实现方法是否有误？

首先，你的段错误不是因为用类代替struct导致的，核心问题是你混淆了对象和指针的使用逻辑，再加上全局对象的不当操作，才引发了内存访问错误。下面逐一拆解问题并给出修复方案：

核心错误点 #

1. palabraTemp是实体对象而非指针，循环逻辑完全失效 #

你定义的palabraTemp是全局的Palabra对象，不是指针：

extern Palabra palabraTemp;

• 用while (&palabraTemp != NULL)作为循环条件永远为真——一个已初始化的实体对象的地址不可能是NULL，循环会无限执行。
• 当执行palabraTemp = *palabraTemp.next;时，如果palabraTemp.next是NULL，解引用空指针直接触发段错误，这就是你标注的错误位置的根源。

2. 直接赋值对象而非操作指针，破坏链表结构 #

当你执行primeraPalabra = *nuevaPalabra;时，这是对象拷贝操作，不是让链表指针指向新对象。全局的primeraPalabra是一个实体对象，拷贝后它的next/prev会复制nuevaPalabra的成员，但后续操作的是这个拷贝出来的对象，而非你传入的prototypo/prototypo1，导致链表的指针关联完全混乱。

3. 全局实体对象的设计风险 #

你定义的三个全局Palabra实体对象，本身就容易引发状态混乱——比如primeraPalabra初始是一个空对象，后续通过赋值覆盖它，会导致原来的对象内存被覆盖，指针指向错误的内存区域。

修复后的完整代码示例 #

调整头文件（palabra.h） #

把全局实体对象改成指针，避免拷贝问题，同时补全缺失的头文件：

#include <iostream>
#include <string> // 原代码漏了string头文件，必须包含
using namespace std;

class Palabra{
public:
    int p_id;
    string p_nombre;
    int p_valor;
    Palabra *next;
    Palabra *prev;

    // 构造函数
    Palabra(int id, string nombre);
    // 方法实现声明
    void printNombre();
    int calcularValorPalabra();
    void agregarSiguiente(Palabra *nuevaPalabra);
    void eliminar();
    void buscar();
};

// 全局指针，初始化为NULL
extern Palabra *primeraPalabra;
extern Palabra *ultimaPalabra;

实现文件（Palabra.cpp） #

修正链表操作逻辑，用指针遍历，不再依赖全局实体对象：

#include "palabra.h"
#include <cctype> // 用于toupper函数

// 初始化全局指针
Palabra *primeraPalabra = NULL;
Palabra *ultimaPalabra = NULL;

Palabra::Palabra(int id, string nombre) {
    p_id = id;
    p_nombre = nombre;
    p_valor = calcularValorPalabra();
    next = NULL;
    prev = NULL;
}

// 补全calcularValorPalabra的实现（原代码仅声明未实现）
int Palabra::calcularValorPalabra() {
    int valor = 0;
    for (char c : p_nombre) {
        valor += toupper(c) - 'A' + 1; // 示例：按字母顺序计算值
    }
    return valor;
}

void Palabra::agregarSiguiente(Palabra *nuevaPalabra) {
    if (primeraPalabra == NULL) {
        // 链表为空，设置首尾节点
        primeraPalabra = nuevaPalabra;
        ultimaPalabra = nuevaPalabra;
    } else {
        // 遍历到链表末尾
        Palabra *temp = primeraPalabra;
        while (temp->next != NULL) {
            temp = temp->next;
        }
        // 添加新节点并维护双向链表指针
        temp->next = nuevaPalabra;
        nuevaPalabra->prev = temp;
        ultimaPalabra = nuevaPalabra;
    }
}

// 补全printNombre的实现
void Palabra::printNombre() {
    cout << p_nombre << endl;
}

// 其他方法（eliminar、buscar）可按需实现

主函数（main.cpp） #

现在可以正确操作链表，同时注意内存管理：

#include "palabra.h"

int main() {
    // 用new创建节点（也可以用栈对象，但链表中建议用堆对象避免生命周期问题）
    Palabra *prototypo = new Palabra(1,"proto");
    Palabra *prototypo1 = new Palabra(2,"proto2");
    
    // 添加第一个节点
    primeraPalabra->agregarSiguiente(prototypo);
    cout << primeraPalabra->p_nombre << endl;
    
    // 添加第二个节点
    primeraPalabra->agregarSiguiente(prototypo1);
    cout << primeraPalabra->next->p_nombre << endl;

    // 释放内存，避免内存泄漏
    delete prototypo1;
    delete prototypo;
    primeraPalabra = NULL;
    ultimaPalabra = NULL;

    return 0;
}

额外优化建议 #

• 尽量避免全局变量，建议封装一个ListaPalabras类来管理链表的首尾节点和所有操作，代码会更清晰，也减少全局状态的风险。
• 操作链表时始终使用指针，不要直接拷贝对象（除非你明确需要复制节点），否则会破坏链表的指针关联。
• 记得给所有声明的方法补全实现，避免未定义行为。

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