Python：我无法在单链表中添加值

Question

代码写在python：

prepend Function 应该在链接列表显示的开头添加一个值 Function 应该以列表格式显示链接列表 python fromarr Function 应该将新节点添加到链接列表

class Node:
    def __init__(self,value=None):
        self.value = value
        self.next = None
    
class SinglyLL:
    def __init__(self):
        self.head = Node()
        self.tail = Node()
    def prepend(self,value):
        newnode  = Node(value)
        newnode.next = self.head
        self.head = newnode
    def append(self,data):
        newnode = Node(data)
        if self.head is None:
            self.head = newnode
            return
        cur = self.head
        while cur.next != None:
            cur = cur.next
        cur.next = newnode
        newnode.next = None
    def display(self):
        lis = []
        cur = self.head
        while cur.next != None:
            cur = cur.next
            lis.append(cur.value)
        return lis
    def fromarr(self,list):
        for i in list:
            temp = Node(i)
            self.append(temp.value)

newsll = SinglyLL()
newsll.fromarr([1,2,3,4,5])
newsll.prepend(10)
print(newsll.display())

预期 output： [10,1,2,3,4,5]

实际 Output： [None,1,2,3,4,5]

Answer 1

好吧，你正在用一个空节点初始化你的头，并显示来自self.head.next的值，所以不是将元素设置为 position 0，而是需要将元素设置为 position 1。

为prepend方法添加以下代码

class Node:
    def __init__(self,value=None):
        self.value = value
        self.next = None

class SinglyLL:
    def __init__(self):
        self.head = Node()
        self.tail = Node()
    def prepend(self,value):
        newnode  = Node(value)
        if self.head is None:
            self.head = newnode
        else:
            tmp = self.head.next
            self.head.next = newnode
            newnode.next = tmp

    def append(self,data):
        newnode = Node(data)
        if self.head is None:
            self.head = newnode
            return
        cur = self.head
        while cur.next != None:
            cur = cur.next
        cur.next = newnode
        newnode.next = None
    def display(self):
        lis = []
        cur = self.head
        while cur.next != None:
            cur = cur.next
            lis.append(cur.value)
        return lis
    def fromarr(self,list):
        for i in list:
            temp = Node(i)
            self.append(temp.value)

newsll = SinglyLL()
newsll.fromarr([1,2,3,4,5])
newsll.prepend(10)
print(newsll.display())

Answer 2

您的代码混合了多种方法来组织链表数据结构，但它们不兼容：

• 您的构造函数为head属性创建了一个虚拟节点，当您打算 append 该虚拟节点之后的所有数据节点时，这可能是一种方法。 display方法与该方法一致，因为它不在结果中包含虚拟节点的值。 但是prepend和append方法都使用这种方法，因为它们将head节点视为数据节点。

• 您的构造函数为tail属性创建了一个虚拟节点。 这是一种可能用于双向链表的方法，但对单链表没有用，因为到 append 列表末尾的新节点的过程无法从该引用中受益。

• 尽管构造函数定义了一个tail属性，但其他方法都没有使用它，也没有在必要时更新它。

其他一些说明：

• 奇怪的是， display方法不显示任何内容。 使用__iter__方法会更合适，因为它会产生链表的值。

• fromarr对于接受列表的方法来说不是一个很好的名字。

• 这是一个实例方法并不直观，顾名思义它将根据这些值创建一个新列表，但实际上您需要先自己创建一个链表实例，然后调用此方法。 我建议放弃此方法并使用可选的 arguments 扩展构造函数，它将用于填充构造的链表。

• list是参数的错误名称，因为此名称已被本机 Python 使用。

• 如果Node构造函数接受用于定义next属性的可选参数，那就太好了。

这是您的代码的更新，其中考虑了这些要点，并使用以下方法：

• 设置并使用tail属性
• 没有创建虚拟节点

class Node:
    def __init__(self, value=None, nxt=None):
        self.value = value
        self.next = nxt
    
class SinglyLL:
    def __init__(self, *values):
        # Don't create nodes here
        self.head = self.tail = None
        # Allow immediate population from arguments
        for value in values:
            self.append(value)
        
    def prepend(self,value):
        self.head = Node(value, self.head)
        self.tail = self.tail or self.head
        
    def append(self, data):
        if not self.tail:
            self.prepend(data)
        else:
            self.tail.next = self.tail = Node(data)

    def __iter__(self):
        cur = self.head
        while cur:
            yield cur.value
            cur = cur.next
        

newsll = SinglyLL(1,2,3,4,5)
newsll.prepend(10)
print(list(newsll))  # [10, 1, 2, 3, 4, 5]