配列がうまく表示されない java

Question

メインクラスの中のOrderをアウトプットしようとしたのですが、うまくいきません。
なせ出力されないのでしょうか。

Pizzaクラス

package Pra3_3;
import java.util.Scanner;

// Class name should be "Pizza" though....
public class Pizza {
		
		private String size;
		private int cheese;
		private int pepperoni;
		private int ham;
		public int cost;
		
		
		public Pizza() {}
		
		public Pizza (String size, int cheese, int pepperoni, int ham) {
			this.size = size;
			this.cheese = cheese;
			this.pepperoni = pepperoni;
			this.ham = ham;
		}
		

	
	/**
		 * @return the size
		 */
		public String getSize() {
			return size;
		}

		/**
		 * @param size the size to set
		 */
		public void setSize(String size) {
			this.size = size;
		}

		/**
		 * @return the cheese
		 */
		public int getCheese() {
			return cheese;
		}

		/**
		 * @param cheese the cheese to set
		 */
		public void setCheese(int cheese) {
			this.cheese = cheese;
		}

		/**
		 * @return the pepperoni
		 */
		public int getPepperoni() {
			return pepperoni;
		}

		/**
		 * @param pepperoni the pepperoni to set
		 */
		public void setPepperoni(int pepperoni) {
			this.pepperoni = pepperoni;
		}

		/**
		 * @return the ham
		 */
		public int getHam() {
			return ham;
		}

		/**
		 * @param ham the ham to set
		 */
		public void setHam(int ham) {
			this.ham = ham;
		}
		
		
		public double calcCost() {
			int pizzaSize;
			int toppings;
			if ((getSize() == "small") && ((getCheese() >= 0) 
					|| (getPepperoni() >= 0) || (getHam() >= 0))) {
				pizzaSize = 10;
				toppings = 2 * getCheese() + 2 * getPepperoni() + 2 * getHam();
				cost = pizzaSize + toppings;
			}
			
			else if((getSize() == "medium") && ((getCheese() > 0) 
					|| (getPepperoni() > 0) || (getHam() > 0))) {
				pizzaSize = 12;
				toppings = getCheese() * 2 + getPepperoni() * 2 + getHam() * 2;
				cost = pizzaSize + toppings;
			}
			
			else if((getSize() == "large") && ((getCheese() >= 0) 
					|| (getPepperoni() >= 0) || (getHam() >= 0))) {
				pizzaSize = 14;
				toppings = 2 * getCheese() + 2 * getPepperoni() + 2 * getHam();
				cost = pizzaSize + toppings;
			}
			return cost;
		}
		
		public String getDescription() {
			return "Pizza size: " + getSize() + " Topping: " + (getCheese() + getPepperoni() + getHam())
					+ " Total cost is: " + calcCost();
		}
		

}

PizzaOrderクラス

package Pra3_4;

public class PizzaOrder {

	private Pizza[] pizzas = new Pizza[3];
	
	/*
	public PizzaOrder(Pizza[] pizzas) {
		this.pizzas = pizzas;
	}
	*/
	
	
	public void setPizzas(Pizza newPizza, int index) {
		pizzas[index] = newPizza;
	}
	
	public Pizza[] getPizzas() {
		return pizzas;
	}
	
	public double calcTotal() {
		double total = 0.0;
		for(int i = 0; i < pizzas.length; i++) {
			total += pizzas[i].calcCost(); 
		}
		return total;
	}
	
	/*
	public int getNumPizzas() {
		
	}
	*/
}

Mainクラス

package Pra3_4;

import java.util.Arrays;

public class Main {

	public static void main(String[] args) {
		
		Pizza pizza1 = new Pizza("large", 1, 0, 1);
		Pizza pizza2 = new Pizza("medium", 2, 2, 0);
		
		
		
		PizzaOrder order = new PizzaOrder();

		order.setPizzas(pizza1,2);
		order.setPizzas(pizza2,0);
		
		//System.out.println(Arrays.toString(order[0].getPizzas()));
		
		for(int i = 0; i < order.getPizzas().length; i++) {
			System.out.println(order);
		}
		//double total = order.calcTotal();
		
		
		//System.out.println(Arrays.toString(order));
		
		//double total = order.calcTotal();
		
		//System.out.printf("Pizza Order total cost $%.2f\n", total);

	}

}

Answer 1

        System.out.println(order);

ループで回してるのに、同じものしか表示させていない、ってことかと

Comments

[coala]

ご回答ありがとうございます。おっしゃられていることが鮮明ではないのですが、アウトプットが”Pra3_4.Pizza@4b1210ee”になります

[y_waiwai]

質問がイマイチ不明ですが、
どう表示されればいいんでしょうか

[coala]

Mainクラスがテストになっており、Pizza pizza1 = (large....), Pizza pizza2 = (medium....)をsetPizzasで配列の中にセットしていると思っていて、その配列の中にはpizza1, pizza2が代入されているプログラミングを書いたつもりです。最終その配列の中を表示させたいのですが、上手に表示されません。

[Luice]

まずはこちらを一読した方が良いと思います。
https://qiita.com/NagaokaKenichi/items/78015f53979b2545c2e6

[dodox86]

＞@質問者さん

視点を変えて、自分自身がPizzaOrderクラスのインスタンスであったと考えてみると良いです。
> PizzaOrder order = new PizzaOrder();
こうした後に
> System.out.println(order);
こうすることで、PizzaOrderクラスのインスタンスであるorderは、多くの場合にString型の引数を求めるSystem.out.println()に対して、いったい何をしたら（表現したら）良いのであろうか、と言うお話です。teratailのJavaの関する質問でも頻出です。teratailトップぺージから「Java toString」と検索してみると関連する話題がヒットすると思います。

[coala]

ご回答ありがとうございます。toString メソッドがキーになること理解致しました。
私の中で配列はSysout.out.println(Arrays.toString(配列));とすれば表示されるからtoStringメソッドは関係ないと思っていました。

[dodox86]

しかしながら本回答にあるy_waiwaiさんの指摘は、その点を越えたところにあります。forで要素数分回しているようだけど、要素全てが入っているであろうorder自身をSystem.out.printlnしようとするのはこれ如何に、ということです。

[coala]

なるほど！本来はorder[i]などにしてどの要素を出力するかを指示する必要があるという理解で大丈夫でしょうか。
ご回答頂いたようにtoStringについて調べてオーバーライドしてみたところ出力されました。
ありがとうございます。

[dodox86]

> 本来はorder[i]などにしてどの要素を出力するかを指示する必要

ここは恐らくプログラマー（コードを設計、書いた人）次第の話で、forなどのループで逐一取り出したいのであればget(index), at(index)のようなメソッドを用意してそれを使うのでしょうし、そうではなく、あくまでインスタンスにとってのtoString()がそのインスタンス全体をStringで表現すべきと判断するのであれば、indexで要素を指定するまでもなく、forで回すまでもなく、toString()で全要素を一挙に出力するようにしても良いのではないでしょうか。

提示のコードが以下のようになっているので、本回答の内容になるのだと思います。

> for(int i = 0; i < order.getPizzas().length; i++) {
> System.out.println(order);
> }

ちなみにこれは拡張for文で以下のようにもできるはずです。
for (Pizza pizza: order.getPizzas()) {
System.out.println(pizza);
//System.out.println(pizza.toString());
}

[coala]

とても参考になるご回答頂きありがとうございます。ぜひ、ベストアンサーに選ばさせて頂きたいのですが、回答欄に再度投稿いただけないでしょうか。
よろしくお願い致します。

[dodox86]

あ、いえ、上記コメントはこちらの回答への補間のつもりでした。オブジェクト（クラスのインスタンス）のある内容をStringで表現することがまぁ、言うなればtoString()の役目であり、それが解決したら本回答に繋がるか、と思ってのコメントでした。長々と書いてすみません。toString()についてもそもそもLuiceさんが先に指摘されていましたし、別回答で投稿しようかと思いましたがやめておきます。

[coala]

とても参考になる補足コメントありがとうございました。また何かありましたら今後も宜しくお願い致します。