java 電卓コードについて

Question

java初心者です。
電卓アプリとして四則演算ができるコードを作成しようとしています。

足し算引き算は問題なくできるようになったのですが、掛け算割り算が現在まだできていません。

文字列を1文字ずつ読み取る処理を書いて、リストの中身を足すことで足し算引き算は処理できたのですが、掛け算割り算のときにif文の処理内容をどのように記述するべきなのかがわかりません。
稚拙な文章とコードで申し訳ありませんが、ご教授いただけますと幸いです。よろしくお願い致します。
最終的には、例として
6+2＊3が入力された時に12となることが理想です。
以下、コードになります。
/**
*
* @author
* 電卓実行クラス
*/
public class Dentaku2 {
public static void main(String[] args) throws IOException {

             BufferedReader br = new BufferedReader(new InputStreamReader(System.in));

             String str = br.readLine();
             String[] strArray = str.split("");

             List<String> numList = new ArrayList<>();

             StringBuilder sb = new StringBuilder();

             for (String s : strArray) {

                  if (s.matches("[0-9]")) {
                        sb.append(s);
                       
                  } else if (s.equals("*")) {
                        numList.add(sb.toString());
                        sb.setLength(0);
                       
                  } else if (s.equals("/")) {
                        numList.add(sb.toString());
                        sb.setLength(0);
                       
                  } else if (s.equals("+")) {
                        numList.add(sb.toString());
                        sb.setLength(0);
                       
                  } else if (s.equals("-")) {
                        numList.add(sb.toString());
                        sb.setLength(0);
                        sb.append("-");
                  }
             }
             numList.add(sb.toString());
             sb.setLength(0);
            
             int sum = 0;
             int sum2=1;
            
             for (String i : numList) {
                  if(i.equals("*")) {
                   sum2 *=Integer.parseInt(i);
                  System.out.println(sum2);
                 
                  }else if(i.equals("+")||(i.equals("-"))){
                  sum += Integer.parseInt(i);
                  System.out.println(sum);
             }    
       }
  }

}

Comments

[Zuishin]

> 掛け算割り算のときにif文の処理内容をどのように記述するべきなのかがわかりません。

記述されているように見えますが、どこの話をしていますか？

[Zuishin]

https://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q11279623676

[Zuishin]

これも同じ質問だったのかな？

https://teratail.com/questions/t97fr7kjdun0zx

[Natsu_chan]

コメントありがとうございます。
記述しているのですが、うまく計算が処理されず、困っている次第です。。

[melian]

以下の部分ですが、numList には演算子が追加されていません。(数字のみ) 仮に演算子が含まれていたとしてもInteger.parseInt(i) でエラーになってしまいます。

for (String i : numList) {
if(i.equals("*")) {
sum2 *=Integer.parseInt(i);
System.out.println(sum2);

[kazuma-s]

> 足し算引き算は問題なくできるようになったのですが、
現在の質問のコードでは足し算引き算もできません。
質問を編集して、足し算と引き算ができるようになったコードを追記してください。

[Natsu_chan]

大変失礼しました。
修正させていただきました。

[jimbe]

コードはコードのマークダウン(```だけの行で囲む)を用いてください。
用いないと、一部の記号等が修飾のマークダウンとして扱われて消えてしまう可能性があります。

>修正させていただきました。

変わらず足し算引き算出来ませんけど。

[kazuma-s]

> 文字列を1文字ずつ読み取る処理を書いて
6+23 は 29 になるんですか？

[jimbe]

削除依頼をされていたようですが、何かで解決したのでしたらご自身で回答を書いてそれをベストアンサーとして解決済みとしてください。
teratail は個人の問題が解決すればそれで良いというのではなく、同様の問題を抱える他の人の為にコンテンツとして残すことを目的にしています。

Answer 1

括弧が無くていいのなら、これで十分でしょう。

import java.io.IOException;
import java.io.BufferedReader;
import java.io.InputStreamReader;

public class Dentaku2 {
    static int num[] = new int[3], op[] = new int[3], k;
    static StringBuilder sb = new StringBuilder();

    static void proc(int opr) {
        num[k] = Integer.parseInt(sb.toString());
        sb.setLength(0);
        op[k] = opr;
        while (k > 0 && op[k-1] >= op[k]-1) {
            if (op[--k] == 2) num[k] += num[k+1];
            else if (op[k] == 3) num[k] -= num[k+1];
            else if (op[k] == 5) num[k] *= num[k+1];
            else if (op[k] == 6) num[k] /= num[k+1];
            op[k] = op[k+1];
        }
        k++;
    }

    public static void main(String[] args) throws IOException {
        BufferedReader br = new BufferedReader(
            new InputStreamReader(System.in));
        String str = br.readLine();
        k = 0;
        String[] strArray = str.split("");
        for (String s : strArray) {
            if (s.matches("[0-9]")) sb.append(s);
            else if (s.equals("+")) proc(2);
            else if (s.equals("-")) proc(3);
            else if (s.equals("*")) proc(5);
            else if (s.equals("/")) proc(6);
        }
        proc(0);
        System.out.println(num[0]);
    }
}

入力が不正な場合のエラー処理はしていません。
理解できない場合は、どこが分からないのかをコメントに書いてください。

Comments

[xebme]

proc(0); が終端であることを示しています。逆ポーランド記法の評価と同じだと思います。
num[],op[]のサイズが3で固定なのは括弧が無い仕様だからですね。

Answer 2

乗徐を優先するならそのように細工する必要があるでしょう。
以下のコードでは list 内で計算を進め、 pri のループで pri=0 の時に乗除、pri=1 の時に加減の計算をするようにしています。
減算も数値を負にして加算するより(どうせ if で演算子を判定するなら)そのまま引いたほうが簡単に思います。
また、 split で分解するなら、(式に問題が無ければ)正規表現を工夫することでループ等せずできます。
先読みと後読みを使ったパターンの記述

import java.io.*;
import java.util.*;

public class Dentaku2 {
  public static void main(String[] args) throws IOException {
    String str = null;
    try(BufferedReader br = new BufferedReader(new InputStreamReader(System.in))) {
      str = br.readLine();
    }
    System.out.println("str=" + str);

    List<String> list = new ArrayList<>(Arrays.asList(
        str.split("(?!\\d)|(?<!\\d)") //数字以外の直前もしくは数字以外の直後で分割
    ));
    System.out.println("list=" + list);

    //演算子に従って list を変化させていくことで計算を進める
    for(int pri=0; pri<2; pri++) {
      for(int i=1; i<list.size(); ) { //奇数番目は演算子
        if(list.get(i).equals("*") && pri == 0) {
          replace(list, i, getInt(list, i-1) * getInt(list, i+1));
        } else if(list.get(i).equals("/") && pri == 0) {
          replace(list, i, getInt(list, i-1) / getInt(list, i+1));
        } else if(list.get(i).equals("+") && pri == 1) {
          replace(list, i, getInt(list, i-1) + getInt(list, i+1));
        } else if(list.get(i).equals("-") && pri == 1) {
          replace(list, i, getInt(list, i-1) - getInt(list, i+1));
        } else {
          i += 2; //次の演算子
        }
      }
    }
    System.out.println(list.get(0));
  }
  //i 番目を数値化して取り出す
  private static int getInt(List<String> list, int i) {
    return Integer.parseInt(list.get(i));
  }
  //i 番目とその前後の 3 要素を v に置き換え
  private static void replace(List<String> list, int i, int v) {
    list.set(i-1, ""+v);
    list.remove(i); list.remove(i); //2つ削除(達磨落とし)
    System.out.println("list=" + list);
  }
}

6+2*3
str=6+2*3
list=[6, +, 2, *, 3]
list=[6, +, 6]
list=[12]
12

Comments

[xebme]

6 + 2 * 3
スペースを含むと動かないようです。スペースは含まない仕様ですか。

[jimbe]

何処まで対応が必要なのかは確認していませんので、「例として6+2＊3が入力された時に12となる」のみを対象としました。

[xebme]

電卓にスペースキーはありませんね。