Moja pierwsza metoda rozszerzenia, czy można ją napisać lepiej?

Question

Ponieważ jest to moja pierwsza próba metodę rozszerzenia, które wydaje się dość użyteczne dla mnie, po prostu chcę się upewnić, jadę w dół właściwą drogę

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

nazywany przez

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

EDYTOWANIE: Kilka doskonałych sugestii, dokładnie tego rodzaju rzeczy, których szukałem. Dzięki

EDIT:

zdecydowałem na poniższym realizacji

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

wolałem użyciu params nad IEnumerable ponieważ uproszczony kod wywołujący

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

daleko na poprzedni

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

Jeszcze raz dziękuję!

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

Dzięki paramom nie trzeba najpierw wciskać łańcuchów w tablicę.

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

Linq ified (JC + ja) z NRE (standard ramowej):

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

Ulepsz swój parametr tokens - tzn. Spraw, aby był to IEnumerable<string>.

Ponadto istnieje już równoważna metoda, która rozszerza się o IEnumerable<>, np. Any:

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

Również Joel jest odpowiedni kurs: może chcesz sprawdzić null wartości przed wykonaniem czynności (kodowanie obronny). Nie jest to bezpieczniejsze, ale ułatwia zlokalizowanie błędu.

Answer 3

aby uprościć korzystanie z EqualsAny można użyć varargs i domyślną strategię 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 
} 

Wywoływana przez

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

Answer 4

Tak! Najpierw musisz sprawdzić s dla wartości null. Ponadto, niech ponosi IEnumerable<string> dla tokenów raczej niż tylko tablicę, a następnie użyć innych operatorów LINQ to zrobić należy sprawdzić:

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

---

myśleć o tym jak radzić sobie null wartości dla S, tam trzeci opcja nikt dotychczas stosowane:

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

---

Wreszcie, w odniesieniu do wybranego realizacji: jeśli masz zamiar mieć przeciążeń, równie dobrze można mieć IEnumerable overloa ds również, i mieć swój kod params te.

Answer 5

Innym rozwiązaniem byłoby. Uprości to twoją stronę wywoławczą, ponieważ jeśli masz kilka ciągów, twoje dopasowanie przeciwko tobie nie będzie musiało tworzyć tablicy ani listy w kodzie.

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

Nie ma nic złego w tym, co robisz. Jednak ten typ funkcjonalności już istnieje w kilku różnych modach.

Przykład:

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