At 06:59 2017-07-13, "Darren" <fox...@ozemail.com.au> wrote:
And another tool I have which helps is to add comments to the end of the
terminating commands  - e.g. ENDIF, ENDFOR, ENDDO, EXIT etc. which help to
identify the branch / loop.

I dislike the clutter intensely, and if the condition changes, there are then two places to change.

[snip]

... With terminating comments...

  DO WHILE .T.

    *- Operate between 1am and 3pm local sydney time
    IF NOT BETWEEN(HOUR(DATETIME()), 1, 15)  && 01:00:00 to 14:59:59

      =messagebox("This function is designed to run between 1am and 3pm
local time (Sydney)" + CHR(13) + CHR(13) + ;
        "The program will now terminate", 48, "Outside of times of
operation", 10000)

      EXIT && DO WHILE .T.

    ENDIF && NOT BETWEEN(HOUR(DATETIME()), 1, 15)  && 01:00:00 to 14:59:59

    SELECT w_bnTrack
    SCAN FOR rssAvailable

      *- Check if this track has runs today.
      SELECT w_usdRace
      LOCATE FOR track_code = w_bnTrack.livetkcode
      IF NOT FOUND()
        SELECT w_bnTrack
        LOOP
      ENDIF && NOT FOUND()

If you really need this to keep your nesting straight, you have too much nesting.

[snip]

Sincerely,

Gene Wirchenko


_______________________________________________
Post Messages to: ProFox@leafe.com
Subscription Maintenance: http://mail.leafe.com/mailman/listinfo/profox
OT-free version of this list: http://mail.leafe.com/mailman/listinfo/profoxtech
Searchable Archive: http://leafe.com/archives/search/profox
This message: 
http://leafe.com/archives/byMID/profox/99cb763296cd638f2f48d19aad910c81@mtlp000086
** All postings, unless explicitly stated otherwise, are the opinions of the 
author, and do not constitute legal or medical advice. This statement is added 
to the messages for those lawyers who are too stupid to see the obvious.

Reply via email to