[VBA] IF And Statement not working

[NessPJ]

Hi there,

For some reason i have an IF AND statement that is giving me trouble. I have tried several other examples on the internet that seem to work...yet the statement i wrote will not work properly. When 1 of the 2 statements if TRUE the code is already executed (which should only happen when both are TRUE).

Can anyone help me please?

Private Sub Tabelkopieren()


Dim TabelX1C1 As String
Dim TabelX1C2 As String


TabelX1C1 = Sheets("Sheet1").Range("L4").Value
TabelX1C2 = Sheets("Sheet1").Range("L5").Value


    If (Len(TabelX1C1) = 0) And (Len(TabelX1C2) = 0) Then
    GoTo Leeg
    Else
    'Do Nothing
    End If

 

[jmacleary]

Re: [VBA] IF And Statement nog working

Hi there. There doesnt seem to be anything wrong with your code. As written, it checks that both L4 and L5 are empty. Is that what you are expecting? In the VB editor, try putting a breakpoint on the if statement and see what the values of TabelX1C1 and TabelX1C2 really are at that point.

 

[NessPJ]

Re: [VBA] IF And Statement nog working

Hi there. There doesnt seem to be anything wrong with your code. As written, it checks that both L4 and L5 are empty. Is that what you are expecting? In the VB editor, try putting a breakpoint on the if statement and see what the values of TabelX1C1 and TabelX1C2 really are at that point.

I read the values from the immediate window.
I tested with: L4 = "" and L5 = "1"

Result from VBA editor:

TabelX1C1 = ""
TabelX1C2 = "1"
Len(TabelX1C1) = 0
Len(TabelX1C2) = 1

The statement is being continued. While i would expect in this case it would execute "GoTo Leeg".
Which...to be clear is fairly simple:

Leeg:


    MsgBox ("It appears there is no data entered. Execution halted!"), vbCritical, "No data!"
    
    Sheets("Sheet1").Protect ("1234")
    
    Application.ScreenUpdating = True
    Application.DisplayAlerts = True
    
    Exit Sub

 

[Fluff]

Re: [VBA] IF And Statement nog working

It will only hit the GoTo Leeg line if both are 0, if you want to execute that line if either are 0 then change And to Or

 

[jmacleary]

Re: [VBA] IF And Statement nog working

You should change the AND to OR if thats the result you want.

 

[NessPJ]

Re: [VBA] IF And Statement nog working

It will only hit the GoTo Leeg line if both are 0, if you want to execute that line if either are 0 then change And to Or

You are absolutely right! Thanks for the help!

 

[Fluff]

Re: [VBA] IF And Statement nog working

Glad we could help & thanks for the feedback