Fix #14, Refactor LC_SampleAPs to remove extraneous if statement

[thnkslprpt]

Checklist

• I reviewed the Contributing Guide.
• I signed and emailed the appropriate Contributor License Agreement to [email protected] and copied [email protected].

Describe the contribution

• Fixes Refactor LC_SampleAPs #14
  • LC_SampleAPs() has been refactored to remove the extraneous if statement.
  • Action points are still sampled in the same way, and if StartIndex == EndIndex, the for loop will only run once (i.e. the same as before).

Testing performed
GitHub CI actions (incl. Build + Run, Unit Tests etc.) all passing successfully.

Expected behavior changes
Intent of the code remains basically unchanged, although the logic has changed slightly.
Main change is that CurrentAPState is checked as valid before any sampling of action points, whereas previously this was only checked in the single action point sample condition - i.e. if (StartIndex == EndIndex)

Contributor Info
Avi @thnkslprpt

[chillfig]

Removes extraneous logic. Looks good!