Способ объединения двух строк в Java

Question

У меня есть метод в Java, который объединяет 2 строки. В настоящее время это работает правильно, но я думаю, что это может быть написано лучше.

public static String concat(String str1, String str2) {
  String rVal = null;
  if (str1 != null || str2 != null) {
    rVal = "";
    if (str1 != null) {
      rVal += str1;
    }
    if (str2 != null) {
      rVal += str2;
    }      
  }    
  return rVal;
}

Вот некоторые требования:

1. Если оба значения str1 и str2 равны нулю, метод возвращает значение null
2. Если str1 или str2 равны нулю, он просто вернет ненулевую строку
3. Если str1 и str2 не равны NULL, он объединяет их
4. Он никогда не добавляет «ноль» к результату

Кто-нибудь может сделать это с меньшим количеством кода?

Answer 1

Использование только простых if предложений:

public static String concat(String str1, String str2) {
    if(str1==null) return str2;
    if(str2==null) return str1;
    return str1 + str2;
}

Или, если у вас глубокая и страстная любовь к скобкам:

public static String concat(String str1, String str2) {
    if(str1==null)
    { 
        return str2;
    }
    if(str2==null) 
    {
        return str1;
    }
    return str1 + str2;
}

Answer 2

Sure:

public static String concat(String str1, String str2) {
  return str1 == null ? str2
      : str2 == null ? str1
      : str1 + str2;
}

Обратите внимание, что это учитывает случай «обоих значений, равных нулю» в первом условии: если str1 равно нулю, то вы либо хотите вернуть значение null (если str2 равно нулю), либо str2 (если str2 не равно нулю) - оба из них обрабатываются просто возвращая str2.

Answer 3

import org.apache.commons.lang.StringUtils;

StringUtils.join([str1, str2]);

Объединяет элементы предоставленного массива в одну строку, содержащую предоставленный список элементов.

Разделитель не добавляется к объединенной строке. Нулевые объекты или пустые строки в массиве представлены пустыми строками.

 StringUtils.join(null)            = null
 StringUtils.join([])              = ""
 StringUtils.join([null])          = ""
 StringUtils.join(["a", "b", "c"]) = "abc"
 StringUtils.join([null, "", "a"]) = "a"

Answer 4

public class ConcatTest extends TestCase {

    // 1. If both str1 and str2 are null, the method returns null
    public void testBothNull() throws Exception {
        assertNull(concat(null, null));
    }

    // 2. If either str1 or str2 is null, it will just return the not null
    // String
    public void testOneNull() throws Exception {
        assertEquals("a", concat(null, "a"));
        assertEquals("b", concat("b", null));
    }

    // 3. If str1 and str2 are not null, it will concatenate them
    public void testNonNull() throws Exception {
        assertEquals("ab", concat("a", "b"));
    }

    // 4. It never adds "null" to the result (not really testable)

    public static String concat(String a, String b) {
        if (a == null && b == null)
            return null;
        return denulled(a) + denulled(b);
    }

    private static String denulled(String s) {
        return s == null ? "" : s;
    }

}

Answer 5

Кажется, что все пропустили условие 1, где, если обе строки равны нулю, он возвращает ноль. Простейшая версия для чтения (IMO) становится:

public static String concat(String str1, String str2) {  
    if(str1==null && str2==null) return null;  
    if(str1==null) return str2;  
    if(str2==null) return str1;  
    return str1 + str2;  
}