I am merrily programming along, and click on "Build and Run" which gives me an error. I am using a switch statement as depicted below, but am getting an error that says "case label not within switch statement" and "Break statement not within Loop or switch statement". What am I doing wrong?

There's nothing wrong with the type of variable itself.

You're using an enum type, and the switch knows you're missing one. In other words,


You'll get that error for not including case ThingThree: or a default:

I'm guessing some syntax error earlier in the file...

Heh. My brain totally interpreted that first error message differently. It's likely not what I wrote.

OneSadCookie Wrote:I'm guessing some syntax error earlier in the file...

I actually did have a parse error that I can't figure out, and a "Undeclared Function" error. Would that have something to do with it?

quite likely. You'll need to post your entire file probably, and all of your error messages.
-wyrmmage

Also, always add default. I've never done this before, but say this happened:


You see? if you left the default out, in this case the switch would not know what to do—none of the conditionals fit the problem. So the compiler yells at you.

Also, about the undefined function and stuff, always track them down. Usually they're indicative of a larger problem—i.e. you spelled the function wrong, forgot to include a header (or included the wrong one), or something else. Lastly, use OSC's compiler flags. They make your code cleaner and generally avoid nasty compiler battle headaches.

On a totally unrelated note: Why aren't urls stylized in some manner by default?

What if you're not using enums? Sometimes I just want to act if a variable is set to a couple of different values, for example:
If, for example, you just needed to run code when the game is starting and ending. Do you still have to include a default? My compiler doesn't tell me that I have to, so I've always just figured it ignored the switch statement if theInt was set to a value other than 0 or 5.
-wyrmmage

You don't have to, it's just to trap something unexpected in case you want to do something (like return an error because it's not 0 or 5).=

Sorry about the short reply's I'm kind of hard-pressed for time to reply. Alright, so first of all, Thank you for all the reply's! It's really helpful. Secondly, I don't know how to add compiler flags, so if you could tell me how to do that, that would be nice. I can't figure out the parse error/undeclared function, so I've attached the source code for that file. Thanks again for your help!

and the errors/warnings

Start by indenting your code properly, that way you'll be able to see at a glance whether you've got the right number of {s and }s...

Wow, that's a lot of stuff.
A. There's many more problems than just that switch statement.
b. it's appendBezierPathWithOvalInRect, not appendBezierPathWithOvalnRect.
c. Look at the bracketing to displayPlaying: you close it WAY after it should have been closed.

That's all I could glean, but I'm sure there's stuff I'm missing.
A note about bracketing: let the ide do the automatic indentation. That way it's pretty easy to figure out if you're missing a bracket somewhere.

Nayr Wrote:Wow, that's a lot of stuff.
A. There's many more problems than just that switch statement.
b. it's appendBezierPathWithOvalInRect, not appendBezierPathWithOvalnRect.
c. Look at the bracketing to displayPlaying: you close it WAY after it should have been closed.

That's all I could glean, but I'm sure there's stuff I'm missing.
A note about bracketing: let the ide do the automatic indentation. That way it's pretty easy to figure out if you're missing a bracket somewhere.

b. That fixed 8 of my errors
c. When should I have closed it?

How do you let it do automatic indentation?

c: Try indenting properly and it'll be obvious. It's obvious even without indenting. In what you pasted, you have code that appears to be outside of a method. That's obviously wrong. What it really is, is bad indenting and an unclosed method.

You were also missing '4' between some plus signs.



Use a lowercase 'a' in AndColumn. It's convention.

Hmm, I was typing what was in the tutorial. But I will start indenting properly(if indeed what was in the tutorial was wrong.)