PHPのif文が反映されません。

前提・実現したいこと

初めまして、deresukeと申します。プログラミングは超がつく初心者です。
PHPを使って簡単なカードゲームを作成しておりました。
トランプを2枚ランダムに引いて、Aと(10,J,Q,K)の組み合わせが出たときのみ
ブラックジャックです!というメッセージが出るように
以下のコードで作成いたしました。

発生している問題・エラーメッセージ

何故かA,10か10,Aの組み合わせの時にしか
メッセージが表示されません。
elseif分ばかりで見にくく申し訳ありません
問題点をご指摘いただければと思います。

該当のソースコード

<?php $card1 = array("A",2,3,4,5,6,7,8,9,10,"J","Q","K"); $card2 = array("A",2,3,4,5,6,7,8,9,10,"J","Q","K"); $deal1 = array_rand($card1,1); $deal2 = array_rand($card2,1); ?> <!DOCTYPE html> <html> <head> <meta charset="utf-8"> <title>CASINO</title> <link rel="stylesheet" type="text/css" href="style.css"> </head> <body> <h1>ブラックジャックゲーム</h1> <p>あなたのカード</p> <?php echo $card1[$deal1]; ?> <?php echo $card2[$deal2]; ?>

<?php if($deal1 == "A" && $deal2 == 10) :?>
<?php echo "ブラックジャックです"; ?>
<?php elseif($deal1 == "A" && $deal2 ==  "J") :?>
<?php echo "ブラックジャックです"; ?>
<?php elseif($deal1 == "A" && $deal2 ==  "Q") :?>
<?php echo "ブラックジャックです"; ?>
<?php elseif($deal1 == "A" && $deal2 ==  "K") :?>
<?php echo "ブラックジャックです"; ?>
<?php elseif($deal1 == 10 && $deal2 ==  "A") :?>
<?php echo "ブラックジャックです"; ?>
<?php elseif($deal1 == "J" && $deal2 ==  "A") :?>
<?php echo "ブラックジャックです"; ?>
<?php elseif($deal1 == "Q" && $deal2 ==  "A") :?>
<?php echo "ブラックジャックです"; ?>
<?php elseif($deal1 == "K" && $deal2 ==  "A") :?>
<?php echo "ブラックジャックです"; ?>
  <?php endif ?>

</body> </head> </html>

補足情報（FW/ツールのバージョンなど）

テキストデータはatomを使用しております。

m.ts10806

2019/10/14 06:02

コードはマークダウンのcode機能にてご提示ください(質問は編集できます)

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

回答3件

ベストアンサー

PHP
1<?php if($deal1 == "A" && $deal2 == 10) :?>
2↓
3<?php if($card1[$deal1] == "A" && $card2[$deal2] == 10) :?>

もしくは

PHP
1<?php if($deal1 == "A" && $deal2 == 10) :?>
2↓
3<?php if($deal1 == 0 && $deal2 == 9) :?>//A&10

としなければいけません

投稿2019/10/14 06:28

jinba

総合スコア310

退会済みユーザー

2019/10/14 06:38

おっしゃるとおり。わたしの回答は間違い。

deresuke

2019/10/14 07:07

jinba様ご指摘いただいたやり方で解決しました。ありがとうございます!

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

本件の原因は以下にあります。
PHPの厳密な緩やかな比較

普段ならマニュアルのリンク貼るのですが、今日は特別仕様ですｗ

投稿2019/10/14 06:32

退会済みユーザー

総合スコア0

退会済みユーザー

2019/10/14 06:40

A だけうまくさばけるんでしょ？

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

数字と文字列が配列の中で混在しているのがしっくりこないので、
わたしならすべて文字列として扱うことを徹底する。

そして、jinbaさんの回答にあるように
PHP: array_rand - Manual

array_rand — 配列から一つ以上のキーをランダムに取得する

キーを取得するのだから、カードがほしければ改めて$card1の値を取得しないといけないので、

php
1<?php
2$card1 = array("A","2","3","4","5","6","7","8","9","10","J","Q","K");
3$card2 = array("A","2","3","4","5","6","7","8","9","10","J","Q","K");
4$deal1 = $card1[array_rand($card1,1)];
5$deal2 = $card2[array_rand($card2,1)];
6?>

とする必要がある。と。

ここから下のわたしの回答は、勘違いに基づく、間違った回答。無視してください。

php
1<?php
2$card1 = array("A","2","3","4","5","6","7","8","9","10","J","Q","K");
3$card2 = array("A","2","3","4","5","6","7","8","9","10","J","Q","K");
4$deal1 = array_rand($card1,1);
5$deal2 = array_rand($card2,1);
6
7?>
8
9<!DOCTYPE html>
10<html>
11<head>
12<meta charset="utf-8">
13<title>CASINO</title>
14<link rel="stylesheet" type="text/css" href="style.css">
15</head>
16
17<body>
18<h1>ブラックジャックゲーム</h1>
19<p>あなたのカード</p>
20<?php echo $card1[$deal1]; ?>
21<?php echo $card2[$deal2]; ?>
22
23<?php 
24    if ($deal1 == "A" and $deal2 == "10") :
25        echo "ブラックジャックです";
26    elseif ($deal1 == "A" and $deal2 ==  "J") :
27        echo "ブラックジャックです";
28    elseif ($deal1 == "A" and $deal2 ==  "Q") :
29        echo "ブラックジャックです";
30    elseif ($deal1 == "A" and $deal2 ==  "K") :
31        echo "ブラックジャックです";
32    elseif ($deal1 == "10" and $deal2 ==  "A") :
33        echo "ブラックジャックです"; 
34    elseif ($deal1 == "J" and $deal2 ==  "A") :
35        echo "ブラックジャックです"; 
36    elseif ($deal1 == "Q" and $deal2 ==  "A") :
37        echo "ブラックジャックです"; 
38    elseif ($deal1 == "K" and $deal2 ==  "A") :
39        echo "ブラックジャックです"; 
40    endif;
41?>
42
43
44</body>
45</head>
46</html> 
47

数字のカードもちゃんと文字列として扱えば問題ないのではないかと。

php
1<?php 
2    if (($deal1 == "A") && ($deal2 == "10")) :
3        echo "ブラックジャックです";
4    elseif (($deal1 == "A") && ($deal2 == "J")) :
5        echo "ブラックジャックです";
6    elseif (($deal1 == "A") && ($deal2 == "Q")) :
7        echo "ブラックジャックです";
8    elseif (($deal1 == "A") && ($deal2 == "K")) :
9        echo "ブラックジャックです";
10    elseif (($deal1 == "10") && ($deal2 == "A")) :
11        echo "ブラックジャックです"; 
12    elseif (($deal1 == "J") && ($deal2 == "A")) :
13        echo "ブラックジャックです"; 
14    elseif (($deal1 == "Q") && ($deal2 == "A")) :
15        echo "ブラックジャックです"; 
16    elseif (($deal1 == "K") && ($deal2 == "A")) :
17        echo "ブラックジャックです"; 
18    endif;
19?>
20