Набросал код:
bdhelp.OpenConnection();
while (true)
{
List<IWebElement> allElms = driver.FindElements(By.XPath("//li[contains(@class, 'j-product-item')]")).ToList();
foreach(var elm in allElms)
{
m++;
VariablesForReport.tovName = driver.FindElement(By.XPath(".//p[contains(@class, 'goods-card__description')]/span|//span[contains(@class, 'goods-name')]")).GetAttribute("innerText");
VariablesForReport.tovPriceWithDiscount = driver.FindElement(By.XPath(".//del[contains(@class, 'goods-card__price-last')]|//span[contains(@class, 'price-old-block')]/del")).GetAttribute("innerText");
VariablesForReport.tovPriceWithoutDiscount = driver.FindElement(By.XPath(".//del[contains(@class, 'goods-card__price-last')]|//span[contains(@class, 'price-old-block')]/del")).GetAttribute("innerText");
VariablesForReport.cardNum = driver.FindElement(By.XPath("//li[contains(@class, 'goods-card')]|//div[contains(@class, 'product-card-overflow')]/div/div[contains(@id,'')]")).ToString();
var query = $@"INSERT INTO `parser_report`(`product_name`) VALUES('{VariablesForReport.tovName}')";
var command = new MySqlCommand(query, bdhelp.Connection);
command.ExecuteNonQuery();
var query1 = $@"UPDATE `parser_report` SET `price_w_discount`='{VariablesForReport.tovPriceWithDiscount}' WHERE `Num` = {m}";
var command1 = new MySqlCommand(query1, bdhelp.Connection);
command1.ExecuteNonQuery();
var query2 = $@"UPDATE `parser_report` SET `price_without_discount`='{VariablesForReport.tovPriceWithoutDiscount}' WHERE `Num` = {m}";
var command2 = new MySqlCommand(query2, bdhelp.Connection);
command2.ExecuteNonQuery();
var query3 = $@"UPDATE `parser_report` SET `vendor_code`='{VariablesForReport.cardNum}' WHERE `Num` = {m}";
var command3 = new MySqlCommand(query3, bdhelp.Connection);
command3.ExecuteNonQuery();
}
}
bdhelp.CloseConnection();
вот верно?
- нет. Вы постоянно ищите по всему дереву элементы, когда должны найти определенный участок, а затем в нем искать уже нужные данные. Вот у вас есть коллекция элементов, в ответе вы вон начали циклом проходится, это правильно, только потом опять началиdriver.FindElement()
, когда должно бытьelm.FindElement()
. Также у вас ерунда в XPath, зачем вы пишете весь путь, когда достаточно, например.//span[contains(@class, 'goods-name')]
. Далее, зачемGetAttribute("innerText")
, если можно просто.Text
? Короче, дума понимаете, что делаете весьма много лишних действий.