6

I have a WinForm application with several input controls on a form. In the validation event handler (either Validating or Validated), I need to determine what control to activate next, based on the validated value.

In Microsoft's documentation of the Validating event, it states:

Caution

Do not attempt to set focus from within the Enter, GotFocus, Leave, LostFocus, Validating, or Validated event handlers. Doing so can cause your application or the operating system to stop responding. For more information, see the WM_KILLFOCUS topic in the "Keyboard Input Reference" section, and the "Message Deadlocks" section of the "About Messages and Message Queues" topic in the MSDN library at http: // msdn.microsoft.com/library.

There is an ActiveControl property for a Form class that allows setting the control that is to become active, and no restrictions are mentioned. I have not found any other solution after several hours of web searches.

Is setting the ActiveControl property (instead of Focus) from my Validated event handler a safe way to positively activate the control I want? If not, are there any solutions?

Because the .NET Compact Framework doesn't have the ActiveControl property, can anyone suggest a solution?

Cody Gray - on strike
  • 239,200
  • 50
  • 490
  • 574
Suncat2000
  • 1,105
  • 1
  • 14
  • 17

4 Answers4

11

Yes, changing the focus during a Validating event is quite troublesome. The event is raised at the exact time the focus changes. The next control has already obtained the focus as far as Windows is concerned but the logical form state still has the focus at the control being validated. When you set e.Cancel to true, Winforms must undo the Windows focus state. When you don't, it must update the logical state after the event. There are a wide variety of things that can go wrong when you change focus yourself.

It is important that you wait until the focus has been sorted out. Best thing to do is to delay your code until everything is done running and the form goes idle again. You can cleanly do so by using the Control.BeginInvoke() method. Something like this:

    private delegate void ChangeFocusDelegate(Control ctl);

    private void textBox1_Validating(object sender, CancelEventArgs e) {
        int value;
        if (!int.TryParse(textBox1.Text, out value)) e.Cancel = true;
        else {
            if (value == 1) this.BeginInvoke(new ChangeFocusDelegate(changeFocus), textBox2);
            else this.BeginInvoke(new ChangeFocusDelegate(changeFocus), textBox3);
        }
    }
    private void changeFocus(Control ctl) {
        ctl.Focus();
    }
Hans Passant
  • 922,412
  • 146
  • 1,693
  • 2,536
  • Wow, clever answer! I thought the delegated method have to call EndInvoke() though, but that was before I knew the difference between Delegate.BeginInvoke() and Control.BeginInvoke(). See [.net - What's the difference between Invoke() and BeginInvoke][1] [1]http://stackoverflow.com/questions/229554/whats-the-difference-between-invoke-and-begininvoke – Suncat2000 Feb 11 '11 at 19:13
3

Have you tried setting the Cancel property of the CancelEventArgs that are passed to the Validating event handler to False?

This is the intended way to keep the focus on the current control and prevent the next control from getting focus if validation fails. For example:

private void TextBox1_Validating(object sender, System.ComponentModel.CancelEventArgs e)
{
    //Make sure that the textbox is not left blank
    if (string.IsNullOrEmpty(TextBox1.Text))
    {
        e.Cancel = true;
    }
}
Cody Gray - on strike
  • 239,200
  • 50
  • 490
  • 574
  • Thanks. I know about this, but it only keeps the focus from changing. It doesn't allow setting focus to a specific control. – Suncat2000 Feb 11 '11 at 14:37
  • @Suncat: Yeah, I couldn't tell from the question that you wanted to set focus to a *different* control. Hans already gave you a good answer on how to do that. – Cody Gray - on strike Feb 12 '11 at 04:09
0

This thread is old, but I have a couple of thoughts:

  1. Every control has a Tag property. What about giving the control you want have focus a unique Tag value, and then create a method that will iterate through the controls to find that control? Then you can set focus to it.

  2. Instead of using the Validating event, why not use Leaving instead? Doesn't seem to have the same quirks.

Uwe Keim
  • 39,551
  • 56
  • 175
  • 291
Mark
  • 1
0

Form.SelectNextControl or Control.SelectNextControl

http://msdn.microsoft.com/en-us/library/system.windows.forms.control.selectnextcontrol.aspx