Metoda konkatenacji 2 ciągów w Javie

Question

Mam metodę w Javie, która łączy 2 ciągi. Obecnie działa poprawnie, ale myślę, że można go napisać lepiej.

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

Oto niektóre z wymagań:

1. Jeśli oba str1 i słowo2 są null, metoda zwraca null
2. Jeśli któryś str1 lub słowo2 jest null, to po prostu zwracają not null String
3. Jeśli str1 i słowo2 nie są nieważne, to złączyć je
4. nigdy nie dodaje "null" do wyniku

Czy ktoś może zrobić to z mniejszym kodem?

Answer 1

Sure:

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

Zauważ, że ta zajmuje się "zarówno zerowej" przypadek w pierwszym warunkiem: jeśli str1 jest zerowy, to albo chce wrócić NULL (jeśli str2 jest null) lub str2 (jeśli str2 nie ma wartości NULL) - oba są obsługiwane po prostu zwracając str2.

Answer 2

Korzystanie tylko zwykły if punktach:

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

Albo, jeśli masz długą i namiętną miłość do nawiasach:

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

Answer 3

import org.apache.commons.lang.StringUtils; 

StringUtils.join([str1, str2]); 

Łączy elementy przewidziane tablicy w jeden Łańcuch zawierający podaną listę elementów.

Do połączonego ciągu nie dodano żadnego separatora. Obiekty puste lub puste ciągi w tablicy są reprezentowane przez puste ciągi.

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

Answer 4

Wszystkim wydaje się, że brakowało Warunek 1, gdzie jeśli oba ciągi są nieważne zwraca NULL. Najprostsza wersja do odczytu (IMO), a następnie staje się:

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

Answer 5

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

}