メソッド内でgetterを複数回使用することによる可読性について

Question

こんにちは。

現在、javaでプログラミングをしているのですが、
メソッドの中でgetterを複数回使用することは、
可読性の面で問題ないか疑問に感じています。

サンプルコードを２つ作成してみました。
どちらのように記述するのがいいのか、
助言を頂けますでしょうか？

宜しくお願致します。

###【サンプルコード１：変数に値を代入して、その変数を使用する】

public void sample01(List<SampleForm> sampleForms) {
    for (SampleForm sampleForm :  sampleForms) {

		String name = sampleForm.getName();
		String mail = sampleForm.getMail();
		String tell = sampleForm.getTell();

		boolean isAllNull = name == null && mail == null && tell == null;
		boolean isAllNotNull = name != null && mail != null && tell != null;

		if (isAllNull || isAllNotNull) {
			continue;
		}
		
        ---以下、name,mail,tellを複数回使用する---

	}
}

###【サンプルコード２：getterをそのまま使用する】

public void sample02(List<SampleForm> sampleForms) {
	for (SampleForm sampleForm :  sampleForms) {

		boolean isAllNull = sampleForm.getName() == null && sampleForm.getMail() == null && sampleForm.getTell() == null;
		boolean isAllNotNull = sampleForm.getName() != null && sampleForm.getMail() != null && sampleForm.getTell() != null;

		if (isAllNull || isAllNotNull) {
			continue;
		}
	        
        ---以下、各getterを複数回使用する---**

	}
}

###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Answer 1

一事実一箇所の原則に従うと、
getterなどの値を変数に入れ直すのはあまりよろしくないととある著名なリファクタリング本で見た覚えがあります。

むしろ可読性という観点で見ると、
全てNullを表すメソッド、
全て非Nullを表すメソッドをFormのメソッドとして定義した方が良いでしょう。

Answer 2

アクセッサメソッドを使うことに対しては可読性の問題はないと思いますよ。

ただ、非同期処理とかで別のスレッドで変更された時などを考慮する必要があるかどうかで分けないといけないでしょうね。

また、質問のコードではNullかどうかの判定メソッドをSampleFormクラスに実装することが最善だと思います。