Python 3.14中Mazo类eliminaCarta方法删除错误元素问题求助

嘿，我找到你代码里的问题了！之所以会删掉第一个元素而不是目标牌，根源在于Carta类的比较方法实现不符合Python3的规范，再加上一个小笔误，导致Python无法正确识别两张牌是否相等。

具体问题拆解 #

1. __eq__方法返回值错误 #

Python3要求__eq__方法必须返回布尔值（True/False），但你现在返回的是1或-1：

def __eq__(self, other):
    if self.valor == other.valor:
        return 1
    else:
        return -1

这会让Python在判断carta in self.cartas时逻辑混乱，错误地把第一个元素判定为和你传入的目标牌“相等”，所以remove()就删错了对象。而且这个方法只比较了牌的数值，没比较花色，这本身就不符合扑克牌的相等逻辑啊！

2. __lt__和__gt__方法返回值错误 #

同样，Python3的<和>运算符对应的魔法方法必须返回布尔值，但你返回的是整数1/-1。虽然这不是直接导致删除错误的原因，但后续如果做牌的排序、比较操作肯定会出问题。

3. Mazo类的__contains__方法笔误 #

你写的return item in self.items，但Mazo类里根本没有items属性，应该是self.cartas。不过这个笔误暂时没影响到你，因为列表的in操作默认会用元素的__eq__方法，但修正它能让代码更规范。

修复后的完整代码 #

修正Carta类的比较逻辑 #

import random
class Carta:
    listaDePalos = ["Tréboles", "Diamantes", "Corazones", "Picas"]
    listaDeValores = ["nada", "As", "2", "3", "4", "5", "6", "7", "8", "9", "10", "Sota", "Reina", "Rey"]
    def __init__(self, palo=0, valor=0):
        self.palo = palo
        self.valor = valor
    def __str__(self):
        return f"{self.listaDeValores[self.valor]} de {self.listaDePalos[self.palo]}"
    
    def __lt__(self, other):
        # 先比花色，花色不同直接按大小判断
        if self.palo != other.palo:
            return self.palo < other.palo
        # 花色相同，处理A的特殊规则（A比其他牌大）
        if self.valor == 1:
            return False  # A不会小于任何牌（除了另一个A）
        if other.valor == 1:
            return True   # 非A牌肯定小于A
        # 最后比数值
        return self.valor < other.valor
    
    def __gt__(self, other):
        # 先比花色
        if self.palo != other.palo:
            return self.palo > other.palo
        # 处理A的特殊规则
        if self.valor == 1:
            return True    # A大于任何非A牌
        if other.valor == 1:
            return False   # 非A牌不会大于A
        # 最后比数值
        return self.valor > other.valor
    
    def __eq__(self, other):
        # 只有花色和数值都相同，才是同一张牌
        return self.palo == other.palo and self.valor == other.valor

修正Mazo类的__contains__方法 #

class Mazo:
    def __init__(self):
        self.cartas = []
        for palo in range(4):
            for valor in range(1, 14):
                self.cartas.append(Carta(palo, valor))
    def muestraMazo(self):
        for carta in self.cartas:
            print(carta)
    def __str__(self):
        s = ""
        for i in range(len(self.cartas)):
            s = s + " "*i + str(self.cartas[i]) + "\n"
        return s
    def mezclar(self):
        nCartas = len(self.cartas)
        for i in range(nCartas):
            j = random.randrange(i, nCartas)
            self.cartas[i], self.cartas[j] = self.cartas[j], self.cartas[i]
    def __contains__(self, item):
        return item in self.cartas  # 修正为正确的属性名
    def eliminaCarta(self, carta):
        if carta in self.cartas:
            self.cartas.remove(carta)
            return 1
        else:
            return 0

测试修复效果 #

现在运行你的测试代码：

carta = Carta(1,13)  # 对应Rey de Diamantes
mazo = Mazo()
mazo.eliminaCarta(carta)
print(mazo)

你会看到Rey de Diamantes被正确从牌组中删除，而不是第一个元素啦！

额外提醒 #

• Python3和Python2在魔法方法的返回值要求上有区别，Python2允许比较方法返回整数，但Python3必须返回布尔值，这是你适配时踩的最大坑。
• 以后写__eq__方法时，要确保逻辑符合实际需求——扑克牌的相等应该是花色和数值都一致，而不是只看数值哦。

内容的提问来源于stack exchange，提问作者Matias Alvarez