파이썬 정렬되지 않은 목록 노드 : __str__ 메서드

Question

목록을 [1,2,3,4,5]와 같은 형식으로 인쇄 할 수있게 해주는 UnorderedList 클래스 용 __str__method를 작성하려고합니다.

하지만 몇 가지 이유로 목록의 첫 번째 위치 만 인쇄합니다. 예. [1] 대신 [1,2,3,4,5]

def __str__(self): 

    result = "[" 
    node = self.head 
    if node != None: 
     result += str(node.data) 
     node = node.next 
     for i in range(self.num_elements - 1): 
      result += ", " + str(node.data) 
      node = node.next 
    result += "]" 
    return result  

여기

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

def getData(self): 
    return self.data 

def getNext(self): 
    return self.next 

def setData(self,newdata): 
    self.data = newdata 

def setNext(self,newnext): 
    self.next = newnext 



class UnorderedList: 

def __init__(self): 
    self.head = None 

def isEmpty(self): 
    return self.head == None 

def add(self,item): 
    temp = Node(item) 
    temp.setNext(self.head) 
    self.head = temp 

def size(self): 
    current = self.head 
    count = 0 
    while current != None: 
     count = count + 1 
     current = current.getNext() 
    return count 

def search(self,item): 
    current = self.head 
    found = False 
    while current != None and not found: 
     if current.getData() == item: 
      found = True 
     else: 
      current = current.getNext() 
    return found 

def remove(self,item): 
    current = self.head 
    previous = None 
    found = False 
    while not found: 
     if current.getData() == item: 
      found = True 
     else: 
      previous = current 
      current = current.getNext() 

    if previous == None: 
     self.head = current.getNext() 
    else: 
     previous.setNext(current.getNext()) 

def __str__(self): 

    result = "[" 
    node = self.head 
    if node != None: 
     result += str(node.data) 
     node = node.next 
     for i in range(0 - 1): 
      result += ", " + str(node.data) 
      node = node.next 
    result += "]" 
    return result 

어떤 도움이 좋지 않을까 ... 내 노드 및 UnorderedList 클래스 메소드의 나머지 부분입니다. 또한 훨씬 더 쉬운 방법으로 쓰여졌습니다. str. 당신은 잘못을 반복하고

Answer 1

(모두 상위 버전에서 range(0 - 1) AKA range(-1) 물론 []입니다으로는, self.num_elements을 설정 한 적이과 하단 하나에 결코 일어날 수있는 루프를 강제로 사용 아마 더 나은 :.

def __str__(self): 
    result = "[" 
    node = self.head 
    if node != None: 
     result += str(node.data) 
     node = node.next 
     while node: 
      result += ", " + str(node.data) 
      node = node.next 
    result += "]" 
    return result  

차이는. 루프 while node:보다는 어떻게 든 신성 또는 관련 요소의 수를 강제로 시도에 귀하의 질문에 대한 접선

Answer 2

def __str__(self): 
    result = "[" 
    node = self.head 
    while node: 
     result += ('' if result == '[' else ", ") + str(node.data) 
     node = node.next 
    result += "]" 

    return result