C++ node类实现问题：this指针异常及节点指向错误求助

问题1：if (this != nullptr) 完全多余且错误 #

C++里，this指针永远不会是nullptr——只要你通过合法对象（栈/堆上的实例）调用成员函数，this就指向这个对象。只有用nullptr->成员函数()这种未定义行为调用时，this才可能为空，但这种写法本身就是非法的，绝对不能这么用。

你的代码里这类判断完全没必要，还会误导你对this的理解。比如AddFront是通过numbers.AddFront(1)调用的，numbers是栈上的合法对象，this必然指向它，不可能为空。

问题2：带参构造后this->next指向自身的异常 #

这本质是类设计逻辑错误：你把单个node节点当成了整个链表来操作，而不是用链表类统一管理节点集合。

错误根源 #

看AddFront最后一行的*this = *temp_node;：
这行把堆上新建的temp_node的所有成员值，直接赋值给当前调用对象（也就是numbers）。temp_node->next原本指向调用AddFront前的numbers（栈上的node(2)），赋值后，当前numbers的next就指向了原来的自己（旧的node(2)）。同时旧node(2)的prev被设为堆上的temp_node，但现在numbers变成了堆节点的拷贝，堆节点本身没人管理，直接造成内存泄漏，还导致链表指针彻底混乱。

同样，AddBack里的if (this == nullptr)也是完全错误的，而且while (end_node->next != nullptr)如果逻辑混乱导致end_node为空，会直接触发崩溃。

修复方案 #

推荐用标准的「链表类+内部节点类」设计，这是链表的常规实现方式：

#pragma once
#include <iostream>

template<typename T>
class LinkedList {
private:
    // 内部节点类，只负责存数据和指针
    struct Node {
        T data;
        Node* next;
        Node* prev;
        Node() : next(nullptr), prev(nullptr) {}
        Node(T _data) : data(_data), next(nullptr), prev(nullptr) {}
    };

    Node* head;  // 链表头节点
    Node* tail;  // 链表尾节点
    int size;    // 链表长度

public:
    // 默认构造：空链表
    LinkedList() : head(nullptr), tail(nullptr), size(0) {}
    // 带参构造：初始化第一个节点
    LinkedList(T first_data) : size(1) {
        head = new Node(first_data);
        tail = head;
    }
    // 析构函数：释放所有节点，避免内存泄漏
    ~LinkedList() {
        Node* curr = head;
        while (curr != nullptr) {
            Node* next_node = curr->next;
            delete curr;
            curr = next_node;
        }
    }

    // 头部添加节点
    void AddFront(T new_data) {
        Node* new_node = new Node(new_data);
        if (head == nullptr) {
            // 空链表时，头尾都指向新节点
            head = tail = new_node;
        } else {
            new_node->next = head;
            head->prev = new_node;
            head = new_node;
        }
        size++;
    }

    // 尾部添加节点
    void AddBack(T new_data) {
        Node* new_node = new Node(new_data);
        if (tail == nullptr) {
            head = tail = new_node;
        } else {
            tail->next = new_node;
            new_node->prev = tail;
            tail = new_node;
        }
        size++;
    }

    // 重载[]运算符，按索引访问
    T& operator[](int index) {
        // 这里可以加索引合法性检查，比如判断index是否在0到size-1之间
        Node* curr = head;
        for (int i = 0; i < index; i++) {
            curr = curr->next;
        }
        return curr->data;
    }

    // 获取链表长度
    int GetSize() {
        return size;
    }
};

主函数调用修改为：

#include "LinkedList.h"

int main() {
    LinkedList<int> numbers(2);
    numbers.AddFront(1);
    numbers.AddBack(3);
    for (int i = 0; i < numbers.GetSize(); i++) {
        std::cout << numbers[i] << ' ';
    }
    // 输出结果：1 2 3
    return 0;
}

额外注意事项 #

• 原node类的析构函数为空，会导致堆节点无法释放，造成内存泄漏，上面的修复方案里链表类的析构函数已经处理了这个问题。
• 原代码中operator[]和GetSize没有实现，修复方案里已经补充完成。

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