独学プログラマーのコードを変えたい

Question

独学プログラマーの本に載っているコードの中で
WarGameというのがあります。

player1,2の名前を入力すると自動でトランプを選び、どっちの数が大きいか、大きかったら勝ちで勝った人の名前が表示されるのを繰り返し、最終的に勝った回数が多い人の名前が表示されます。

**
コードの中で疑問があります。**

名前を入力した後、

ゲームを始めます。終わるときはｑ、startはq以外を入力

と表示されます。

もし、いきなり終了を選択した場合、

ｹﾞｰﾑを終わります。【ひきわけが勝ちました。】

となります。
ｹﾞｰﾑを繰り返したあと、最後に表示される文章と同じですが、すぐ終了した場合は引き分けとなり、それ以外は勝った人の名前がはいります。
もしこれを一回目ｹﾞｰﾑを始める前に終了するときは
【ひきわけ。】で終わるにはどうしたらいいですか？

本には日本語で載っていますが、本で紹介されているサイトの英語版のコードです。

from random import shuffle


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):
        """suit + value are ints"""
        self.value = v
        self.suit = s

    def __lt__(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


class Deck:
    def __init__(self):
        self.cards = []
        for i in range(2, 15):
            for j in range(4):
                self.cards\
                    .append(Card(i,
                                 j))
        shuffle(self.cards)

    def rm_card(self):
        if len(self.cards) == 0:
            return
        return self.cards.pop()


class Player:
    def __init__(self, name):
        self.wins = 0
        self.card = None
        self.name = name


class Game:
    def __init__(self):
        name1 = input("p1 name ")
        name2 = input("p2 name ")
        self.deck = Deck()
        self.p1 = Player(name1)
        self.p2 = Player(name2)

    def wins(self, winner):
        w = "{} wins this round"
        w = w.format(winner)
        print(w)

    def draw(self, p1n, p1c, p2n, p2c):
        d = "{} drew {} {} drew {}"
        d = d.format(p1n,
                     p1c,
                     p2n,
                     p2c)
        print(d)

    def play_game(self):
        cards = self.deck.cards
        print("beginning War!")
        while len(cards) >= 2:
            m = "q to quit. Any " + \
                "key to play:"
            response = input(m)
            if response == 'q':
                break
            p1c = self.deck.rm_card()
            p2c = self.deck.rm_card()
            p1n = self.p1.name
            p2n = self.p2.name
            self.draw(p1n,
                      p1c,
                      p2n,
                      p2c)
            if p1c > p2c:
                self.p1.wins += 1
                self.wins(self.p1.name)
            else:
                self.p2.wins += 1
                self.wins(self.p2.name)

        win = self.winner(self.p1,
                         self.p2)
        print("War is over.{} wins"
              .format(win))

    def winner(self, p1, p2):
        if p1.wins > p2.wins:
            return p1.name
        if p1.wins < p2.wins:
            return p2.name
        return "It was a tie!"

game = Game()
game.play_game()

Answer 1

ゲームの終了判定をしている部分に着目して下さい。

if response == 'q':
break

ここで単にbreakするのではなく、
適当にメッセージを出力してから関数から脱出してやれば良いです。

teratailでのコードの書き方について

teratailには、コードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押して下さい。

特にPythonの場合、インデントが崩れるとコードの意味が変わってしまいます。

Comments

[NNA]

ありがとうございます!!!!!やってみます。
コードの書き方も指摘していただきありがとうございます。書き直してみました!!!!!!!!

[KojiDoi]

書き直ってないじゃん。

[dodox86]

いつも拝見して思いますが、LouiS0616さんの「teratailでのコードの書き方について」の動画gif付きの説明、teratail公式で使ってもらいたいですね。ヘルプやFAQの冒頭に。

[LouiS0616]

@dodox86 さん
お褒め頂き光栄です。
今のところ四種類用意してあるので、必要ならじゃんじゃん転用して貰って大丈夫です。

> teratail公式で使ってもらいたいですね。ヘルプやFAQの冒頭に。

以前似たような要望を出したのですが、特に変化は無いんですよね。

[dodox86]

> 今のところ四種類用意してあるので、必要ならじゃんじゃん転用して貰って大丈夫です。
！！何と！LouiS0616さん、どうもありがとうございます。適時、使わせていただこうと思います。よろしくお願いします。

[tanishi_a]

こういうときにコメント欄にもいいねボタンが欲しい！と思います。横から失礼しました

Answer 2

もともとのWar is over.It was a tie! winsというメッセージもいまいちなので、後ろの方を、

        
        print("War is over. "+win)

    def winner(self, p1, p2):
        if p1.wins > p2.wins:
            return p1.name + " wins."
        if p1.wins < p2.wins:
            return p2.name + " wins."
        return "It was a tie!"

ですかね。
少し考えれば、全然難しい話ではないです。

Comments

[NNA]

ありがとうございました!!!!!