Перевірте два аргументи в Java, або обидва не нульові, або обидва нульові елегантно

Я використовував весняне завантаження, щоб розробити проект оболонки, який використовується для надсилання електронної пошти, наприклад

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

Якщо аргументи 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Умова буде вірним , якщо тільки одна змінна дорівнює нулю.

Але я думаю, що зазвичай краще використовувати дві 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");
}

Він легше читається і набагато простіше зрозуміти, і для цього потрібно лише трохи додаткового набору тексту.

Ну, це здається, що ви намагаєтеся перевірити, чи однакова умова обох цих двох чи ні. Ви можете використовувати:

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()метод.

Ось загальне рішення для будь-якої кількості нульових перевірок

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

Додатковою перевагою є те, що якщо будь-який з ваших стандартних значень буде недійсним, це також буде виявлено.

Сказавши це, загалом я вважаю, що використання 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. Однак це вимагає, щоб ви були трохи докладнішими, але вгору ви можете використовувати спеціальні Винятки, які я запропонував, щоб отримати більш змістовне повідомлення про помилку.

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)

Добре працює для перевірки цього конкретного стану, але не узагальнює добре минулі дві змінні.

Як я бачу ваші наміри, не потрібно завжди перевіряти обидва виняткові недоліки, але перевіряти, чи passwordнемає, якщо і лише якщо fromне є нульовим. Ви можете ігнорувати даний 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 або non-null, і якщо це не null, обидва його поля мають ненульові значення.

Однією з переваг такого підходу є те, що помилка введення одного поля, а не іншого поля недійсне, спрацьовує там, де обліковий запис спочатку отримано, а не тоді, коли працює за допомогою облікового запису код. На час виконання коду, який використовує обліковий запис, дані не можуть бути недійсними.

Ще одна перевага цього підходу є більш читаною, оскільки вона забезпечує більше семантичної інформації. Крім того, цілком імовірно, що ви потребуєте імені та пароля разом в інших місцях, тому вартість визначення додаткового амортизаційного класу за декілька звичок.