Prevent Movement after Auto

[Technologyman00]

Add code to prevent the robot from moving after auto as the robot will attempt to rotate back to the initial heading as desired by the default command

[thenetworkgrinch]

Great idea! I disagree with the implementation though. I try to "Do not repeat yourself" as much as possible. Perhaps a bool attribute (see debounce) instead so execute pulls the current orientation instead of using the controller after initialize sets the debounce and execute unsets it.

[Technologyman00]

I changed it around. I don't know that I love this option but it doesn't repeat code.

[Technologyman00]

Alright should be good now

[thenetworkgrinch]

It may look uglier but it follows more rules than it breaks. This also uses less lines and less cycles so it is more performant.

[thenetworkgrinch]

You did forget to set firstLoop to true in initialize however. initialize will run every time the commabd is loaded.

[Technologyman00]

Ok I can fix that. Should I set it to private or public?

[Technologyman00]

I set it to private based on the minimal amount of research I did.

[thenetworkgrinch]

Private is correct.