I'm new to robotc and programming, yet am trying to navigate a maze. The following portion of my code shows errors as listed. Any thoughts on what's wrong?

if (!Walls[0]) {
if (Map[MyPosX+RelPosX(LeftOf(MyDir))][MyPosY+RelPosY(LeftOf(MyDir))]==0)
NextMove = 0;
} else {
DecreaseOne(Map[MyPosX][MyPosY]);
DecreaseOne(Map[MyPosX+RelPosX(LeftOf(MyDir))][MyPosY+RelPosY(LeftOf(MyDir))]);
}

Errors Line 2:
Unexpected error token-> '?'
Unmatched left parenthesis ')'
Expected , found :
Unexpected error token :
Unexpected error token 0
Unexpected error token ,
Unexpected ; fund ,
Unexpected error token ==

other errors
The fifth and sixth lines (DecreaseOne) lines have the same types of errors as above. Thanks for any help you can provide.

try that instead. You were missing a whole bunch of curly braces. If this is not the intended structure of your code, then you should learn to make better use of indentation.

- Xander

Thanks for the previous help. My maze navigation program is getting there, but I'm having one last error with my code. I get the following message in the line " MyDir = RightOf(MyDir);" in the switch statement.

**Severe*:Internal:: 'assign location' is same as "expression location' on simple assignment. Recursive proc call of 'cast" may cause this.

Well, without knowing what tDirection is, I can't really tell. Perhaps there is a better way to handle this multiple casting thing, it looks really nasty.

Xander

Here is tDirection. I define North to be the direction the robot is when it enters the maze.

What is wrong with using something like:

- and -



If you are not reusing RightOf and LeftOf, consider inserting the code there directly, it will make your code a lot more readable.

Xander

Just curious, but way wouldn't this work?
#define RightOf(Dir) (Dir - 1) % 4

if Dir=0, North it will return 3, East, etc.

LennyB,

If Dir == 0, you will end up doing (0 - 1) % 4 which is -1, not 3.

- Xander

I think it would work better to add 3 instead of subtracting 1.

#define RightOf(Dir) (Dir + 3) % 4