मैं इस विधि को कैसे बढ़ा सकता हूं जो एक स्ट्रिंग से टेक्स्ट हटा देता है?

Question

मैंने पर निम्न विधि लिखा है तारों से ब्रैकेट में नामस्थान हटा दें।

मैं इसे तेज़ जितना संभव हो उतना बनाना चाहता हूं।

क्या निम्न कोड को गति देने का कोई तरीका है?

using System; 

namespace TestRemoveFast 
{ 
    class Program 
    { 
     static void Main(string[] args) 
     { 
      string[] tests = { 
      "{http://company.com/Services/Types}ModifiedAt", 
      "{http://company.com/Services/Types}CreatedAt" 
          }; 

      foreach (var test in tests) 
      { 
       Console.WriteLine(Clean(test)); 
      } 

      Console.ReadLine(); 
     } 

     static string Clean(string line) 
     { 
      int pos = line.IndexOf('}'); 
      if (pos > 0) 
       return line.Substring(pos + 1, line.Length - pos - 1); 
      else 
       return line; 
     } 
    } 
} 

Answer 1

आप समांतरता का प्रयास कर सकते हैं क्योंकि ऐसा लगता है कि आपको एक तुल्यकालिक उपचार की आवश्यकता नहीं है। PLINQ के साथ समानांतर foreach चाल करेगा।

लेकिन अगर आप जब तक VS2010 आधिकारिक तौर पर बाहर है इंतजार नहीं कर सकता, तो आप Poor Man's Parallel.ForEach Iterator by Emre Aydinceren

Answer 2

आप Regex के साथ इस की कोशिश की और/या एक स्ट्रिंग के बजाय एक stringbuilder का उपयोग किया है?

Answer 3

अपने foreach के बजाय

की कोशिश कर सकते ...

for(int i=0;i<tests.Length;i++) 
    Console.WriteLine(tests[i].Replace("}",null)); 

Answer 4

यदि आप गति अंतरिक्ष के लिए पाश, आप कर सकते हैं एक बार में बदल दिया गया सरणी, और '{। *}' के अलावा अन्य वर्णों की प्रतिलिपि बनाएँ। इससे दो कॉल (IndexOf() और .Substring()) को बचाया जाएगा।

Answer 5

दृष्टिकोण तेजी से शांत लगता है। लेकिन आपके पास स्ट्रिंग से, मैंने निष्कर्ष निकाला है कि नाम आमतौर पर {URL} छोटा होता है। आप .LastIndexOf() विधि का उपयोग कर सकते हैं। मुझे लगता है कि यह स्ट्रिंग

Answer 6

के अंत से शुरू होता है मान लें कि आपको यहां एक जवाब मिल गया है। मुझे लगता है कि आपको इस बात पर विचार करने की ज़रूरत है कि आपके कोड को देखने वाले अगले व्यक्ति के लिए "समाधान" कैसा दिख रहा है।

मैं किसी भी दिन कुछ मिलीसेकंड बनाम अधिक पठनीय कोड ले जाऊंगा।

Answer 7

कंसोल। राइटलाइन() तक, वहां तक ​​की सबसे धीमी चीज है। निम्नलिखित उदाहरण लें:

public void TestCleanSpeed() 
    { 
     var start = DateTime.Now; 
     for (var i = 0; i < 10000; i++) 
     { 
      string[] tests = { 
           "{http://company.com/Services/Types}ModifiedAt", 
           "{http://company.com/Services/Types}CreatedAt" 
          }; 

      foreach (var test in tests) 
      { 
       Console.WriteLine(Clean(test)); 
      } 
     } 
     var end = DateTime.Now; 

     var ts = end - start; 
     Console.WriteLine(ts); 
    } 

यदि आप इसे चलाते हैं, तो इसमें लगभग छह सेकंड लगते हैं। फिर, Console.WriteLine को हटाएं और इसके बजाय var newTest = Clean(test); असाइन करें। मेरे परीक्षण में, 10000 निष्पादन के लिए इसमें 0.02 सेकंड लग गए।

Answer 8

एकमात्र अन्य दृष्टिकोण line.Remove(0, pos + 1); का उपयोग करना होगा, लेकिन मुझे लगता है कि आंतरिक रूप से अधिक जटिल तो सबस्ट्रिंग निकालें, इस तथ्य के कारण, निकालें बीच में कुछ भी कटौती कर सकता है।

तो सबस्ट्रिंग() उपवास होना चाहिए।

Answer 9

आप अपनी स्ट्रिंग को XName पर परिवर्तित कर सकते हैं और निम्नानुसार नाम अनुभाग प्राप्त कर सकते हैं।

((System.Xml.Linq.XName)"{http://company.com/Services/Types}ModifiedAt").LocalName 

Answer 10

क्या आप वाकई अपने कोड में एक बाधा है? इस विधि की समय आवश्यकताओं के लिए आपका विनिर्देश क्या है? क्या आपने यह कोड देखने के लिए अब कोड को प्रोफाइल किया है, चाहे वह उन विनिर्देशों को पूरा करता हो?

मुझे लगता है कि आपके पास जो कोड है वह लगभग इष्टतम है। IndexOf और Substring दोनोंकोड को फैंसी अनुकूलन के सभी प्रकार के लिए आमंत्रित करते हैं जो आपके लिए तब तक उपलब्ध नहीं होंगे जब तक आप unsafe मार्ग भी नहीं जाते।यदि आप ऐसा करते हैं, तो आप अभी भी IndexOf और Substring को फिर से लिखने जा रहे हैं।

तो जब तक यह कोड आपके कोड में निश्चित रूप से बाधा नहीं है और आपके पास इस विधि के लिए समय आवश्यकताओं का उचित विनिर्देश है, तो मैं आपके प्रयासों को कहीं और केंद्रित करूंगा।

Answer 11

line.Substring(line.IndexOf('}') + 1); 

मामूली तेज़ी से।