Python面向对象：通过方法实现对象间属性修改的正确实践

class Character:
    def __init__(self, health, power, char_range, position):
        self.health = health
        self.power = power
        self.range = char_range # 'range'是Python内置函数，建议避免直接使用
        self.position = position

    def get_health(self):
        return self.health

    def set_health(self, value):
        # 这里的实现存在冗余和逻辑问题
        self.health = self.get_health() # 这一行是多余的，相当于 self.health = self.health
        self.health -= value # 实际的扣血逻辑

    def punch(self, value):
        # 这个方法修改的是传入的局部变量'value'，而不是目标对象的health
        value -= self.power
        return value

# 初始使用示例
# from character import Character # 假设Character类在单独的文件中

char1 = Character(100, 25, 5, 3)
char2 = Character(100, 20, 3, 4)

# 尝试让char1攻击char2
char2.health = char1.punch(char2.get_health())

print(char2.get_health())

class Character:
    def __init__(self, health, power, char_range, position):
        self.health = health
        self.power = power
        # 避免使用Python内置关键字 'range' 作为变量名
        self.range = char_range
        self.position = position

    def get_health(self):
        """获取当前角色的生命值。"""
        return self.health

    def set_health(self, value):
        """
        设置或更新当前角色的生命值。
        这个方法应该直接接收新的生命值，或生命值的变化量。
        这里我们设计为直接设置生命值。
        """
        self.health = value

    def take_damage(self, damage_amount):
        """
        角色受到伤害。
        这是一个更符合封装原则的方法，由角色自身处理伤害。
        """
        self.health -= damage_amount
        if self.health < 0:
            self.health = 0 # 确保生命值不为负
        print(f"{self.__class__.__name__}在{self.position}受到{damage_amount}点伤害，当前生命值：{self.health}")

    def punch(self, target_character):
        """
        当前角色攻击另一个角色。
        直接操作目标角色的方法来改变其状态。
        """
        if not isinstance(target_character, Character):
            print("攻击目标必须是Character类型。")
            return

        damage_dealt = self.power
        # 调用目标角色的方法来处理伤害，而不是直接修改其属性
        target_character.take_damage(damage_dealt)
        print(f"{self.__class__.__name__}在{self.position}攻击了{target_character.__class__.__name__}在{target_character.position}，造成{damage_dealt}点伤害。")
        return damage_dealt # 可以选择返回造成的伤害值

# 优化后的使用示例
char1 = Character(100, 25, 5, 3)
char2 = Character(100, 20, 3, 4)

print(f"初始状态：char1生命值={char1.get_health()}，char2生命值={char2.get_health()}")

# char1攻击char2
char1.punch(char2)

print(f"攻击后：char1生命值={char1.get_health()}，char2生命值={char2.get_health()}")

# 再次攻击
char1.punch(char2)
print(f"再次攻击后：char2生命值={char2.get_health()}")