[RESOLVED] If... Then (without using = true or = false)

[Jose_VB]

I'm reading the following code:

Dim fRecording as Boolean

If fRecording Then
...
End If

I thought the use of If... Then using a Boolean variable should be

If fRecording = true then
....
End If

What means If fRecording Then... without the use of = false or = true?

[techgnome]

All that needs to be between the If and Then is a boolean expression. Meaning it can be anything that evaluates down to a true or false value. If you want to see if a variable equals 2, you do this, right? If x = 2 Then .... you don't write If (x=2)=True Then ... right? Why not? as long as the expression, what ever it is, results in a true/false value, it can be what ever you need it to be.

-tg

[dilettante]

Something like:

If fRecording = True then

Is sort of silly and redundant. Almost as silly as faux C hungarian using "f" ("flag?) for a Boolean.

For that matter the expression must be Boolean or else the compiler coerces it to a Boolean value, which is why even when testing a Long, etc. any non-zero value takes the True branch.

[MarkT]

Something like:

If fRecording = True then

Is sort of silly and redundant. Almost as silly as faux C hungarian using "f" ("flag?) for a Boolean.

For that matter the expression must be Boolean or else the compiler coerces it to a Boolean value, which is why even when testing a Long, etc. any non-zero value takes the True branch.

I would agree with redundant but silly could be a little harsh. Some may even argue that

If fRecording = True then

is more readable than

If fRecording Then

[Bonnie West]

Some may even argue that

If fRecording = True then

is more readable than

If fRecording Then

IMHO, a more descriptive variable name would be both readable and efficient:

Dim IsRecording As Boolean

If IsRecording Then

Personally, I have nothing against using Hungarian notation (such as fRecording). In fact, I find it very helpful when reading code using an editor other than the VB6 IDE.

[Tanner_H]

Something like:

If fRecording = True then

Is sort of silly and redundant. Almost as silly as faux C hungarian using "f" ("flag?) for a Boolean.

Agreed, "f" has always struck me as an odd choice. True hungarian uses "f" for "float" and "b" for "boolean", which makes it all the more confusing.

IMHO, a more descriptive variable name would be both readable and efficient:

Dim IsRecording As Boolean

If IsRecording Then

This is great advice. Not sure if others would agree, but I also prefer this when evaluating conditions to False, e.g. instead of...

If IsRecording = False Then...

use:

If Not IsRecording Then...

[Jose_VB]

Thank you very much to all for the answers. I understand it now.

[dilettante]

Welcome to Nitpickers Anonymous!

[JuggaloBrotha]

Dim IsRecording As Boolean
 
'This
If IsRecording Then
 
'Is the same as
If IsRecording = True Then
 
'Just as this
If Not IsRecording Then
 
'Is the same as
If IsRcording = False Then

In both cases I recommend avoiding the double evaluation, don't go with the choice involving an = True or = False in it when the variable is already a boolean.