PHP 簡易的な計算機問題

Question

●簡単な計算機作成問題
現在以下の問題が解けずに苦しんでいます。
お力添えお願い致します。

問題点
・一応エラーにはならず動作はします。

・「3」→「+」→「5」→「-」
この-の段階で計算が実行されてしまい、rasultの中身が8になって欲しいのに-2になってしまう。

・「5」→「＝」
resultが5になるべきなのに0になってしまう。

すごい初歩的なロジックで悩んでしまっているのですが、なかなか抜け出せなくなってしまっているの
で助言をお願いしたいです。よろしくお願い致します。

＜問題＞

• or - 演算子を利用して、入力された演算子に沿った計算を行うようにし、

演算子の入力ごとに現在の値が表示されるようにしてください。
また、= が入力された場合は計算結果が表示されて処理を終了させます。

入力の順番は原則 数値 -> 演算子 -> 数値 -> 演算子… となります。
数値の入力時には数字以外の入力を受け付けないようにチェックして数値以外の場合は再度数値入力を求めるようにしましょう。
ctype_digit()関数を利用することで数値かどうかをチェックすることが可能です。
また、演算子の入力受付時に該当しないキーが入力された場合も再度、演算子の入力を求めるようにしましょう。

表示例

数値を入力してください。
3 (←入力値)

演算子を入力してください。
+:足し算　-:引き算　=:終了
+ (←入力値)
計: 3

数値を入力してください。
hello

数値以外が入力されました。
数値を入力してください。
5 (←入力値)

演算子を入力してください。
+:足し算　-:引き算　=:終了
- (←入力値)
計: 8

数値を入力してください。
4 (←入力値)

演算子を入力してください。
+:足し算　-:引き算　=:終了
= (←入力値)
計算結果は 4 です。

$result=0;


while (true){
    while (true) {
        echo "数値を入力してください。\n";
        $value=trim(fgets(STDIN));
        if(ctype_digit($value)){
            break;
        }
        else{
            echo "数値以外が検出されました。\n";
        }
    }

    while(true){
        echo"演算子を入力してください。\n
        +:足し算　-:引き算　=:終了\n";
        $value2=trim(fgets(STDIN));
        if($value2==="+"||$value2==="-"||$value2==="="){
            break;
        }
    }

    if($value2==="+"){
        $result+=$value;
        echo "計:".$result,"\n";
    }
    else if($value2==="-"){
        $result-=$value;
        echo "計:".$result,"\n";
    }
    
    else if($value2==="="){
        echo "計算結果は".$result."です。\n";
        break;
    }
}

Comments

[m.ts10806]

https://teratail.com/questions/241434
こちらなぜ削除依頼されたのか分かりませんが、質問は編集できるので同件であれば質問投稿しなおす必要はありませんよ。

[ryota_4645]

申し訳ございません。以後気をつけて修正いたします。

Answer 1

構造が根本的に間違っています。

演算を行うのは、演算子が入力されたときじゃなくて、数値が入力されたときです。

初期化：
　前回結果 = 0
　演算子 = "+"

ループ：
　数値入力
　前回結果 = 前回結果 演算子 入力値
　演算子入力
　"="なら終わり

乗算・除算が入って、優先度が入ってくると途端に難しくなりますが。

Comments

[ryota_4645]

回答ありがとうございました。無事ちゃんとした動作をするようになりました。

現在次の問題で、おっしゃっていた掛け算、割り算、剰余算を
組み込む問題で掛け算が正常に機能せず苦しんでおります。。

[otn]

> 掛け算、割り算、剰余算を
さらに、括弧とか。・・・・ (1+2)*(3+4)=

私も、この回答のようにすぐにすらすらとは書けないです。
変数に保存しておくんじゃなくて、スタックに積むんでしょうね。
あるいは、再帰呼び出しか。