Thread: Segmentation Fault -_-

Have you tried to print out the values of newX and newY right before the following line:

Code:

        while (((grid[newX + 1][newY] || 
                 grid[newX - 1][newY] || 
                 grid[newX][newY - 1] || 
                 grid[newX][newY + 1]) != 1)
              )

This is the line causing your segmentation fault. The problem I saw was newX was equal to 9 so 9 + 1 would be out of bounds.

Jim

Ooga,

While that code does eliminate all the segmentation errors, I am still facing the massive problem that it does not check if the "adjacent" element horizontally and vertically, has a value equal to 1.

The loop is meant to end as soon as newX and newY are adjacent to a particle with say, a and b co-ordinates. The way I've designed it is by assigning particle(a,b) with a value 1 (aka the element grid[a][b] = 1, and all other elements in **grid = 0. The while loop does a random walk, checking random elements in the grid based on this random walk to see if it does end up next to this particle, and then stops.

Try this.

Code:

        while (!(newX < size-1 && grid[newX+1][newY] ||
                 newX > 0      && grid[newX-1][newY] ||
                 newY > 0      && grid[newX][newY-1] ||
                 newY < size-1 && grid[newX][newY+1])) {

Ooga,

It still doesn't work man. Segmentation faults are killing me arghh.

Grumpy,

It may seem "trivial" to you because you have years of experience. However I'm still starting out and have been studying C for one month. It takes me longer to solve problems like this. So stop being an ass.

The newY below should be newX.
Fix that and use the while condition I gave you.

Code:

            else if(rnd == 2)
             {
                 if(newY == 1)
                 {
                         step = step * (-1);
                 }
                 newX = newX - (step);
             }

Also, it newX should actually be compared to 0, not 1.

Yess! It works now man! Thank you so much for your help Ooga dude. And the rest of you guys for your time.

I made this pic with your program (size=50, noPars=250)

Code:

                              *    *
                              * ****
                              * *
                          *   ***
                          *   * *
                    *    ******
             *      *      *
             *     **      *
             *      **   * *
             **      **  ***
              *  **   ** ****
    ** *      ******   **** *
    ****           *     *  *         **
       *           ****  *            **
       *     *       *** *           **
       * *****     *  ****   ***      *  *
    ****** * **********  * ****  *  * *****
    *****  *  *   *   * ************* **    *
   ** *          **   * ** * * **  ******** *
    * **          *   *         **        ***
 **** *          **   *         *         *
 **   *        *****  **        *         *
  *   **         * *  *        **
     **          *    *     *****
      *        ****            ***
      *          *             * *
                              **
                             ***
                              *
                              **
                               *

I just realized that by putting all my source code up here that I might get plagiarized. This was meant to be a final project for one of my modules at college...
Is there any way to remove the sections where I posted my full code and just leave the section of code where I posted the while loop? Otherwise I'm gonna be screwed if everyone starts handing assignments with my code all over it. Please reply moderators as this is kind of urgent, we have severe consequences for any plagiarism. The only reason why I posted the full code is so that everyone who was helping me could get an idea of my initializations of arrays and variables used in my loop, otherwise I just would have posted the loop by itself...

The only thing I needed help with was my while loop, not all of my code. I realize now that I shouldn't have posted my full code in the first place, it was pointless and unnecessary . Is there no way you can just keep the while loop which I included by itself in post no.8 and just delete my FULL source code that i've posted? No one is going to learn anything from it and it'd make my life much easier.