Сообщений в теме: 7

[dmitryberezin]

Здравствуйте!

 

Суть моей программы в том, что она должна каждые две секунды обновлять страницу сайта и и парсить с нее  одну строку.

 

Вот, собственно, код части программы, в которой возникает проблема:

 

while(true) {        

      driver.get("адрес сайта");

     number = driver.findElements(By.tagName("b"));   // записываем в список элементы с тегом b
     string = number.get(2).getText();                             // записываем в строку текст элемента с индексом 2

        if (string.trim().length() == 0) {                             // проверяем если строка пустая, то выводим в консоль ошибку

                    System.out.println("ошибка");
        } else{
                 System.out.println(string);                         // если же нет, то выводим в консоль строку
        }

           try {
             Thread.sleep(2000);                                     //останавливаем на 2 секунды
        } catch (Exception e){};

}

    Дело в том, что строка иногда остается пустой и в консоль выводится "ошибка", но в основном все работает как надо (в консоль выводится строка)

 

    То есть, 20 раз в консоль может вывестись строка как положено, а потом раз 5 ошибка, дальше все снова продолжает работать как нужно

    

    Буду благодарен если кто-нибудь поможет решить проблему.

 

 

 

 

[Freiman]

наверно, первое, что можно сделать - вставить между открытием страницы и поиском элемента ожидание. Сейчас оно у вас почему-то в конце цикла и try/catch.

[dmitryberezin]

наверно, первое, что можно сделать - вставить между открытием страницы и поиском элемента ожидание. Сейчас оно у вас почему-то в конце цикла и try/catch.

Спасибо! Помогло

[BadMF]

но теперь, если у вас это не тест, а тригер к оперативного принятия решения, вы начали опаздывать на 2 секунды =)

что конечно лучше чем на 12 секунд =)

[Noksa]

Да даже если это тест. 

Если в каком-то тесте есть sleep, значит он есть и в других.

 

Страшно представить, сколько времени тупо в никуда уходит от таких решений.

[Freiman]

Уважаемые опытные тестировщики, вы постановку задачи топикстартера вообще прочитали? :)

Суть моей программы в том, что она должна каждые две секунды обновлять страницу сайта и и парсить с нее  одну строку.

[Alex]

 

Уважаемые опытные тестировщики, вы постановку задачи топикстартера вообще прочитали? :)

Суть моей программы в том, что она должна каждые две секунды обновлять страницу сайта и и парсить с нее  одну строку.

 

Ну если придираться, то все равно такой хардкорный слип некорректно. Каждые две секунды - значит надо заводить таймер и запускать следующую итерацию по его окончании. Учитывая характер действий в цикле, даже сейчас маловероятно уложиться в две секунды на итерацию. 

[Noksa]

 

Уважаемые опытные тестировщики, вы постановку задачи топикстартера вообще прочитали? :)

Суть моей программы в том, что она должна каждые две секунды обновлять страницу сайта и и парсить с нее  одну строку.

 

 

 

В таком случае надо писать по-другому, а не использовать thread.sleep напрямую.

 

В C# я бы реализовал это так. Здесь мы точно пойдём на следующую итерацию после двух секунд после загрузки страницы, а не после всех действий на ней.

var sw = new Stopwatch();
DriverManager.Driver.Navigate().GoToUrl("http://url.ru"); 
while (true)
            {                            
                sw.Start();
                var number = DriverManager.Driver.FindElements(By.TagName("b"));
                var str = number[2].Text;
                Console.WriteLine(string.IsNullOrEmpty(str) ? "Error" : str);
                sw.Wait().Until(_ => _.Elapsed.TotalSeconds >= 2);
                sw.Stop(); 
                sw.Reset();
                DriverManager.Driver.Navigate().Refresh(); 
             }

               

В Java есть что-то похожее. И да, вместо Thread.Sleep тогда уж использовали бы TimeUnit.