Javaで仕様書通りにかけない。。。

Question

前提・実現したいこと

お知恵をおかしください。

前提・実現したいこと

Java初心者です。Eclipseで実装しています。
Java でコンソールから複数の数値（整数・実数）を受け取って、入力された数値の積を表示する
<分からないこと>
文字列を数値に変換して前回の数値と今回入力された数値の積を出す記述方法や
※入力はscannar以外を使用して必ずIOExceptionを発生させる
＜仕様＞
・数値以外が入力された場合は、「数値を入力してください」を表示する
・入出力例外が発生した場合は、「入出力例外が発生しました」を表示して、処理を終了する
・改行のみが入力された時点で入力を終了する
・演算の結果として、「入力された値の積 :（演算結果）」を表示する
・積算は「*」オペレータを使用し、double型の精度まで表示する
・演算結果は、3桁カンマ編集して表示する
・入力が１つもない場合の演算結果は「1」とする

■ 出力例
積を求める数値を入力してください
13
9.3
a
数値を入力してください
32
6.7

入力された値の積 : 25,920.96

該当のソースコード

言語名JAVA

import java.io.*;

public class Exercise3 {

public static void main(String[] args) {
	// TODO Auto-generated method stub
	int x;
	System.out.println("入力された値の積 ::");
	//keyboard inputs
	InputStreamReader isr = new InputStreamReader(System.in);
	BufferedReader br = new BufferedReader(isr);
	
	
	try {
		String buf;
		while(true) {
	 buf = br.readLine();//Arise an IOException intentionally
	 if(buf.contentEquals("\n"))//if the value is the only new line , the input wraps up
		 break;
	//if it is a blank, the result is "1"
	 if(buf.contentEquals(""))
	 
		 
		 System.out.println(buf);
	 }
	try {//Exception handling
		x = Integer.parseInt(buf);//arise an Exception if input value wasn't number 
		
	}catch (NumberFormatException e) {
		x = 0;
		System.out.println("数値を入力してください");
	}
	
	
		
	}catch(IOException e) {//handle the exception and print line the stings below
		System.out.println("入出力例外が発生しました");
	}
	
		
		
	
	
}

}
追記:
現在のソースコード:
while (!(str = br.readLine()).isEmpty())//入力された値が空なら入力終了
try {
product *= Double.valueOf(str);//受け取った値を数値(Double type)に変換して変数"product"に代入
} catch (NumberFormatException e) {//数値以外の入力なら例外発生
System.out.println("数値を入力してください");//その例外が発生したら出力
}

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

java.lang.NullPointerException
原因:Ctr+Zを押して入力終了をさせたら上記のエア-がかえった。

試したこと

while (true) {
str = br.readLine();
if (str == null || str.isEmpty()) break;
にすると、今度は複数の値を読み込まれません。

while ((str = br.readLine()) != null && !str.isEmpty());

⇒こっちは、aを入れても例外処理されないし、ctr+Z押して終了させると、やはりNullPointerExceptionが発生しますね

Comments

[gentaro]

仕様書貰ってるって事は、少なくとも仕事でやってるんですよねたぶん。
それであれば、まず職場の人、特に教育を担当している人に相談されてはどうでしょう。
ここで聞いて乗り切っても、次の仕様書が来たらまたちんぷんかんぷんになりますよ。

[javahack]

会社で与えられた課題だとして、teratailのような外部のサイトで質問することを会社は許可しているのでしょうか。

[jimbe]

> 大至急
こちらはそういう場所では無いと思います.

[hoshi-takanori]

この時期にあわててやってるってことは、入社前の課題（なので相談する相手がいない）かもしれませんね。
マジレスしようかと思ったけど、仕様書が結構難しい（ひっかけが多いというか）ことに気づきました。
Exercise3 なので、1 と 2 の内容と理解度が分からないと無責任な回答になっちゃうし。

> 入力が１つもない場合の演算結果を1に戻るとことかチンプンカンプンです。
「1に戻る」んじゃなくて「最初から1」なんですよね…。掛け算の単位元って、学校では習わないのかも。
掛け算じゃなくて足し算だったらできるのかな。それとも、無駄な回り道はしたくないとか言われちゃうのかな。

[meg_]

コードは「コードの挿入」で記入してください。

[jimbe]

ご質問を大きく編集変更してしまうと, 回答と食い違ってしまいます.
進捗により内容を変えたい場合は, 追記するようにしてください.

> while (!(str = br.readLine()).isEmpty()){
> をWhile((str = br.readLine())!= null){
> にしてみましたがダメでした。
どう「ダメ」だったのでしょうか.
元々 "{" は付いていませんし, "While" では文法エラーです. それを直せば Ctrl-z でも NullPointerException は出ませんでした.

[退会済みユーザー]

追記しました。
あと、どう駄目だったか⇒複数の値を入力できなくなってしまったり、Ctr+Z押しても入力終了ができなかったり、色々です。あと、"While"は"{"で囲むようにと教授から指示がありました。

[kazuma-s]

質問を編集するなら、マウスでコード部分を選択し <code>ボタンを押してください。
または、コードの前に ```Java を挿入し、コードの後に ``` を追加してください。
エアーはエラーの間違いですね。
質問のコードは最初の質問のコードで、現在のコードが追加されていません。
現在おかしくなっているのは while の行の最後にセミコロンを付けているからです。
{ を付けたければ、; の代わりに { を書き、while の範囲の最後に } を挿入してください。

Answer 1

・積算は「*」オペレータを使用し、double型の精度まで表示する

・演算結果は、3桁カンマ編集して表示する

これは私も疑問だったので、調べてみました。
double の精度は 10進16桁だとします。
カンマ編集は String.format メソッドでできるようです。
小数点以下で後続の 0 や小数点の削除を追加してみました。

class Main {
	public static void main(String[] args) {
		double y = 1, x[] = { 13, 9.3, 32, 6.7 };
		for (int i = 0; i < x.length; i++) y *= x[i];
		
		String s = String.format("%,.16g", y);
		if (s.indexOf('.') >= 0) {
			s = s.replaceAll("0*$", "");
			s = s.replaceAll("\.$", "");
		}
		System.out.println("入力された値の積 : " + s);
	}
}

これで仕様を満たすかな？

import java.io.*;

public class Exercise3 {
	public static void main(String[] args) {
		try {
			BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
			double product = 1;
			String str;
			while (!(str = br.readLine()).isEmpty())
				try {
					product *= Double.valueOf(str);
				} catch (NumberFormatException e) {
					System.out.println("数値を入力してください");
				}
			str = String.format("%,.16g", product);
			if (str.indexOf('.') >= 0)
				str = str.replaceAll("0*$", "").replaceAll("\.$", "");
			System.out.println("入力された値の積 : " + str);
		} catch (IOException e) {
			System.out.println("入出力例外が発生しました");
		}
	}
}

追記

・入力が１つもない場合の演算結果は「1」とする

という仕様を満たしていませんでした。

while (!(str = br.readLine()).isEmpty())
を
while ((str = br.readLine()) != null && !str.isEmpty())
に修正してください。

1行にまとめているのが分かりにくいなら、次のように書いてもいいでしょう。

            while (true) {
                str = br.readLine();
                if (str == null || str.isEmpty()) break;
                try {
                    product *= Double.valueOf(str);
                } catch (NumberFormatException e) {
                    System.out.println("数値を入力してください");
                }
            }

Comments

[退会済みユーザー]

ありがとうございました。おかげさまで、仕様書通りに実現出来ました。

[kazuma-s]

本当に理解できましたか？
疑問点はもうないのですか？

[junzi]

自分もString.formatを使ってうまいこと行かなかったので「%,」で出来ることに大変勉強になりました！

ですが、
double型は15桁まで有効なはずなので、
13*9.3*32*6.7は
25,920.96ではなく
25,921.960000000003
となるはずなので、
str = String.format("%,.16f", product);
じゃないですかね？
例題も入力された値の積 : 25,920.96となっていますが、double型の精度を満たせていないように思えます。。
丸め誤差を仕様にしたかったのではないかと思います。

[kazuma-s]

https://ja.wikipedia.org/wiki/IEEE_754
Wikipedia の IEEE 754 を見ると、double(倍精度)の有効桁数は、
2進で (52+1)ビット、十進換算で 15.95桁となっています。
これは 2の 53乗が 9007199254740992 であって、
16桁の最大数 9999999999999999にわずかに届かないためです。
したがって、double の精度は 15桁だと言っていいかもしれません。

13*9.3*32*6.7 は、5920.96であって、25921.960000000003でないのは明らかです。
25921.960000000003 は 17桁です。15.95桁しか精度がない double を無理やり
それ以上の桁数表示させると、誤差が表れてそのようになります。

%.16f" の 16 は「精度(有効桁数)」ではなく、「小数点以下の表示桁数」です。
25921.96 は整数部分は 5桁であるため、小数点以下は 10桁または 11桁でなければ
ならないのに それ以上表示させたため誤差の 3が表れています。

[junzi]

ご回答ありがとうございます。
整数部分含めての有効桁数ということを勘違いしておりました。
＞小数点以下は 10桁または 11桁でなければならないのに
10桁以下の3は誤差になるのですね。

わかりやすく教えていただきありがとうございます。
データ型など基本的な知識が足りていないことが分かったので勉強してみます！

[退会済みユーザー]

私もその点についてお聞きしたかったのですが、お答え頂いてたみたいなので、理解できました。
ただ、以下のコードが理解出来ているかわかりません。
str = str.replaceAll("0*$", "").replaceAll("\.$", "");
これは、末尾の値が0なら⇒""に置換する、エスケープ処理で.(小数点)も置換するという感じであっているでようか？

[退会済みユーザー]

Ctr+Zを押して入力終了させると、 java.lang.NullPointerExceptionが発生しました。(!(str = br.readLine()).isEmpty())を
((str = br.readLine())!＝null)に変更しても駄目でした。どなたか知っていれば教えてください。

[jimbe]

> if (str.indexOf('.') >= 0)
> str = str.replaceAll("0*$", "").replaceAll("\.$", "");
は
str = str.replaceAll("\.?0+$", "");
でも出来るかと思います.

[kazuma-s]

> str = str.replaceAll("\.?0+$", "");

それだと、1234567890123400 が 1,234,567,890,123,4 になってしまいます。
if (str.indexOf('.') >= 0) を残せばよいでしょう。

[jimbe]

ごめんなさい, if まで入れちゃいました.

if 無しで出来るか試しましたが,
replaceAll("((?<=\.\d{1,15}?)0+|\.0+)$", "")
となってしまいました.
結局場合分けするなら if のほうが良さそうです.
失礼しました.

[退会済みユーザー]

while (true) {
str = br.readLine();
if (str == null || str.isEmpty()) break;
にすると、今度は複数の値を読み込まれません。

while ((str = br.readLine()) != null && !str.isEmpty());

こっちは、aを入れても例外処理されないし、ctr+Z押して終了させると、やはりNullPointerExceptionが発生しますね

[kazuma-s]

質問を編集するなら、マウスでコード部分を選択し <code>ボタンを押してください。
または、コードの前に ```Java を挿入し、コードの後に ``` を追加してください。
エアーはエラーの間違いですね。
質問のコードは最初の質問のコードで、現在のコードが追加されていません。
現在おかしくなっているのは while の行の最後にセミコロンを付けているからです。
{ を付けたければ、; の代わりに { を書き、while の範囲の最後に } を挿入してください。

[kazuma-s]

while (true) { にしたとき、それを閉じる } はどこに書きましたか？
try-catch の後に書きましたか？

[kazuma-s]

KEIponさんが何も返答しないせいか
spacemanさんという方が
https://teratail.com/questions/251709
「NullpointerExceptionが発生していまう←これだけが解決せんばい」
で、同じ質問をしています。
私は、KEIponさんの質問には真摯に応えているつもりです。
返答をお願いします。

[退会済みユーザー]

すみません、返信遅れました。
try-catchのあとに書きました。

[kazuma-s]

それでも複数の数値が読み込めないんですね。そのコード全体を質問に追加してもらえませんか？
コードを ```Java と ``` で挟んで。

それから、while のセミコロンの問題はどうなりましたか？

Answer 2

とりあえずやってみたのですが、、、
double精度で小数点+カンマ区切りってなんかいい方法ないのか、うまく出来なかった。
知ってる人教えてください。。

package testJava;

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

public class Exercise3 {

	public static void main(String[] args) {
		String inputStr;
		double result = 1;
		System.out.println("積を求める数値を入力してください");

		try {
			while (true) {
				double inputDub;
				inputStr = new BufferedReader(new InputStreamReader(System.in)).readLine();
				if (inputStr.contentEquals(""))
					break;
				try {
					inputDub = Double.parseDouble(inputStr);
				} catch (NumberFormatException e) {
					System.out.println("数値を入力してください");
					continue;
				}
				result = result * inputDub;
			}
		} catch (IOException e) {
			System.out.println("入出力例外が発生しました");
		}

		System.out.print("入力された値の積 :");
		DecimalFormat decimalFormat = new DecimalFormat("#,###");
		System.out.print(decimalFormat.format(result));
		String resultStr = String.valueOf(result);
		int index = resultStr.indexOf(".");
		if (!resultStr.substring(index).equals(".0")) {
			System.out.println(resultStr.substring(index));
		}
	}
}

Comments

[退会済みユーザー]

ありがとうございました。