Setfocus is not working after a looping through the textbox

[LucianoOliveira]

I have an "Input data" form and all I'd like to do was to add a code so when the user press the button Ok it'd check each texbox and if it's empty show a error message and setfocus back to the empty textbox.<?xml:namespace prefix = o ns = "urn:schemas-microsoft-comfficeffice" /><o></o>
<o></o>
I'm a new vba user, however I managed to check the textboxes using a looping through the controls "For Each cCount...", but for any reason the cCount.Setfocus doesn't work.<o></o>
<o></o>
Please, any help would be greatly appreciated.<o></o>
<o></o>
Cheers<o></o>

 

[Jolly_Nolly]

Hi LucianoOliveira
Welcome on board.

Can you post your code so that we can assist you.

Nolly

 

[LucianoOliveira]

Hi Nolly, thanks for your quick repply

This is the part of the code that I was refering:

For Each cCont In Me.Controls
If TypeName(cCont) = "TextBox" Then
If cCont = vbNullString Then
MsgBox "Please you still have empty fields"
cCont.SetFocus
'Cancel = True
End If
End If
Next cCont

I hope you can help me
Thanks

 

[Jolly_Nolly]

Your code is fine except it keeps looking through all the text boxes and showing the message box for each empty text box before reaching the end of the code.

Just add this one line and it will stop at the first empty textbox.

For Each cCont In Me.Controls
    If TypeName(cCont) = "TextBox" Then
        If cCont = vbNullString Then
        MsgBox "Please you still have empty fields"
        cCont.SetFocus
        Exit Sub [COLOR=SeaGreen]'##Add this line##[/COLOR]
        End If
    End If
Next cCont

I hope this is what you needed.
Nolly

 

[LucianoOliveira]

Thank you very much Nolly

It's working well now.

Have a good day, Cheers