0

下記のような実装についてうまい修正方法を教えていただきたいです

private boolean isChange(String oldStr, String newStr){
 if(StringUtils.isEmpty(oldStr)){
  if(StringUtils.isEmpty(newStr)){
    return true;
  }else{
    return false;
  }
 } else if (StringUtils.isEmpty(newStr)){
  return true;
 }
 if(!oldStr.equals(newStr)){
  return true;
 }
 return false;
}
1
  • 1
    if(StringUtils.isEmpty(oldStr)){ if(StringUtils.isEmpty(newStr)){ return true; }else{ return false;} の部分ですが、oldStrnewStr が両方ともに空文字列の場合に true を返して、oldStr が空文字列で newStr が空文字列ではない場合に false を返します。これは逆ではないでしょうか?
    – user39889
    Commented 2020年7月22日 13:05

2 件の回答 2

1

true が返る条件は,newが空であり、old.epual(new)がfalseの場合
つまり

private boolean isChange(old, new){
  if(StringUtils.isEmpty(new)){
    return true;
  }else{
    return !old.equal(new)
  }
}

もっと短くするのであれば下でも行けるかと。
可読性下がるのでおすすめはしませんが。

return StringUtils.isEmpty(new) ? (true) : (!old.equal(new))

あと、"new"は予約語なので適切な変数名にしたほうが良いです。
引数の型宣言もないので適切にする必要があります。

2
  • ぬるぽ例外が意外と罠なんですかね?
    – Jogenara
    Commented 2020年7月22日 11:02
  • 2
    コンパイルエラーがあります。equalはequalsの間違いですね。isChange("", "aaa")の場合、質問された方のコードと異なる動きをします。 metropolisさんのコメントどおり、回答された方のコードの方が正しいと思います。isChange("", "")のときtrueになるのは質問された方も回答のコードも誤りだと思います。 Commented 2020年7月22日 22:44
1

下記のような実装についてうまい修正方法を教えていただきたいです

isChangeのメソッド名から、新旧に変動があった場合はtrue、変動がなかったらfalseを返すメソッドを作りたいのだと勝手に解釈しました。

以下はコードの修正例です。StringUtilsは使っていません。
null""空文字列を同一視しません。nullから""に変化する場合、その逆も変動したと判定します。

private boolean isChange(String oldStr, String newStr) {
    if (oldStr == null && newStr == null) {
        return false;   //変動なし
    }
    else if (oldStr == null || newStr == null) {
        return true;    //変動あり
    }
    else if (newStr.equals(oldStr)) {
        return false;   //変動なし
    } else {
        return true;    //変動あり
    }
}


metropolisさんのコメントを参考にStringUtilsを調べてみました。
次のコードはnull""空文字列を同一視します。nullから""に変化する場合、その逆も変動なしと判定します。

if (StringUtils.isEmpty(oldStr) && StringUtils.isEmpty(newStr)) {
    return false;
}
else if (StringUtils.equals(oldStr, newStr)) {
    return false;
} else {
    return true;
}

次のコードはnull""空文字列を同一視しません。nullから""に変化する場合、その逆も変動ありと判定します。

private  boolean isChange(String oldStr, String newStr) {
    return !StringUtils.equals(oldStr, newStr);
}
3
  • 1
    StringUtils.equals() は "nulls are handled without exceptions" ですので、newStr.equals(oldStr)StringUtils.equals(oldStr, newStr) に書き替えると、else if (oldStr == null || newStr == null) { return true; } の部分は不要になります。
    – user39889
    Commented 2020年7月23日 6:32
  • StringUtils.equals()を調べてみました。使ったことがなく勉強になりました。metropolisさん、ありがとうございました。 Commented 2020年7月23日 8:56
  • とてもわかりやすい、説明ありがとうございます!
    – amiu
    Commented 2020年7月24日 8:34

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。