クラスのプロパティは属性を制限しません

Question

私はクラスを作って、その属性hpは常に0とmaxhpの間にとどまっています 理論では、hpのプロパティは私に希望の結果を与える必要があります。

属性をリンクする方法はありますか？だから私はユニットクラスオブジェクトの位置を格納しています。 2つの場所では、[x、y]配列を含む属性位置ともう1つの属性属性x、yに格納され、それぞれにintが格納されます。 self.xまたはself.yを変更すると、self.positionが変更され、逆も同様です。あなたの__init__で

class units(object): 

    def __init__(self,typus, position, stats): 
     self.type = typus 

     #they should be linked both directions 
     self.position = position 
     self.x = self.position[0] 
     self.y = self.position[1] 

     self.attack = stats[0] 
     self.defense = stats[1] 
     self.maxhp = stats[2] 
     self.hp = self.maxhp 

    def __repr__(self): 
     text = "This a %s at position [%s,%s].\n Attack: %s \n Defense: %s \n Hp : %s/%s \n " \ 
       % (self.type,self.position[0],self.position[1], self.attack, self.defense, self.hp, self.maxhp) 
     return text 


    # hp set to always be in between 0 and maxhp 
    @property 
    def hp(self): 
     return self.__hp 

    @hp.setter 
    def hp(self, hp): 
     if hp < 0: 
      self.__hp = 0 
     if hp > self.maxhp: 
      self.__hp = self.maxhp 
     else: 
      self.__hp = hp 

    def takedmg(self,dmg): 
     self.hp -= max(dmg-self.defense, 0) 
     if self.hp <= 0: 
      self.alive = False 
     return self.hp 



p = units("peasant", [1,1], [2,0,30]) 
p.takedmg(100) 
print (p.hp)  # it should be 0! 

Answer 1

、ラインself.hp = self.maxhpはself.__hp = self.maxhpでなければなりません。そうすれば、それは@propertyの方法でのみ設定/取得されます。

hpの場合と同じ方法で、postion,x、yを処理します。 getterおよびsetterの値に対応するには、_postion,_xおよび_yを内部的に使用します。それぞれのセッターに_propの値をすべて設定します。例としてpositionを使用して：私は思うものの、xとyのための同様

@property 
def position(self): 
    return self._position 

@position.setter 
def position(self, position): 
    self._position = position # do checking before this if needed 
    self._x = position[0] 
    self._y = position[1] 

あなたはposition経由でそれを行うのみ必要があり、次のように

@property 
def x(self): 
    return self._x 

@x.setter 
def x(self, x): 
    self._x = x 
    # self._y remains unchanged 
    self._position[0] = x 

ところで、hpセッターを再書き込むことができます：

@hp.setter 
def hp(self, hp): 
    self.__hp = max(0, min(hp, self.maxhp)) 

Answer 2

もう1つの問題はhp.setterです。

@hp.setter 
def hp(self, hp): 
    if hp < 0: 
     self.__hp = 0 
    elif hp > self.maxhp: 
     self.__hp = self.maxhp 
    else: 
     self.__hp = hp