我正在调整text adventure game tutorial,github以适应python 2.7。我正在为我的IDE使用PyCharm 4.5.4社区版。当我不覆盖父方法时,它给我一个错误:
类WolfRoom必须实现所有抽象方法
首先要消除这个错误,我将缺少的方法def modify_player(self, the_player):
定义为pass
,但我很快意识到我的方法是什么,而不是我想要的东西。现在,如果我只是从WolfRoom类中删除该方法,我会收到一个IDE错误,如上所示,但是当我运行游戏时它似乎工作得很好。我应该退出此方法还是定义它并使用super()
?
以下是一些代码段:
class MapTile(object):
"""The base class for all Map Tiles"""
def __init__(self, x, y):
"""Creates a new tile.
Attributes:
:param x: The x coordinate of the tile.
:param y: The y coordinate of the tile.
"""
self.x = x
self.y = y
def intro_text(self):
"""Information to be displayed when the player moves into this tile."""
raise NotImplementedError()
def modify_player(self, the_player):
"""Process actions that change the state of the player."""
raise NotImplementedError()
def adjacent_moves(self):
"""Returns all move actions for adjacent tiles."""
moves = []
if world.tile_exists(self.x + 1, self.y):
moves.append(actions.MoveEast())
if world.tile_exists(self.x - 1, self.y):
moves.append(actions.MoveWest())
if world.tile_exists(self.x, self.y - 1):
moves.append(actions.MoveNorth())
if world.tile_exists(self.x, self.y + 1):
moves.append(actions.MoveSouth())
return moves
def available_actions(self):
"""Returns all of the available actions in this room"""
moves = self.adjacent_moves()
moves.append(actions.ViewInventory())
return moves
...
class EnemyRoom(MapTile):
def __init__(self, x, y, enemy):
self.enemy = enemy
super(EnemyRoom, self).__init__(x, y)
def intro_text(self):
pass
def modify_player(self, the_player):
if self.enemy.is_alive():
the_player.hp = the_player.hp - self.enemy.damage
print("Enemy does {} damage. You have {} HP remaining.".format(self.enemy.damage, the_player.hp))
def available_actions(self):
if self.enemy.is_alive():
return [actions.Flee(tile=self), actions.Attack(enemy=self.enemy)]
else:
return self.adjacent_moves()
...
class WolfRoom(EnemyRoom):
def __init__(self, x, y):
super(WolfRoom, self).__init__(x, y, enemies.Wolf())
def intro_text(self):
if self.enemy.is_alive():
return """
A grey wolf blocks your path. His lips curl to expose canines as white as
the nights sky. He crouches and prepares to lunge.
"""
else:
return"""
The corpse of a grey wolf lays rotting on the ground.
"""
答案 0 :(得分:8)
我认为这实际上是由于PyCharm检查员在查看是否存在任何未实现的方法会引发NotImplementedError时发生错误,或至少是关于PEP 8样式的可疑决定。考虑一个非常相似的简单示例:
class Base(object):
def foo(self):
raise NotImplementedError
def bar(self):
return 0
class Child(Base):
def foo(self):
return 0
class GrandChild(Child):
def bar(self):
return 1
my_grand_child = GrandChild()
print my_grand_child.foo()
上面的代码成功地将0输出到输出,因为当Python在GrandChild中找不到foo()的实现时,它会查找继承链并在Child中找到它。但是,出于某种原因,PyCharm检查器期望所有引发NotImplementedError的类都在继承链的所有级别中实现。
如果你要在具有大型继承结构的程序中遵循这种风格,那么当你不需要时,你会发现自己在整个链中实现方法和调用super时非常冗长。就个人而言,我只是忽略了错误,并认为如果PyCharm找到在它正在检查的类的任何超类中实现的方法,那么它应该更新为不显示它。
答案 1 :(得分:2)
简单地从方法中提升NotImplementedError
不会完全使其成为一种抽象方法。您仍然可以实例化一个不覆盖其所有继承的伪抽象方法的类,您只能调用方法。 (或者更确切地说,如果你在NotImplementedError
语句中抓住try
,你甚至可以打电话给他们。)
您可以使用abc.ABCMeta
使类真正抽象化;元类机制阻止你甚至使用un-overriden抽象方法实例化一个类。
import abc
class MapTile(object):
"""The base class for all Map Tiles"""
__metadata__ = abc.ABCMeta
def __init__(self, x, y):
"""Creates a new tile.
Attributes:
:param x: The x coordinate of the tile.
:param y: The y coordinate of the tile.
"""
self.x = x
self.y = y
@abc.abstractmethod
def intro_text(self):
"""Information to be displayed when the player moves into this tile."""
pass
# etc.
答案 2 :(得分:0)
是的,您必须在Python中实现所有抽象方法,将它们实例化为对象(标有@abstractmethod
的对象等)。但是,如何实现这些完全取决于您。如果您不打算实例化,则不需要覆盖所有这些内容。
例如:
class Animal(object):
__metaclass__ = ABCMeta
@abstractmethod
def eat(thing):
pass
class Slug(Animal):
def eat(thing):
pass
这意味着每个可实例化的Animal
必须能够吃,但Slugs
吃东西时什么也不做。