Indents please

[GlenW]

Another thing I do and would like opinions on:
I only fully left align;

General declarations - not sure if this includes Function & Sub declarations but I mean it to.

Navigational labels

Debug.Prints
Debug MsgBox lines

The last 2 so they are easy to find and remove when the prog is finished.

Any comments?

[HaxSoft]

Originally posted by GlenW
Another thing I do and would like opinions on:
I only fully left align;
.
.
.
Debug.Prints
Debug MsgBox lines

The last 2 so they are easy to find and remove when the prog is finished.

Any comments?

Very good, I will do the same from now on. I always have problems finding those and deleting them after testing/debugging.

Thanks for a good tip.

[MerrionComputin]

...Debug.Prints
Debug MsgBox lines

You should define a compile time constant DEBUG and put such lines in:

#IF DEBUG Then
     MsgBox "This is a debug thingy" 
#End If

That way to remove the msgbox you just unset the compile time directive and compile it...

[darre1]

i do exactly the same.

makes code dead easy to read too

[cafeenman]

I only put them in when I need them, solve the problem and then take them out again. Then I don't have to worry about it.

[GlenW]

Originally posted by MerrionComputin


You should define a compile time constant DEBUG and put such lines in:

#IF DEBUG Then
     MsgBox "This is a debug thingy" 
#End If

That way to remove the msgbox you just unset the compile time directive and compile it...

Does this not produce an unnecessary run time overhead?

[cafeenman]

No, it doesn't get compiled.

[darre1]

i think the term is "Conditional Compilation"

[GlenW]

I learn something new every day
Every day in every way I'm getting better and better.