C# LinqでTryParseしたい

Question

前提・実現したいこと

文字列のコレクションに対して、「TryParseに成功したものを変換された状態で取得する」という処理を書きたいです。

Linqを使って書いてみた結果以下のようになりましたが、WhereとSelectで2度変換している点や、Whereで変換したDateTimeを捨てている点が気になります。

より良い書き方はないでしょうか？

該当のソースコード

// 2回変換するのはどうか・・・
var results = strings.Where(p => DateTime.TryParse(p, out DateTime q))
                     .Select(p => DateTime.Parse(p));

// かといって行が増えるのもちょっと・・・
var results = strings.Select(p => DateTime.TryParse(p, out DateTime q) ? (DateTime?)q : null)
                     .Where(p => p != null);
                     .Select(p => (DateTime)p);

Answer 1

気にするほどの行数ではないと思いますが、気になるのであれば拡張メソッドにしてみては？

        public static IEnumerable<DateTime> ParseOfDateTime(this IEnumerable<string> strings)
        {
            DateTime q;
            return strings.Select(p => new { f = DateTime.TryParse(p, out q), d = q }).Where(p => p.f).Select(p => p.d);
        }

Comments

[YamakawaJunichi]

拡張メソッド化も検討してみます、ありがとうございました。

Answer 2

var xs = new[]{"2018/01/01","hoge","2018/02/02"};
DateTime q; // 実行環境がC#6.0


var ys = xs.SelectMany(p => DateTime.TryParse(p, out q) ? new[]{q} : new DateTime[]{});
        
foreach(var y in ys) Console.WriteLine(y);

Comments

[YamakawaJunichi]

なるほど、空の配列にすることで値がないこと表現するんですね。
新たな気づきになりました、ありがとうございました。