with thisform * 1: .cErrorMessages = .checkAppExe( 'ABC.EXE' ) ; + .checkRequiredFiles() ; + .checkCompatibility() * 2: .cErrorMessages = .checkAppExe( 'ABC.EXE' ) ; + .checkRequiredFiles() ; + .checkCompatibility() * 3: .cErrorMessages = .checkAppExe( 'ABC.EXE' ) + ; .checkRequiredFiles() + ; .checkCompatibility() * 4: .cErrorMessages = '' .cErrorMessages = .cErrorMessages + .checkAppExe( 'ABC.EXE' ) .cErrorMessages = .cErrorMessages + .checkRequiredFiles() .cErrorMessages = .cErrorMessages + .checkCompatibility() endwithI don't like case 1. Case 2 looks better than case 1. Case 3 is what I do a lot. Case 4 is what I do occassionally.
* No spacing ln1=val(left(lcT1,2))*3600+val(right(lcT1,2))*60 * Spacing ln1 = val( left( lcT1, 2 ) ) * 3600 + val( right( lcT1, 2 ) ) * 60Ofcourse,spacingimprovesreadability. Ofcourse(spacing(improves(readability))).
* Straightforward lcC1 = ltrim( str( lnRec ) ) lcC2 = ltrim( str( int( lnRec / tnXis^3 ) % tnXis + 1000 ) ) lcC3 = ltrim( str( int( lnRec / tnXis^2 ) % tnXis + 1000 ) ) lcC4 = ltrim( str( int( lnRec / tnXis ) % tnXis + 1000 ) ) lcC5 = ltrim( str( lnRec % tnXis + 1000 ) ) * Justified in a special manner lcC1 = ltrim( str( lnRec ) ) lcC2 = ltrim( str( int( lnRec / tnXis^3 ) % tnXis + 1000 ) ) lcC3 = ltrim( str( int( lnRec / tnXis^2 ) % tnXis + 1000 ) ) lcC4 = ltrim( str( int( lnRec / tnXis ) % tnXis + 1000 ) ) lcC5 = ltrim( str( lnRec % tnXis + 1000 ) )The differences between the formulas are much clearer if the special justification is applied.