ローカル変数を別メソッドで使用

Question

前提

クラスの先頭で配列
static int count = 0;
Position[] arrayposition = new Position[count +1];
を宣言し、それをメソッドaddPosition()で配列の値を定義しました。
その上で同じクラス内の別のメソッドfindPositionでその値を利用したいのですが、どのようにすれば利用することが可能でしょうか。

<エラーメッセージ>
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 1 out of bounds for length 1
at Portfolio.addPosition(Issue4.java:223)
at Issue100.main(Issue4.java:32)

実際に書いたコードはこちらです。
また全部のコードも記載致します。

コード
class Portfolio extends Position{
	static int count = 0;
	Position[] arrayposition = new Position[count +1];
	
	public void addPosition(Position position){
		if(count == 0){
			arrayposition[count] = position;
			count++;
		}
		else if(count > 0){
			for(int i = 0; count > i;i++){
				boolean bool = position.equals(arrayposition[i]);
				if(bool == true){
					amount++;
					break;
				}
				else if(bool == false){
					if(i == count -1){
						arrayposition[count] = position;
         			    count++;
					}
				}
			}
		}
	}

	public Position findPosition(Issue4 issue){
		boolean bool2;
			for(int i = 0;i < count + 1;i++){
			bool2 = issue.equals(arrayposition[i]);
				if(bool2 == true){
					return arrayposition[i];
				}
				else{
				}
			}
		return null;
	}
	
	@Override
	public int hashCode(){
		int result = 1;
		result = 31*result + ((issue == null)? 0:issue.hashCode());
		return result;
	}
	@Override
	public boolean equals(Object obj){
		if(this == obj){
			return true;
		}
		if(obj == null){
			return false;
		}
		if(getClass() != obj.getClass()){
			return false;
		}
		Portfolio port = (Portfolio)obj;
		
		if(!issue.equals(port.issue)){
			return false;
		}
		return true;
	}
}
java
コード全体
import java.io.*;

class Issue100{
	public static void main(String[] args){
		
		Issue4 issue1 = new Issue4("1111","AAAA");
		Issue4 issue2 = new Issue4("2222","BBBB");
		Issue4 issue3 = new Issue4("3333","CCCC");
		Issue4 issue4 = new Issue4("4444","DDDD");
		Issue4 issue5 = new Issue4("5555","EEEE");
		Issue4 issue6 = new Issue4("6666","FFFF");
		Issue4 issue7 = new Issue4("1111","AAAA");
		
		Stock stock1 = new Stock("1111","AAAA",Market.TSE);
		Stock stock2 = new Stock("2222","BBBB",Market.OSE);
		Stock stock3 = new Stock("3333","CCCC",Market.NSE);
		Bond bon1 = new Bond("1111","AAAA",20001111,1.0);
		Bond bon2 = new Bond("2222","BBBB",20002222,1.5);
		Bond bon3 = new Bond("3333","CCCC",20003333,2.0);
		
		
		Position pos1 = new Position(issue1,0);
		Position pos2 = new Position(issue2,0);
		Position pos3 = new Position(issue3,0);
		Position pos4 = new Position(issue4,0);
		Position pos5 = new Position(issue5,0);
		Position pos6 = new Position(issue6,0);
		
		Portfolio por = new Portfolio();
		
		por.addPosition(pos1);
		por.addPosition(pos2);
		/*
		Position a = por.findPosition(issue7);
		System.out.println(a);
		*/
	}
}
class Issue4{
	
	protected String code;
	protected String name;
	
	public Issue4(String code,String name)throws IllegalArgumentException{
		if(code == null || name == null){
			IllegalArgumentException e = new IllegalArgumentException();
			throw e;
		}
		else{
			this.code = code;
		    this.name = name;
		}
	}
	public String getCode(){
		return this.code;
	}
	public String getName(){
		return this.name;
	}
}

class Bond extends Issue4{
	
	private int maturity;
	private double coupon;
	
	public Bond(String code,String name,int maturity,double coupon)throws IllegalArgumentException{
		super(code,name);
		if(maturity <= 20000101 || maturity >= 29991231){
			IllegalArgumentException e = new IllegalArgumentException();
			throw e;
			}
		else{
			if(coupon < 0){
				IllegalArgumentException e = new IllegalArgumentException();
		    	throw e;
				}
			else{
            	this.maturity = maturity;
            	this.coupon = coupon;
				}
			}
	}
	public int getMaturity(){
		return this.maturity;
	}
	public double getCoupon(){
		return this.coupon;
	}
	public BondType getBondType(){
		if(coupon == 0){
			BondType bondtype = BondType.ZERO_COUPON_BOND;
			return bondtype;
		}
		else{
			BondType bondtype = BondType.COUPON_BOND;
			return bondtype;
		}
	}
	@Override
	public String toString(){
		return "コード :" + code + "\n" + "名称 :" + name + "\n" + "償還月日 :"+ this.maturity +
		"\n" + "クーポンレート :" + this.coupon ;
	}
	@Override
	public int hashCode(){
		double result = 1;
		result = 31*result + ((code == null)? 0:code.hashCode());
		result = 31*result + this.maturity;
		result = 31*result + this.coupon;
		int result2 = (int)result;
		return result2;
	}
	@Override
	public boolean equals(Object obj){
		if(this == obj){
			return true;
		}
		if(obj == null){
			return false;
		}
		if(getClass() != obj.getClass()){
			return false;
		}
		Bond bond = (Bond)obj;
		
		if(!code.equals(bond.code)){
			return false;
		}
		if(maturity != bond.maturity){
			return false;
		}
		if(coupon != bond.coupon){
			return false;
		}
		return true;
	}
}
class Stock extends Issue4{
	private Market market;
	
	public Stock(String code,String name,Market market){
		super(code,name);
		this.market = market;
	}
	public Market getMarket(){
		return this.market;
	}
	@Override
	public String toString(){
		return "コード :" + code + "\n" + "名称 :" + name + "\n" + "上場市場 :"+ this.market;
	}
	@Override
	public int hashCode(){
		int result = 1;
		result = 31*result + ((code == null)? 0:code.hashCode());
		result = 31*result + ((this.market == null)? 0:market.hashCode());
		return result;
	}
	@Override
	public boolean equals(Object obj){
		if(this == obj){
			return true;
		}
		if(obj == null){
			return false;
		}
		if(getClass() != obj.getClass()){
			return false;
		}
		Stock stock = (Stock)obj;
		
		if(!code.equals(stock.code)){
			return false;
		}
		if(market != stock.market){
			return false;
		}
		return true;
	}
}
class Position{
	
	protected Issue4 issue;
	protected double amount;
	
	public Position(){
	}
	public Position(Issue4 issue,double amount){
		this.issue = issue;
		this.amount = amount;
	}
	public Issue4 getIssue4(){
		return this.issue;
	}
	public double getAmount(){
		return this.amount;
	}
	public void setIssue4(Issue4 issue){
		this.issue = issue;
	}
	public void setAmount(double amount){
		this.amount = amount;
	}
}
class Portfolio extends Position{
	static int count = 0;
	Position[] arrayposition = new Position[count +1];
	
	public void addPosition(Position position){
		if(count == 0){
			arrayposition[count] = position;
			count++;
		}
		else if(count > 0){
			for(int i = 0; count > i;i++){
				boolean bool = position.equals(arrayposition[i]);
				if(bool == true){
					amount++;
					break;
				}
				else if(bool == false){
					if(i == count -1){
						arrayposition[count] = position;
         			    count++;
					}
				}
			}
		}
	}
/*
	public Position findPosition(Issue4 issue){
		boolean bool2;
			for(int i = 0;i < count + 1;i++){
			bool2 = issue.equals(arrayposition[i]);
				if(bool2 == true){
					return arrayposition[i];
				}
				else{
				}
			}
		return null;
	}
	*/
	@Override
	public int hashCode(){
		int result = 1;
		result = 31*result + ((issue == null)? 0:issue.hashCode());
		return result;
	}
	@Override
	public boolean equals(Object obj){
		if(this == obj){
			return true;
		}
		if(obj == null){
			return false;
		}
		if(getClass() != obj.getClass()){
			return false;
		}
		Portfolio port = (Portfolio)obj;
		
		if(!issue.equals(port.issue)){
			return false;
		}
		return true;
	}
}

Comments

[jimbe]

コードのご提示にはマークダウン記法を用いてください。

[javahack]

メソッドだけでなく、クラス全体をマークダウン記法で囲ってほしい。

[BeatStar]

エラーメッセージは省略せずに提示してください。お願いします。
エラーメッセージは「コンパイラ等からのメッセージ」です。

Answer 1

また全部のコードも記載致します。

全部ではないようです。Market.TSE などが未定義シンボルです。

	static int count = 0;
	Position[] arrayposition = new Position[count +1];

これでは arrayposition.length が 1 で、arrayposition[0] しか参照できません。
count++; を実行しても配列の要素数が勝手に増えることはありません。
最初に要素数を 100 にしてみると、次のようになります。

import java.io.*;

enum Market { TSE, OSE, NSE };						// ★
enum BondType { ZERO_COUPON_BOND, COUPON_BOND };	// ★

class Issue100{
	public static void main(String[] args){
		
		Issue4 issue1 = new Issue4("1111","AAAA");
		Issue4 issue2 = new Issue4("2222","BBBB");
		Issue4 issue3 = new Issue4("3333","CCCC");
		Issue4 issue4 = new Issue4("4444","DDDD");
		Issue4 issue5 = new Issue4("5555","EEEE");
		Issue4 issue6 = new Issue4("6666","FFFF");
		Issue4 issue7 = new Issue4("1111","AAAA");
		
		Stock stock1 = new Stock("1111","AAAA",Market.TSE);
		Stock stock2 = new Stock("2222","BBBB",Market.OSE);
		Stock stock3 = new Stock("3333","CCCC",Market.NSE);
		Bond bon1 = new Bond("1111","AAAA",20001111,1.0);
		Bond bon2 = new Bond("2222","BBBB",20002222,1.5);
		Bond bon3 = new Bond("3333","CCCC",20003333,2.0);
		
		
		Position pos1 = new Position(issue1,0);
		Position pos2 = new Position(issue2,0);
		Position pos3 = new Position(issue3,0);
		Position pos4 = new Position(issue4,0);
		Position pos5 = new Position(issue5,0);
		Position pos6 = new Position(issue6,0);
		
		Portfolio por = new Portfolio();
		
		por.addPosition(pos1);
		por.addPosition(pos2);

		Position a = por.findPosition(issue7);					// ★
		System.out.println(a);									// ★
		Issue4 i4 = a.getIssue4();								// ★
		System.out.println(i4.getCode() + ", " + i4.getName());	// ★
	}
}
class Issue4{
	
	protected String code;
	protected String name;
	
	public Issue4(String code,String name)throws IllegalArgumentException{
		if(code == null || name == null){
			IllegalArgumentException e = new IllegalArgumentException();
			throw e;
		}
		else{
			this.code = code;
		    this.name = name;
		}
	}
	public String getCode(){
		return this.code;
	}
	public String getName(){
		return this.name;
	}
}

class Bond extends Issue4{
	
	private int maturity;
	private double coupon;
	
	public Bond(String code,String name,int maturity,double coupon)throws IllegalArgumentException{
		super(code,name);
		if(maturity <= 20000101 || maturity >= 29991231){
			IllegalArgumentException e = new IllegalArgumentException();
			throw e;
			}
		else{
			if(coupon < 0){
				IllegalArgumentException e = new IllegalArgumentException();
		    	throw e;
				}
			else{
            	this.maturity = maturity;
            	this.coupon = coupon;
				}
			}
	}
	public int getMaturity(){
		return this.maturity;
	}
	public double getCoupon(){
		return this.coupon;
	}
	public BondType getBondType(){
		if(coupon == 0){
			BondType bondtype = BondType.ZERO_COUPON_BOND;
			return bondtype;
		}
		else{
			BondType bondtype = BondType.COUPON_BOND;
			return bondtype;
		}
	}
	@Override
	public String toString(){
		return "コード :" + code + "\n" + "名称 :" + name + "\n" + "償還月日 :"+ this.maturity +
		"\n" + "クーポンレート :" + this.coupon ;
	}
	@Override
	public int hashCode(){
		double result = 1;
		result = 31*result + ((code == null)? 0:code.hashCode());
		result = 31*result + this.maturity;
		result = 31*result + this.coupon;
		int result2 = (int)result;
		return result2;
	}
	@Override
	public boolean equals(Object obj){
		if(this == obj){
			return true;
		}
		if(obj == null){
			return false;
		}
		if(getClass() != obj.getClass()){
			return false;
		}
		Bond bond = (Bond)obj;
		
		if(!code.equals(bond.code)){
			return false;
		}
		if(maturity != bond.maturity){
			return false;
		}
		if(coupon != bond.coupon){
			return false;
		}
		return true;
	}
}
class Stock extends Issue4{
	private Market market;
	
	public Stock(String code,String name,Market market){
		super(code,name);
		this.market = market;
	}
	public Market getMarket(){
		return this.market;
	}
	@Override
	public String toString(){
		return "コード :" + code + "\n" + "名称 :" + name + "\n" + "上場市場 :"+ this.market;
	}
	@Override
	public int hashCode(){
		int result = 1;
		result = 31*result + ((code == null)? 0:code.hashCode());
		result = 31*result + ((this.market == null)? 0:market.hashCode());
		return result;
	}
	@Override
	public boolean equals(Object obj){
		if(this == obj){
			return true;
		}
		if(obj == null){
			return false;
		}
		if(getClass() != obj.getClass()){
			return false;
		}
		Stock stock = (Stock)obj;
		
		if(!code.equals(stock.code)){
			return false;
		}
		if(market != stock.market){
			return false;
		}
		return true;
	}
}
class Position{
	
	protected Issue4 issue;
	protected double amount;
	
	public Position(){
	}
	public Position(Issue4 issue,double amount){
		this.issue = issue;
		this.amount = amount;
	}
	public Issue4 getIssue4(){
		return this.issue;
	}
	public double getAmount(){
		return this.amount;
	}
	public void setIssue4(Issue4 issue){
		this.issue = issue;
	}
	public void setAmount(double amount){
		this.amount = amount;
	}
}
class Portfolio extends Position{
	static int count = 0;
	Position[] arrayposition = new Position[100];	// ★
	
	public void addPosition(Position position){
		if(count == 0){
			arrayposition[count] = position;
			count++;
		}
		else if(count > 0){
			for(int i = 0; count > i;i++){
				boolean bool = position.equals(arrayposition[i]);
				if(bool == true){
					amount++;
					break;
				}
				else if(bool == false){
					if(i == count -1){
						arrayposition[count] = position;
         			    count++;
					}
				}
			}
		}
	}

	public Position findPosition(Issue4 issue){
		boolean bool2;
		for(int i = 0;i < count + 1;i++){
			bool2 = issue.getCode().equals(arrayposition[i].getIssue4().getCode());	// ★
			if(bool2 == true){
				return arrayposition[i];
			}
			else{
			}
		}
		return null;
	}

	@Override
	public int hashCode(){
		int result = 1;
		result = 31*result + ((issue == null)? 0:issue.hashCode());
		return result;
	}
	@Override
	public boolean equals(Object obj){
		if(this == obj){
			return true;
		}
		if(obj == null){
			return false;
		}
		if(getClass() != obj.getClass()){
			return false;
		}
		Portfolio port = (Portfolio)obj;
		
		if(!issue.equals(port.issue)){
			return false;
		}
		return true;
	}
}

findPositionメソッドも変更が必要ですが、これでいいのでしょうか？

Comments

[icebreak]

恐らくfindPosition()メソッドも変更が必要だとは思うのですが、どのように変更すれば良いのか見当がつかない状態です。

[kazuma-s]

code が一致すれば見つかったとしていいんですか？
code と name の両方が一致しないとだめなんですか？
for文の i < count + 1 は間違いで、i < count でしょう。

[icebreak]

返信遅くなり、申し訳ございません。
基本的にissueが同じ場合一致したと判定するため、codeとnameの両方が一致する必要があります。
また、最初の質問に戻るのですが、あらかじめ配列の要素数を１００と定義するやり方以外にメソッドが呼び出された回数分だけ配列の要素数を作成することは出来ないのでしょうか。

Answer 2

そのクラス内であれば、arraypositionはそのまま使えます

基本的に、定義された変数は定義されたブロック｛｝内で有効です

Comments

[icebreak]

addPosition()メソッドで配列の値を定義するのですが、そこで定義した値はfindPosition()メソッドで使用可能でしょうか。もし使えない場合はどうすれば利用できるようになるのかご教授いただければ幸いです。

[y_waiwai]

arraypositionはそれらの関数の外で定義されてるんで、どちらの関数からでも使えます
しかし、関数の実行の順番には気をつけよう

で、それでなんか問題はあるの？

[icebreak]

このコードのままだと実行時に　 Index 1 out of bounds for length 1　というエラーが発生するため、恐らく配列が存在しないことになっているのではないかと考えています。

[y_waiwai]

それは配列のサイズが１しかないのに、１のインデックスでは範囲を超えてる、というエラーです。

存在しないというエラーではないです