Перетворення 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;
}

PS: mc.CopyTo(strArray,0)видає виняток:

Принаймні один елемент у вихідному масиві не може бути відведений до цільового масиву.

Спробуйте:

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

Відповідь Дейва Біша хороша і працює належним чином.

Варто зазначити, хоча ця заміна 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 () працює повільніше.

Я провів той самий орієнтир, який опублікував Алекс, і виявив, що іноді це Castбуло швидше, а іноді OfTypeшвидше, але різниця між ними була незначною. Однак, хоча і потворний, цикл for стабільно швидший, ніж обидва інші два.

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

Можна також скористатися цим методом розширення, щоб впоратися з роздратуванням через 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;
    }
}

Розглянемо наступний код ...

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