Question

我有三个字符串

String firstName; 
String lastName;
String companyName;

这是我的代码，我试图看看我是否可以更好地修改和重写

  if(firstName!=null && lastname == null) { comanyName = firstName;}
  if(lastname !=null && firstName== null) { comanyName = lastname ;}
  if(firstName!=null && lastname != null) { companyName = firstName.concat(lastName)}

谢谢

Answer 1

将所有字符串初始化为空字符串：

String firstName = ""; 
String lastName = "";
String companyName = "";

companyName = firstName + lastName;

现在涵盖了所有执行路径。其中一个是空的，它将具有另一个的值，如果两个都不为空，那么它将被连接。

Answer 2

最有效的方法取决于最常见的情况。重写这个以使用if-elseif结构，首先包括最常见的情况，然后是最常见的情况，等等。这样，当你的代码至少运行时，它将是最有效的。

Answer 3

您可以考虑使用if else而不是if。这样，并非所有if语句都能通过，因为它们是3个不同的情况。

     if(firstName!=null && lastname == null) { comanyName = firstName;}
     else if(lastname !=null && firstName== null) { comanyName = lastname ;}
     else if(firstName!=null && lastname != null) { companyName = firstName.concat(lastName)}

这是一种更清晰，更好的方法。

Answer 4

if (firstName != null) {
    if (lastName != null) {
        companyName = firstName.concat(lastName);
    }
    else {
        companyName = firstName;
    }
}

else {
    if (lastName != null) {
        companyName = lastName;
    }
    else {
        //What if they're both null?
    }
}

有效使用if语句

4 个答案: