How to capture values in a string using regex in c#

Question

Below I have a sample of a log file that I am working with:

5/21/2015 11:55:56 PM | Batch 6|386/767|50.33 %|CH2M-R|Processed NXRMN5...Checking refundable and non-refundable fares. Traditional Booking. Inside ticketing window. Minimum Savings Required: $131.00. Actual Savings: $257.18. Savings found: $257.18 (11.55 %). Savings were previously found.

Almost every line is like this, but some will say Savings not found. Below is the most important part of this sample:

Savings found: $257.18

I am trying to write a piece of code that will look through this log file, search through the whole file, and if savings were found, it will record that number into a variable, that way when I total it up, it will just fall into a variable.

The problem I am having right now is getting my code to display just that number. Below is what I have been working on thus far:

foreach (string line in gdsGroup)
        {
            Match m = Regex.Match(line, @"Savings\s+found:\s*\$(?<savings>\d+\.\d+)\s*\(\s*(?<percent>\d+\.\d+)\s*%");
            if (m.Success)
            {
                decimal gdsNumberSavings = decimal.Parse(m.Groups["savings"].Value, CultureInfo.InvariantCulture);
                decimal gdsNumberPercent = decimal.Parse(m.Groups["percent"].Value, CultureInfo.InvariantCulture);
                string prefix = string.Empty;

                if (gdsNumberPercent >= 30)
                {
                    if (gdsNumberSavings >= 500)
                        prefix = "**";
                    else
                        prefix = "*";
                }
                Console.WriteLine(prefix + line + "\n");
                Console.WriteLine(gdsNumberSavings);
            }
        }
    }

The problem that is happening is that after the line that gives me what I have above, it prints out the of the savings for that line. So my question is, should I continue to try to go about a variable way, or do I need a regular expression to isolate that value, and if I need a regular expression what would the expression be?

"The problem I am having right now is getting my code to display just that number? What is "THAT" number? — Shar1er80
– Shar1er80, Commented Jul 6, 2015 at 19:29
@Shar1er80 the savings found is that number. So in the sample I provided, that $257.18 is an example of that number, minus the dollar sign of course — user4970927
– user4970927, Commented Jul 6, 2015 at 19:38
And what do you want to do with that number? You've already successfully obtained it with the Regex you're using. — Shar1er80
– Shar1er80, Commented Jul 6, 2015 at 19:39
I want that to be added to the other numbers in the log file. For example: if I were to run this program, at the end of my program I would want it to say "the total savings I found were: " and the blank space being the total number it finds — user4970927
– user4970927, Commented Jul 6, 2015 at 19:41
So if I had that sample number twice in this file, and those were the only savings found, it would say "The total savings found were: 514.36" — user4970927
– user4970927, Commented Jul 6, 2015 at 19:42

Shar1er80 · Accepted Answer · 2015-07-06 19:47:32Z

There's nothing wrong with the Regex you are using to obtain the value you are wanting from each line in your foreach.

You just need an accumulator initialized before you start the foreach and add gdsNumberSavings to it whenever there is a match.

string[] lines = 
{
    "5/21/2015 11:55:56 PM | Batch 6|386/767|50.33 %|CH2M-R|Processed NXRMN5...Checking refundable and non-refundable fares. Traditional Booking. Inside ticketing window. Minimum Savings Required: $131.00. Actual Savings: $257.18. Savings found: $257.18 (11.55 %). Savings were previously found.",
    "5/21/2015 11:55:56 PM | Batch 6|386/767|50.33 %|CH2M-R|Processed NXRMN5...Checking refundable and non-refundable fares. Traditional Booking. Inside ticketing window. Minimum Savings Required: $131.00. Actual Savings: $257.18. Savings found: $257.18 (11.55 %). Savings were previously found.",
    "5/21/2015 11:55:56 PM | Batch 6|386/767|50.33 %|CH2M-R|Processed NXRMN5...Checking refundable and non-refundable fares. Traditional Booking. Inside ticketing window. Minimum Savings Required: $131.00. Actual Savings: $257.18. Savings found: $257.18 (11.55 %). Savings were previously found.",
    "5/21/2015 11:55:56 PM | Batch 6|386/767|50.33 %|CH2M-R|Processed NXRMN5...Checking refundable and non-refundable fares. Traditional Booking. Inside ticketing window. Minimum Savings Required: $131.00. Actual Savings: $257.18. Savings found: $257.18 (11.55 %). Savings were previously found."
};

// Initialize total
decimal total = 0;
foreach (string line in lines)
{
    Match m = Regex.Match(line, @"Savings\s+found:\s*\$(?<savings>\d+\.\d+)\s*\(\s*(?<percent>\d+\.\d+)\s*%");
    if (m.Success)
    {
        decimal gdsNumberSavings = decimal.Parse(m.Groups["savings"].Value, CultureInfo.InvariantCulture);
        decimal gdsNumberPercent = decimal.Parse(m.Groups["percent"].Value, CultureInfo.InvariantCulture);
        string prefix = string.Empty;

        if (gdsNumberPercent >= 30)
        {
            prefix = gdsNumberSavings >= 500 ? "**" : "*";
        }
        Console.WriteLine(prefix + line + "\n");
        Console.WriteLine(gdsNumberSavings);

        // Accumulate into a total
        total += gdsNumberSavings;
    }
}
// Display total
Console.WriteLine("Total Savings: {0}", total);

Constuntine · Accepted Answer · 2015-07-06 19:16:54Z

0

I'm not sure as for the variable side of the question. But as for the regular expression, this will find your dollar amounts: \$[0-9]+[.]\d{2} You have three dollar amounts in that line, so just take the second instance that would be returned from the expression and that would be your total savings.

answered Jul 6, 2015 at 19:16

Constuntine

5182 silver badges16 bronze badges

2 Comments

user4970927 Over a year ago

Could you go a little farther in depth as to what you mean by second instance?

Constuntine Over a year ago

When a regular expression is used, it will return everything in the string that matches the expression (starting from the beginning). In your sample string that you gave, the expression will have three matches (the three different dollar amounts). So you use the second match returned for your total savings.

Collectives™ on Stack Overflow

How to capture values in a string using regex in c#

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related