Readability is most important.
The second seems more natural to me. It seems misleading to use the equal sign and then adding the Not
to it.
The Not
IMHO is best used with boolean functions like IsNumeric(...)
which lends itself to a more natural reading of the code, E.g.,
If Not IsNumeric(myNumberString) Then
And, I personally prefer using Not
to = False
for Boolean tests. E.g., I like...
If bFlag1 Then
If Not bFlag2 Then
...more than...
If bFlag1 = True Then ' <-- (This is totally redundant) = True
If bFlag2 = False Then
If a <> b = True Then ' <-- please don't ever do this