Periksa dua argumen di Jawa, keduanya tidak nol atau keduanya nol dengan elegan

Saya menggunakan spring boot untuk mengembangkan proyek shell yang digunakan untuk mengirim email, misalnya

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

Jika fromdan passwordargumen tidak ada, saya menggunakan pengirim dan kata sandi default, misalnya noreply@bar.comdan 123456.

Jadi, jika pengguna melewati fromargumen mereka juga harus melewati passwordargumen dan sebaliknya. Dengan kata lain, keduanya bukan nol, atau keduanya nol.

Bagaimana saya memeriksanya dengan elegan?

Sekarang jalan saya

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

Ada cara menggunakan operator ^( XOR ):

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

The ifKondisi akan benar jika hanya satu variabel adalah null.

Tapi saya pikir biasanya lebih baik menggunakan dua ifkondisi dengan pesan pengecualian yang berbeda. Anda tidak dapat menentukan apa yang salah dengan menggunakan satu kondisi.

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

Ini lebih mudah dibaca dan jauh lebih mudah dipahami, dan hanya perlu sedikit mengetik tambahan.

Yah, sepertinya Anda mencoba memeriksa apakah kondisi "nullity" keduanya sama atau tidak. Anda bisa menggunakan:

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

Atau membuatnya lebih eksplisit dengan variabel pembantu:

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

Secara pribadi, saya lebih suka dibaca daripada elegan.

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

Masukkan fungsi itu dalam metode 2 argumen dengan tanda tangan:

void assertBothNullOrBothNotNull(Object a, Object b) throws RuntimeException

Ini menghemat ruang dalam metode aktual yang Anda minati dan membuatnya lebih mudah dibaca. Tidak ada yang salah dengan nama metode yang sedikit verbose dan tidak ada yang salah dengan metode yang sangat pendek.

Solusi Java 8 akan digunakan Objects.isNull(Object), dengan asumsi impor statis:

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

Untuk Java <8 (atau jika Anda tidak suka menggunakan Objects.isNull()), Anda dapat dengan mudah menulis isNull()metode Anda sendiri .

Berikut ini adalah solusi umum untuk sejumlah cek nol

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

Penggunaan

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

Karena Anda ingin melakukan sesuatu yang istimewa (gunakan default) ketika pengirim dan kata sandi tidak ada, tangani terlebih dahulu.
Setelah itu, Anda harus memiliki pengirim dan kata sandi untuk mengirim email; lempar pengecualian jika salah satu tidak ada.

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

Manfaat tambahan adalah bahwa, jika salah satu dari default Anda menjadi nol, itu akan terdeteksi juga.

Karena itu, secara umum saya berpikir bahwa penggunaan XOR harus diizinkan ketika itu adalah operator yang Anda butuhkan. Ini adalah bagian dari bahasa, bukan hanya beberapa trik yang bekerja karena bug-compiler misterius.
Saya pernah punya sapi-orker yang menemukan operator ternary terlalu membingungkan untuk digunakan ...

Saya ingin menyarankan alternatif lain yaitu bagaimana saya benar-benar menulis potongan kode ini:

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

Bagi saya ini sepertinya cara paling alami untuk mengekspresikan kondisi ini yang membuatnya mudah dimengerti bagi seseorang yang mungkin harus membacanya di masa depan. Hal ini juga memungkinkan Anda untuk memberikan pesan diagnostik yang lebih bermanfaat dan memperlakukan kata sandi yang tidak perlu sebagai kurang serius dan membuatnya mudah untuk memodifikasi yang agak mungkin untuk kondisi seperti itu. Yaitu, Anda mungkin mengetahui bahwa memberikan kata sandi sebagai argumen baris perintah bukanlah ide terbaik dan mungkin ingin membolehkan membaca kata sandi dari input standar secara opsional jika argumen tersebut tidak ada. Atau Anda mungkin ingin mengabaikan argumen kata sandi berlebihan secara diam-diam. Perubahan seperti ini tidak mengharuskan Anda untuk menulis ulang semuanya.

Selain itu hanya menjalankan perbandingan minimum, sehingga tidak lebih mahal daripada alternatif yang lebih "elegan" . Meskipun kinerja sangat tidak mungkin menjadi masalah di sini karena memulai proses baru sudah jauh lebih mahal daripada pemeriksaan nol tambahan.

Saya pikir cara yang tepat untuk menangani ini adalah dengan mempertimbangkan tiga situasi: baik 'dari' dan 'kata sandi' disediakan, tidak ada yang disediakan, campuran keduanya disediakan.

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

Kedengarannya seperti pertanyaan asli yang ingin memutus aliran jika inputnya salah, tetapi kemudian mereka harus mengulang beberapa logika nanti. Mungkin pernyataan lemparan yang baik mungkin:

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

Berikut ini adalah cara yang relatif lurus ke depan yang tidak melibatkan Xor dan waktu panjang. Namun itu mengharuskan Anda untuk sedikit lebih bertele-tele, tetapi pada terbalik, Anda dapat menggunakan Pengecualian khusus yang saya sarankan untuk mendapatkan pesan kesalahan yang lebih bermakna.

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

Sepertinya tidak ada yang menyebut operator ternary :

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

Bekerja dengan baik untuk memeriksa kondisi khusus ini, tetapi tidak menyamaratakan melewati dua variabel.

Seperti yang saya lihat niat Anda, tidak perlu selalu memeriksa kedua nullities eksklusif tetapi untuk memeriksa apakah passworditu null jika dan hanya jika fromtidak nol. Anda dapat mengabaikan passwordargumen yang diberikan dan menggunakan default Anda sendiri jikafrom nol.

Ditulis dalam pseudo harus seperti ini:

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
}

Saya terkejut tidak ada yang menyebutkan solusi sederhana membuat fromdan passwordbidang kelas dan memberikan referensi ke contoh kelas itu:

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

Sini from bisa nol atau non-nol dan jika bukan nol, kedua bidangnya memiliki nilai bukan nol.

Salah satu keuntungan dari pendekatan ini adalah bahwa kesalahan membuat satu bidang tetapi bukan bidang lain nol dipicu di mana akun awalnya diperoleh, bukan ketika kode menggunakan akun berjalan. Pada saat kode menggunakan akun dieksekusi, data tidak mungkin menjadi tidak valid.

Keuntungan lain dari pendekatan ini lebih mudah dibaca karena memberikan lebih banyak informasi semantik. Juga, kemungkinan Anda memerlukan nama dan kata sandi bersama di tempat lain sehingga biaya untuk menentukan kelas tambahan diamortisasi selama beberapa penggunaan.