>>>Hi
>>>
>>>I have this code snippet, lots in fact, where the code goes something like this:
>>>
LOCAL llRetVal
>>>
>>>llRetVal = DODEFAULT()
>>>
>>>IF llRetVal
>>> llRetVal = THISFORM.pgf.Page1.txtIno.CheckForBadNextNo()
>>>
>>> IF llRetVal
>>>...
>>> ENDIF
>>>ENDIF
>>>
>>>RETURN llRetVal
>>>
>>>Any suggestion where the code can be more linear.
>>>
>>>Or do I RETURN as soon as I get a false value?
>>>Or do I IF Condition()?
>>>
>>>Please advise.
>>
>>I don't think you need llRetVal variable.
>>
>>return ;
>> dodefault() and ;
>> THISFORM.pgf.Page1.txtIno.CheckForBadNextNo() and ;
>> ...
>>
>>
>
>Cetin, attention to NULL returns :)
>
>>If you like if series and do not obey 'one return point' rule (I don't):
>>
>>
>>if !dodefault()
>> return .f.
>>endif
>>if !...
>> return .f.
>>endif
>>
Cetin
Fabio,
I would also check type of returns but original post guarantees a logical (not null) return as I get it.
Cetin