Kiểm tra hai đối số trong Java, cả hai đều không null hoặc cả null

Tôi đã sử dụng boot mùa xuân để phát triển một dự án shell được sử dụng để gửi email, vd

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

Nếu thiếu fromvà passwordđối số, tôi sử dụng một người gửi và mật khẩu mặc định, ví dụ noreply@bar.comvà 123456.

Vì vậy, nếu người dùng vượt qua fromđối số, họ cũng phải vượt qua passwordđối số và ngược lại. Điều đó có nghĩa là, cả hai đều không có giá trị hoặc cả hai đều là null.

Làm thế nào để tôi kiểm tra điều này một cách thanh lịch?

Bây giờ cách của tôi là

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

Có một cách sử dụng toán tử ^( XOR ):

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

Điều ifkiện sẽ đúng nếu chỉ có một biến là null.

Nhưng tôi nghĩ thông thường tốt hơn là sử dụng hai ifđiều kiện với các thông báo ngoại lệ khác nhau. Bạn không thể xác định những gì đã sai bằng một điều kiện duy nhất.

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

Nó dễ đọc hơn và dễ hiểu hơn nhiều, và chỉ cần gõ thêm một chút.

Chà, có vẻ như bạn đang cố kiểm tra xem điều kiện "vô hiệu" của hai người có giống nhau hay không. Bạn đã có thể sử dụng:

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

Hoặc làm cho nó rõ ràng hơn với các biến của trình trợ giúp:

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

Cá nhân, tôi thích đọc hơn thanh lịch.

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

Đặt chức năng đó trong phương thức 2 đối số bằng chữ ký:

void assertBothNullOrBothNotNull(Object a, Object b) throws RuntimeException

Điều này tiết kiệm không gian trong phương pháp thực tế mà bạn quan tâm và làm cho nó dễ đọc hơn. Không có gì sai với các tên phương thức hơi dài dòng và không có gì sai với các phương thức rất ngắn.

Một giải pháp Java 8 sẽ được sử dụng Objects.isNull(Object), giả sử nhập tĩnh:

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

Đối với Java <8 (hoặc nếu bạn không thích sử dụng Objects.isNull()), bạn có thể dễ dàng viết isNull()phương thức của riêng mình .

Đây là một giải pháp chung cho bất kỳ số lượng kiểm tra 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));
}

Công dụng

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

Vì bạn muốn làm một cái gì đó đặc biệt (sử dụng mặc định) khi cả người gửi và mật khẩu đều vắng mặt, hãy xử lý trước.
Sau đó, bạn nên có cả người gửi và mật khẩu để gửi e-mail; ném một ngoại lệ nếu một trong hai là mất tích.

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

Một lợi ích bổ sung là, nếu một trong hai mặc định của bạn là null, điều đó cũng sẽ được phát hiện.

Phải nói rằng, nói chung tôi nghĩ rằng việc sử dụng XOR nên được cho phép khi đó là toán tử bạn cần. Nó là một phần của ngôn ngữ, không chỉ là một số mẹo hoạt động được do lỗi trình biên dịch phức tạp.
Tôi đã từng có một người chăn bò, người tìm thấy toán tử ternary quá khó hiểu để sử dụng ...

Tôi muốn đề xuất một phương án khác, đó là cách tôi thực sự viết đoạn mã này:

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

Đối với tôi đây dường như là cách tự nhiên nhất để diễn đạt tình trạng này, điều này giúp dễ hiểu cho những người có thể phải đọc nó trong tương lai. Nó cũng cho phép bạn đưa ra các thông báo chẩn đoán hữu ích hơn và coi mật khẩu không cần thiết là ít nghiêm trọng hơn và giúp dễ dàng sửa đổi, điều này có khả năng xảy ra đối với tình trạng như vậy. Tức là bạn có thể phát hiện ra rằng việc cung cấp mật khẩu làm đối số dòng lệnh không phải là ý tưởng tốt nhất và có thể muốn cho phép đọc mật khẩu từ đầu vào tiêu chuẩn tùy chọn nếu thiếu đối số. Hoặc bạn có thể muốn âm thầm bỏ qua đối số mật khẩu không cần thiết. Những thay đổi như thế này sẽ không yêu cầu bạn viết lại toàn bộ.

Bên cạnh đó, nó chỉ thực hiện số lượng so sánh tối thiểu, do đó, nó không đắt hơn các lựa chọn thay thế "thanh lịch" hơn. Mặc dù hiệu suất rất khó xảy ra ở đây vì bắt đầu một quy trình mới đắt hơn nhiều so với kiểm tra null.

Tôi nghĩ rằng một cách chính xác để xử lý vấn đề này là xem xét ba tình huống: cả 'từ' và 'mật khẩu' đều được cung cấp, không được cung cấp, cả hai đều được cung cấp.

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

Nghe có vẻ như câu hỏi ban đầu muốn phá vỡ dòng chảy nếu đó là đầu vào không chính xác, nhưng sau đó họ sẽ phải lặp lại một số logic sau đó. Có lẽ một tuyên bố ném tốt có thể là:

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

Đây là một cách tương đối đơn giản mà không liên quan đến bất kỳ if dài nào của Xor. Tuy nhiên, nó đòi hỏi bạn phải dài dòng hơn một chút, nhưng về mặt ưu điểm, bạn có thể sử dụng Ngoại lệ tùy chỉnh mà tôi đề xuất để nhận thông báo lỗi có ý nghĩa hơn.

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

Dường như không ai đề cập đến toán tử ternary :

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

Hoạt động độc đáo để kiểm tra điều kiện đặc biệt này, nhưng không khái quát quá hai biến.

Theo tôi thấy ý định của bạn, không cần phải luôn luôn kiểm tra cả hai giá trị độc quyền mà chỉ kiểm tra xem có phải passwordlà null hay không và nếu fromkhông phải là null. Bạn có thể bỏ qua passwordđối số đã cho và sử dụng mặc định của riêng bạn nếu fromlà null.

Viết bằng giả phải như thế này:

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
}

Tôi ngạc nhiên không ai đề cập đến giải pháp đơn giản về việc tạo fromvà passwordcác trường của một lớp và chuyển một tham chiếu đến một thể hiện của lớp đó:

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

Đây from có thể là null hoặc không null và nếu nó không null, cả hai trường của nó đều có giá trị khác null.

Một ưu điểm của phương pháp này là lỗi tạo một trường nhưng không phải trường khác null được kích hoạt ở nơi tài khoản ban đầu được lấy, không phải khi mã sử dụng tài khoản chạy. Vào thời điểm mã sử dụng tài khoản được thực thi, dữ liệu không thể không hợp lệ.

Một ưu điểm khác của phương pháp này là dễ đọc hơn vì nó cung cấp nhiều thông tin ngữ nghĩa hơn. Ngoài ra, có thể bạn yêu cầu tên và mật khẩu cùng nhau ở những nơi khác, do đó, chi phí xác định một lớp bổ sung sẽ được khấu hao theo nhiều cách sử dụng.