Replace Multiple Elements String trong C#

Question

Có cách nào tốt hơn để làm điều này ...

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

Tôi đã kéo dài chuỗi lớp để giữ nó xuống một công việc nhưng có cách nào nhanh hơn?

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

---

Just for fun (và để ngăn chặn các đối số trong các ý kiến) Tôi đã xô đẩy một ý chính lên điểm chuẩn các ví dụ khác nhau dưới đây.

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

Điểm lựa chọn regex khủng khiếp; tùy chọn từ điển xuất hiện nhanh nhất; phiên bản kéo dài của trình tạo chuỗi thay thế nhanh hơn một chút so với tay ngắn.

Answer 1

Nhanh hơn - không. Hiệu quả hơn - vâng, nếu bạn sử dụng lớp StringBuilder. Với việc thực hiện của bạn, mỗi thao tác tạo ra một bản sao của một chuỗi trong các trường hợp có thể làm giảm hiệu suất. Các chuỗi là bất biến đối tượng để mỗi thao tác chỉ trả về một bản sao đã sửa đổi.

Nếu bạn mong đợi phương pháp này được chủ động gọi trên nhiều Strings có độ dài đáng kể, có thể tốt hơn là "di chuyển" triển khai của nó lên lớp StringBuilder. Với nó bất kỳ sửa đổi được thực hiện trực tiếp trên trường hợp đó, vì vậy bạn phụ tùng hoạt động sao chép không cần thiết.

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 2

này sẽ hiệu quả hơn:

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 3

Có lẽ một chút dễ đọc hơn?

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; 
     } 
    } 

Ngoài ra thêm mới Trong gợi ý Town về StringBuilder ...

Answer 4

tôi đang làm một cái gì đó tương tự, nhưng trong trường hợp của tôi, tôi đang làm serialization/De-serialization vì vậy tôi cần để có thể đi cả hai hướng. Tôi thấy việc sử dụng chuỗi [] [] gần như giống với từ điển, bao gồm khởi tạo, nhưng bạn cũng có thể đi theo hướng khác, trả lại giá trị thay thế cho giá trị ban đầu của chúng, một thứ mà từ điển thực sự không được thiết lập để thực hiện.

Chỉnh sửa: Bạn có thể sử dụng Dictionary<Key,List<Values>> để có được cùng một kết quả như string [] []

Answer 5

Nếu bạn chỉ đơn giản là sau khi một giải pháp khá và không cần phải tiết kiệm một vài nano giây, làm thế nào về một số đường 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

Có một thứ có thể được tối ưu hóa trong các giải pháp được đề xuất. Có nhiều cuộc gọi đến Replace() làm cho mã thực hiện nhiều lần vượt qua cùng một chuỗi. Với chuỗi rất dài, các giải pháp có thể chậm vì dung lượng bộ nhớ cache CPU bị mất. Có thể là một trong những nên xem xét 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

Một lựa chọn khác sử dụng LINQ là

[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); 
}