Breaking out of external loop

[agh]

In one of my programs, I have code like this:

void f()
{
    //...
    for (/* blah blah*/)
    {
        switch (/* blah blah */) {
            case X:
                //do stuff...
            case Y:
                //do stuff...
             //...
        }
    }
    //...
}

In some cases inside the switch statement, I want the program to get out of the for loop and continue in the function. What's a good way to do this?

I tried this:

void f()
{
    //...
    bool quit = false;
    for (/* blah blah*/)
    {
        if (quit) {
            //some code...
            break;
        }
        switch (/* blah blah */) {
            case X:
                //do stuff...
                quit = true;
                break;
            case Y:
                //do stuff...
             //...
        }
    }
    //...
}

But if case X is found at the last iteration of the loop, then the "some code..." part inside the if statement won't be reached because the program will break out of the loop and won't check wether quit == true.

Then I did this:

void f()
{
    //...
    class Stop_Loop {};
    for (/* blah blah*/)
    {
        switch (/* blah blah */) {
            case X:
                //do stuff...
                throw Stop_Loop();
            case Y:
                //do stuff...
             //...
        }
    }
    catch (Stop_Loop) {};
    //...
}

And it works fine. But I want to know if there's a better way than using exceptions to do this. Maybe it's really easy to do but I haven't come up with another way yet.

[abc123]

for(x)
top_of_for: {
  switch(j){
       case 1:
          break top_of_for;
   }
}

i tink that will work

or something similiar

i'm *sure* someone will correct me if im wrong however

[palin]

put the condition check with the for statement thats the way I would do it and have done it.

[agh]

A condition check inside the for statement won't work. There are too many different cases where the loop needs to be broken. I can't put all of that inside for().
Besides, some code needs to get executed before breaking out of the loop on each case.

Thanks for the help, though.

[agh]

abc123:

Thanks, I'll try that.

[abc123]

no, it doesn't work ..

could always do this:

for ( x ) {
  switch(z){
    case x:
       goto abc;
   }

}

abc:

[alpha]

maybe putting the

if(quit)

stuff under the switch statement...?

[agh]

Originally posted by alpha
maybe putting the

if(quit)

stuff under the switch statement...?

Of course! I'm so dumb heh... :S