Infinite Loop in my code.

[Vincent Mule]

If you saw my earlier post, we have to navigate a 10x10 array using 1's as a path. When a 1 is selected, we have to change the current position to -1 so that way we don't keep retracing our steps. When I run the program, it keeps saying STEPPED TO [2,3] over and over again. Can someone please identify where my infinite loop is? I can't seem to find it.

/*
	Program9.java

*/

import java.io.*;
import java.util.*;


public class Program9
{
    public static void main(String[] args)
    {
		int myRow=2, myCol=3;  // you are dropped into the swamp at: [2][3]

		int[][] swamp1 =
		{
			{ 0,0,0,0,0,0,0,0,0,0 },
			{ 0,0,0,0,0,0,0,0,0,1 },
			{ 0,0,0,1,0,0,0,0,1,0 },
			{ 0,0,0,1,0,0,0,1,0,0 },
			{ 0,0,0,0,1,0,1,0,0,0 },
			{ 0,0,0,0,1,0,0,1,0,0 },
			{ 0,0,0,0,1,0,0,0,1,0 },
			{ 0,0,0,0,0,1,0,1,0,0 },
			{ 0,0,0,0,0,0,1,0,0,0 },
			{ 0,0,0,0,0,0,0,0,0,0 }
		};

		printSwamp( "SWAMP1",swamp1 );
		printEscapePath( swamp1, myRow,myCol );


		// Now swamp2

		myRow=2; myCol=8;  // you are dropped into the swamp at [2][8]

		int[][] swamp2 =
		{
			{ 0,0,0,0,0,0,0,0,0,0 },
		 	{ 0,0,0,0,0,0,0,0,0,0 },
	 		{ 0,0,1,0,0,0,0,1,1,0 },
		 	{ 0,0,0,1,0,0,1,0,0,0 },
	 		{ 0,0,0,0,1,1,0,0,0,0 },
		 	{ 0,0,0,0,0,0,0,0,0,0 },
	 		{ 0,0,0,0,0,0,0,0,0,0 },
		 	{ 0,0,0,0,0,0,0,0,0,0 },
	 		{ 0,0,0,0,0,0,0,0,0,0 },
		 	{ 0,0,0,0,0,0,0,0,0,0 }
	 	};

		printSwamp( "\nSWAMP2",swamp2 );
		printEscapePath( swamp2, myRow,myCol );

	} // END MAIN

	// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

	// 	DO NOT MODIFY THIS METHOD - USE AS GIVEN

    private static void printSwamp(String label, int[][] swamp )
    {

        System.out.println( label );
        for(int r = 0; r < swamp.length; r++)
		{
            for(int c = 0; c < swamp[r].length; c++)
                 System.out.print( swamp[r][c] + " ");
            System.out.println();
        }
    }


    private static void printEscapePath(int[][] swamp, int myRow, int myCol)
    {

		int[] coords = new int[2];
		coords[0]=myRow;
		coords[1]=myCol;

		System.out.println("STARTING AT: " + "["+ myRow + "][" + myCol + "]");

		while ( !onEdge( swamp, coords ) )
		{
			
			if ( nextStep( swamp, coords ) )
			{
				System.out.println( "STEPPED  TO: [" + coords[0] + "][" + coords[1] + "]" );
			}
			else 
			{
				System.out.println( "STRANDED AT: [" + coords[0] + "][" + coords[1] + "]" );
				return;

			}
			
		}

		System.out.println( "ESCAPED  AT: [" + coords[0] + "][" + coords[1] + "]" );

	} 


	
	private static boolean nextStep( int[][] swamp, int[] coords )
	{
	int myRow = coords[0];
	int myCol = coords[1];
	int temp = 0;
	int x = 0;
	int y = 0;
	y = myRow;
	x = myCol;
		if (swamp[x+1][y] == 1)//Moving North
		{
			temp = swamp[x-1][y];
			swamp[x][y] = -1;
			swamp[x][y] = swamp[x+1][y];
			return true;
		}
		if (swamp[x+1][y] == 1)//Moving South
		{
			temp = swamp[x][y];
			swamp[x][y] = -1;
			swamp[x][y] = swamp[x+1][y];
			return true;
		}
		if (swamp[x][y-1] == 1)//Moving East
		{
			temp = swamp[x][y-1];
			swamp[x][y] = -1;
			swamp[x][y] = swamp[x][y-1];
			return true;
		}
		if (swamp[x][y+1] == 1)//Moving West
		{
			
			swamp[x][y] = -1;
			swamp[x][y] = swamp[x][y+1];
			return true;
		}
		if( swamp[x-1][y-1] == 1)//Moving NorthWest [-1][-1]
		{
			temp = swamp[x][y];
			swamp[x][y] = -1;
			swamp[x][y] = swamp[x-1][y-1];
			return true;
		}
		if(swamp[x-1][y+1] == 1)//Moving NorthEast [-1][+1}
		{
			temp = swamp[x][y];
			swamp[x][y] = -1;
			swamp[x][y] = swamp[x-1][y+1];
			return true;
		}
		if(swamp[x+1][y-1] == 1)//Moving SouthWest [+1][-1]
		{
			temp = swamp[x][y];
			swamp[x][y] = -1;
			swamp[x][y] = swamp[x+1][y-1];
			return true;
		}
		if(swamp[x+1][x+1] == 1)//Moving SouthEast [+1][+1]
		{
			temp = swamp[x][y];
			swamp[x][y] = -1;
			swamp[x][y] = swamp[x+1][y+1];
			return true;
		}
			

		else 
		{
			return false;  // just to make it compile you edit as needed
		}
	}


	

	private static boolean onEdge( int[][] swamp, int[] coords )
	{
		if (coords[0] == 0)
		{
			return true;
		}
		if (coords[0] == 10)
		{
			return true;
		}
		if (coords[1] == 0)
		{
			return true;
		}
		if (coords[1] == 10)
		{
			return true;
		}
		
		else 
		{
			return false;  
		}
	}
}

[Ivan Franko]

After each step, you don't change coordinates, why?

I think it is a root of the issue.

[Vincent Mule]

So I changed it to read

int myRow = coords[0];
	int myCol = coords[1];
	int temp = 0;
	int x = 0;
	int y = 0;
	x = myRow;
	y = myCol;
		if (swamp[x+1][y] == 1)//Moving North
		{
			swamp[x][y] = -1;
			swamp[x][y] = swamp[x+1][y];
			coords[0] = x+1;
			coords[1] = y;
			return true;
		}

The changes in X and Y are reflected through the rest of the code, but I am still getting an infinite loop when attempting to move diagonally. It keeps jumping back and forth between spaces.