現在学校のjavaの課題電卓を作っています

質問

import java.io.*;
class dentaku
{
public static void main(String[] args)throws IOException
{
BufferedReader r =
new BufferedReader(new InputStreamReader(System.in));

System.out.print("式を入力");
String input = r.readLine();

String sum[] = new String[input.length()];
String number = "";
int s = 0;
int goukei =0;
char enzan = ' ';

for(int i=0; i < input.length(); i++)
{
//System.out.print(str[i]);
//文字列から1文字ずつ抽出
char num = input.charAt(i);
if(i == input.length()-1)
{
sum[s] = number;
}
else
{
if(num == '+' || num == '-'|| num == '*'|| num == '/')
{
//配列sumに代入
sum[s] = number;
s++;
sum[s] = String.valueOf(num);
s++;
}
else
{
//変数numberにnumを代入
number = number + num;
}
}
}
for(int j=0; j<sum.length; j++)
{
if(sum[j] == null)
{
break;
}
if(j == 0)
{
goukei = Integer.parseInt(sum[j]);
}
else
{
if(sum[j].equals("+") || sum[j].equals("-")|| sum[j].equals("*")|| sum[j].equals("/"))
{
enzan = sum[j].charAt(0);
}
else
{
switch(enzan)
{
case '+':
goukei += Integer.parseInt(sum[j]);
break;
case '-':
goukei -= Integer.parseInt(sum[j]);
break;
case '*':
goukei *= Integer.parseInt(sum[j]);
break;
case '/':
goukei /= Integer.parseInt(sum[j]);
break;
}
}
}
}
System.out.print(goukei);
}
}

までできたのですが　実行し　実際に 1+1+1とうってみると答えが23になってしまいます。
どのようにしたらいいのでしょうか

回答No.1

NNori · ベストアンサー

プログラムを見ると
前半：式を分解してsum[] という配列にいれている。
後半：sum[]という配列から前から読み取って演算している。
というようですね。
こういうときは、一度に作らずに前半だけ作ってから後半を作りましょう。

前半が終わったところで、sum[]という配列にちゃんと思ったような文字が入っていることを
確認すべきです。sum[]を確認してNULLを入れているところで、sum[]の内容を出力してみましょう。

sum[]に自分が思ったような文字列が入力できるようになったら、演算の部分を見ていきましょう。
switch で演算しているところで、goukei と sum[j]の内容を出力して思ったように動作しているか
を確認しましょう。

以下、怪しいところを列挙してみます。

//変数numberにnumを代入
number = number + num;
⇒これって、文字列型の number に読み取った文字列 num を連結（結合）しているのですが、注釈が「代入」となっていて気持ち悪いです。

//配列sumに代入
sum[s] = number;
s++;
sum[s] = String.valueOf(num);
s++;
⇒たぶん、ここが間違い。sum[s]にnumberを代入したところで、次に備えてnumber = "" とする必要がある。

あと、全体に非常に見にくい。
sum という配列は、合計を意味するので文字列を格納するのはやめたほうがいい。
number num といういかにも数字が入りそうな変数名に文字列を格納するのはやめたほうがいい。
まあコンピュータは、忠実なので、宣言された変数の型にはなるのだけど、人間は読みにくいですねぇ。
自分も間違えちゃうので早めに癖は直した方がいいですよ。