Javaで2つの引数をチェックします。どちらもnullでないか、どちらもエレガントにnullです

私は春のブートを使用して、メールの送信に使用するシェルプロジェクトを開発しました

sendmail -from foo@bar.com -password  foobar -subject "hello world"  -to aaa@bbb.com

fromおよびpassword引数がない場合は、デフォルトの送信者とパスワードを使用します（例：noreply@bar.comと）123456。

したがって、ユーザーがfrom引数を渡す場合、password引数も渡す必要があり、その逆も同様です。つまり、両方がnullでないか、両方がnullです。

これをエレガントにチェックするにはどうすればよいですか？

今私の方法は

if ((from != null && password == null) || (from == null && password != null)) {
    throw new RuntimeException("from and password either both exist or both not exist");
}

^（XOR）演算子を使用する方法があります：

if (from == null ^ password == null) {
    // Use RuntimeException if you need to
    throw new IllegalArgumentException("message");
}

if1つの変数のみがnullの場合は条件が真となります。

しかし、通常は、if例外メッセージが異なる2つの条件を使用する方が良いと思います。単一の条件を使用して問題の原因を定義することはできません。

if ((from == null) && (password != null)) {
    throw new IllegalArgumentException("If from is null, password must be null");
}
if ((from != null) && (password == null)) {
    throw new IllegalArgumentException("If from is not null, password must not be null");
}

読みやすく、理解しやすく、追加のタイピングが少しだけ必要です。

さて、2つの「無効」条件が同じであるかどうかを確認しようとしているようです。あなたは使うことができます：

if ((from == null) != (password == null))
{
    ...
}

または、ヘルパー変数を使用してより明示的にします。

boolean gotFrom = from != null;
boolean gotPassword = password != null;
if (gotFrom != gotPassword)
{
    ...
}

個人的には、エレガントよりも可読性を好みます。

if (from != null && password == null) {
    throw new RuntimeException("-from given without -password");
}
if (from == null && password != null) {
    throw new RuntimeException("-password given without -from");
}

その機能をシグネチャを使用して2つの引数を持つメソッドに配置します。

void assertBothNullOrBothNotNull(Object a, Object b) throws RuntimeException

これにより、関心のある実際のメソッドのスペースが節約され、読みやすくなります。少し冗長なメソッド名でも問題はなく、非常に短いメソッドでも問題はありません。

Java 8ソリューションはObjects.isNull(Object)、静的インポートを想定してを使用することになります。

if (isNull(from) != isNull(password)) {
    throw ...;
}

Java <8の場合（またはを使用したくない場合Objects.isNull()）、独自のisNull()メソッドを簡単に作成できます。

これは、任意の数のnullチェックの一般的な解決策です

public static int nulls(Object... objs)
{
    int n = 0;
    for(Object obj : objs) if(obj == null) n++;
    return n;
}

public static void main (String[] args) throws java.lang.Exception
{
    String a = null;
    String b = "";
    String c = "Test";

    System.out.println (" "+nulls(a,b,c));
}

用途

// equivalent to (a==null & !(b==null|c==null) | .. | c==null & !(a==null|b==null))
if (nulls(a,b,c) == 1) { .. }

// equivalent to (a==null | b==null | c==null)
if (nulls(a,b,c) >= 1) { .. }

// equivalent to (a!=null | b!=null | c!=null)
if (nulls(a,b,c) < 3) { .. }

// equivalent to (a==null & b==null & c==null)
if (nulls(a,b,c) == 3) { .. }

// equivalent to (a!=null & b!=null & c!=null)
if (nulls(a,b,c) == 0) { .. }

送信者とパスワードの両方がない場合に特別なことをしたいので（デフォルトを使用）、最初にそれを処理します。
その後、電子メールを送信するには送信者とパスワードの両方が必要です。どちらかがない場合は例外をスローします。

// use defaults if neither is provided
if ((from == null) && (password == null)) {
    from = DEFAULT_SENDER;
    password = DEFAULT_PASSWORD;
}

// we should have a sender and a password now
if (from == null) {
    throw new MissingSenderException();
}
if (password == null) {
    throw new MissingPasswordException();
}

もう1つの利点は、デフォルトのいずれかがnullの場合でも検出されることです。

そうは言っても、一般的には、XORの使用は、それが必要な演算子である場合は許容できると思います。それは言語の一部であり、難解なコンパイラバグのために機能するトリックだけではありません。
私はかつて、三項演算子を使用するには混乱しすぎていることに気付いた牛職人がいました...

私が実際にこのコードを書く方法である別の代替案を提案したいと思います：

if( from != null )
{
    if( password == null )
        error( "password required for " + from );
}
else
{
    if( password != null )
        warn( "the given password will not be used" );
}

私にとって、これはこの状態を表現する最も自然な方法であるように思われ、将来それを読む必要があるかもしれない誰かにとって理解しやすくします。また、より有用な診断メッセージを提供し、不要なパスワードをそれほど深刻ではないものとして扱うことができ、そのような状態になりそうな変更を簡単に行うことができます。つまり、コマンドライン引数としてパスワードを指定するのは最善の方法ではなく、引数がない場合はオプションで標準入力からパスワードを読み取れるようにすることもできます。または、余計なパスワード引数を黙って無視することもできます。このような変更では、全体を書き直す必要はありません。

その上、最小数の比較のみを実行するので、より「優雅な」代替よりも高価ではありません。新しいプロセスを開始することは、余分なnullチェックよりもはるかに高価であるため、パフォーマンスが問題になることはほとんどありません。

これを処理する正しい方法は、3つの状況を検討することだと思います。「from」と「password」の両方が提供され、どちらも提供されず、2つの組み合わせが提供されます。

if(from != null && password != null){
    //use the provided values
} else if(from == null && password == null){
    //both values are null use the default values
} else{
   //throw an exception because the input is not correct.
}

元の質問は、入力が正しくない場合はフローを中断したいようですが、後でロジックの一部を繰り返す必要があります。おそらく、適切なスローステートメントは次のようになります。

throw new IllegalArgumentException("form of " + form + 
    " cannot be used with a "
    + (password==null?"null":"not null") +  
    " password. Either provide a value for both, or no value for both"
);

Xor ogの長いifを含まない比較的簡単な方法を次に示します。ただし、少し冗長にする必要がありますが、利点として、私が提案したカスタムの例外を使用して、より意味のあるエラーメッセージを取得できます。

private void validatePasswordExists(Parameters params) {
   if (!params.hasKey("password")){
      throw new PasswordMissingException("Password missing");
   }
}

private void validateFromExists(Parameters params) {
   if (!params.hasKey("from")){
      throw new FromEmailMissingException("From-email missing");
   }
}

private void validateParams(Parameters params) {

  if (params.hasKey("from") || params.hasKey("password")){
     validateFromExists(params);
     validatePasswordExists(params);
  }
}

三項演算子については誰も言及していないようです。

if (a==null? b!=null:b==null)

この特定の条件をチェックするにはうまくいきますが、2つの変数をあまり一般化していません。

私があなたの意図を理解しているように、常に両方の排他的無効性をチェックする必要はありませんがpassword、fromがあります。がnullのpassword場合、指定された引数を無視して独自のデフォルトを使用できますfrom。

疑似で書かれたものは次のようでなければなりません：

if (from == null) { // form is null, ignore given password here
    // use your own defaults
} else if (password == null) { // form is given but password is not
    // throw exception
} else { // both arguments are given
    // use given arguments
}

クラスのとフィールドを作成fromしpassword、そのクラスのインスタンスへの参照を渡すという簡単な解決策について誰も言及しなかったことに驚いています。

class Account {
    final String name, password;
    Account(String name, String password) {
        this.name = Objects.requireNonNull(name, "name");
        this.password = Objects.requireNonNull(password, "password");
    }
}

// the code that requires an account
Account from;
// do stuff

ここに from nullまたは非nullになると、それは非ヌルだ場合、その両方のフィールドがnull以外の値を持つことができます。

このアプローチの利点の1つは、アカウントを使用するコードが実行されたときではなく、アカウントが最初に取得されたときに、一方のフィールドをnullにしないというエラーが発生することです。アカウントを使用するコードが実行されるまでに、データが無効になることはありません。

この方法のもう1つの利点は、セマンティックな情報が提供されるため、読みやすくなります。また、他の場所で名前とパスワードを一緒に必要とする可能性があるため、追加のクラスを定義するコストは、複数の使用に渡って償却されます。