Skip to content
This repository has been archived by the owner on Oct 20, 2023. It is now read-only.

Latest commit

 

History

History
24 lines (16 loc) · 841 Bytes

refactoring.md

File metadata and controls

24 lines (16 loc) · 841 Bytes

Refactoring Steps

Finished steps are strikethroughed

  1. Have one Constants file

    • Hatchbot example uses a constants class with static final classes nested inside (in the same file)
    • Should be easy to do
    • Dylan didn't specifically say to do this but its in hatchbot
  2. Hungarian notation

    • Python script is done but will apply last just in case
  3. Remove commented out bloat code

  4. Organize subsystems

    • Combine everybot subsystems with actual subsystems
  5. Change static setup methods to be constructors

  6. Migrate from timing based to command based

    • Timing methods in everybot should be moved to robotcontainer using JoystickButton.whenpressed
  7. Change references to static methods to call from the class rather than the instance

    • Or rather, change all methods to be called from the instance