guidの比較でlinqにforeach

Question

このメソッドのforeachをlinq式に変換したいと思います。私は2つのtry/catchを持っています。なぜなら、guidのリストまたはguidToFindの中で渡された文字列を有効なguid文字列とみなすことはできないからです。

他の解決策のように簡潔

public static bool IsGuidInList(List<string> guids, string guidToFind) 
    { 
     try 
     { 
      var guid = new Guid(guidToFind.Trim()); 
      foreach (var g in guids) 
      { 
       try 
       { 
        var g2 = new Guid(g.Trim()); 
        if (g2 == guid) 
         return true; 
       } 
       catch {} // swallow exception 
      } 
     } 
     catch{} // swallow exception 
     return false; 
    } 

Answer 1

public static bool IsGuidInList(List<string> guids, string guidToFind) 
{ 
    try 
    { 
     var guid = new Guid(guidToFind.Trim()); 
     return 
      guids 
      .Select(x => 
      { 
       Guid result; 
       return Guid.TryParse(x, out result) ? (Guid?)result : null; 
      }) 
      .Where(x => x.HasValue) 
      .Any(x => x.Value == guid); 
    } 
    catch { } // swallow exception 
    return false; 
} 

Answer 2

var tg = Guid.Empty; 
    guids.Any(g=> g!= null 
     && Guid.TryParse(g.Trim(), out tg) && new Guid(g.Trim()) == guid) 

Answer 3

ませんが、私は偉大な答えを、それが読める

public static bool IsGuidInList(List<string> guids, string guidToFind) 
    { 
     Guid outGuid; 

     var inGuidToFind = Guid.TryParse(guidToFind, out outGuid) ? outGuid : Guid.Empty; 
     if (inGuidToFind == Guid.Empty) 
      return false; 

     var inGuids = new List<Guid>(); 
     guids.Where(i => Guid.TryParse(i, out outGuid)).ForEach(i => inGuids.Add(new Guid(i))); 

     return inGuids.Contains(inGuidToFind); 
    } 

Answer 4

Thxを見つけるには、しかし、私はジェイソンBoydsの答えに行きましたが、と最後のtry/catchを取り除くために私自身のわずかな変更。私は私が期待される結果とこれらのテストを実行した：

• 悪いguidToFind列
• 良いguidToFind文字列、すべての有効なGUID文字列

• 良いguidToFind文字列、ほとんど有効なGUID文字列のリストのリスト

  public static bool IsGuidInList(List<string> guids, string guidToFind) 
  { 
      Guid guid; 
      if (!Guid.TryParse(guidToFind.Trim(), out guid)) 
       return false; 
  
      return 
       guids 
        .Select(x => 
        { 
         Guid result; 
         return Guid.TryParse(x, out result) ? (Guid?)result : null; 
        }) 
        .Where(x => x.HasValue) 
        .Any(x => x.Value == guid); 
  } 
  

UPDATE： 上記の方法でコードレビューからのフィードバックの後、私はより多くのrefinemenを作ることができましたTSこれが結果です：

は

public static bool IsGuidInList(IEnumerable<string> guids, string guidToFind) 
    { 
     Guid guid; 
     if (!Guid.TryParse(guidToFind.Trim(), out guid)) 
      return false; 

     return guids.Any(x => { Guid result; 
      Guid.TryParse(x, out result); 
      return result == guid; 
     }); 
    } 

Answer 5

この答えはすでに掲載他の人には特に違いはありませんが、私は個人的にこれを使用します。

public static bool ContainsGuid(this IEnumerable<string> guids, string guidToFind) 
    { 
     if (guids == null) throw new ArgumentNullException(nameof(guids)); 
     if (guidToFind == null) throw new ArgumentNullException(nameof(guidToFind)); 

     if (!Guid.TryParse(guidToFind, out var guid)) 
      throw new ArgumentException($"Could not convert '{guidToFind}' to a GUID"); 

     return guids 
      .Select(s => Guid.TryParse(s, out var result) ? (Guid?)result : null) 
      .Contains(guid); 
    } 

違いがあること：

• 拡張メソッド（個人的な好み）
• 新しいC＃の特徴「うちのvar結果」（個人的な好み）
• 明示的に有効なパラメータは、（あなたが後にしている行動に依存しますが、私は大ファンです）
が

最後の項目は、一般的なポイントに私をリード： の場合あなたのプログラムは、GUIDであることを意図した文字列の集合を入力として受け取り、その時点でそれらを検証せずにGuidのIEnumerableを保持するのはなぜですか？検索するGUIDと同じことをして、コードは：

IEnumerable<Guid> guidList = // validated elsewhere 
var found = guidList.Contains(guid);