コンストラクタに関する課題

Question

■問題
キーボードから円の半径と円周率を入力させ円の面積と演習を計算して出力するプログラムを作成してください。なお『Circle』クラスを作成しコンストラクタで半径と円周率を設定してください。ただし円周率の入力が省略された場合は『Circle』クラスの定数『PI』に『3.14』を代入しておきその定数を利用してください。また半径と円周率に数値以外のものが入力された場合『半径は数値を入力してください』とメッセージを出力してください。可能であれば面積、円周の値は小数第2位までの出力としてください。

実行例

半径>5
円周率>3
面積：75.0 円周：30.0

この問題を解いているのですがいまだ解けていません...

package en12;

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

public class Ex0512 {


	public static void main(String[] args) throws IOException {
		BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
		try {
			System.out.print("半径>");
			String str = br.readLine();
			double radius = Double.parseDouble(str);
			try {
				System.out.print("円周率>");
				String str1 = br.readLine();
				if(str1 == null || str1.length() == 0){
					Circle C= new Circle(radius);
					C.area();
				}
				double pi = Double.parseDouble(str1);
				Circle B = new Circle(radius, pi);
				B.area();
			} catch (NumberFormatException e) {
				System.out.println("円周は数値を入力してください");
			}
		} catch (NumberFormatException e) {
			System.out.println("半径は数値を入力してください");
		}
	}
}

package en12;

public class Circle {
	private double pi, radius;
	final double PI = 3.14;

	public Circle(double radius) {
		this.pi = 3.14;
	}

	public Circle(double radius, double pi) {
		super();
		this.radius = radius;
		this.pi = pi;
	}

	public double getPi() {
		return pi;
	}

	public void setPi(double pi) {
		this.pi = pi;
	}

	public double getRadius() {
		return radius;
	}

	public void setRadius(double radius) {
		this.radius = radius;
	}

	void area() {
		System.out.print("面積：" + (radius * radius * pi));
	}
}

Comments

[LouiS0616]

https://teratail.com/questions/90875 これと内容同じでは？

[aikosu]

そうです...いまだ解けません...

[LouiS0616]

せめて、前回の回答で何が理解出来なかったか書きましょう。

[aikosu]

円周率を入力しなかった場合どうコードを書けばよいのかわかりません

[aikosu]

コメントミスしたので消しました

[LouiS0616]

さすがにこれは読めません。質問を編集してください。編集するときは、コード部分を選択した状態で<code>ボタンを押すようにしてください。読みやすくなります。

[aikosu]

ありがとうございます。作成したコードを追加しました

[LouiS0616]

なにをもって、『円周率の入力を省略』とみなすのでしょう？

[aikosu]

コンソール上で文字数字を打たずにEnterを押すことです

Answer 1

上手くいかない理由

コンストラクタCircle(double radius)で、入力値が処理されていないからです。
また、3.14という即値をここに書くのはおそらく仕様に反するでしょう。

public Circle(double radius) {
    this.pi = 3.14;
}

さらに、入力が省略された際に、省略されていないときの処理をスキップ出来ていません。
if-elseで分離する必要があります。

質問する際には、どのように上手くいかないのか書いておくとスムーズです。

楽をするなら

とりあえず、円周率の入力省略を感知できれば良いと仮定します。

こういう静的メソッドを組むと楽が出来るかもしれません。
ただし、負の数値の入力は受け付けず、入力を省略されたときは-1を返すようにしてあります。

private static double readPositiveDouble(String varName) throws IOException {
    while(true) {
        System.out.print(varName + "> ");
        try {
            String input = br.readLine();
            if(input.length() == 0) return -1;

            double ret = Double.parseDouble(input);
            if(ret < 0) {
                System.out.println(varName + "は正数を入力してください。");
                continue;
            }

            return ret;

        } catch(NumberFormatException e) {
            System.out.println(varName + "は数値を入力してください。");
        }
    }
}

使い方に関しては、そんなに難しくないので適宜試してみてください。
どのような処理をしているかも、じっくり考えればわかるはずです。

ついでに

日頃から処理を細かく切り分ける癖をつけておくと、いろいろ工夫できます。
たとえば、入力の省略を禁止したり、

private static double readPositiveDoubleByForce(String varName) throws IOException {
    double ret;
    do {
        System.out.println(varName + "の入力は必須です。");
        ret = readPositiveDouble(varName);
    } while(ret == -1);
    return ret;
}

デフォルト値を設定したり。

private static double readPositiveDouble(String varName, double defaultValue) throws IOException {
    double ret = readPositiveDouble(varName);
    return ret == -1 ? defaultValue : ret;
}

今回の課題の場合、後者は不適ですが。