別クラスからのgetter呼び出しについて

Question

privateで宣言した変数とそれの getter setterを作成したクラスがあり、
別のクラスからgetterを呼び出して実行した際に、値が入らずnullになってしまいます。

試しに同じクラス内で、getterを呼び出したときは正しく値が取得できているみたいです。

public class A {
    private File document;
    
    public void hoge(){
        File file = getDocument(); //ここには変数が正しく設定される
        File file = document;      //1行上と同じ結果
    }
    public File getDocument() {
        return document;
    }
    public void setDocument(File document) {
        this.document = document;
    }
}

public class B {
    public void execute(){
    A a = new A();
    File file = a.getDocument();  //ここでは値が設定されずnullになる
    }
}

正直、正しく理解してないと思うので説明もおかしいかもしれません。。
どなたかご教授頂けたら幸いです。。

・・・
追記：失礼しました。ご指摘頂いた箇所につきまして、編集しました。

Comments

[moonphase]

コードは ``` で囲ってください。

[moonphase]

あと、同じクラス内で値が取得されている場合のコードも記載してください。

Answer 1

クラスAのコンストラクタで、documentの値の初期化は行っていますか？
クラスBのexecute()の内容が提示されているものだけで、クラスAのコンストラクタでも初期化を行っていないのなら、setDocument()で値の設定を行っていないのでnullになるのは正常な動作です。

クラスBのexecute()内では、クラスAのインスタンスを新たに生成しています。
このインスタンスはexecute()を実行する際に初めて生まれた実体です。
たとえクラスBの外側で一度クラスAのインスタンス（仮にa2）を生成していても、それとは別の実体になるので、documentの値は初期化時のままです。

public void main(){

File documentFile = new File();
A a2 = new A();
a2.setDocument(documentFile);
//a2が指すインスタンスに、documentFileを設定

B b = new B();
b.execute();
//exexute()の中でAのインスタンス a を生成するが、
//これは上のa2とは別の存在なので、documentFileを持ってはいない

}

Comments

[corazon]

そうですね・・・。
コンストラクタは作ってないと思います。
新しく生成されたインスタンス内でセッターが行われてないから＝nullということですよね。
クラスBで新しいインスタンスを生成せずにクラスAの変数documentを他クラスから直接使用できるようにすることもできますよね。
そちらの線で試行錯誤してみようかなぁ。
ご回答ありがとうございます。大変参考になりました。

[KaedeKazane]

>クラスBで新しいインスタンスを生成せずにクラスAの変数documentを他クラスから直接使用できるようにすることもできますよね。

直接使用というと語弊があるかもしれません。
変数documentはクラスA（のインスタンス）に属している変数なので、操作は基本的にクラスAのインスタンスを介して行われます。
どちらかと言えば、「既に変数documentに値を持っているAのインスタンスから情報をもらう」、という考え方です（例の場合は a2 から）。

クラス間での値の受け渡しや相互利用は、オブジェクト指向の理解が曖昧だと混乱するかもしれません。
この辺りに不安があれば、いちど通して解説している参考書等を読むことをおすすめします。

Answer 2

うまく行ってる方の呼び出し元では、A#setDocumentを呼んでいると思いますが・・・
あと、うまく行ってる方はコンパイルエラーになりませんか？変数が重複定義されています。

File file = getDocument(); //ここには変数が正しく設定される
File file = document;      //1行上と同じ結果

Comments

[corazon]

あ、失礼しました。 変数は変えてます。
たしかにこれでやるとコンパイルエラーになります。

Answer 3

setDocumentで値を設定してないからnullになっているだけでは？