regex не делает то что хочу.

[Aleksandr H.]

Метод тянет курс НБУ за параметром currency - USD, EUR, RUB
по ссылке
Возникает проблема с курсами для которых поле "Кількість одиниць" = 10. Ошибку дает в строке

var dAmount = Convert.ToInt32(Amount);

An exception of type 'System.FormatException' occurred in mscorlib.dll but was not handled in user code

Additional information: Входная строка имела неверный формат.

переменная matchAmount получается пустой.

public static string GetNBURate(string currency)
        {
            try
            {
                using (WebClient web = new WebClient())
                {
                    string lnk = "http://bank.gov.ua/control/uk/curmetal/currency/search?"+
                                @"formType=searchFormDate&time_step=daily&date=" + DateTime.Now.ToShortDateString() + 
                                @"&execute";
                    string html = web.DownloadString(lnk);
                    string pat = @"<tr>\s{1,}<td[^>]*>(.+?)</td>\s{1,}" +
                        @"<td[^>]*>"+currency.ToUpper()+@"</td>\s{1,}" +
                     @"(?<Amount><td[^>]*>([0-9]+)</td>\s{1,})" +
                     @"(?<Currency><td[^>]*>(.+?)</td>\s{1,})" +
                     @"(?<Price><td[^>]*>([0-9\.]+)</td>)";
                    
                    Regex regex = new Regex(pat);
                    Match match = regex.Match(html);
                    while(match.Success)
                    {
                        string Price = match.Groups["Price"].ToString();
                        string Amount = match.Groups["Amount"].ToString();
                        
                        Regex regexPrice = new Regex(@"\d+.\d+");
                        Match matchPrice = regexPrice.Match(Price);
                        var dPrice = Convert.ToDecimal(matchPrice.ToString(), new CultureInfo("en-US"));

                        Regex regexAmount = new Regex(@"\d+");
                        Match matchAmount = regexPrice.Match(Amount);
                        Amount = matchAmount.ToString();
                        var dAmount = Convert.ToInt32(Amount); // ERROR HERE :(

                        double Answer = Math.Round(Convert.ToDouble(dPrice) / dAmount,4);
                        return "Станом на сьогодні курс НБУ для " + currency.ToUpper() + 
                                " становить " + Answer.ToString();
                    }
                }
            }
            catch (WebException ex)
            {
                HttpWebResponse httpResp = (HttpWebResponse)ex.Response;
                if (ex.Status == WebExceptionStatus.ProtocolError && httpResp.StatusCode == HttpStatusCode.NotFound)
                {
                    // PageNotFound
                }
                else throw;

            }
            return "Інформація щодо валюти " + currency.ToUpper() + " відсутня.";
        }

пробовал на сайте
http://regexstorm.net/tester
паттерн

\d+

для текста

<td class=\"cell_c\">10</td>\r\n

отрабатывает нормально.

Нужна подсказка.

[Serge_Bliznykov]

банальная опечатка?

Regex regexAmount = new Regex(@"\d+");
                        Match matchAmount = regexPrice.Match(Amount);

наверное, должно быть:

Match matchAmount = regexAmount.Match(Amount);

[Alex11223]

Для курсов есть куча API, зачем так извращаться?
https://bank.gov.ua/control/en/publi...rt_id=25365630

Ну и если уж парсить HTML, то не регекспами, а HTML парсером. Например HtmlAgilityPack.

[Aleksandr H.]

Serge_Bliznykov, как всегда саппорт неоценим. досадная очепятка.


методом тыка вариантов в инете, получил такой вариант для результата JSONа из линка сообщения 3

public static string NBURateAPI(string currency)
        {
            string Answer = null;
            HttpWebResponse response = null;

            try
            {
                using (WebClient web = new WebClient())
                {
                    
                    DateTime dt = DateTime.Now;

                    string date = DateTime.Now.ToString("yyyyMMdd", System.Globalization.CultureInfo.GetCultureInfo("en-US"));
                    string link = @"https://bank.gov.ua/NBUStatService/v1/statdirectory/exchange?valcode=" + currency +
                            @"&date=" + date + "&json";
                    HttpWebRequest request = (HttpWebRequest)WebRequest.Create(link);

                    request.Method = "GET";
                    request.Accept = "application/json";


                    response = (HttpWebResponse)request.GetResponse();
                    StreamReader reader = new StreamReader(response.GetResponseStream());
                    String output = reader.ReadToEnd();
                    if (output != "[]")
                    {
                        var examples = Newtonsoft.Json.JsonConvert.DeserializeObject<Response[]>(output);
                        
                        Answer = "Станом на сьогодні курс "+examples[0].cc+" ("+examples[0].txt+") становить "+examples[0].rate;
                    }
                    else
                        Answer = "Інформація щодо валюти " + currency.ToUpper() + " відсутня.";
                }
            }
            catch (WebException ex)
            {
                HttpWebResponse httpResp = (HttpWebResponse)ex.Response;
                if (ex.Status == WebExceptionStatus.ProtocolError && httpResp.StatusCode == HttpStatusCode.NotFound)
                {
                    // PageNotFound
                }
                else throw;

            }
            finally
            {
                if (response != null)
                {
                    response.Close();
                }
                
            }
            return Answer;
        }


        public class Response
        {
            [JsonProperty("r030")]
            public int r030 { get; set; }
            [JsonProperty("txt")]
            public string txt { get; set; }
            [JsonProperty("rate")]
            public double rate { get; set; }
            [JsonProperty("cc")]
            public string cc { get; set; }
            [JsonProperty("exchangedate")]
            public string exchangedate { get; set; }
        }

Alex11223, а не могли бы вы на досуге накидать пример для HtmlAgilityPack конкретно для парсинга и поиска данных на странице по ссылки из сообщения 1? Чтобы делало то же, что и regex.

[pu4koff]

public string GetNBURate(string currency)
        {
            var web = new HtmlAgilityPack.HtmlWeb();
            var doc = web.Load(@"http://bank.gov.ua/control/uk/curmetal/currency/search?formType=searchFormDate&time_step=daily&date=31.08.2017&execute");
            var row = doc.DocumentNode.SelectSingleNode($"//table[4]//tr/td[text()=\"{currency.ToUpper()}\"]/../.");
            var amountNode = row?.SelectSingleNode(@"td[3]");
            var priceNode = row?.SelectSingleNode(@"td[5]");

            if (row != null && amountNode != null && priceNode != null &&
                decimal.TryParse(priceNode.InnerText, NumberStyles.AllowDecimalPoint, System.Globalization.CultureInfo.InvariantCulture, out var dPrice)
                && int.TryParse(amountNode.InnerText, out var dAmount))
            {
                return $"Станом на сьогодні курс НБУ для {currency.ToUpper()} становить {Math.Round(Convert.ToDouble(dPrice) / dAmount, 4)}";
            }

            return "Інформація щодо валюти " + currency.ToUpper() + " відсутня.";
        }

скачивать можно и WebClient'ом и грузить строку в HtmlDocument. XPath можно другие написать

[Aleksandr H.]

Сложно И внятных примеров НАР с комментариями на русском не могу найти.

[Alex11223]

Это просто XPath, не нужны примеры по HAP. По сути там из важного только метод SelectNodes (или SelectSingleNode), который принимает XPath выражение и возвращает HTML элементы.

?. и $"{....}" это фичи C# с VS 2015 если что.

Ну а без английского программировать вообще сложно.

[Aleksandr H.]

Посоветуете ресурс для обучения написания ХPath?

[Alex11223]

Я просто посмотрел несколько примеров на w3schools или типа того, а потом гуглил по необходимости (например "xpath attribute contains").

там же не так много часто используемых вещей
имя элемента, атрибуты в [ ], parent/child, наличие или отсутствие //

Например

//div[@id='myId']/span[contains(@class, 'myClass') and @data-attr]

<div id="myId">
    <span>hello</span>
    <span class="myClass notMyClass" data-attr="whatever" data-attr2="whatever2">world</span>
    <span class="myClass">!</span>
</div>

найдет второй span тут, в любом месте документа.

Если без //, то искать от текущего элемента, как td в примере выше.

/.. для перехода к родителю (как в файловой системе), например чтоб получить тут div

//span[contains(@class, 'myClass') and @data-attr]/..

Для быстрой проверки выражения можно использовать $x() в Хроме.

1.html - Google Chrome 2017-09-03 11.52.44.png

[Aleksandr H.]

Глупый вопрос "что почитать". Берешь и читаешь одно другое третье и что-то прояснится.

А лучше спросить конкретные вопросы что где непонятно

В этой строке, как объяснить для себя?

var row = doc.DocumentNode.SelectSingleNode($"//table[4]//tr/td[text()=\"{currency.ToUpper()}\"]/../.");

В row записываем все ноды по критерию:
а) из 4-й таблицы table, (//table[4])
б) найти все теги tr, (//tr)
в) в которых есть дети td c текстом <currency>, (/td[text()=\"{currency.ToUpper()}\"])
г) взять всех ихних детей (/..)
д) ??? (/.)

Почему 4-ая table если через просмотр кода, она 6-ая
att.jpg

var amountNode = row?.SelectSingleNode(@"td[3]");
var priceNode = row?.SelectSingleNode(@"td[5]");

в amountNode передаем текст из 3-го тега td?
в priceNode передаем текст из 5-го тега td?
@ значит "взять текст"?
row? - это что?