Можете ли вы улучшить этот код регулярного выражения C #?

Question

В программе я читаю некоторые файлы данных, часть из которых отформатирована в виде серии записей, каждая в квадратных скобках. Каждая запись содержит заголовок раздела и серию пар ключ / значение.

Первоначально я написал код для обхода и извлечения значений, но решил, что это можно сделать более элегантно с помощью регулярных выражений. Ниже приведен мой полученный код (я только что его взломал в консольном приложении - поэтому знайте, что имена переменных не так уж хороши и т. Д.)

Можете ли вы предложить улучшения? Я чувствую, что нет необходимости делать два совпадения и подстроку, но не могу понять, как сделать все это за один большой шаг:

string input = "[section1 key1=value1 key2=value2][section2 key1=value1 key2=value2 key3=value3][section3 key1=value1]";

MatchCollection matches=Regex.Matches(input, @"\[[^\]]*\]");
foreach (Match match in matches)
{
    string subinput = match.Value;

    int firstSpace = subinput.IndexOf(' ');
    string section = subinput.Substring(1, firstSpace-1);
    Console.WriteLine(section);

    MatchCollection newMatches = Regex.Matches(subinput.Substring(firstSpace + 1), @"\s*(\w+)\s*=\s*(\w+)\s*");
    foreach (Match newMatch in newMatches)
    {
        Console.WriteLine("{0}={1}", newMatch.Groups[1].Value, newMatch.Groups[2].Value);
    }
}

Answer 1

Я предпочитаю именованные снимки, хорошее форматирование и четкость:

string input = "[section1 key1=value1 key2=value2][section2 key1=value1 key2=value2 key3=value3][section3 key1=value1]";
MatchCollection matches = Regex.Matches(input, @"\[
                                                    (?<sectionName>\S+)
                                                      (\s+                                                            
                                                         (?<key>[^=]+)
                                                          =
                                                         (?<value>[^ \] ]+)                                                    
                                                      )+
                                                  ]", RegexOptions.IgnorePatternWhitespace);

foreach(Match currentMatch in matches)
{
    Console.WriteLine("Section: {0}", currentMatch.Groups["sectionName"].Value);
    CaptureCollection keys = currentMatch.Groups["key"].Captures;
    CaptureCollection values = currentMatch.Groups["value"].Captures;

    for(int i = 0; i < keys.Count; i++)
    {
        Console.WriteLine("{0}={1}", keys[i].Value, values[i].Value);           
    }
}

Answer 2

Вы должны воспользоваться коллекциями, чтобы получить каждый ключ. Так вот как то так:

        string input = "[section1 key1=value1 key2=value2][section2 key1=value1 key2=value2 key3=value3][section3 key1=value1]";

        Regex r = new Regex(@"(\[(\S+) (\s*\w+\s*=\s*\w+\s*)*\])", RegexOptions.Compiled);

        foreach (Match m in r.Matches(input))
        {
            Console.WriteLine(m.Groups[2].Value);
            foreach (Capture c in m.Groups[3].Captures)
            {
                Console.WriteLine(c.Value);
            }
        }

Результирующий вывод:

section1
key1=value1
key2=value2
section2
key1=value1
key2=value2
key3=value3
section3
key1=value1

Answer 3

Вы должны быть в состоянии сделать что-то с вложенными группами, как это:

pattern = @"\[(\S+)(\s+([^\s=]+)=([^\s\]]+))*\]"

Я не тестировал его на C # и не просматривал матчи, но результаты выглядят прямо на rubular.com

Answer 4

Это будет соответствовать всем парам ключ / значение ...

var input = "[section1 key1=value1 key2=value2][section2 key1=value1 key2=value2 key3=value3][section3 key1=value1]";

var ms = Regex.Matches(input, @"section(\d+)\s*(\w+=\w+)\s*(\w+=\w+)*");

foreach (Match m in ms)
{
    Console.WriteLine("Section " + m.Groups[1].Value);

    for (var i = 2; i < m.Groups.Count; i++)
    {
        if( !m.Groups[i].Success ) continue;
        var kvp = m.Groups[i].Value.Split( '=' );
        Console.WriteLine( "{0}={1}", kvp[0], kvp[1] );
    }
}