!HELP! IM STUCK IN A SUBROUTINE!!! !HELP!

[cdennisxlx2]

I know it works because it works in all the other modes, just not the one with the subroutines, we have a dip switch to choose autonomous modes. but here are the user_routines_fast.c and user_routines.h files (attached)

[Alan Anderson]

	if (rc_dig_in11 = 1) /* 1st Autonomy code */

There's your problem right there. You just tried to assign the value 1 to the variable rc_dig_in11. To test for equality, you must use the == symbol, with two equal signs.

The code you posted will always run the BASH routine. You need to change all those single = to double == in your if and else if statements.

[JamesBrown]

Originally Posted by Alan Anderson

	if (rc_dig_in11 = 1) /* 1st Autonomy code */

There's your problem right there. You just tried to assign the value 1 to the variable rc_dig_in11. To test for equality, you must use the == symbol, with two equal signs.

The code you posted will always run the BASH routine. You need to change all those single = to double == in your if and else if statements.

Yep, that would be it.

As a note to any one else who ever needs help the more code you post the better we can help, don't worry about people stealing it, it wont work on our robots anyway.

James

[Dave Scheck]

I agree with Alan and James that a problem exists in the use of the assignment operator, but the initial problem stated that the code was stuck in Left. I don't see how this could be reached. How was it determined that it was getting stuck in Left?

[cdennisxlx2]

i had a student telling me what directions the wheels were turning and i took his word for it instead of double checking, but thank you all for the help

[Matt Krass]

I still think we need to see the rest of the code, or at least how the time variable is defined. If it isn't defined static, the variable will be re-initialized everytime its host entity is restarted. So if it's at the top of your function, and this function runs over and over every loop, it will be reinitialized every loop to 0, no matter how much you increment it. Or, time can be a global and it will retain its value, but if you're assigning 0 to it every loop that will also ruin your day. I think you have a pretty simple (and somewhat hard to notice) problem that I'm unfortunately very familiar with, as Mark McLeod can tell you, how many times did I hit Chris with the robot again?

[cdennisxlx2]

the .c file is attached above, but i know for a fact thats not the problem because ive defined "time" and incrimented it the same way for 3 years now and i have had no problem with that at all.

[Alan Anderson]

Originally Posted by ghhs_1527

...ive defined "time" and incrimented it the same way for 3 years now and i have had no problem with that at all.

You probably have had problems, but you didn't recognize them as being caused by the way you're initializing your time variable. The way you're doing it, the initialization happens once when the robot powers up. You won't be able to rerun the autonomous mode correctly unless you reset the robot first. Better would be set time to zero explicitly when you enter the User_Autonomous_Code() function, just before the while (autonomous_mode) loop.

[Dave Flowerday]

Originally Posted by Matt Krass

I still think we need to see the rest of the code, or at least how the time variable is defined. If it isn't defined static, the variable will be re-initialized everytime its host entity is restarted.

It's there, line 25 of user_routines_fast.c, as a global, so it should be fine. I'm not exactly sure what you mean by "If it isn't defined static, the variable will be re-initialized everytime its host entity is restarted."

[Matt Krass]

Originally Posted by Dave Flowerday

It's there, line 25 of user_routines_fast.c, as a global, so it should be fine. I'm not exactly sure what you mean by "If it isn't defined static, the variable will be re-initialized everytime its host entity is restarted."

I missed the attached posting, my fault for trying to do too much at once, sorry about that.

As far as what I meant, I meant that a local variable inside of a function would be reset everytime the function ended and started again. Not sure why I worded it that way, let me just chalk it up to a goofy morning eh?