Stringクラスがnullか空文字列かを判定する方法

Question

はじめに

いつもお世話になっております。
表題の事で質問があります。

あるListに格納されたStringクラスを判定して
null もしくは 空文字列の場合は falseを返す様にしたいです。

リストから取ってくる箇所は簡略して以下の様に修正しました。
しかしこの修正はプロっぽくないのでは？と思いました。
特に判定の前後を入れ替えるとExceptionが発生するとかありえないのでは？と思い
この判定はありなのか？

あるいは他の方法があるのかを教えて頂きたく投稿しました。
よろしくお願いいたします。

public class Test{
	
	/* st が null か空文字列の場合にerrorを表示する */
	public static boolean main(String []args){
	
		String st = null;
				
		/* これだと間抜けっぽい気がする */
//		if(st == null){
//			return  false;
//		}
//		if(st.isEmpty()){
//			return  false;
//		}

		/* これだと正しく動くけど判定が逆になるとExceptionを起こすから可読性や保守に問題がある様な気がする */
		if( st==null || st.isEmpty() ){
			System.out.print("result value is false");
			return false;
		}
		
//		/* これだとstにnullが入った場合はNullPointerExceptionを起こす */
//		if( st.isEmpty() || st==null ){
//		System.out.print("result value is false");
// 			return false;
//		}	
	}
}

---

遅くなって申し訳ありませんでした。
確認させて頂きました。
全員の方から問題ないとお墨付きを頂きとても安心しております。

ベストアンサーを選ぶのはとても迷いましたが、プロの目から見て間違いないと言い切ってくれた方につけさせて頂きました。

本当にありがとうございました。

Answer 1

    /* これだと正しく動くけど判定が逆になるとExceptionを起こすから可読性や保守に問題がある様な気がする */

    if( st==null || st.isEmpty() ){

この書き方で特に問題ありません。プロから見ても普通のコーディングだと思います。
ひと目みて逆に書くと Exception を起こすことがわからないような素人が保守した時点でもっと不幸なことがたくさん起きると思います。

Answer 2

「アリかナシか」
で言うとアリですが、メソッド化しておくと、なお良いと思います。

一度、メソッドを作ってしまえば後はそれを呼び出すだけで何度でも再利用できますし、
以下のようにコメントしておけば、下手な修正を入れられることも防げると考えるからです。

public static boolean isNullOrEmpty(String str) {
    /*
     * 判定が逆になるとExceptionを起こすから、逆にするなよ？
     * 絶対にするなよ？
     */
    return str == null || str.isEmpty();
}

もし、Jarファイルの追加が手間でなければ、Apache Commons の`Commons Lang'ライブラリが便利です。

https://commons.apache.org/proper/commons-lang/

StringUtilsクラスのisEmptyメソッドが、まさにご質問のケースで利用できます。
https://commons.apache.org/proper/commons-lang/javadocs/api-release/org/apache/commons/lang3/StringUtils.html#isEmpty-java.lang.CharSequence-

Checks if a CharSequence is empty ("") or null.

以下がその実装ですが、ご質問のコードとほぼ、同じことをしていますね。
http://grepcode.com/file/repo1.maven.org/maven2/commons-lang/commons-lang/2.6/org/apache/commons/lang/StringUtils.java#StringUtils.isEmpty%28java.lang.String%29

Answer 3

/* これだと正しく動くけど判定が逆になるとExceptionを起こすから可読性や保守に問題がある様な気がする */
        if( st==null || st.isEmpty() ){

プロなら短絡評価の考えは知っているべきで、不用意に判定を逆にするというのはしないはずです。プロであるならむしろ上記のような書き方を理解したうえで書いていそうなものです（個人的な感想）

Answer 4

StringUtils.isEmpty()を使えば、nullまたは""だった場合にtrueになります。

Answer 5

if(st==null || st.isEmpty())

if(st!=null && !st.isEmpty())

この手の書き方はもはや常套句として使っていいのでは？
逆になると例外になるとか心配していますが、むしろもっと簡単な「|や&をひとつ消す」という危険を考えないのはなぜなんだろう…

Answer 6

if (st == null ? true : st.isEmpty()) {

のように3項演算子で判定もできますが、if( st==null || st.isEmpty() ){の方がシンプルですね。