配列を用いた実装（addメソッド、findメソッド）

Question

前提・実現したいこと

もともと以下のPositionクラスが存在します。
/*
package issues;
public class Position{
private Issue issue;
private double amouunt;

public Positon(Issue issue,double amount){
	this.issue=issue;
	this.amount=amount;
}

public Issue getIssue(){
	return issue;
}

public double getAAmount(){
	return amouunt;
}

public void setIssue(Issue issue){
	if(issue == null){
		throw new IllegalArgumentException("issue is null");
	}
	this.issue = issue;
}

public void setAmount(double amount){
	if(amount == null){
		throw new IllegalArgumentException("amount is null");
	}
	this.amount = amount;
}

*/
そこで、複数のPositionを保持するためにPortfolioクラスを作成したいです。

今回の実装は配列を用いて実装してください。

次の2つのメソッドを用いて作成しろという問題です。
・public void addPosition(Position p)
PortfolioにPosition pを追加する。
同じIssueのPositionがすでにある場合には、それにamountを加算する。

・public Position findPosition(Issue issue)
Portfolioに含まれるPositionから、issueが同じPositionを探し出して返す。見つからない場合はnullを返す。

発生している問題

findPositionに関してはなんとなく作れるのですが、addPositionの方がどのような形になるのか分かりません。

どのようなコードになるのか、なぜそのような形になるのかまで教えていただければありがたいです。

よろしくお願いいたします。

Answer 1

制御の部分だけコード例を示しますね。

public void addPosition(Position p) {
    Position found = findPosition(p);
    if (found == null) {
      // p を配列に保存するコードをここに書く
    } else {
      // found の amount に p の amount を加算するコードをここに書く
    }
}

「なぜ」、については、上のコードの制御ロジックを理解できれば、特に説明の必要はないように思えますが、どうでしょうか。上で直接書いていない部分（コメントで説明だけしてある箇所）がわからないのであれば、そこにどのようなコードを書いたのかを具体的に示して、どのような結果になって困っているのかを書かれた方が、回答を得やすいと思いますよ。

なお、問題がそうなっているのなら仕方がありませんが、 addPosition メソッドは public でなければならないのでしょうか。もしクラスの外に公開する意図がないか、もっと狭い範囲で良いなら、public よりも狭い範囲のアクセス修飾子にされる方が良いです。(カプセル化とか、そいういう観点で)

Answer 2

findPosition作れるなら、見つけたPositionと検索対象のPositionのamountの合計を見つけたPositionにセットすればいいのでは？