इस दोहराए गए फ़ंक्शन को बेहतर ढंग से लिखने के लिए कैसे करें

Question

मेरे पास निम्न फ़ंक्शन है जो पैरामीटर के रूप में स्ट्रिंग लेता है और इसे कई बार (पैरामीटर भी) दोहराता है। मुझे लगता है कि यह ऐसा कुछ है जो ढांचे में पहले से ही है या कम से कम बेहतर किया जा सकता है। कोई सुझाव?

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

Answer 1

कार्यात्मक प्रोग्रामिंग शैली दृष्टिकोण:
(की आवश्यकता कम से कम सी # 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; 
    } 
} 

उपयोग:

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

(निश्चित रूप से नहीं सबसे कारगर समाधान है, और वहाँ हमेशा count के लिए अनुचित रूप से मान अधिक ढेर अतिप्रवाह के खतरे है प्रत्यावर्तन के कारण; लेकिन फिर भी मैं थोड़ा अलग, आसानी से समझने वाला दृष्टिकोण साझा करना चाहता था।)

Answer 2

यदि आपका दोहराना पैरामीटर बहुत बड़ा है तो आप स्ट्रिंगबिल्डर का उपयोग करने पर विचार करना चाहेंगे।

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

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

Answer 3

मैं तार के एक StringBuilder जैसा कि वर्तमान में आप संभावित आवंटन कर रहे हैं और बहुत सारे deallocating का उपयोग करेंगे:

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

या और भी बेहतर अगर स्ट्रिंग केवल एक ही है चरित्र:

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

Answer 4

नहीं सबसे कुशल, लेकिन संक्षिप्त:

नेट 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

अपने इनपुट वास्तव में एक भी चरित्र नहीं बल्कि एक स्ट्रिंग से है, तो आप बस यह कर सकते हैं:

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

अन्यथा, मुझे नहीं लगता कि वहाँ एक तरफ करने के लिए और कुछ है ओम StringBuilder बजाय का उपयोग कर संयोजन की:

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

आपके कार्य में सबसे महत्वपूर्ण सुधार यह एक वर्णनात्मक नाम देना है।

Answer 9

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