Java에서 두 개의 인수를 확인하십시오. null이 아니거나 우아하게 null입니다.

스프링 부트를 사용하여 이메일을 보내는 데 사용되는 쉘 프로젝트를 개발했습니다.

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

는 IF from및 password인수 누락, 나는 기본 발신자와 비밀번호, 예를 들어, 사용 noreply@bar.com및 123456.

따라서 사용자가 from인수를 전달하면 인수도 전달해야 password하며 그 반대도 마찬가지입니다. 즉, 둘 다 널이 아니거나 둘 다 널입니다.

이것을 우아하게 확인하려면 어떻게해야합니까?

이제 내 길은

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");
}

if하나의 변수가 null 인 경우 조건이 true가됩니다.

그러나 일반적으로 if예외 메시지가 다른 두 가지 조건 을 사용하는 것이 좋습니다 . 단일 조건을 사용하여 무엇이 잘못되었는지 정의 할 수 없습니다.

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");
}

더 읽기 쉽고 이해하기 쉬우 며 약간의 추가 입력 만하면됩니다.

음, 둘의 "nullity"조건이 같은지 아닌지 확인하려고하는 것 같습니다. 당신은 사용할 수 있습니다 :

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

이를 통해 실제 실제 방법으로 공간을 절약하고 더 읽기 쉽습니다. 약간 장황한 메소드 이름에는 문제가 없으며 매우 짧은 메소드에는 문제가 없습니다.

Objects.isNull(Object)정적 가져 오기를 가정하면 Java 8 솔루션은을 사용합니다 .

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();
}

추가 이점은 기본값 중 하나가 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" );
}

나에게 이것은 미래에 그것을 읽을 필요가있는 누군가를 이해하기 쉽게 만드는이 상태를 표현하는 가장 자연스러운 방법 인 것 같습니다. 또한보다 유용한 진단 메시지를 제공하고 불필요한 암호를 덜 심각하게 취급 할 수 있으며 이러한 조건에 적합한 것으로 쉽게 수정할 수 있습니다. 즉, 명령 행 인수로 비밀번호를 제공하는 것이 최선의 아이디어가 아니며 인수가 누락 된 경우 선택적으로 표준 입력에서 비밀번호를 읽도록 허용 할 수 있습니다. 또는 불필요한 암호 인수를 자동으로 무시할 수 있습니다. 이와 같은 변경으로 전체 내용을 다시 작성할 필요는 없습니다.

그 외에도 최소한의 비교 만 수행하므로 "우아한" 대안 보다 비싸지 않습니다 . 새 프로세스를 시작하는 것이 이미 추가 널 검사보다 훨씬 비싸기 때문에 성능이 문제가되지는 않습니다.

나는 이것을 처리하는 올바른 방법이 세 가지 상황을 고려하는 것이라고 생각합니다.

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 long 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)

이 특정 조건을 확인하는 데는 효과적이지만 두 변수를 지나쳐 일반화하지는 않습니다.

당신의 의도를 볼 때, 항상 독점적 인 passwordnull 을 모두 검사 할 필요 from는 없지만 null이 아닌 경우 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이 아닌 경우 두 필드 모두 null이 아닌 값을 갖습니다.

이 방법의 한 가지 장점은 계정을 사용하는 코드가 실행될 때가 아니라 계정을 처음 가져올 때 하나의 필드를 만들지 만 다른 필드를 null로 만드는 오류가 발생한다는 것입니다. 계정을 사용하는 코드가 실행될 때까지는 데이터가 유효하지 않습니다.

이 방법의 또 다른 장점은 더 의미적인 정보를 제공하기 때문에 더 읽기 쉽습니다. 또한 다른 장소에서 이름과 비밀번호를 함께 요구할 가능성이 있으므로 추가 클래스를 정의하는 데 드는 비용이 여러 용도로 사용됩니다.