python if文のreturnについて

Question

書籍でpythonを勉強しているのですはが、トランプゲームの戦争について書かれており
そのカードを表すCardクラスのところで大小比較のところで一部分疑問に思ったので質問しました。

Cardクラスのソースは以下です。

class Card:
    suits = ["spades","hearts","diamonds","clubs"]
    values = [None,None,
              "2","3","4","5","6","7","8","9","10",
              "Jack","Queen","King","Ace"]

    def __init__(self,v,s):
        self.value = v
        self.suit = s

    def __it__(self,c2):
        if self.value < c2.value:
            return True

        if self.value == c2.value:
            if self.suit < c2.suit:
                return True
            else:
                return False
        return False#ここ！

    def __gt__(self,c2):
        if self.value > c2.value:
            return True
        
        if self.value == c2.value:
            if self.suit > c2.suit:
                return True
            else:
                return False
        return False#ここ！

    def __repr__(self):
        v = self.values[self.value] + "of" + self.suits[self.suit]
        return v


card1 = Card(10,2)
card2 = Card(11,3)

print(card1 > card2)

「ここ！」とコメントしている行なのですが、これは

【def it(self,c2)の場合】
self.value < c2.valueかつself.value == c2.valueでない
（つまりself.value > c2.value）のときFalse

で

【def gt(self,c2)の場合】
self.value > c2.valueかつself.value == c2.valueでない（
つまりself.value < c2.value）のときFalse

という意味でしょうか？

Answer 1

それであってます。

__it__というのは名前が意味不明なので、__lt__の間違いでは？

Comments

[gontaros0302]

伸びシロですね。。

Answer 2

次の二つの関数の動作は同じです。

def func(...):
    ...

    if 条件1:
        return AAA
    if 条件2:
        return BBB
   
    return CCC

def func(...):
    ...

    if 条件1:
        return AAA
    elif 条件2:
        return BBB
    else:
        return CCC

これは...という意味でしょうか？

おそらくその理解で正しいです。
『AかつBでない』というのが、『Aではなく、かつBでもない』という意味ならば。

---

なお、__it__は__lt__のタイポかと思います。
それぞれ less than と greater than の略ですので。

Comments

[gontaros0302]

『Aではなく、かつBでもない』という意味でした！
完全に__it__だと思い込んでいました。。。

特殊メソッドも勉強しなきゃですね！
ありがとうございます????

Answer 3

        if self.value > c2.value:
            return True

        if self.value == c2.value:
            if self.suit > c2.suit:
                return True
            else:
                return False
        return False#ここ！

は、次のほうが良いかもしれません。

        if self.value > c2.value:
            return True

        if self.value == c2.value:
            if self.suit > c2.suit:
                return True
        return False#ここ！

まずは数字で比較する、数字が同じ時はスーツ(種類)で比較する。

True を返す条件を　or や and でつなげて
return True と Return False が それぞれ 1 箇所だけにすることも可能ですね。

数字 * 100 +スーツの数　でカードを点数化すれば、その点数の比較をするだけで済ますこともできそうです。

質問文にあるコードの最適化を進めていくと

if (self.value > c2.value) or (self.value == c2.value and self.suit > c2.suit):
  return True
return False

となります。(reutrn True か return False でメソッドが終了する)

でも つぎのように

if self.value == c2.value:
  return self.value > c2.value
return self.suit > c2.suit

と、return スーツの比較結果 か return 数字の比較結果 でメソッドが終了するのも良いかもしれません。

Comments

[gontaros0302]

数字 * 100 +スーツの数　でカードを点数化すれば、その点数の比較をするだけで済ますこともできそうです。

なるほど！確かにそうですね！
それも試してみます。ありがとうございます????

[katoy]

点数化せず、カードの強さ順に配列にならべて、カードの順位を調べるの良いかもしれません。
配列 は ["C1", "C2", ...."S10", "SJ", "SQ", "SK"] のようにしておけば、
index("C1") は 0, index("C2") は 1 が得られます。

[LouiS0616]

次のように書いた方が、尚良いような。
if self.value == c2.value:
__return self.suit > c2.suit
return False