Breaking Outer loop when Condition become true in Inner loop

Question

3.33/5 (3 votes)

See more:

hello friends,

suppose we have nested loops as below....

for (int y = 0; y < someLength; y++)
            {
                for (int x = 0; x <somelength;x++)          
                 {
                    if (condition becomes true)
                    {
                       Break both loops inner loop and outer loop.......
                    }
                }
            }

is there any standard way to break both loops..........when condition becomes true in just Inner loop

as we know.... can break loop in which we are nicely...... using break...

Thanks........

Posted 24-May-11 23:43pm

Pritesh Aryan

Updated 24-May-11 23:46pm

v5

Add a Solution

3 solutions

Solution 3

below code will not return but come out of loop and will execute next code after loops

C#

bool loopbreak=new bool()  ;
           loopbreak = false;
           for (int y = 0; y < 10 && loopbreak==false; y++)
           {
               for (int x = 0; x <10;x++)
                {
                   if (x==5)
                   {
                      loopbreak=true;
                      break;
                   }

               }
          }

Posted 25-May-11 0:29am

hitesh_tech

Comments

Sergey Alexandrovich Kryukov 25-May-11 10:48am

Will work, but I would prefer a simple solution by John. Please see my comments.
--SA

Solution 2

try this code

C#

for (int i = 0; i < 5; i++)
           {
               for (int j = 0; j < 5; j++)
               {

                   if (i == 1)
                   {
                       return;
                   }
                   MessageBox.Show("j" + j.ToString());
               }
               MessageBox.Show("i" +i.ToString());
           }

Posted 24-May-11 23:59pm

ambarishtv

Updated 25-May-11 0:00am

v2

Comments

Sergey Alexandrovich Kryukov 25-May-11 10:49am

You should show the method you're returning from, otherwise it's not quite clear. This is what I would prefer to do.
Will you improve you solution like that (see also my comments to John's solution)?
--SA

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

#realJSOP · Accepted Answer · 2011-05-24T23:54:00

The only thing I can think of is to put the nested loops into their own method, and if the appropriate condition is met, simply return from the method.

C#

private void LooperMethod()
{
    for (int i = 0; i < somevalue; i++)
    {
        for (int j = 0; j < someOtherValue; j++)
        {
            if (conditionExists)
            {
                if (specialConditionExists)
                {
                    return;
                }
                break;
            }
            if (outerConditionExists)
            {
                break;
            }
        }
    }
}

If you can't abstract the loop code into its own method, your only real choice is to maintain a variable that indicates when to break the outer loop:

C#

bool breakOuter = false;
for (int i = 0; i < somevalue; i++)
{
    for (int j = 0; j < someOtherValue; j++)
    {
        if (conditionExists)
        {
            if (specialConditionExists)
            {
                breakOuter = true;
            }
            break;
        }
        if (outerConditionExists || breakOuter)
        {
            break;
        }
    }
}