じゃんけんゲームのデバックをしています。

やりたい事

数字を手にしたい
引き分け以外の勝敗を表示させたい。

引き分け時

現状出した手が数字になってしまいます。
引き分けの時は、勝敗は引き分けです。となります

それ以外

でも、それ以外の時は出力されません。

試した事

ジャンケンの手に変換される前に、$me->getChoice()と、$enemy->getChoice()を呼び出しているのが原因かと思いました。
Battle　クラス内が原因だと思いますので、色々試しています。
分かる方居ましたらお願い致します。

コード

<?php
// デバック練習問題
// あらかじめ壊れているプログラムを用意してあります。
// コードを読みデバックしつつジャンケンゲームを完成させてください。
// 判定が勝った時のみ勝利回数を表示させてください。
// 例)
// 山田太郎はグーを出しました。
// 相手はチョキを出しました。
// 勝敗は勝ちです。
// 3回目の勝利です。


if (!isset($_SESSION['result'])) {
    $_SESSION['result'] = 0;
}

class Player
{
    public function jankenConverter(int $choice): string
    {
        $janken = '';
        switch ($choice) {
        case 1:
                $janken = 'グー';
            break;
        case 2:
                $janken = 'チョキ';
        case 3:
                $janken = 'パー';
            break;
        default:
            break;
        }
        return $janken;
    }
    
}

class Me
{
    private $name;
    private $choice;

    public function __construct(string $lastName, string $firstName, int $choice)
    {
        $this->name   = $lastName.$firstName;
        $this->choice = $choice;
    }

    public function getName(): string
    {
        return $this->name;
    }

    public function getChoice(): string
    {
        return $this->choice;
    }
}

class Enemy
{
    private $choice;
    public function __construct()
    {
        $this->choice = random_int(1, 3);
    }

    public function getChoice(): string
    {
        return $this->choice;
    }
}

class Battle
{
    public $first;
    public $second;
    public function __construct(Me $me, Enemy $enemy)
    {
        $this->first  = $me->getChoice();
        $this->second = $enemy->getChoice();
    }

    public function judge()
    {
        if ($this->first === $this->second) {
            return '引き分け';
        }

        if ($this->first === 'グー' && $this->second === 'チョキ') {
            return '勝ち';
        }

        if ($this->first === 'グー' && $this->second === 'パー') {
            return '負け';
        }

        if ($this->first === 'チョキ' && $this->second === 'グー') {
            return '負け';
        }

        if ($this->first === 'チョキ' && $this->second === 'パー') {
            return '勝ち';
        }

        if ($this->first === 'パー' && $this->second === 'グー') {
            return '勝ち';
        }

        if ($this->first === 'パー' && $this->second === 'チョキ') {
            return '負け';
        }
    }

    public function countVictories()
    {
        if ($this->judge() === '勝ち') {
            $_SESSION['result'] = 1;
        }
    }

    public function getVitories()
    {
        return $_SESSION['result'];
    }

    public function showResult()
    {
        return $this->judge();
    }
}

if (!empty($_POST)) {
    $me    = new Me($_POST['last_name'], $_POST['first_name'], $_POST['choice'], $_POST['choice']);
    $enemy = new Enemy();
    echo $me->getName().'は'.$me->getChoice().'を出しました。';
    echo '<br>';
    echo '相手は'.$enemy->getChoice().'を出しました。';
    echo '<br>';
    $battle = new Battle($me, $enemy);
    echo '勝敗は'.$battle->showResult().'です。';
    if ($battle->showResult() === '勝ち') {
        echo '<br>';
        echo $battle->getVitories().'回目の勝利です。';
    }
}

?>
<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="utf-8">
<title>デバック練習</title>
</head>
<body>
    <section>
    <form action='./debug03.php' method="post">
        <label>姓</label>
        <input type="text" name="last_name" value="<?php echo '山田' ?>" />
        <label>名</label>
        <input type="text" name="first_name" value="<?php echo '太郎' ?>" />
        <select name='choice'>
            <option value=0 >--</option>
            <option value=1 >グー</option>
            <option value=2 >チョキ</option>
            <option value=3 >パー</option>
        </select>
        <input type="submit" value="送信する"/>
    </form>
    </section>
</body>
</html>

行動規範の内容に同意します

回答1件

ベストアンサー

https://teratail.com/questions/8xin4x3bclfa58

以前の質問で指摘されていますが、修正方法が間違っています。

元のコードに戻して、class Me と class Enemy で class Player を継承してください。

php
1<?php
2...
3
4class Me extends Player
5{
6...
7}
8
9class Enemy extends Player
10{
11...
12}

投稿2022/06/30 10:52

arcxor

総合スコア2859

RiaFeed

2022/06/30 15:20 編集

まだいくつかありますが都度都度聞いて丸投げするつもりなのかなぁ…

行動規範の内容に同意します

あなたの回答

tips

プレビュー

行動規範の内容に同意します

質問の解決につながる回答をしましょう。サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。また、読む側のことを考えた、分かりやすい文章を心がけましょう。

15分調べてもわからないことは
teratailで質問しよう！

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう！

じゃんけんゲームデバック時、出した手が数字になってしまう。

やりたい事

引き分け時

それ以外

試した事

コード

関連した質問