Javaプログラミング

Question

前提・実現したいこと

Javaを個人で学習していて、練習問題で解答例が載っていないので教えてもらいたい。
円や球などの面積や体積を求めるプログラミングです。

出力としては、
半径1.0の円の面積：3.1
半径1.0の球の面積：4.2
一辺1.0の正方形の面積：1.0
一辺1.0の立方体の面積：1.0

となるようなプログラミングを行いたい。

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

CircleクラスとSquareクラスで今まではreturnで計算式を返せば、よかったがそれぞれのクラスで二つ式が必要でどうすればいいか手詰まりになった。

該当のソースコード

interface calcFigure{
	double calcArear( double x );
	double calcVolume( double x );
}

CircleとSquareクラスを作成

class FigCalculator{
	public static void main( String [] args){
		Circle cc = new Circle();
		Square sq = new Sqare();
		double x= Double.parseDouble(args[0]);
		Sestem.out.printf("半径%.1fの円の面積:%.1f\n", x, cc.calcArea(x));
		Sestem.out.printf("半径%.1fの球の体積:%.1f\n", x, cc.calcvolume(x));
		Sestem.out.printf("一辺%.1fの正方形の面積:%.1f\n", x, sq.calcArea(x));
		Sestem.out.printf("一辺%.1fの立方体の体積:%.1f\n", x, sq.calcVolume(x));
	}
}
		 
		

試したこと

returnで計算式を返そうとしたが一つの式しか返せなかった。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

エラーメッセージ

該当のソースコード

ソースコード

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[swordone]

球や立方体の面積…？

Answer 1

クラスは４つ必要ですね。計算式は 円、正方形は面積、 球,立方体は体積ですね。

interface calcFigure2D {
    public double calcArea(double x);
}
interface calcFigure3D {
    public double calcVolume(double x);
}

// クラスを作成
class Circle implements calcFigure2D {
    public double calcArea(double x) {
	return Math.PI * x * x;
    }
}
class Square implements calcFigure2D {
    public double calcArea(double x) {
	return x * x;
    }
}
class Ball implements calcFigure3D {
    public double calcVolume(double x) {
	return 3.0 / 4 * Math.PI * x * x * x;
    }
}
class Cube implements calcFigure3D {
    public double calcVolume(double x) {
	return x * x * x;
    }
}

public class FigCalculator{
    public static void main(String [] args){
        Circle circle = new Circle();
	Ball ball = new Ball();
        Square square = new Square();
	Cube cube = new Cube();

        double x = Double.parseDouble(args[0]);
        System.out.printf("半径%.1fの円の面積:%.1f\n", x, circle.calcArea(x));
        System.out.printf("半径%.1fの球の体積:%.1f\n", x, ball.calcVolume(x));
        System.out.printf("一辺%.1fの正方形の面積:%.1f\n", x, square.calcArea(x));
        System.out.printf("一辺%.1fの立方体の体積:%.1f\n", x, cube.calcVolume(x));
    }
}

Answer 2

球はCircleではないし、立方体もSquareではないので、設計が根本的に間違っている気がする。

Comments

[bochan2]

質問者さんは円と正方形のプログラムは出来たけれど球のクラスが出来ないと言いたいのではないでしょうか？
なぜかバグで低評価ボタンが押せないので代わりに高評価押しときます

[koko_u]

低評価ボタン、めっちゃ押せそうな気がするけど試してない。

[Zuishin]

低評価は一日に使える限度があります。

Answer 3

class Circle{
    private double radius;
    public Circle(double radius){
        this.radius=radius;
    }
    public double GetArea(){
        return radius*radius*Math.PI;
　 　}
    public double getRadius(){
        return radius;
    }

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

}

class Sphere{
    public double radius;
    public Sphere(double radius){
        this.radius=radius;
    }
    public double GetArea(){
        return radius*radius*Math.PI*4;
　 　}
    public double GetVolume(){
        return radius*radius*radius*Math.PI*4/3;
    }
    public double getRadius(){
        return radius;
    }

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

}

普通に宣言すれば関数は複数作れますよ

Comments

[LouiS0616]

radiusフィールドをpublicにしているのはどうしてでしょうか？

[bochan2]

面積と表面積を求めるだけ外部から変更しても差し支えが無いのでpublicにしました。
構文的に冗長的になっている気もしますがある程度はまともだと思います。

[LouiS0616]

積極的な理由が無い限りフィールドをpublicにするのは避けた方が良いかと。
そもそも外部から自在に半径を設定したいのならば、getArea及びgetVolumeを非staticにする正当な理由が無いです。

[Zuishin]

getArea や getVolume は呼ばれる度に一々計算していますが、それなら引数に radius を取る関数で十分です。
radius を設定するタイミングまたは radius が設定された後はじめて呼ばれたタイミングで計算し、後はキャッシュを返すだけにした方が効率的ですし、オブジェクト指向らしくなります。
その意味でも radius は公開せず getter setter を公開するのが良いと思います。

[bochan2]

LouiSさん、Zuishinさんアドバイスありがとうございます。
radiusを公開せずにgetter setterをpublicにする形に編集しておきます。

[LouiS0616]

radiusの値を外部からいじれるようにするくらいなら、引数として受け取った方が良いという話です。
Zuishinさんの言うとおりキャッシュを用意するならその限りではありませんが。