配列に格納した情報を他のクラスから参照したい

Question

前提・実現したいこと

配列に格納した情報を他のクラスから参照したい

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

他のクラスから参照するとnullになってしまう

該当のソースコード

package test2;

public class Main {
	public static void main(String args[]){
		SetInfo setin = new SetInfo();
		OutInfo outin = new OutInfo();
		
		setin.setI();
		outin.outI();
	}
}

package test2;

public class SetInfo {
	String[] name = {"yoshida","soken","moruboru"};
	int[] id = {10,20,30};
	
	Kakunou kaku = new Kakunou();
	
	public void setI(){
		for(int i = 0 ; i < 3 ; i++){
			kaku.setName(i, name[i]);
			kaku.setId(i, id[i]);
		}
	}

}

package test2;

public class OutInfo {
	Kakunou kaku = new Kakunou();
	public void outI(){
		for(int i=0;i<3;i++){
			System.out.println(kaku.getId(i) + ":" + kaku.getName(i));
		}
	}
}

package test2;

public class Kakunou {
	private String[] name = new String[3];
	private int[] id = new int[3];
	
	public void setName(int i,String name){
		this.name[i] = name;
	}
	
	public void setId(int i,int n){
		this.id[i] = n;
	}
	
	public String getName(int i){
		return this.name[i];
	}
	
	public int getId(int i){
		return this.id[i];
	}
}

試したこと

Kakunouの変数をpublicにしてkaku.name[i]で呼び出してみてもnullになってしまう

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

JDK 8
Version: Neon.3 Release (4.6.3)

Comments

[jimbe]

コードは, 入力欄の上にある <code> を押して出てくる ``` の間の"コード"を置き換える形にして頂けますでしょうか. (マークダウン)
そうすると専用の枠内でインデントが有効になって表示されるようになり, 枠内右上に[+]と表示されてコードがクリップボードにコピーできるようになります.
また, "ここに言語を入力" という文字列を "java" に書き換えますと, java のキーワードのフォントが変わる等, さらに見やすくなります.

[m.ts10806]

コードブロック(マークダウン)はファイルごとに分けてください。でないと正しく再現確認できませんので。(できればPackageも書いてほしい)

[moyashikun]

失礼しました
修正しました

Answer 1

setin.setI() でセットした内容を、 outin.outI() で出力したいと理解しました。

もしそうであれば、SetInfo クラスに、 getKoku() メソッドを作って、Kokunou クラスを戻りの型に指定して、 SetInfo クラスが保持する Kokunou クラスのインスタンスを返すようにしましょう。いわゆる getter を追加するということですね。

次に OutInfo クラスの outI() メソッドに引数を追加します。 outI(Kokunou koku) ですね。そして、outI() メソッド内の Kokunou koku = new Kokunow() の行を削除します。

最後に outin.outI() に、 setin.getKoku() の結果を渡します。 outin.outI(setin.getKoku()) のように。

これでコンパイルして実行すれば、outI メソッドの処理で、渡された Kokunou クラスのインスタンスの内容を表示してくれるはずです。

何が問題だったかというと、new 演算子で新しいインスタンスを作成するごとに、中身は初期状態のものになってしまっていたという点です。つまり SetInfo クラスの setI メソッドで作成した Kokunou クラスのインスタンスとは別のものです。
OutInfo クラスの outI メソッドで新しい Kokunou クラスのインスタンスを作ってその中身を表示していたため、中身は初期状態のままでしかないのです。

ですので、上の対処方法で示したのは次のような考え方です。

1. setI メソッドで作成した Kokunou クラスのインスタンスを取得できるようにして
2. 取得した Kokunou クラスのインスタンスを OutInfo クラスの outI メソッドに引き渡せるようにし
3. outI メソッドは引数の Kokunou クラスのインスタンスの中身を表示する、

Comments

[moyashikun]

ありがとうございます！

Answer 2

単純に, Kakunou オブジェクトを外で作り, setInfo/outInfo に共有させては如何でしょうか.

public class Main {
  public static void main(String args[]){
    Kakunou kaku = new Kakunou(); //追加
    SetInfo setin = new SetInfo(kaku); //パラメータ追加
    OutInfo outin = new OutInfo(kaku); //パラメータ追加

    setin.setI();
    outin.outI();
  }

  static class Kakunou {
    private String[] name = new String[3];
    private int[] id = new int[3];

    public void setName(int i,String name){
      this.name[i] = name;
    }

    public void setId(int i,int n){
      this.id[i] = n;
    }

    public String getName(int i){
      return this.name[i];
    }

    public int getId(int i){
      return this.id[i];
    }
  }

  static class SetInfo {
    private String[] name = {"yoshida","soken","moruboru"};
    private int[] id = {10,20,30};
    private Kakunou kaku;
    SetInfo(Kakunou kaku){ //コンストラクタ追加
      this.kaku = kaku;
    }

    public void setI(){
      for(int i = 0 ; i < 3 ; i++){
        kaku.setName(i, name[i]);
        kaku.setId(i, id[i]);
      }
    }
  }

  static class OutInfo {
    private Kakunou kaku;
    OutInfo(Kakunou kaku){ //コンストラクタ追加
      this.kaku = kaku;
    }
    public void outI(){
      for(int i=0;i<3;i++){
        System.out.println(kaku.getId(i) + ":" + kaku.getName(i));
      }
    }
  }
}

Comments

[moyashikun]

修正したプログラムまでありがとうございます！