Java'da iki argümanı kontrol edin (ikisi de boş değil veya her ikisi de zarif olarak boş)

E-posta göndermek için kullanılan bir kabuk projesi geliştirmek için bahar önyükleme kullandım, örn.

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

Eğer fromve passwordargümanlar eksik, bir varsayılan gönderen ve şifre, örneğin kullanın noreply@bar.comve 123456.

Dolayısıyla, kullanıcı fromargümanı geçerse, argümanı da geçmelidir passwordve tersi de geçerlidir. Yani her ikisi de boş değil ya da her ikisi de boş.

Bunu zarif bir şekilde nasıl kontrol edebilirim?

Şimdi benim yolum

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

^( XOR ) operatörünü kullanmanın bir yolu vardır :

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

ifSadece bir değişken null ise koşul doğru olacaktır.

Ancak genellikle iffarklı istisna mesajlarıyla iki koşul kullanmak daha iyidir . Neyin yanlış gittiğini tek bir koşul kullanarak tanımlayamazsınız.

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

Daha okunabilir ve anlaşılması çok daha kolaydır ve sadece biraz fazladan yazma gerektirir.

Eh, ikisinin "sıfırlık" durumunun aynı olup olmadığını kontrol etmeye çalışıyorsunuz. Kullanabilirsin:

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

Veya yardımcı değişkenlerle daha açık hale getirin:

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

Şahsen, okunabilir ve zarif olmayı tercih ederim.

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

Bu işlevselliği imzalı 2 bağımsız değişken yöntemine yerleştirin:

void assertBothNullOrBothNotNull(Object a, Object b) throws RuntimeException

Bu, ilgilendiğiniz gerçek yöntemde yer tasarrufu sağlar ve daha okunabilir hale getirir. Biraz ayrıntılı yöntem adlarında yanlış bir şey yoktur ve çok kısa yöntemlerde yanlış bir şey yoktur.

Bir Java 8 çözümü Objects.isNull(Object), statik bir içe aktarma varsayarak kullanmak olacaktır :

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

Java <8 için (veya kullanmayı sevmiyorsanız Objects.isNull()) kendi isNull()yönteminizi kolayca yazabilirsiniz .

İşte herhangi bir sayıda boş kontrol için genel bir çözüm

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

Kullanımları

// 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) { .. }

Hem gönderen hem de şifre olmadığında özel bir şey yapmak istediğinizden (varsayılanları kullanın), önce bunu ele alın.
Bundan sonra, e-posta göndermek için hem gönderen hem de parolanız olmalıdır; her ikisi de eksikse bir istisna atar.

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

Ek bir avantaj, varsayılan değerlerinizden herhangi birinin boş olması durumunda, bunun da algılanmasıdır.

Bunu söyledikten sonra , genel olarak ihtiyacınız olan operatör bu olduğunda XOR kullanımına izin verilmesi gerektiğini düşünüyorum. Bu , dilin bir parçası, sadece bir sır derleyici hatası nedeniyle çalışan bir numara değil.
Bir keresinde üçlü operatörü kullanmak için kafa karıştırıcı bulan bir inek orkum vardı ...

Aslında nasıl bu kod parçası yazmak istiyorsunuz başka bir alternatif önermek istiyorum:

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

Bana göre bu, bu durumu ifade etmenin en doğal yolu gibi görünüyor, bu da gelecekte okuması gerekebilecek biri için anlaşılmasını kolaylaştırıyor. Ayrıca daha yararlı tanılama mesajları vermenize ve gereksiz parolayı daha az ciddi olarak ele almanıza olanak tanır ve bu tür bir durum için oldukça olası olan değiştirmeyi kolaylaştırır. Yani bir komut satırı argümanı olarak bir parola vermenin en iyi fikir olmadığını ve argüman eksikse isteğe bağlı olarak parolanın standart girdiden okunmasına izin vermek isteyebilirsiniz. Veya gereksiz parola bağımsız değişkenini sessizce yok saymak isteyebilirsiniz. Bu gibi değişiklikler, her şeyi yeniden yazmanızı gerektirmez.

Bunun yanı sıra, sadece asgari sayıda karşılaştırma yapar, bu yüzden daha "zarif" alternatiflerden daha pahalı değildir . Her ne kadar performans burada pek olası bir sorun olmasa da, yeni bir işlem başlatmak zaten boş bir kontrolden çok daha pahalı.

Bunu ele almanın doğru bir yolunun üç durumu göz önünde bulundurmak olduğunu düşünüyorum: 'hem' hem de 'şifre' sağlandı, ikisi de sağlanmadı, ikisinin bir karışımı sağlandı.

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.
}

Orijinal soru yanlış giriş ise akışı kırmak istiyor gibi görünüyor, ancak daha sonra mantığın bir kısmını tekrarlamak zorunda kalacaklar. Belki de iyi bir atış ifadesi şunlar olabilir:

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

İşte herhangi bir Xor og uzun ifs içermeyen nispeten basit bir yol. Bununla birlikte, biraz daha ayrıntılı olmanızı gerektirir, ancak tersine, daha anlamlı bir hata mesajı almak için önerdiğim özel İstisnaları kullanabilirsiniz.

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

Hiç kimse üçlü operatörden bahsetmemiş gibi görünüyor :

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

Bu özel durumu kontrol etmek için iyi çalışır, ancak iki değişkeni genelleştirmez.

Niyetlerinizi gördüğüm gibi, her zaman iki özel null'ı da kontrol etmenize gerek yoktur, ancak passwordnull olup olmadığını ve yalnızca null değilse kontrol edin from. Verilen passwordbağımsız değişkeni yok sayabilir ve fromnull ise kendi varsayılanınızı kullanabilirsiniz .

Sözde yazılmış böyle olmalı:

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
}

Kimsenin bir sınıfın basit bir çözümünü fromve passwordalanlarını ve bu sınıfın bir örneğine referans vermesini söylemediğine şaşırdım :

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

Burada fromnull veya null olmayabilir ve null değilse, her iki alanında da null olmayan değerler bulunur.

Bu yaklaşımın bir avantajı, hesabı kullanan kod çalıştığında değil, hesabın başlangıçta alındığı yerde bir alanı yapma, diğer alanı null yapma hatasının tetiklenmesidir. Hesabı kullanan kod yürütüldüğünde, verilerin geçersiz olması mümkün değildir.

Bu yaklaşımın bir diğer avantajı daha semantik bilgi sağladığı için daha okunabilir. Ayrıca, başka yerlerde ad ve şifreyi birlikte kullanmanız gerekebilir, bu nedenle ek bir sınıf tanımlama maliyeti birden fazla kullanım için amorti olur.