インスタンスに格納した図形の情報を別のインスタンスにランダムに入れ直して実行する方法

Question

#いくつかの図形の面積を求めるサブクラスを作り、別のクラスに入れてランダムに面積を表示するプログラムを作りたいです。

実行したいコードは以下の通りです

```java
package object;

import java.util.Arrays;
import java.util.Random;
import java.util.concurrent.ThreadLocalRandom;

public class Figures{
	public static void main(String[] args) {
		Figure[] figures1 = new Figure[9];
		Figure[] figures2 = figures1.clone();	
		
		Rectangle r1 = new Rectangle();
		Rectangle r2 = new Rectangle(30, 20);	
		int i = 0;
		figures1[i++] = r1;
		figures1[i++] = r2;
		System.out.println("The area of r1 is" + r1.getArea());
		System.out.println("The area of r2 is" + r2.getArea());
		
		Circle1 c1 = new Circle1();
		Circle1 c2 = new Circle1(25);
		figures1[i++] = c1;
		figures1[i++] = c2;
						
		System.out.println("The area of c1 is" + c1.getArea());
		System.out.println("The area of c2 is" + c2.getArea());
		
		Triangle t1 = new Triangle();
		Triangle t2 = new Triangle(30, 20);
		Triangle t3 = new Triangle(300, 200);
		System.out.println("The area of t1 is" + t1.getArea());
		System.out.println("The area of t2 is" + t2.getArea());
		System.out.println("The area of t3 is" + t3.getArea());
		
		figures1[i++] = t1;
		figures1[i++] = t2;
		figures1[i++] = t3;
		
		Square2 s1 = new Square2();
		Square2 s2 = new Square2(20);
		System.out.println("The area of s1 is " + s1.getArea());
		System.out.println("The area of s2 is " + s2.getArea());
		
		figures1[i++] = s1;
		figures1[i++] = s2;
	}
		public static void shuffle(int[] figures1) {	       
	        Random rnd = ThreadLocalRandom.current();
	        for (int k = figures1.length - 1; k > 0; k--) {
	            int index = rnd.nextInt(k + 1);
	            // 要素入れ替え(swap)
	            int tmp = figures1[index];
	            figures1[index] = figures1[k];
	            figures1[k] = tmp;
	        }
	        
for(int k=0; k<figures1.length;k++) {
	        	System.out.println(figures1[k].getName()+":"+figures1[k].getArea());
	        	 
	        }
	    }
		
	}

ここの

for(int k=0; k<figures1.length;k++) {
	        	System.out.println(figures1[k].getName()+":"+figures1[k].getArea());
	        	 
	        }

という部分が

「この行に複数マーカーがあります

• getName() を基本タイプ int で起動できません
• getArea() を基本タイプ int で起動できません」

というエラーコードを出してうまく動きません。
既出の質問を参考にさせてもらっています。
図形配列をシャッフルして別の配列に格納する方法

##各図形のコード

 package object;

import java.util.concurrent.atomic.AtomicInteger;

public class Rectangle extends Figure {
	private double height = 0;
	private double width = 0;
	
	static final String NAME = Rectangle.class.getSimpleName();
	static final java.util.concurrent.atomic.AtomicInteger  counter = new AtomicInteger(0);
	
	String name = "Rectangle";
	
	Rectangle(){
		this.width = 20;
		this.height = 10;
	}
	Rectangle(double w, double h){
		this.width = w;
		this.height = h;
	}	
	void setWidth(double w) {
		this.width = w;
	}
	void setHeight(double h) {
		this.height = h;
	}
	void setSize(double w, double h) {
		setWidth(w);
		setHeight(h);
	}
	double getWidth() {
		return this.width;
	}
	double getHeight() {
		return this.height;
	}
	
	Rectangle(int width, int height){
		super(NAME + counter.addAndGet(1));
		this.width = width;
		this.height = height;
	}
	
	@Override
	double getArea() {
		return width * height;
	}
	void identify() {
		System.out.println("I am a rectangle with area" + getArea() + ".");
	}
}

###スーパークラス

package object;

public abstract class Figure {
	final String name;
	public Figure(){
		this.name = "";
	}
	public Figure(String name) {
		this.name = name;
	}	
	
	abstract double getArea();
	
	@Override
	public String toString() {
		return name + ":" + String.format("%.1f", getArea());
	}
	public void identify() {
		// TODO 自動生成されたメソッド・スタブ
		
	}
	
	
	
	
}

どこで間違えているかわかるかたがいらっしゃいましたら教えていただきたいです。

Answer 1

figures1[k].getName()の部分にエラーが出ているのであれば、
まずは、figures1[k]の型を調べます。
コードを辿ると……
public static void shuffle(int[] figures1) {
ここで、
int[] figures1
と定義されていますので、figures1[k]の型は、int型となります。
int型は、基本型なので.getName()のようなメソッドは持ちません。

それをまとめると、getName() を基本タイプ int で起動できませんのエラーになります。

これを解決するためには、適切な型を指定する必要があります。
何の型にすべきかは、何をしたいか次第ですので、それに合わせて相応しいものを選んでください。

Comments

[shimajiro1917]

とても勉強になったのでベストアンサーに選ばせていただきました。
しかし、私の近いが甘いものでfigures1とfigures2にはどのような数値？が入っているのかいまいち理解できていない状態です。
System.out.println("The area of r1 is" + s1.getArea());
の中身をランダムにfigures2の中に入れて実行しているものと思っていたので、どのように型を変換すれば実行できるのかわからないのでご助言をいただけると幸いです。

[amiya]

何が入っているのか？は、何を入れたか？(入れたいか？)という話になります。
どうなっているのか、よりも前に、どうしたいのか？の方から、コードをそれに寄せてゆくものかと。

とりあえず、現行ではshuffleメソッドは独立しているので、外側のfigures1、figures2とは、無関係になっています。

まずは、このメソッドが何をどうしたいのか？という点から考えると良いかと思います。

[shimajiro1917]

figures1の内容をfigures2に入れたいのでその方向で再度調べて考えてみます。
理解が浅いので再度検討し直します。ご丁寧にご回答していただきありがとうございます。

Answer 2

public static void shuffle(int[] figures1) {

の部分を直せばよいです。

Comments

[shimajiro1917]

そこなんですね。
どのように間違えているのか検討がつかないのでもう少し詳細に教えていただけると幸いです。