In C#, what is the best way to parse out this value from a string?

Question

I have to parse out the system name from a larger string. The system name has a prefix of "ABC" and then a number. Some examples are:

ABC500
ABC1100
ABC1300

the full string where i need to parse out the system name from can look like any of the items below:

ABC1100 - 2ppl
ABC1300
ABC 1300
ABC-1300
Managers Associates Only (ABC1100 - 2ppl)

before I saw the last one, i had this code that worked pretty well:

string[] trimmedStrings = jobTitle.Split(new char[] { '-', '–' },StringSplitOptions.RemoveEmptyEntries)
                           .Select(s => s.Trim())
                           .ToArray();

return trimmedStrings[0];

but it fails on the last example where there is a bunch of other text before the ABC.

Can anyone suggest a more elegant and future proof way of parsing out the system name here?

Answer 1

One way to do this:

string[] strings =
{
    "ABC1100 - 2ppl",
    "ABC1300",
    "ABC 1300",
    "ABC-1300",
    "Managers Associates Only (ABC1100 - 2ppl)"
};

var reg = new Regex(@"ABC[\s,-]?[0-9]+");

var systemNames = strings.Select(line => reg.Match(line).Value);

systemNames.ToList().ForEach(Console.WriteLine);

prints:

ABC1100
ABC1300
ABC 1300
ABC-1300
ABC1100

demo

Answer 2

You really could leverage a Regex and get better results. This one should do the trick [A-Za-z]{3}\\d+ , and here is a Rubular to prove it . Then in the code use it like this:

var matches = Regex.Match(someInputString, @"[A-Za-z]{3}\d+");
if (matches.Success) {
    var val = matches.Value;
}

Answer 3

You can use a regular expression to parse this. There may be better expressions, but this one works for your case:

using System;
using System.Text.RegularExpressions;

namespace ConsoleApplication1
{
  class Program
  {
    static void Main(string[] args)
    {
      string txt="ABC500";

      string re1="((?:[a-z][a-z]+))";   
      string re2="(\\d+)"

      Regex r = new Regex(re1+re2,RegexOptions.IgnoreCase|RegexOptions.Singleline);
      Match m = r.Match(txt);
      if (m.Success)
      {
            String word1=m.Groups[1].ToString();
            String int1=m.Groups[2].ToString();
            Console.Write("("+word1.ToString()+")"+"("+int1.ToString()+")"+"\n");
      }
    }
  }
}

Answer 4

You should definitely use Regex for this. Depending on the exact nature of the system name, something like this could prove to be enough:

Regex systemNameRegex = new Regex(@"ABC[0-9]+");

If the ABC part of the name can change, you can modify the Regex to something like this:

Regex systemNameRegex = new Regex(@"[a-zA-Z]+[0-9]+");