सी # में एकाधिक स्ट्रिंग तत्वों की जगह

Question

वहाँ ऐसा करने का एक बेहतर तरीका है ...

MyString.Trim().Replace("&", "and").Replace(",", "").Replace(" ", " ") 
     .Replace(" ", "-").Replace("'", "").Replace("/", "").ToLower(); 

मैं एक ही काम करने के लिए इसे कम रखने के लिए स्ट्रिंग वर्ग बढ़ाया लेकिन वहाँ एक तेज तरीका है है?

public static class StringExtension 
{ 
    public static string clean(this string s) 
    { 
     return s.Replace("&", "and").Replace(",", "").Replace(" ", " ") 
       .Replace(" ", "-").Replace("'", "").Replace(".", "") 
       .Replace("eacute;", "é").ToLower(); 
    } 
} 

---

बस मस्ती के लिए मैं नीचे विभिन्न उदाहरणों बेंचमार्किंग एक सार अप shoved है (और टिप्पणियों में बहस को रोकने के लिए)।

https://gist.github.com/ChrisMcKee/5937656

regex विकल्प स्कोर बहुत; शब्दकोश विकल्प सबसे तेज़ आता है; स्ट्रिंगबिल्डर प्रतिस्थापन का लंबा घुमावदार संस्करण छोटा हाथ से थोड़ा तेज़ है।

Answer 1

तेज - नहीं। अधिक प्रभावी - हाँ, यदि आप StringBuilder कक्षा का उपयोग करेंगे। आपके कार्यान्वयन के साथ प्रत्येक ऑपरेशन एक स्ट्रिंग की एक प्रति उत्पन्न करता है जो परिस्थितियों में प्रदर्शन को खराब कर सकता है। स्ट्रिंग्स अपरिवर्तनीय ऑब्जेक्ट्स हैं इसलिए प्रत्येक ऑपरेशन केवल एक संशोधित प्रतिलिपि देता है।

यदि आप इस विधि को सक्रिय रूप से Strings पर सक्रिय रूप से बुलाए जाने की अपेक्षा करते हैं, तो इसके कार्यान्वयन को StringBuilder कक्षा पर "माइग्रेट" करना बेहतर हो सकता है। इसके साथ किसी भी संशोधन को सीधे उस उदाहरण पर किया जाता है, इसलिए आप अनावश्यक प्रतिलिपि संचालन को छोड़ देते हैं।

public static class StringExtension 
{ 
    public static string clean(this string s) 
    { 
     return new StringBuilder(s) 
       .Replace("&", "and") 
       .Replace(",", "") 
       .Replace(" ", " ") 
       .Replace(" ", "-") 
       .Replace("'", "") 
       .Replace(".", "") 
       .Replace("eacute;", "é") 
       .ToString() 
       .ToLower(); 
    } 
} 

Answer 2

इस और अधिक कुशल हो जाएगा?

public static class StringExtension { 

     private static Dictionary<string, string> _replacements = new Dictionary<string, string>(); 

     static StringExtension() { 
      _replacements["&"] = "and"; 
      _replacements[","] = ""; 
      _replacements[" "] = " "; 
      // etc... 
     } 

     public static string clean(this string s) { 
      foreach (string to_replace in _replacements.Keys) { 
       s = s.Replace(to_replace, _replacements[to_replace]); 
      } 
      return s; 
     } 
    } 

इसके अलावा StringBuilder के बारे में टाउन के सुझाव में नए जोड़ने ...

Answer 3

हो सकता है कि एक छोटे से अधिक पठनीय:

public static class StringExtention 
{ 
    public static string clean(this string s) 
    { 
     StringBuilder sb = new StringBuilder (s); 

     sb.Replace("&", "and"); 
     sb.Replace(",", ""); 
     sb.Replace(" ", " "); 
     sb.Replace(" ", "-"); 
     sb.Replace("'", ""); 
     sb.Replace(".", ""); 
     sb.Replace("eacute;", "é"); 

     return sb.ToString().ToLower(); 
    } 
} 

Answer 4

मैं ऐसी ही कुछ कर रहा हूँ, लेकिन मेरे मामले में मैं क्रमबद्धता/डी-क्रमबद्धता कर रहा हूँ तो मैं जाने के लिए सक्षम होना चाहिए दोनों दिशाओं में। मुझे एक स्ट्रिंग का उपयोग करना पड़ता है [] [] प्रारंभिक समेत शब्दकोश के लगभग समान रूप से काम करता है, लेकिन आप दूसरी दिशा भी जा सकते हैं, विकल्प को अपने मूल मूल्यों पर वापस कर सकते हैं, जो कुछ भी वास्तव में करने के लिए सेट नहीं किया गया है।

संपादित करें: आप क्रम में Dictionary<Key,List<Values>> उपयोग कर सकते हैं स्ट्रिंग के रूप में एक ही परिणाम प्राप्त करने के लिए [] []

Answer 5

आप एक सुंदर समाधान के बाद बस कर रहे हैं और कुछ नैनोसेकंड को बचाने की जरूरत नहीं है, तो कैसे कुछ LINQ चीनी के बारे में ?

var input = "test1test2test3"; 
var replacements = new Dictionary<string, string> { { "1", "*" }, { "2", "_" }, { "3", "&" } }; 

var output = replacements.Aggregate(input, (current, replacement) => current.Replace(replacement.Key, replacement.Value)); 

Answer 6

एक ऐसी चीज है जिसे सुझाए गए समाधानों में अनुकूलित किया जा सकता है। Replace() पर कई कॉल करने के लिए कोड एक ही स्ट्रिंग पर एकाधिक पास करने के लिए बनाता है। बहुत लंबे तारों के साथ समाधान सीपीयू कैश क्षमता याद आती है क्योंकि समाधान धीमा हो सकता है। किसी को replacing multiple strings in a single pass पर विचार करना चाहिए।

Answer 7

string input = "it's worth a lot of money, if you can find a buyer."; 
for (dynamic i = 0, repl = new string[,] { { "'", "''" }, { "money", "$" }, { "find", "locate" } }; i < repl.Length/2; i++) { 
    input = input.Replace(repl[i, 0], repl[i, 1]); 
} 

Answer 8

एक अन्य विकल्प LINQ का उपयोग कर

[TestMethod] 
public void Test() 
{ 
    var input = "it's worth a lot of money, if you can find a buyer."; 
    var expected = "its worth a lot of money if you can find a buyer"; 
    var removeList = new string[] { ".", ",", "'" }; 
    var result = input; 

    removeList.ToList().ForEach(o => result = result.Replace(o, string.Empty)); 

    Assert.AreEqual(expected, result); 
}