I know that a similar answer has been asked many times, but my case isn't that simple.
I have a recursive method which can call itself 4 times ("Worst case"). I'm trying hard to avoid recursion since it leads to a StackOverFlowException but I cannot find a way to replace it with a while loop or something similar.
Is this even possible? As far as I've come with my knowledge, you can only move in one direction using a while loop instead of "flowing" in all directions (depth-first-search in reality).
Here is the code:
private static void searchNeighboringPixels(int x, int y, int[][] arr) {
arr[y][x] = 2;
if (x+1 < arr[y].length && arr[y][x+1] == 1) {
searchNeighboringPixels(x+1, y, arr);
//...do other things
}
if (x-1 > 0 && arr[y][x-1] == 1) {
searchNeighboringPixels(x-1, y, arr);
//...do other things
}
if (y+1 < arr.length && arr[y+1][x] == 1) {
searchNeighboringPixels(x, y+1, arr);
//...do other things
}
if (y-1 > 0 && arr[y-1][x] == 1) {
searchNeighboringPixels(x, y-1, arr);
//...do other things
}
}
What I am doing here:
You can always avoid a recursion by using a Stack :
instead of making a recursive call to searchNeighboringPixels(x, y, arr)
, put the point (x,y) in a Stack.
wrap your 4 conditions with a while loop, which runs until the Stack is empty.
each iteration pops the top of the Stack, and treats that point as the current point.
Something like this :
private static void searchNeighboringPixels(int x, int y, int[][] arr) {
Stack<Point> points = new Stack<>();
points.push(new Point(x,y));
while (!points.isEmpty()) {
Point p = points.pop();
x = p.getX();
y = p.getY();
arr[y][x] = 2;
if (x+1 < arr[y].length && arr[y][x+1] == 1) {
points.push(new Point(x+1,y);
//...do other things
}
if (x-1 > 0 && arr[y][x-1] == 1) {
points.push(new Point(x-1,y);
//...do other things
}
if (y+1 < arr.length && arr[y+1][x] == 1) {
points.push(new Point(x,y+1);
//...do other things
}
if (y-1 > 0 && arr[y-1][x] == 1) {
points.push(new Point(x,y-1);
//...do other things
}
}
}