MatchCollectionを文字列配列に変換する

Question 1

これよりもMatchCollectionを文字列配列に変換するためのより良い方法はありますか？

MatchCollection mc = Regex.Matches(strText, @"\b[A-Za-z-']+\b");
string[] strArray = new string[mc.Count];
for (int i = 0; i < mc.Count;i++ )
{
    strArray[i] = mc[i].Groups[0].Value;
}

PS：mc.CopyTo(strArray,0)例外をスローします：

ソース配列の少なくとも1つの要素を、宛先配列タイプにキャストできませんでした。

Question 2

試してみてください：

var arr = Regex.Matches(strText, @"\b[A-Za-z-']+\b")
    .Cast<Match>()
    .Select(m => m.Value)
    .ToArray();

Question 3

デイブビッシュの答えは良く、正しく機能します。

に置き換えるCast<Match>()と処理OfType<Match>()速度が向上しますが、注目に値します。

コードは次のようになります。

var arr = Regex.Matches(strText, @"\b[A-Za-z-']+\b")
    .OfType<Match>()
    .Select(m => m.Groups[0].Value)
    .ToArray();

結果はまったく同じです（そしてOPの問題にまったく同じ方法で対処します）が、巨大な文字列の場合はより高速です。

テストコード：

// put it in a console application
static void Test()
{
    Stopwatch sw = new Stopwatch();
    StringBuilder sb = new StringBuilder();
    string strText = "this will become a very long string after my code has done appending it to the stringbuilder ";

    Enumerable.Range(1, 100000).ToList().ForEach(i => sb.Append(strText));
    strText = sb.ToString();

    sw.Start();
    var arr = Regex.Matches(strText, @"\b[A-Za-z-']+\b")
              .OfType<Match>()
              .Select(m => m.Groups[0].Value)
              .ToArray();
    sw.Stop();

    Console.WriteLine("OfType: " + sw.ElapsedMilliseconds.ToString());
    sw.Reset();

    sw.Start();
    var arr2 = Regex.Matches(strText, @"\b[A-Za-z-']+\b")
              .Cast<Match>()
              .Select(m => m.Groups[0].Value)
              .ToArray();
    sw.Stop();
    Console.WriteLine("Cast: " + sw.ElapsedMilliseconds.ToString());
}

出力は次のとおりです。

OfType: 6540
Cast: 8743

以下のために非常に長い文字列のキャスト（）ので遅いです。

Question 4

Alexが投稿したのとまったく同じベンチマークを実行したところ、Cast速い場合と速い場合がありましたOfTypeが、両者の違いはごくわずかでした。ただし、醜いですが、forループは他の2つよりも一貫して高速です。

Stopwatch sw = new Stopwatch();
StringBuilder sb = new StringBuilder();
string strText = "this will become a very long string after my code has done appending it to the stringbuilder ";
Enumerable.Range(1, 100000).ToList().ForEach(i => sb.Append(strText));
strText = sb.ToString();

//First two benchmarks

sw.Start();
MatchCollection mc = Regex.Matches(strText, @"\b[A-Za-z-']+\b");
var matches = new string[mc.Count];
for (int i = 0; i < matches.Length; i++)
{
    matches[i] = mc[i].ToString();
}
sw.Stop();

結果：

OfType: 3462
Cast: 3499
For: 2650

Question 5

この拡張方法を利用して、MatchCollection一般的でないという煩わしさに対処することもできます。それは大したことではありませんが、これはほぼ確実に、OfTypeまたはよりもパフォーマンスが高くCastなります。これは、列挙しているだけなので、どちらも実行する必要があります。

（サイドノート：それは作るために.NETチームのために可能である場合、私は疑問に思うMatchCollectionの継承ジェネリックICollectionとIEnumerable？将来的にはその後、我々はすぐにLINQが使用可能に変換するには、この余分なステップを必要としません）。

public static IEnumerable<Match> ToEnumerable(this MatchCollection mc)
{
    if (mc != null) {
        foreach (Match m in mc)
            yield return m;
    }
}

Question 6

次のコードを検討してください...

var emailAddress = "joe@sad.com; joe@happy.com; joe@elated.com";
List<string> emails = new List<string>();
emails = Regex.Matches(emailAddress, @"([a-zA-Z0-9_\-\.]+)@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.)|(([a-zA-Z0-9\-]+\.)+))([a-zA-Z]{2,4}|[0-9]{1,3})")
                .Cast<Match>()
                .Select(m => m.Groups[0].Value)
                .ToList();