Лучший способ разбора int из строки

Question

Я пытаюсь понять, есть ли другой / лучший способ анализа строки, которая у меня есть.

Строка "#def xyz[timer=50, fill=10]". Из этой строки я пытаюсь извлечь таймер и заполнить значения.

Код, который у меня сейчас есть:

string def = "#def xyz[timer=50, fill=10]";
string _timer = def.Remove(def.IndexOf(","));
_timer = _timer.Remove(0, _timer.IndexOf("=", _timer.IndexOf("timer")) + 1);

string _fill = def.Remove(def.IndexOf("]"));
_fill = _fill.Remove(0, _fill.IndexOf("=", _fill.IndexOf("fill")) + 1);

int timer = Int32.Parse(_timer);
int fill = Int32.Parse(_fill);

Какие-либо предложения?

Заранее спасибо!

Answer 1

Я бы, наверное, использовал регулярное выражение. Например:

using System;
using System.Text.RegularExpressions;

class Test
{
    static void Main()
    {
        // You can create the regex once and reuse it, of course. Adjust
        // as necessary if the name isn't always "xyz" for example.
        Regex regex = new Regex(@"^#def xyz\[timer=(\d+), fill=(\d+)\]$");
        string input = "#def xyz[timer=50, fill=10]";
        Match match = regex.Match(input);
        if (match.Success)
        {
            int fill = int.Parse(match.Groups[1].Value);
            int timer = int.Parse(match.Groups[2].Value);
            Console.WriteLine("Fill={0}, timer={1}", fill, timer);
        }
    }
}

Заметки:

• Это касается только неотрицательных целых чисел
• Он потерпит неудачу (за исключением), если значение выходит за пределы диапазона int

Я бы сказал, что это показывает, что вы делаете более четко, чем те, Remove хотя звонит...

Answer 2

      Match m = Regex.Match("#def xyz[timer=50, fill=10]", "timer=([0-9]+?), fill=([0-9]+?)[]]");

      string timer = m.Result("$1");
      string fill = m.Result("$2");

Answer 3

Мне нравится использовать split, когда я могу, в большинстве случаев это намного быстрее, чем регулярное выражение - я не проверял, но я ожидаю, что здесь будет быстрее. Конечно, в этом коде очень мало проверок ошибок.

void Main()
{
  string def = "#def xyz[timer=50, fill=10]";

  string [] inBracket = def.Split("[]".ToCharArray());

  string [] elements = inBracket[1].Split(",".ToCharArray());

  int timer = int.Parse(elements[0].Split("=".ToCharArray())[1]);

  int fill = int.Parse(elements[1].Split("=".ToCharArray())[1]);

  Console.WriteLine("timer = "+timer.ToString());
  Console.WriteLine("fill = "+fill.ToString());

}