String型の変数の中身がNullになってしまう

Question

前提・実現したいこと

String型の変数に受け取った文字列を代入したいのですが中身がNullになってしまいます。
変数にstaticを使用しないようにしたくて一週間ぐらい色々試してみたのですがどうしてNullになってしまうのか分かりません。
どなたか助けて頂けると助かります。

該当のソースコード

public class MainHandler implements ActionListener {
	
	UI ui;
	String position;//正しく入れば使用可能、代入出来てない？
	String test;
	
	public MainHandler(UI userInterface) {		
		ui = userInterface;	
	}

	public void select_location(String sl) {
		this.position = sl;//ここが間違っている？
		this.test = "test";//ここが間違っている？
		System.out.println(sl);//ここは問題なし
		System.out.println(test);//ここは問題なし
		
	}
public void actionPerformed(ActionEvent event) {
		
		String yourChoice = event.getActionCommand();
		
		Atorie lc = new Atorie(ui);
		
		System.out.println(this.position);//ここがNullになってしまう
		System.out.println(this.test);//ここがNullになってしまう
		
		switch(this.position) {//ここがNullになってしまう

		case"atorie_townGate":
				switch(yourChoice) {
				case"talk":lc.atorie_TalkGuard();break;
				}
		break;
		
		case"atorie_TalkGuard":
				switch(yourChoice) {
				case"talk":lc.atorie();break;
				}
		break;			
		}			
	}		
}

Comments

[maisumakun]

「select_location」はどこから実行するものでしょうか？

[clothblue]

別パッケージのクラスから実行しております。
質問の答えになっていなかったらすみません。

[maisumakun]

actionPerformedは、確実にselect_locationを実行したあとに実行されますか？

[clothblue]

申し訳ないのですがプログラミングにまだ慣れていなくて流れを追って確認するのに時間がかかってしまいそうです。
一度実行順序に問題がないかしっかり確認してみたいと思います。
ご親切にありがとうございます。

[swordone]

唐突にAtorieというのが出てきていますが、これはなんですか？

[clothblue]

AtorieクラスでactionPerformedが実行される前に一度実行されていました。
実行結果なのですがactionPerformedが実行されるとnullになってしまうようです。

実行結果
atorie_townGate//position変数の中身
test//test変数の中身
null//position変数の中身
null//test変数の中身

[clothblue]

Atorieクラス内にswitch以下で実行されるメソッドが書かれており、その各メソッドの中でselect_locationが実行され更新する流れになっております。

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

[clothblue]

大変失礼致しました。修正致します。

[Kaleidoscope]

>実行結果
その順番で出力されるのであれば、同じものを操作しているつもりで別のインスタンスを操作している可能性がありますね。

[clothblue]

おっしゃって頂いた可能性が高いので確認させて頂きます。ありがとうございます。

Answer 1

Atorieクラス内にswitch以下で実行されるメソッドが書かれており、その各メソッドの中でselect_locationが実行され更新する流れになっております。

でしたら、1回目にactionPerformedへ突入したときにはAtorieクラス内のメソッドは実行前ですので、this.positionやthis.testは初期値のnullのままになるのが必然かと思います。

Comments

[clothblue]

こちらactionPerformedへ入る前に別のクラスでselect_locationを一度実行しているのですが、なぜ初期値のnullになってしまうのでしょうか？。
初期化されていないから初期値のnullになってしまう事は理解いたしました。
根本的なことが分かっていないみたいでご迷惑おかけします。

[maisumakun]

> こちらactionPerformedへ入る前に別のクラスでselect_locationを一度実行しているのですが

それを先に言ってください（そんな事情を知らない上での回答です）。

[clothblue]

すみません、頂いた質問の返事として記載させて頂いたのですが上手くお伝え出来ていなかったようです。
大変失礼致しました。以後気を付けます。

[clothblue]

度々すみません、返事の内容がとても分かりにくかったです。伝わらなくて当然でした。申し訳ないです。
厳密にはAtorieクラスとは別のクラスで一度実行しているとう事でした。

[maisumakun]

select_locationを実行したのは、別なMainHandlerのインスタンスに対してだった、ということはありませんか？

[clothblue]

確認致します。

[clothblue]

position変数に初期値を入れる事でnullは回避出来て問題なく動いたのですが、今度は条件分岐のためにpositionを更新することが出来なくなってしまいました。常に最初の初期値が入ってしまいす。
やはり基本が分かっていないようなので一度最初から見直してみたいと思います。