我是否必须在python 2.7中实现所有抽象方法?

时间:2015-10-01 04:52:32

标签: python python-2.7 oop pycharm

我正在调整text adventure game tutorialgithub以适应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.
            """

3 个答案:

答案 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吃东西时什么也不做。