Có luôn luôn là một ý tưởng tồi để sử dụng + để nối chuỗi

Question

tôi có mã như sau:

String s = ""; 
for (My my : myList) { 
    s += my.getX(); 
} 

FindBugs luôn báo cáo lỗi khi tôi làm điều này.

Answer 1

Tôi sẽ sử dụng + nếu bạn đang tự concatenating,

String word = "Hello"; 
word += " World!"; 

Tuy nhiên, nếu bạn đang đi làm lại và concatenating tôi sẽ đề nghị StringBuilder,

StringBuilder sb = new StringBuilder(); 
for (My my : myList) { 
    sb.append(my.getX()); 
} 

Answer 2

Đối tượng chuỗi không thay đổi được trong Java. Mỗi + có nghĩa là một đối tượng khác. Bạn có thể sử dụng StringBuffer để giảm thiểu số lượng các đối tượng đã tạo.

Answer 3

Trình biên dịch có thể tối ưu hóa một số điều như

"foo" + "bar"

Để

StringBuilder s1 = new StringBuilder(); s1.append ("foo"). Chắp thêm ("thanh");

Tuy nhiên điều này vẫn còn tối ưu vì nó bắt đầu với kích thước mặc định là 16. Như với nhiều thứ, bạn nên tìm cổ chai lớn nhất của mình và tìm cách xuống danh sách. Nó không bị tổn thương khi ở trong habbit sử dụng một mẫu SB từ khi bắt đầu, đặc biệt nếu bạn có thể tính toán kích thước khởi tạo tối ưu.

Answer 4

Tối ưu hóa sớm có thể xấu cũng như thường làm giảm khả năng đọc và thường hoàn toàn không cần thiết. Sử dụng + nếu nó dễ đọc hơn trừ khi bạn thực sự có mối quan tâm trọng yếu.

Answer 5

Nó không phải là 'luôn luôn xấu' để sử dụng "+". Sử dụng StringBuffer ở khắp mọi nơi có thể làm cho mã thực sự cồng kềnh.

Nếu ai đó đặt nhiều "+" ở giữa vòng lặp chuyên sâu, thời gian quan trọng, tôi sẽ cảm thấy khó chịu. Nếu ai đó đặt rất nhiều "+" trong một đoạn mã hiếm khi được sử dụng, tôi sẽ không quan tâm.

Answer 6

Tôi có thể nói sử dụng cộng trong những điều sau đây:

String c = "a" + "b"

Và sử dụng lớp StringBuilder nơi khác. Như đã đề cập trong trường hợp đầu tiên nó sẽ được tối ưu hóa bởi trình biên dịch và nó dễ đọc hơn.

Answer 7

Mỗi lần bạn làm string+=string, nó gọi phương pháp như thế này:

private String(String s1, String s2) { 
    if (s1 == null) { 
     s1 = "null"; 
    } 
    if (s2 == null) { 
     s2 = "null"; 
    } 
    count = s1.count + s2.count; 
    value = new char[count]; 
    offset = 0; 
    System.arraycopy(s1.value, s1.offset, value, 0, s1.count); 
    System.arraycopy(s2.value, s2.offset, value, s1.count, s2.count); 
} 

Trong trường hợp của StringBuilder, nói đến:

final void append0(String string) { 
    if (string == null) { 
     appendNull(); 
     return; 
    } 
    int adding = string.length(); 
    int newSize = count + adding; 
    if (newSize > value.length) { 
     enlargeBuffer(newSize); 
    } 
    string.getChars(0, adding, value, count); 
    count = newSize; 
} 

Như bạn có thể kết luận rõ ràng, string + string tạo ra rất nhiều trên không, và theo ý kiến ​​của tôi nên tránh nếu có thể.Nếu bạn nghĩ rằng sử dụng StringBuilder là cồng kềnh hoặc để lâu bạn chỉ có thể thực hiện một phương pháp và sử dụng nó một cách gián tiếp, như:

public static String scat(String... vargs) { 
    StringBuilder sb = new StringBuilder(); 

    for (String str : vargs) 
     sb.append(str); 

    return sb.toString(); 
} 

Và sử dụng nó như:

String abcd = scat("a","b","c","d"); 

Trong C# Tôi đang nói về mình giống như string.Concat();. Trong trường hợp của bạn nó sẽ là khôn ngoan để viết quá tải cho SCAT, như:

public static String scat(Collection<?> vargs) { 
    StringBuilder sb = new StringBuilder(); 

    for (Object str : vargs) 
     sb.append(str); 

    return sb.toString(); 
} 

Sau đó, bạn có thể gọi nó với:

result = scat(myList) 

Answer 8

Một trong những lý do tại sao FindBugs nên tranh luận về việc sử dụng toán tử nối (được nó "+" hoặc "+ =") là khả năng cục bộ. Trong ví dụ này bạn đã cho nó không phải là quá rõ ràng, nhưng trong trường hợp của đoạn mã sau đó là:

String result = "Scanning found " + Integer.toString(numberOfViruses) + " viruses"; 

Nếu điều này có vẻ hơi quen thuộc, bạn cần thay đổi phong cách mã hóa của bạn. Vấn đề là, nó sẽ âm thanh tuyệt vời bằng tiếng Anh, nhưng nó có thể là một cơn ác mộng đối với người dịch. Đó là bởi vì bạn không thể đảm bảo rằng thứ tự của câu sẽ vẫn như cũ sau khi dịch - một số ngôn ngữ sẽ được dịch sang "1 blah blah", một số thành "blah blah 3". Trong những trường hợp như vậy, bạn nên luôn sử dụng MessageFormat.format() để xây dựng các câu ghép và sử dụng toán tử nối là lỗi quốc tế hóa rõ ràng.

BTW. Tôi đặt một lỗi i18n khác ở đây, bạn có thể phát hiện ra nó không?

Answer 9

Thời gian chạy của hai chuỗi là tỷ lệ thuận với độ dài của chuỗi. Nếu nó được sử dụng trong một thời gian chạy vòng lặp luôn luôn tăng lên. Vì vậy, nếu nối là cần thiết trong một vòng lặp của nó tốt hơn để sử dụng StringBuilder như Anthony đề nghị.