javaで、配列の中身を追加するクラスを作って、それを用いて別クラスから作った配列を表示したい

Question

###追記
こちら課題であり、「ArrayListやMapなどのコレクションは使用しないように」という指定があります。

Position,Stock,Bondはそれぞれ
Position(Issue issue, double amount)
Stock(String code, String name, Market market)
であり、
Issue(String code, String name)
は、Position,Stockのスーパークラスとなっています。

前提・実現したいこと

javaで、Portfolioクラスで、配列(portfolio)に中身(Position)を追加するメソッドを用いて、
PortfolioTestクラスにて、適当なPositionをSystem.out．prinntlnで表示したい

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

エラー: for-eachは式のタイプに使用できません
		for(Position p : portfolio){
		                 ^
  期待値: 配列またはjava.lang.Iterable
  検出値: Portfolio
エラー1個

該当のソースコード

public class Portfolio{
	private Position[] portfolio = new Position[0];
	public void addPosition(Position p){
		Position samePosition = findPosition(p.getIssue());
		if(samePosition == null){
			Position[] newPortfolio = new Position[portfolio.length + 1];
			System.arraycopy(portfolio, 0, newPortfolio, 0, portfolio.length);
			newPortfolio[portfolio.length] = p;
			this.portfolio = newPortfolio;
		}else{
			int i = 0;
			do{
				i ++;
				}while( portfolio[i].getIssue().getCode().equals(p.getIssue().getCode()) && portfolio[i].getIssue().getName().equals(p.getIssue().getName()) );
			double sum = samePosition.getAmount() + p.getAmount();
			portfolio[i].setAmount(sum);
		}
	}
	
	public Position findPosition(Issue issue){
		for(Position p : portfolio){
			if( p.getIssue().getCode().equals(issue.getCode()) && p.getIssue().getName().equals(issue.getName()) ){
				return p;
			}
		}
		return null;
	}
}

class PortfolioTest{
	public static void main(String[] args){
		//(2)
		Stock stock1 = new Stock("92010", "A", Stock.Market.TSE);
		Stock stock2 = new Stock("68610", "B", Stock.Market.OSE);
		Stock stock3 = new Stock("72030", "C", Stock.Market.NSE);
		Bond bond1 = new Bond("00611273", "D", 20310907, 0.110);
		Bond bond2 = new Bond("33070153", "F", 20261120, 0.001);
		Bond bond3 = new Bond("090650948", "G", 20231120, 0.001);
		
		Position positionStock1 = new Position();
		positionStock1.setIssue(stock1);
		positionStock1.setAmount(0.1);
		Position positionStock2 = new Position();
		positionStock2.setIssue(stock2);
		positionStock2.setAmount(0.2);
		Position positionStock3 = new Position();
		positionStock3.setIssue(stock3);
		positionStock3.setAmount(0.3);
		
		Position positionBond1 = new Position();
		positionBond1.setIssue(bond1);
		positionBond1.setAmount(0.075);
		Position positionBond2 = new Position();
		positionBond2.setIssue(bond2);
		positionBond2.setAmount(0.120);
		Position positionBond3 = new Position();
		positionBond3.setIssue(bond3);
		positionBond3.setAmount(0.205);

		//(3)
		Portfolio portfolio = new Portfolio();
		portfolio.addPosition(positionStock1);
		portfolio.addPosition(positionStock2);
		portfolio.addPosition(positionStock3);
		portfolio.addPosition(positionBond1);
		portfolio.addPosition(positionBond2);
		portfolio.addPosition(positionBond3);
		
		for(Position p : portfolio){
			System.out.println(p);
		}
	}
}

試したこと

各Positionは正しく作れていることを確認できました。
エラーから、配列にしなきゃと思い、//(3)にて、
Portfolio[] portfolio = new Portfolio[1];
としてみたのですが、addPositionが使えませんでした。

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

java version "16.0.1" 2021-04-20
Java(TM) SE Runtime Environment (build 16.0.1+9-24)
Java HotSpot(TM) 64-Bit Server VM (build 16.0.1+9-24, mixed mode, sharing)

Position、Stock、Bondはそれぞれ
Position(Issue issue, double amount)
Stock(String code, String name, Market market)
Bond(String code, String name, int maturity, double coupon)
となっており、
Issue(String code, String name)
は、 Stock,Bondのスーパークラスです。

Comments

[jimbe]

まず基本的に、可変長な配列をわざわざ作るよりは、List を使ったほうが良いのではないでしょうか。

テストの Main がやけにごちゃごちゃしていますが、 Position, Stock, Bond 各クラスの定義はどうなっているのでしょうか。

[knt715]

＠jimbe さん
・Listは使ってはいけないという指定があり、使えません。（補足説明忘れていました????‍♂️）

・それぞれ
Position(Issue issue, amount)
Stock(String code, String name, Market market)
Bond(String code, String name, int maturity, double coupon)
となっています。ちなみに、
Issue(String code, String name)
であり、Position、Stock、BondはIssueのサブクラスとなっております。
よろしくお願いします。

[dodox86]

Portfolioクラスのフィールドのportfolioと、PortfolioTestクラスのmainメソッド内のportfolio 変数と、ごっちゃにしていると思います。

[knt715]

@dodox86 さん
Portfolioクラスのportfolioを使うには、どうしたら良いのでしょうか。
現状、配列を作れていないということでしょうか。

[jimbe]

> Issue(String code, String name) は、 Posittion,Stock,Bondのスーパークラスです。

Position は Issue を保持するのに、 Position 自身が Issue でもあるのですか。
クラスの構造が実際の言語的定義と合っていないということでしょうか。

[knt715]

＠jinbe さん
Positionはサブクラスではありませんでした。
たびたび申し訳ないです。

Answer 1

外部のクラスに対し実装("配列")を直接見せるのは良いことではありませんので、表示だけであれば専用メソッドを作ってしまえば良いと思います。

---

コレクション不可ということですので配列に戻しました。
そのままというのも勿体ない(?)ので、段階的拡張としてみました。
専用メソッドも課題的に問題視されるかもしれませんので、元の形にしました。
Portfolio を for-each ループで使えるように Iterable 化しました。

package teratail_java.q373056;

import java.util.*;

public class Portfolio implements Iterable<Position> {
  private Position[] positions = new Position[0];
  private int length = 0;
  public void addPosition(Position p) {
    Position exists = findPosition(p.getIssue());
    if(exists == null) {
      if(length >= positions.length) expandArray();
      positions[length++] = p;
    } else {
      exists.addAmount(p.getAmount());
    }
  }
  private void expandArray() {
    Position[] newArray = new Position[positions.length + 5];
    System.arraycopy(positions, 0, newArray, 0, length);
    positions = newArray;
  }
  private Position findPosition(Issue issue) {
    for(Position p : this) if(p.getIssue().equals(issue)) return p;
    return null;
  }
  public int getLength() { return length; }
  public Position getPosition(int index) {
    if(index < 0 || length <= index) throw new ArrayIndexOutOfBoundsException(index);
    return positions[index];
  }
  @Override
  public Iterator<Position> iterator() {
    return new Iterator<Position>() {
      private int index = 0;
      @Override
      public Position next() {
        return positions[index++];
      }
      @Override
      public boolean hasNext() {
        return index < length;
      }
    };
  }

  public static void main(String[] args) {
    Position positionStock1 = new Position(new Stock("92010", "A", Stock.Market.TSE), 0.1);
    Position positionStock2 = new Position(new Stock("68610", "B", Stock.Market.OSE), 0.2);
    Position positionStock3 = new Position(new Stock("72030", "C", Stock.Market.NSE), 0.3);
    Position positionBond1 = new Position(new Bond("00611273", "D", 20310907, 0.110), 0.075);
    Position positionBond2 = new Position(new Bond("33070153", "F", 20261120, 0.001), 0.120);
    Position positionBond3 = new Position(new Bond("090650948", "G", 20231120, 0.001), 0.205);

    Portfolio portfolio = new Portfolio();
    portfolio.addPosition(positionStock1);
    portfolio.addPosition(positionStock2);
    portfolio.addPosition(positionStock3);
    portfolio.addPosition(positionBond1);
    portfolio.addPosition(positionBond2);
    portfolio.addPosition(positionBond3);

    for(Position p : portfolio) {
      System.out.println(p);
    }
  }
}

class Issue {
  protected String code;
  protected String name;
  Issue(String code, String name) {
    if(code == null || name == null) throw new NullPointerException();
    this.code = code;
    this.name = name;
  }
  String getCode() { return code; }
  String getName() { return name; }

  @Override
  public int hashCode() {
    return Objects.hash(code, name);
  }
  @Override
  public boolean equals(Object obj) {
    if(this == obj) return true;
    if(obj == null) return false;
    if(getClass() != obj.getClass()) return false;
    Issue other = (Issue) obj;
    return code.equals(other.code) && name.equals(other.name);
  }
}

class Stock extends Issue {
  enum Market {
    TSE, OSE, NSE;
  }
  private Market market;
  Stock(String code, String name, Market market) {
    super(code, name);
    if(market == null) throw new NullPointerException();
    this.market = market;
  }
  @Override
  public String toString() {
    return new StringJoiner(",","Stock [","]")
        .add("code="+code)
        .add("name="+name)
        .add("Market="+market)
        .toString();
  }
}

class Bond extends Issue {
  private int maturity;
  private double coupon;

  Bond(String code, String name, int maturity, double coupon) {
    super(code, name);
    this.maturity = maturity;
    this.coupon = coupon;
  }
  @Override
  public String toString() {
    return new StringJoiner(",","Bond [","]")
        .add("code="+code)
        .add("name="+name)
        .add("maturity="+maturity)
        .add("coupon="+coupon)
        .toString();
  }
}

class Position {
  private Issue issue;
  private double amount;

  Position() {}
  Position(Issue issue, double amount) {
    setIssue(issue);
    setAmount(amount);
  }

  void setIssue(Issue issue) { this.issue = issue; }
  Issue getIssue() { return issue; }

  void setAmount(double amount) { this.amount = amount; }
  void addAmount(double amount) { this.amount += amount; }
  double getAmount() { return amount; }

  @Override
  public String toString() {
    return new StringJoiner(",","Position [","]")
        .add("issue="+issue)
        .add("amount="+amount)
        .toString();
  }
}

実行結果

Position [issue=Stock [code=92010,name=A,Market=TSE],amount=0.1]
Position [issue=Stock [code=68610,name=B,Market=OSE],amount=0.2]
Position [issue=Stock [code=72030,name=C,Market=NSE],amount=0.3]
Position [issue=Bond [code=00611273,name=D,maturity=20310907,coupon=0.11],amount=0.075]
Position [issue=Bond [code=33070153,name=F,maturity=20261120,coupon=0.001],amount=0.12]
Position [issue=Bond [code=090650948,name=G,maturity=20231120,coupon=0.001],amount=0.205]

Comments

[knt715]

回答ありがとうございます。
とても参考になります。ですが、今回「Array Listを使ってはいけない」という指定がありますので、
配列を使った方法を探しています。
補足に書いたのですが、わかりづらいですかね。すみません????‍♂️

[jimbe]

重要なことはご質問を編集して追記して頂けると分かり易いかと思います。

なぜ ArrayList を使ってはいけないのでしょうか。何かの課題ということですか。
使ってはいけないのは List(ArrayList) **だけ** でしょうか?

[knt715]

＠jinbe さん
・課題です。
・「ArrayListやMapなどのコレクションは使用しないように」との指定があります。

・追記してみたのですが、書く場所あってますでしょうか？他に追記専用の場所がありますか？

[jimbe]

追記の方法に関しましては特に決まり事はありません。
単純に…といっていいのか分かりませんが、ご質問として必要な情報が(出来れば分かり易く)ご提示されているかという点だけお考え下さると良いかと思います。

また、 teratail は課題そのもの("分からないので全部教えて"的な)に関するご質問は非推奨です。課題を行う過程においてのコード上の不明点であれば個人的にはぎりぎり(?)OKかな?というつもりですが、一応「課題」に対しては敏感な部分ですのでご留意ください。

[knt715]

@jinbe さん
ありがとうございます!!
色々と、以後気をつけます。(今回は課題の一部でした。)

[jimbe]

BA 付けられた後ですが、
for(Position p : portfolio){
System.out.println(p);
}
でも動作するように修正してみました。
こんなもんですかね。課題がんばってください。

[knt715]

@jimbe さん
ありがとうございます????