Làm thế nào để viết lại hàm lặp lại này tốt hơn

Question

Tôi có chức năng sau đây lấy một chuỗi làm tham số và lặp lại nó một số lần (cũng là một tham số). Tôi cảm thấy như đây là một cái gì đó đã có trong khung hoặc ít nhất có thể được thực hiện tốt hơn. Bất kỳ đề xuất?

private string chr(string s, int repeat) 
{ 
    string result = string.Empty; 
    for (int i = 0; i < repeat; i++) 
    { 
     result += s; 
    } 
    return result; 
} 

Answer 1

chức năng tiếp cận lập trình theo kiểu:
(yêu cầu ít nhất là C# 3,0)

static class StringRepetitionExtension 
{ 
    public static string Times(this int count, string what) 
    { 
     return count > 0 ? string.Concat(what, (count-1).Times(what)) 
         : string.Empty; 
    } 
} 

Cách sử dụng:

3.Times("Foobar") // returns "FoobarFoobarFoobar" 

(Chắc chắn không phải là giải pháp hiệu quả nhất, và do sự đệ quy luôn có sự nguy hiểm của stack overflow với các giá trị bất hợp lý lớn cho count; nhưng tôi vẫn muốn chia sẻ một cách tiếp cận hơi khác, dễ hiểu.)

Answer 2

Bạn có thể muốn xem xét sử dụng StringBuilder nếu tham số lặp lại của bạn rất lớn.

http://msdn.microsoft.com/en-us/library/system.text.stringbuilder.aspx

http://channel9.msdn.com/forums/TechOff/14294-C-string-vs-StringBuilder/

Answer 3

Tôi muốn sử dụng một StringBuilder như hiện nay bạn đang có khả năng phân bổ và deallocating nhiều chuỗi:

private string chr(string s, int repeat) 
{ 
    StringBuilder result = new StringBuilder(); 
    for (int i = 0; i < repeat; i++) 
    { 
     result.Append(s); 
    } 
    return result.ToString(); 
} 

Hoặc thậm chí tốt hơn nếu chuỗi duy nhất là một đĩa đơn ký tự:

private string chr(char s, int repeat) 
{ 
    StringBuilder result = new StringBuilder(); 
    result.Append(s, repeat); 
    return result.ToString(); 
} 

Answer 4

Không phải là hiệu quả nhất, nhưng súc tích:

NET 4:

String.Join(String.Empty, Enumerable.Repeat(s, repeat)); 

NET 3.0/3.5:

String.Join(String.Empty, Enumerable.Repeat(s, repeat).ToArray()); 

Answer 5

Nếu đầu vào của bạn là thực sự là một nhân vật duy nhất chứ không phải là một chuỗi, bạn chỉ cần làm điều này:

var someChar = 'f'; 
var repeat = 10; 
var repeated = new String(someChar, repeat); 

Nếu không, tôi không nghĩ rằng còn nhiều việc khác để làm om sử dụng StringBuilder thay vì nối:

private string chr(string s, int repeat) 
{ 
    var result = new StringBuilder(s.Length * repeat); 
    for (int i = 0; i < repeat; i++) 
    { 
     result.Append(s); 
    } 
    return result.ToString(); 
} 

Answer 6

private string chr (string s, int repeat) { 
    string result = new String(' ', repeat).Replace(" ", s); 
    return result; 
} 

Answer 7

private string chr(string s, int repeat) 
{ 
    return Enumerable.Range(0, repeat) 
     .Aggregate(new StringBuilder(), (sb, i) => sb.Append(s)).ToString(); 
} 

Answer 8

Cải tiến quan trọng nhất mà bạn có thể thực hiện cho chức năng của mình là đặt tên mô tả.

Answer 9

return new System.Text.StringBuilder().Insert(0,"repeatme",count).ToString()