Моя кнопка требует двух кликов вместо одного

Question

Я работаю над небольшим приложением на C#, которое обновляет веб-страницу до тех пор, пока не будут выполнены некоторые условия. У меня есть «Огонь» = начать освежающую кнопку и «Стоп!». который должен остановить операцию. Моя проблема состоит в том, что требуется 2 попытки нажать кнопку остановки вместо 1. Ниже мой код:

Обновленный код с таймером. Тем не менее, я думаю, что может быть лучше использовать таймер, я думаю, что он не обновляется каждую секунду после первых 2-3 обновлений или вообще не обновляется. Есть ли недостаток в моем коде, который я не могу обнаружить?

private void FireButtonClick(object sender, EventArgs e) 
{ 

    try 
    { 
     if (webBrowser1.Url.ToString().StartsWith("some url")) 
     { 
      _stopped = false; 
      _timer.Tick += new EventHandler(RefreshBrowser); 
      _timer.Interval = (1000) * (1); 
      _timer.Enabled = true; 
      _timer.Start(); 
     } 
     else 
     { 
      MessageBox.Show("You must logon first."); 
      return; 
     } 
    } 
    catch (Exception ex) 
    { 
     MessageBox.Show(ex.Message); 
    } 
} 

private void RefreshBrowser(object sender, EventArgs e) 
{ 
    string content = "disabled"; 
    string baseUrl = @"http://some url"; 
    string newUrl = string.Empty; 
    string buttonXpath = @"/html/body/div/div[6]/table/tr/td[2]/table/tr/td/table/tr/td/table/tr[3]/td[2]/div[4]/a"; 
    webBrowser1.Refresh(); 
    _proceed = false; 
    if (!content.ToLower().Equals("disabled") && !_stopped) 
    { 

     if (!_stopped) 
     { 
      HtmlAgilityPack.HtmlDocument htmlDocument = new HtmlAgilityPack.HtmlDocument(); 
      htmlDocument.LoadHtml(webBrowser1.DocumentText); 
      HtmlNode node = htmlDocument.DocumentNode.SelectSingleNode(buttonXpath); 
      content = node.GetAttributeValue("disabled", string.Empty); 
      newUrl = node.GetAttributeValue("href", string.Empty); 
     } 
    } 
    else 
    { 

     webBrowser1.Navigate(baseUrl + newUrl); 
    } 


} 

private void webBrowser1_Navigated(object sender, WebBrowserNavigatedEventArgs e) 
{ 
    _proceed = true; 
    urlTextBox.Text = webBrowser1.Url.ToString(); 
} 

private void MainPageButtonClick(object sender, EventArgs e) 
{ 
    try 
    { 
     webBrowser1.Navigate(_mainPage); 
    } 
    catch (Exception ex) 
    { 
     MessageBox.Show(ex.Message); 
    } 
} 

private void GoButtonClick(object sender, EventArgs e) 
{ 
    try 
    { 
     webBrowser1.Navigate(urlTextBox.Text); 
    } 
    catch (Exception ex) 
    { 
     MessageBox.Show(ex.Message); 
    } 

} 

private void StopButtonClick(object sender, EventArgs e) 
{ 
    _timer.Stop(); 
    _proceed = true; 
    _stopped = true; 
} 

}

Answer 1

Вы должны действительно изменить код, чтобы использовать таймер. Когда вы злоупотребляете потоком графического интерфейса пользователя и выполняете периодические вызовы на Application.DoEvents();, клик может обрабатываться только в этом месте, то есть вам нужно ввести цикл для обработки события.

Это абсолютное отсутствие программирования GUI. Вместо этого используйте таймер.

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

Answer 2

Я предполагаю, что вы застряли в цикле For и не можете остановиться, пока этот цикл не завершится. Вот почему, по-видимому, требуется два клика, и он просто ждет остановки.

Попробуйте вместо этого использовать поток, который выполняет обновление веб-страницы и вызывает thread.run() в вашей кнопке Run Button и thread.stop() в клике Stop. Поскольку обновление веб-страницы будет происходить в отдельном потоке, оно никогда не сможет помешать взаимодействию с пользовательским интерфейсом.

Answer 3

Мне кажется, что это происходит из-за того, что вы выполняете все в одном потоке, поэтому, когда ваш код работает, взаимодействие с пользовательским интерфейсом возможно (= первый щелчок), и, хотя страница перезагружает взаимодействие с пользовательским интерфейсом (= вторая нажмите). Если это проблема, выполните свою логику refresch в отдельном потоке.

Что-то вроде:

private void FireButtonClick(object sender, EventArgs e) 
{ 
    Thread worker = new Thread(new ThreadStart(delegate() 
    { 
     //your code 
    }); 
    worker.IsBackground = true; //so it does not block the app from being closed 
    worker.Start(); 
} 

Если доступ к UI-элементы в тему вы также должны использовать Invoke!