У меня есть словарь объектов класса. Мне нужно вытащить объект, чтобы он выполнил одну из своих функций. У объектов нет имен переменных, они создаются циклом for
.
Вот что я пробовал:
class Pawn(object):
def legal_moves(self):
...
return moves_list
...
# main
dictionary = {'a2': <class '__main__.Pawn'>, 'b2': <class '__main__.Pawn'> etc.}
pawn_obj = dictionary['a2']
moves = pawn_obj.legal_moves()
ПОЛНАЯ ВЕРСИЯ КОДА:
class Pawn(Piece):
def __init__(self, color):
self.type = "P"
super(Pawn, self).__init__(color, self.type)
def legal_moves(self, position, map):
self.l_moves = []
file = int(position[:1])
rank = int(position[1:])
if self.color == "w" and rank == 2:
move1 = str(rank + 1) + str(file)
move2 = str(rank + 2) + str(file)
self.l_moves.append(move1)
self.l_moves.append(move2)
return self.l_moves
#main
b = Board(white_view=True)
p = Pawn("w")
p = b.map.get("12")
print(type(p))
moves = p.legal_moves("12", b.map)
print(moves)
ВОЗВРАТ:
<class '__main__.Pawn'> File "C:/Users/" line 173, in <module> moves = p.legal_moves("12", b.map) TypeError: 'list' object is not callable Process finished with exit code 1```
Также добавьте полное сообщение об ошибке.
Проблема не в pawn_obj
, а в pawn_obj.do_a_thing
. Где-то в вашем коде вы делаете что-то вроде self.do_a_thing = <some list>
Что показывает print(pawn_obj.do_a_thing)
?
Бармар, вы правы. ошибка выдается в строке pawn_obj.do_a_thing. do_a_thing возвращает список допустимых ходов для этой конкретной пешки
Еще не проблема, но как только текущая ошибка будет исправлена, это будет: вы создали do_a_thing()
метод экземпляра. Ваш словарь содержит объекты класса, а не экземпляры Pawn
. Это означает, что pawn_obj.do_a_thing()
потерпит неудачу с TypeError: do_a_thing() missing 1 required positional argument: 'self'
. Вы можете исправить это, используя декораторы @classmethod
или @staticmethod
на do_a_thing()
и определив его с параметром cls
или без параметров соответственно.
Я согласен с Комментарий Бармара.
Я бы CTRL-F проект для self.legal_moves
искал место, где его значение установлено в список.
Опубликованный вами дополнительный код показывает, что Pawn
имеет атрибут l_moves
, который является списком. Может быть, этот атрибут начал свою жизнь как Pawn.legal_moves
до того, как вы создали функцию Pawn.legal_moves()
и где-то пропустили ее переименование?
Попробуйте добавить эти строки отладки, чтобы получить подсказку о том, что действительно содержит этот атрибут в вашем случае:
p = b.map.get("12")
print(type(p))
# new lines
print(type(p.legal_moves)) # if <class 'list'>, we're on the right track; if <class 'method'> we need to look elsewhere
print(p.legal_moves)) # if it's a list, its contents might give you a clue about where it's being set
moves = p.legal_moves("12", b.map)
Вы были правы, в суперклассе я случайно определил «legal_moves» как список. Поменял и теперь работает. Спасибо!
вам необходимо предоставить минимальный воспроизводимый пример. откуда вылетает эта ошибка? трассировка стека должна содержать точную строку. Единственный звонок, который вы делаете, это
pawn_obj.do_a_thing
, поэтому вам следует проверить тип этого