What's wrong with this?

I am trying to get this thing work out but i am completely helpless now. I have problem in that bold part.

Code:

---


        for (i=0; i<N; i++)
              {
                       
                a[i+1]=a[i];
                b[i+1]=b[i];
                val=random()%MAX;
                printf("val=%d\n",val);
                if (val==0)
                        {
                                x=x+1;
                                a[i+1]=x;
                        }
                        else if (val==1)
                        {
                                y=y+1;
                                b[i+1]=y;
                        }
                        else if (val==2)
                        {       
                                x=x-1;
                                a[i+1]=x;
                        }
                        else {
                                y=y-1;
                                b[i+1]=y;
                                }
                for (q=0; q<i+1; q++)
                        {
                                if (a[q]==a[i+1] && b[q]==b[i+1])
                                goto found;                                                               
                        }
                               
        found:
        printf("exit2\n");
                        }

---

What are you trying to do and what does it do?

Sorry i posted on worng part . I am writing C code not C++.
I am trying to check final value instored at a and b with their previous values and just got to found and print exit2.

and how does it not work?

can you post a compilable example?

Basically the random number help to get me the values of a[] and b[]. Then with and for statement i am just trying to get those values of a [] and b[] and trying to compare.

can you post a compilable example?

Code:

---



#include <stdio.h>
#include <stdlib.h>


#define MAX 4
#define ISEED 1000
#define N 3                         
#define M 1                       

double a[N+1], b[N+1];
int main()
        {
                int val, i, j,v,f, q;
               
               
                srandom(ISEED);

                for (j=0; j<M; j++)
                {
                       
                        for (v=0; v<N+1; v++)
                                {
                                        a[v]=0; b[v]=0;
                                }
                        int x=0,y=0;
                        for (i=0; i<N; i++)
                                {
                       
                                a[i+1]=a[i];
                                b[i+1]=b[i];
                                val=random()&#37;MAX;
                                printf("val=%d\n",val);
                                if (val==0)
                                        {
                                                x=x+1;
                                                a[i+1]=x;
                                                }
                                        else if (val==1)
                                                {
                                                        y=y+1;
                                                        b[i+1]=y;
                                                }
                                        else if (val==2)
                                                {       
                                                        x=x-1;
                                                        a[i+1]=x;
                                                }
                                        else {
                                                        y=y-1;
                                                        b[i+1]=y;
                                                }
                                for (q=0; q<i+1; q++)
                                        {
                                                if (a[q]==a[i+1] && b[q]==b[i+1])
                                                        goto found;
                                                               
                                                       
                                        }
                               
                                found:
                                       
                                printf("exit2\n");
                        }

        }

---

it should print exit2 once only but it's not so.

Well, it's repeating because your found label is inside a loop.

i am just testing my program thru that printf statement. I just want to break both the loop i.e i and q when the condition is true.

if (a[q]==a[i+1] && b[q]==b[i+1])
when this condition is true the loop should break and the loop of for(i.... ) should also break.

Ok, then put the found label on the other side of the }.

Then introduce a flag,

Code:

---

int flag = 1;

for (i = 0; (i < N) && (flag != 0); i++)
{
    for(q = 0; (q < i + 1) && (flag != 0); q++)
    {

    }
}

---

Don't use goto.

Thanx robwhit that worked. I was just wandering and thinking it in complicated ways.

Some people recommend using a goto to break out of nested loops.