Question

Could someone please help meto improve this part of code?

The string is something like this: Current Hourly Price (HOEP): $20.09/MWh (2.01¢/kWh). this is one line on website where data of 20.09 and 2. 01 changes within time

static void get_HOEP(string web)
{
    int x = web.IndexOf("Current Hourly Price");
    ...
}

I want just to display something like: Current Hourly Price:   2.01   ¢/kWh

Thanks for help

Answer 1

Use a regex, result will be in the variable result.

var source="Current Hourly Price (HOEP): $20.09/MWh (2.01¢/kWh)";
var regex=new Regex(@"Current Hourly Price \(HOEP\): \$\d+\.\d\d/MWh \((\d+\.\d\d)¢/kWh\)");
var result=regex.Replace(source,"Current Hourly Price $1 ¢/kWh");

–EDIT– Full Class Version

public static class PriceParser {

  private const string MATCH_STRING = @"Current Hourly Price \(HOEP\): \$\d+\.\d\d/MWh \((\d+\.\d\d)¢/kWh\)";
  private const string REPLACE_STRING = @"Current Hourly Price $1 ¢/kWh";
  private static readonly Regex regex=new Regex(MATCH_STRING,RegexOptions.Compiled);
  private static readonly Regex entirePageRegex=new Regex(string.Format("^.*{0}.*$",MATCH_STRING),RegexOptions.Compiled|RegexOptions.Singleline);

  public static void get_HEOP1(string web) {
    Console.WriteLine(regex.Replace(web,REPLACE_STRING));
  }

  public static void get_HEOP2(string web) {
    Console.WriteLine(entirePageRegex.Replace(web,REPLACE_STRING));
  }
}

PriceParser.get_HEOP1(web) simply replaces the match in the search string

PriceParser.get_HEOP2(web) replaces the entirity of web with the replacement string