VbyesNO....need more help. Add few more line..Thanks

[pedie]

My current code looks like this...
I just want someone to please add one more condition to it...

If sheet1.Range("d5").Value = "Stop" then same as below..
or
If sheet1.Range("d5").Value = "" then
msgbox "it is empty...."
exit sub
' Stop the code from running any more
Otherwise my code...

Sub try1()
Dim Responce As VbMsgBoxResult
If sheet1.Range("d5").Value = "Stop" Then
Responce = MsgBox("my message....?", vbYesNo, "Question")
Else
GoTo skip:
End If
If Responce = vbYes Then
skip:
' code
Else
End If
End Sub

 

[kpark91]

Sub try1()
Dim Responce As VbMsgBoxResult
If sheet1.Range("d5").Value = "Stop" Then
Responce = MsgBox("my message....?", vbYesNo, "Question")
ElseIf Sheet1.Range("d5").Value = vbNullString Then
MsgBox ("It is empty...")
Else
GoTo skip
End If
If Responce = vbYes Then
skip:
' code
Else
' code
End If
End Sub

 

[Norie]

You could try Select Case.

Option Explicit
 
Sub try1()
Dim Response As VbMsgBoxResult
 
    Select Case Sheet1.Range("d5").Value
    
        Case "Stop"
            
            Response = MsgBox("my message....?", vbYesNo, "Question")
        
        Case ""
            Response = MsgBox("my message....?", vbYesNo, "Question")
            
        ' add other cases if required
        
        Case Else
            ' do nothing
    End Select
    
    ' do other things based on response
    
End Sub

 

[pedie]

Thank you so much for helping....
Now the code is giving error If there is some other value in instead of "Stop" in D5. error runtime error 13, type mismtach...

Lemmi try the Case case of Norie...

 

[kpark91]

pedie.
The error might be happening because of your process (skip).

Do you want to access your process (skip) only when the user have answered yes?

Then you should delete GoTo skip in else statement.

 

[pedie]

kpark91, it is working now. I was doing it wrong.
But just one more question...

Can we put a condition for range("D1:D5") instead of D5 only???

 

[kpark91]

Sub try1()
Dim Responce As VbMsgBoxResult
For c = 1 To 5
    If sheet1.Range("d" & c).Value = "Stop" Then
        Responce = MsgBox("my message....?", vbYesNo, "Question")
    ElseIf Sheet1.Range("d" & c).Value = vbNullString Then
        MsgBox ("It is empty...")
    End If
Next c
    If Responce = vbYes Then
    skip:
        ' code
    Else
        ' code
    End If
End Sub

 

[pedie]

kpark91, thank you for your assistance today. It worked. Thank you so much!!!

Pedie