__str__ для печати строк по методу

Question

Я пытаюсь улучшить игру «броненосец», которую вы делаете в курсе codecademy для python, и я решил, что первым шагом будет реализация классов.

Мой ожидаемый результат от кода составляет пять строк от O O O O O. Это достижение, когда я использую только оператор печати внутри для цикла, но он выдает ошибку, заявив:

Traceback (most recent call last): 
File "D:/PythonProjects/Battleship/Main.py", line 21, in <module> 
    print(board) 
TypeError: __str__ returned non-string (type NoneType) 

И когда я покину код, как это не бросать и ошибки, но печатает только одну строку выходов

код в вопросе:

class Board(object): 
    """Creates the game board""" 
    board = [] 

    def __init__(self, size): 
     self.size = size 
     for x in range(size): 
      self.board.append(["O"] * size) 

    def __str__(self): 
     for row in self.board: 
      display = (" ".join(row)) 

     return display 

board = Board(5) 
print(board) 

Answer 1

ваш метод __str__ присваивает новое значение display с каждой итерации цикла. Вместо этого вы могли бы сделать что-то вроде:

def __str__(self): 
    accumulator = "" 
    for row in self.board: 
     accumulator += " ".join(row) + "\n" 
    return accumulator 

Или более лаконично

def __str__(self): 
    return "\n".join([" ".join(row) for row in self.board]) 

В полной стороне от кого-то, кто предназначен это снова и снова, чтобы помочь каждому новый набор студентов CS вдаваясь в Интро ООП , этот класс становится намного проще, если он наследует от list. Это контейнер для списков, которые описывают игровое поле - сделать это сам список ....

class Board(list): 
    def __init__(self, size): 
     """Square board of length `size`""" 
     for row in range(size): 
      self.append(["O"] * size) 

    def __str__(self): 
     return "\n".join([" ".join(row) for row in self]) 

Answer 2

Прежде всего, вы не должны использовать свойство класса для board. Это позволит обмениваться экземпляром во всех экземплярах Board, поэтому вы продолжаете добавлять в тот же список. Вместо этого создайте новый список для каждого объекта вы создаете:

class Board(object): 
    def __init__(self, size): 
     self.size = size 
     self.board = [] 
     for x in range(size): 
      self.board.append(["O"] * size) 

Теперь, чтобы ответить на ваши проблемы, вы не должны на самом деле получить эту ошибку типа, потому что в то время как вы держите перезапись display в петле в __str__, это получить по крайней мере одно значение, которое не равно None, при условии, что board не пуст (чего нет в вашем примере).

Но то, что вы хотите сделать вместо этого собрать все строки и присоединиться к ним с новой строки:

def __str__(self): 
    display = [] 
    for row in self.board: 
     display.append(" ".join(row)) 
    return '\n'.join(display) 

Или в одной строке:

def __str__(self): 
    return '\n'.join([' '.join(row) for row in self.board])