Q&A
このソースをさらに簡潔にすることはできるのでしょうか？
lang
1<html> 2<head> 3<meta http-equiv="content-type" content="text/html;charset=utf-8"/> 4<title>jankenform</title> 5<style> 6img{border-style: none;} 7a{text-decoration: none;} 8</style> 9</head> 10<body> 11<div align="center"> 12<div><h1>じゃんけんゲーム</h1></div> 13<?php 14 //「グー、チョキ、パー」を配列で準備 15 $hand = array("グー","チョキ","パー"); 16?> 17<p>あなたが出す手を選んでください。</p> 18<?php 19 for($i=0;$i<3;$i++){ 20 echo"<a href='jankenresult_6.php?choice=".$i."'><img src='img/". $i .".png' height='100px'/></a>"; 21 } 22?> 23</div> 24</body> 25</html>
lang
1<?php 2 session_start(); 3?> 4<html> 5<head> 6<meta http-equiv="content-type" content="text/html;charset=utf-8"/> 7<title>jankenresult</title> 8<style> 9img{border-style: none;} 10a{text-decoration: none;} 11td{text-align: center;} 12</style> 13</head> 14<body> 15<div align="center"> 16<?php 17 if(!isset($_SESSION["userhand"])){ 18 $_SESSION["user_cnt"]=0; 19 $_SESSION["com_cnt"]=0; 20 $_SESSION["userhand"]=array(); 21 $_SESSION["comhand"]=array(); 22 $_SESSION["fight_u"]=array(); 23 $_SESSION["fight_c"]=array(); 24 } 25 //「グー、チョキ、パー」を配列で準備 26 $hand = array("グー","チョキ","パー"); 27 $judge = array("あいこ","勝ちです","負けです"); 28 29 //ユーザーの手を変数に格納 30 $userhand = $_GET["choice"]; 31 32 //コンピューターの手を0～2の数値でランダムに発生させ、変数に格納 33 $comhand = mt_rand(0,2); 34 $fight = $userhand - $comhand; 35 array_push($_SESSION["userhand"],$userhand); 36 array_push($_SESSION["comhand"],$comhand); 37 38 //じゃんけんの勝敗を表示 39 if($fight == 0){ 40 echo "<h1>あいこです！！</h1><br/>"; 41 array_push($_SESSION["fight_u"],0); 42 array_push($_SESSION["fight_c"],0); 43 }else if($fight == 2 || $fight== -1){ 44 45 ++$_SESSION["user_cnt"]; 46 echo "<h1>勝ちです！！</h1><br/>"; 47 array_push($_SESSION["fight_u"],1); 48 array_push($_SESSION["fight_c"],2); 49 }else{ 50 ++$_SESSION["com_cnt"]; 51 echo "<h1>負けです！！</h1><br/>"; 52 array_push($_SESSION["fight_u"],2); 53 array_push($_SESSION["fight_c"],1); 54 } 55 echo "あなたは <img src='img/". $userhand .".png' height='100px'/> をだしました。 "; 56 echo "コンピューターは <img src='img/". $comhand .".png' height='100px'/> をだしました。<br/>"; 57 58 $pub = ""; 59 $pub .= "<br/><table><tr><th style='text-align:left;'>あなたの手：</th>"; 60 foreach ($_SESSION["userhand"] as $u){ 61 $pub .= "<td><img src='img/". $u .".png' height='30px'/></td>"; 62 } 63 $pub .= "</tr><tr><th style='text-align:left;'>コンピューターの手：</th>"; 64 foreach ($_SESSION["comhand"] as $c){ 65 $pub .= "<td><img src='img/". $c .".png' height='30px'/></td>"; 66 } 67 68 //５回先に勝ったときの表示、また５回未満のときの表示 69 if($_SESSION["user_cnt"]==5){ 70 echo "<p><font size='20px'>あなたの勝ちです。<br/>☆☆☆☆☆<br/><br/>勝率 : ". (int)(5/sizeof($_SESSION["userhand"]) * 100) ."%</font></p><br/><a href=\"jankenform_6.php\">もう一度じゃんけんする！</a>"; 71 echo $pub; 72 echo "</tr><tr><th style='text-align:left;'>あなたの勝敗：</th>"; 73 foreach ($_SESSION["fight_u"] as $f_u){ 74 echo "<td>". $judge[$f_u] ."</td>"; 75 } 76 echo "</tr></table>"; 77 $_SESSION = array(); 78 }else if($_SESSION["com_cnt"]==5){ 79 echo"<p><font size='20px'>コンピューターの勝ちです。<br/>★★★★★<br/><br/>勝率 : ". (int)(5/sizeof($_SESSION["comhand"]) * 100) ."%</font></p><br/><a href=\"jankenform_6.php\">もう一度じゃんけんする！</a>"; 80 echo $pub; 81 echo "</tr><tr><th style='text-align:left;'>コンピューターの勝敗：</th>"; 82 foreach ($_SESSION["fight_c"] as $f_c){ 83 echo "<td>". $judge[$f_c] ."</td>"; 84 } 85 echo "</tr></table>"; 86 $_SESSION = array(); 87 }else { 88 echo"あなたの勝ち数："; 89 for($i=0;$i<$_SESSION["user_cnt"];$i++){ 90 echo "☆"; 91 } 92 echo"<br/>"; 93 echo"コンピューターの勝ち数："; 94 for($i=0;$i<$_SESSION["com_cnt"];$i++){ 95 echo "★"; 96 } 97 echo"<br/>"; 98 echo"<p>あなたが出す手を選んでください。</p>"; 99 for($i=0;$i<3;$i++){ 100 echo"<a href='jankenresult_6.php?choice=".$i."'><img src='img/". $i .".png' height='100px'/></a>"; 101 } 102 } 103?> 104</div> 105</body> 106</html>
回答2件
0
簡潔にする前段階として
__HTML__部分をできるだけPHPの中に入れないような作りにすることをオススメします。
HTMLがコード内で重複している部分がなくなります。
lang
1if ($_SESSION["user_cnt"] == 5) { 2 $winner = "あなた"; 3} 4... 5<p><font size='20px'><?=$winner?>の勝ちです。<br /> 6... 7</body> 8</html>
また簡潔に__コードの読みやすさ__も含めるとすると__array()を[]__にするなど新しめの書き方を覚えるのはどうでしょう？（参考書が古いとarray()となっているかもしれません）
lang
1$_SESSION["userhand"] = [];
投稿2015/04/29 15:54
総合スコア51
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。