web-dev-qa-db-ja.com

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

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

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

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

c#arraysregex
68
Vil

試してください：

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

Dave Bishの答えは適切で、適切に機能します。

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

非常に長い文字列の場合、Cast（）は遅くなります。

26
Alex

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
5
David DeMar

この拡張メソッドを利用して、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;
    }
}
1
Nicholas Petersen

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

var emailAddress = "[email protected]; [email protected]; [email protected]";
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();

幸運を！

0
gpmurthy