Ngoài sự tò mò - có cách tiếp cận thực hiện tốt hơn để thực hiện chuỗi này thay thế không?

Question

Tôi có phương pháp dưới đây, nơi tôi cần phải kiểm tra một số chuỗi nhất định có thể trong bất kỳ trường hợp nào và sau đó xóa chúng. Chỉ cần tự hỏi nếu có một cách thực hiện tốt hơn?

private void MyMethod(string Filter) 
{ 
    //need to remove <Filter> and </Filter> case in-sensitive 
    var result = Filter.ToLower().Replace("<filter>",""); 
    result = Filter.ToLower().Replace("</filter>,""); 

    ........................... 
} 

Answer 1

Một vấn đề với cách tiếp cận đó là nó sẽ biến toàn bộ chuỗi thành chữ thường, không chỉ làm cho một trường hợp không bị thay thế.

Bạn có thể sử dụng một biểu thức chính quy để làm một trường hợp insensetive trận đấu:

string result = Regex.Replace(
    Filter, 
    "</?filter>", 
    String.Empty, 
    RegexOptions.IgnoreCase 
); 

Một lựa chọn khác là sử dụng phương pháp IndexOf để xác định vị trí các dây, vì nó có thể làm một trường hợp tìm kiếm insensetive:

string result = Filter; 
int index; 
while ((index = IndexOf("<filter>", StringComparison.OrdinalIgnoreCase)) != -1) { 
    result = result.Remove(index, 8); 
} 
while ((index = IndexOf("</filter>", StringComparison.OrdinalIgnoreCase)) != -1) { 
    result = result.Remove(index, 9); 
} 

Answer 2

Replace gọi tới mã không được quản lý được triển khai bằng C++ mà tôi cho là khó đánh bại.

Tuy nhiên, tôi có thể thấy bạn tiếp tục sử dụng .ToLower() mà bạn có thể cắt giảm xuống một cuộc gọi và giữ chuỗi.

Answer 3

Kiểm tra câu trả lời này: Is there an alternative to string.Replace that is case-insensitive?

Bạn có thể muốn so sánh với kiểm tra hiệu suất. Hồ sơ này với một hồ sơ. Đó là cách duy nhất để thực sự biết, những gì là nhanh hơn.

Nhưng thành thực: Hiệu suất có thực sự quan trọng không? Bạn có thường xuyên làm việc này không? Tôi thực sự không thể thấy bạn thực hiện việc này thường xuyên như vậy, hiệu suất đó sẽ trở thành vấn đề ...

Bạn có thể thử Regex.Replace, với sự thay thế không phân biệt chữ hoa chữ thường. Đây không phải là nhanh hơn. Nhưng nó không phân biệt chữ hoa chữ thường.

Answer 4

Trong mọi trường hợp, bạn sắp xếp chuỗi gốc ở đây thấp hơn, đây có thể không phải là điều tốt?

Answer 5

Nếu mã được cung cấp làm việc cho bạn, hơn thế này sẽ nhanh hơn:

private void MyMethod(string Filter) 
{ 
    //need to remove <Filter> and </Filter> case in-sensitive 
    var result = Filter.ToLower().Replace("</filter>",""); 

    ........................... 
} 

kết quả báo cáo kết quả đầu tiên được bỏ qua.

Answer 6

Nó phụ thuộc vào một vài điều, chuỗi Bộ lọc là bao lâu, vv ..
Vì vậy, bạn sẽ phải đo lường.

Nhưng tôi mong đợi một (đơn!) RegEx sẽ nhanh hơn ở đây.