PHP　計算プログラムのコードについて

前提・実現したいこと

form.phpで入力し、calculate.phpで処理し、エラーがあれば表示、あるいは計算結果を表示させます。

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

処理自体は行えているのですが、calculate.phpの８行目と２３行目に割る数が０でないかの同じチェックをしています。
８行目の方はエラーメッセージを表示するため、２３行目の方は割る数が０の場合を除いて、割り算処理を行います。
同じような内容を繰り返しで行っているので、先にエラーチェックをして、よりシンプルに処理する方法がないか考えているのですが。思い浮かびません。何かアドバイスいただけませんでしょうか？よろしくお願いします。

エラーメッセージ

該当のソースコード

"PHP"
(form.php)

<!DOCTYPE html> <html lang="ja"> <head> <meta charset="UTF-8"> <title>入力ページ</title> </head> <body> <h3>入力フォーム</h3> <form method="post" action="calculate.php"> <label>入力欄1:</label><input type="text" name="num1"><br> <select name="calcu"> <option value="plus">+</option> <option value="minus">-</option> <option value="times">×</option> <option value="divide">÷</option> </select><br> <label>入力欄2:</label><input type="text" name="num2"><br> <input type="submit" value="計算する"> </form> </body> </html>

(calculate.php)

<?php $num1 = $_POST['num1']; $num2 = $_POST['num2']; $calcu = $_POST['calcu']; if (!is_numeric($num1) || !is_numeric($num2)) { $message1 = "数字以外が含まれているため、計算できません。"; } if ($calcu === "divide" && $num2 == 0) { $message2 = "割る数が0のため、計算できません。"; } if (is_numeric($num1) && is_numeric($num2)) { switch ($calcu) { case "plus": $answer = $num1 + $num2; break; case "minus": $answer = $num1 - $num2; break; case "times": $answer = $num1 * $num2; break; case "divide": if (!$num2 == 0) { $answer = $num1 / $num2; } break; } } ?> <!DOCTYPE html> <html lang="ja"> <head> <meta charset="UTF-8"> <title>計算結果表示</title> </head> <body> <div> <?php if (isset($message1)) { echo $message1 . "<br>"; } if (isset($message2)) { echo $message2; } if (isset($answer)) { echo "計算結果: " . $answer; } ?> </div> </body> </html>

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

m.ts10806

2019/12/17 01:29

コードはマークダウンのcode機能を利用してご提示ください。 https://teratail.com/help#about-markdown

m.ts10806

2019/12/17 01:32

また現在のコードでも「先にエラーチェックをして」の部分は実装されていると思いますが、主にどういう方向性を目指したいのでしょうか。チェックの内容も違うのでそこまで同じようなことを繰り返し書いているような印象はありませんが

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

回答1件

エラーハンドリングのメソッドを作成し、計算箇所の処理ではエラーメッセージが存在していなければ計算をするような処理に変更すればswith文の中でif文書く必要が無くなるのではないでしょうか？

動作確認はしてないですがこんな感じに関数をつくってみました。

PHP
1function errorCheck($left_num,$right_num,$culc=null)
2{
3    $message = array();
4    
5    if(!is_numeric($left_num) || !is_numeric($right_num))
6    {
7        $message[] = "数字以外が含まれているため、計算できません。";
8    }
9
10    if(!is_null($culc))
11    {
12        switch($culc)
13        {
14            case "divide":
15                if(!$right_num)
16                {
17                    $message[] = "割る数が0のため、計算できません。";
18                }
19                break;
20        }
21    }
22
23    return $message;
24}