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

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

ただいまの
回答率

87.49%

phpで簡易電卓を作成中です。Notice: Undefined variable: answer とエラーが出ます。

受付中

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,105

score 2

前提・実現したいこと

phpで簡易電卓を作成しています。

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

$num2に0を入力した際、”分母計算できません”と表示をさせていますが、それと同時に
Notice: Undefined variable: answer in /home/www/satou179/calculate.php on line 27
と表示されます。
こちらのエラーを解決したいです。

該当のソースコード

<?php
$num1 = $_POST['number1'];
$num2 = $_POST['number2'];
$ope = $_POST['selope'];

switch($ope){
case"+":
$answer = $num1+$num2;
break;
case"-":
$answer = $num1-$num2;
break;
case"×":
$answer = $num1*$num2;
break;
case"÷":
if ($num2 == 0) {
echo  "母数が0なので計算できません";
} else {
$answer = $num1/$num2;
}
break;
}

if (is_numeric($num1) && is_numeric($num2)) {
echo $answer;
} else {
echo '数字以外が含まれています';
}
?>

<!doctype html>
<html lang="ja">
<head>
<meta charset="utf-8">
</head>
<body>
<form name="form1" action="calculate.php" method="post">
<p>
<input type="text" name="number1">
<select name="selope" size=1>
<option value="+">+</option>
<option value="-">-</option>
<option value="×">×</option>
<option value="÷">÷</option>
</select>
<input type="text" name="number2">
<input type="submit" value="=">
</p>
</form>
</body>
</html>

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

+1

過去質問にありますよ。
 [電卓 tag:PHP] の検索結果: 22件 

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/11 16:59

    ご回答、ありがとうございます。
    問題は解決しましたが、新たな問題が発生しました。
    前回答のコメント欄に追加の質問をいたしました。

    キャンセル

  • 2019/08/11 17:02

    追加質問している段階では解決とは言えません。
    一度解決済みを解除し、要件を質問本文を編集して追加するか、もう少し自身で考えた上で新しく質問をたててください。

    ※なお必ず私が提示した過去質問は参考にしてください。

    キャンセル

  • 2019/08/11 18:35

    承知いたしました。

    キャンセル

+1

echo $num1 + $num2; なので足し算限定になっていますよね。

$_POST['演算'] の内容でそこを変えれば良いと思います。 
※ '演算' というMultibyte文字をKeyに使うのはちょっと不要な問題を発生させるので、<select name="op"> とかにして $_POST['op'] として使うほうが無難だとは思いますが...

例えば、以下みたいなイメージでしょうか。 

if (is_numeric($num1) && is_numeric($num2)) {
    $op = $_POST['演算'];
    if ($op === '+') {
        echo $num1 + $num2;
    } else if ($op === '-') {
        echo $num2 - $num1;
    } else if ...
} else {
    echo '数字以外が含まれています';
}

※ こういうケースでPHPだと if-else をつなげるより switch-case を使うほうが美しいかもしれないです

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/10 18:04

    はい、あくまで奥の手(というか悪手)ですね。

    キャンセル

  • 2019/08/11 16:58

    ご回答、ありがとうございます。
    問題は解説しましたが、新たな問題が発生しました。
    $num2に0が入力された場合、”母数が0なので計算できません”と表示したいのですが、エラーになってしまいます。

    ソースコード↓

    <?php
    $num1 = $_POST['number1'];
    $num2 = $_POST['number2'];
    $ope = $_POST['selope'];


    switch($ope){
    case"+":
    $answer = $num1+$num2;
    break;
    case"-":
    $answer = $num1-$num2;
    break;
    case"×":
    $answer = $num1*$num2;
    break;
    case"÷":
    $answer = $num1/$num2;
    break;
    }

    if (is_numeric($num1) && is_numeric($num2)) {
    echo $answer;
    } elseif ($ope == "/") {
    if ($num2 == 0) $answer = "母数が0なので計算できません";
    else $answer = $num1 / $num2;
    } else {
    echo '数字以外が含まれています';
    }
    ?>

    <!doctype html>
    <html lang="ja">
    <head>
    <meta charset="utf-8">
    </head>
    <body>
    <form name="form1" action="calculate.php" method="post">
    <p>
    <input type="text" name="number1">
    <select name="selope" size=1>
    <option value="+">+</option>
    <option value="-">-</option>
    <option value="×">×</option>
    <option value="÷">÷</option>
    </select>
    <input type="text" name="number2">
    <input type="submit" value="=">
    </p>
    </form>
    </body>
    </html>

    キャンセル

  • 2019/08/11 17:09

    その
    if ($num2 == 0) $answer = "母数が0なので計算できません";
    を入れるなら、
    case"÷":
    のところでしょうね。(そこのブロックで既に 0 除算が発生しているので)。

    キャンセル

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

  • ただいまの回答率 87.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る