Phương pháp mở rộng đầu tiên của tôi, nó có thể được viết tốt hơn không?

Question

Bởi vì đây là nỗ lực đầu tiên của tôi tại một phương pháp mở rộng mà dường như khá hữu ích với tôi, tôi chỉ muốn chắc chắn rằng tôi đang đi xuống con đường đúng

public static bool EqualsAny(this string s, string[] tokens, StringComparison comparisonType) 
    { 
     foreach (string token in tokens) 
     { 
      if (s.Equals(token, comparisonType)) 
      { 
       return true; 
      } 
     } 

     return false; 
    } 

gọi bởi

if (queryString["secure"].EqualsAny(new string[] {"true","1"}, StringComparison.InvariantCultureIgnoreCase)) 
{ 
    parameters.Protocol = Protocol.https; 
} 

EDIT: Một số gợi ý xuất sắc thông qua, chính xác loại điều tôi đang tìm kiếm. Cảm ơn

EDIT:

tôi đã quyết định về việc thực hiện sau

public static bool EqualsAny(this string s, StringComparison comparisonType, params string[] tokens) 
{ 
    // for the scenario it is more suitable for the code to continue 
    if (s == null) return false; 

    return tokens.Any(x => s.Equals(x, comparisonType)); 
} 

public static bool EqualsAny(this string s, params string[] tokens) 
{ 
    return EqualsAny(s, StringComparison.OrdinalIgnoreCase, tokens); 
} 

tôi ưa thích sử dụng params qua IEnumerable vì nó đơn giản hóa mã gọi

if (queryString["secure"].EqualsAny("true","1")) 
{ 
    parameters.Protocol = Protocol.https; 
} 

Một xa khóc trên trước đây

if (queryString["secure"] != null) 
{ 
    if (queryString["secure"] == "true" || queryString["secure"] == "1") 
    { 
     parameters.Protocal = Protocal.https; 
    } 
} 

Cảm ơn bạn một lần nữa!

Answer 1

public static bool EqualsAny(
    this string s, 
    StringComparison comparisonType, 
    params string[] tokens) 
{ 
    foreach (string token in tokens) 
    { 
     if (s.Equals(token, comparisonType)) 
     { 
      return true; 
     } 
    } 
    return false; 
} 

Với thông số, trước tiên bạn không phải buộc chuỗi của bạn vào mảng.

var match = "loool".EqualsAny(StringComparison.Ordinal, "hurf", "Durf"); 

LINQ-ified (JC + tôi) với một TNMT (tiêu chuẩn khuôn khổ):

public static bool EqualsAny(
    this string s, 
    StringComparison comparisonType, 
    params string[] tokens) 
{ 
    if(s == null) throw new NullReferenceException("s"); 
    return tokens.Any(x=> s.Equals(x, comparisonType)); 
} 

Answer 2

Làm cho thông số tokens của bạn tổng quát hơn - tức là biến nó thành IEnumerable<string>.

Ngoài ra, một phương thức tương đương đã tồn tại kéo dài IEnumerable<>, ví dụ: Any:

public static bool EqualsAny(this string s, IEnumerable<string> tokens, StringComparison comparisonType) 
{ 
    return tokens.Any(t => s.Equals(t, comparisonType)); 
} 

Ngoài ra, Joel là tất nhiên đúng: bạn có thể muốn kiểm tra null giá trị trước khi thực hiện các hành động (mã hóa phòng thủ). Điều này không an toàn hơn nhưng nó làm cho lỗi dễ dàng hơn để bản địa hóa.

Answer 3

để đơn giản hóa việc sử dụng của EqualsAny bạn có thể sử dụng varargs và chiến lược mặc định cho StringComparison:

public static bool EqualsAny(this string s, params string[] tokens) { 
    return EqualsAny(s, StringComparison.InvariantCultureIgnoreCase, tokens); 
} 

public static bool EqualsAny(this string s, 
          StringComparison stringComparison, 
          params string[] tokens) { 
    // your method 
} 

gọi bởi

if (queryString["secure"].EqualsAny("true", "1")) { 
    parameters.Protocol = Protocol.https; 
} 

Answer 4

Có! Trước tiên, bạn cần phải kiểm tra s cho null. Ngoài ra, để cho nó chấp nhận bất kỳ IEnumerable<string> cho thẻ chứ không phải chỉ là một mảng, và sau đó sử dụng các toán LINQ khác để làm việc kiểm tra:

public static bool EqualsAny(this string s, IEnumerable<string> tokens, StringComparison comparisonType) 
{ 
    if (s== null) return false; 
    return tokens.Any(t => s.Equals(t, comparisonType)); 
} 

---

Suy nghĩ về làm thế nào để xử lý một giá trị null cho s, có một phần ba lựa chọn không ai được sử dụng nào:

public static bool EqualsAny(this string s, IEnumerable<string> tokens, StringComparison comparisonType) 
{ 
    if (s== null) return tokens.Any(t => t == null); 
    return tokens.Any(t => s.Equals(t, comparisonType)); 
} 

---

Cuối cùng, về việc thực hiện lựa chọn của bạn: nếu bạn đang đi để có quá tải, bạn cũng có thể có overloa IEnumerable ds là tốt, và có mã số params của bạn.

Answer 5

tùy chọn khác sẽ được. Điều này sẽ đơn giản hóa trang web cuộc gọi của bạn vì nếu bạn có một vài chuỗi phù hợp với bạn, bạn sẽ không phải tạo mảng hoặc danh sách trong mã.

public static bool EqualsAny(this string s,StringComparison comparisonType, param string[] tokens) 
{ 
    return EqualsAny(s,comparisonType,tokens); 
}  

public static bool EqualsAny(this string s,StringComparison comparisonType, IEnumerable<string>tokens)  
{ 
    //Throw nullReference to keep the semantics aligned with calling an instance member 
    if (s==null) throw new NullReferenceException();  
    foreach (string token in tokens)   
    {    
     if (s.Equals(token, comparisonType))    
     {     
      return true;    
     }   
    }   
    return false;  

} 

Answer 6

Không có gì sai với những gì bạn đang làm. Tuy nhiên, loại chức năng này đã tồn tại trong một số thời trang khác nhau.

Ví dụ:

var candidates = List<SomeObject>(); 
if (candidates.Count(c=> string.Compare(c.PropertyValue, queryString["secure"], StringComparison.InvariantCultureIgnoreCase) == 0) > 0) 
{ 
parameters.Protocol = Protocol.https; 
}