2つのfor文を1つのfor文に置き換えたい。もっとカッコいいコードを書きたい

Question

前提・実現したいこと

関数next内の前半のfor文と後半の入れ子になっているfor文を一つの入れ子になっているfor文にまとめたいです。
なんかかっこ悪いのでスマートな書き方があったら教えてください。
hyoはvかけるvの二次元配列
now_gyo,now_retuはグローバル変数
気持ちとしては今の座標をもとに次？の座標(-1,0でないもの)をもとめる感じです
例えばv=5の以下の表でnow_gyo=1 , now_retu=2なら実行後はnow_gyo=2 , now_retu=4になってほしいです。
0 -1 -1 5 -1
-1 0 -1 4 5
-1 -1 0 2 -1
-1 -1 -1 0 -1
-1 -1 -1 -1 0

該当のソースコード

		
　　　　import java.util.Arrays;

public class hogehoge {
	public static int now_gyo=1;
	public static int now_retu=2;
	public static int v=5;
	public static void main(String[] args) {
		next();

	}
	public static void next() {
		
		int[][] hyo=new int[][]{
				{0 ,-1, -1 , 5 ,-1},
				{-1 , 0, -1 , 4 , 5},
				{-1 ,-1,  0 , 2 ,-1},
				{-1 ,-1, -1 , 0 ,-1},
				{-1 ,-1, -1 ,-1 , 0}
		};
		for (int i = now_retu+1; i < v; i++) {
            if(hyo[now_gyo][i]==-1){
                now_retu=i;
                return;
            }
        }
        for (int i = now_gyo+1; i < v; i++) {
            for (int j = i+1; j < v; j++) {
                if(hyo[i][j]==-1){
                    now_gyo=i;
                    now_retu=j;
                    return;
                }
            }
        }
	}
}

Comments

[kazuma-s]

質問のコードには、now_gyo も now_retu もありません。
return していますが、x が結果ですか？
スマートに書く前に、2つの for文を使った、動くコードがあるはずです。
そのコード全体を見せてください。質問は編集できます。

[jimbe]

2次元がダメなら1次元の重なりと見れば良いのではないでしょうか。

[koklopkiujio]

返信ありがとうございます
コード全体に書き換えました。
以前の部分的なコードはまちがっていてのｘをnow_gyo、ｙをnow_retuに置換したものが正しかったです。
よくみるとfor文2つという日本語が正しいかは微妙ですね・・・あとで質問文を変えます。
やりたいことは前半のfor文と後半の入れ子になっているfor文を一つの入れ子になっているfor文にまとめたいので、多分、1次元の重なりと見る必要はないと思います。入れ子部分を入れ子ではない文にしたいわけではないので・・・
入れ子部分を入れ子ではない文にしたい場合、1次元の重なりとみる案はありだと思います。数学的にはできそうです。
ただ、hyoの左下の-1でできた三角形の部分を無視したいので for文内のi, jをどう変えていけばいいかよくわかんないです。具体的なやり方があればそれについても教えてください

[jimbe]

> now_gyo=1 , now_retu=2なら実行後はnow_gyo=2 , now_retu=4
となっていますが、例えば hyo が
{ 0, 1, 1, 5, 1 },
{ -1, 0, -1, 4, 5 },
{ -1, -1, 0, 2, 1 },
{ -1, -1, -1, 0, -1 },
{ -1, -1, -1, -1, 0 }
として now_gyo=0 , now_retu=3なら、now_gyo=1 , now_retu=2となるのは正解でしょうか。

Answer 1

質問のコードでは、next が呼び出されてから hyo を新たに作成し、
main に戻ると hyo は無くなります。
また、最後の -1 のあとは now_gyo と1 now_retu は変化しなくていいんですね。

hyo を static にしてみました。

class Main {
	public static int now_gyo = 0;
	public static int now_retu = 1;
	public static int v = 5;

	public static int[][] hyo = new int[][] {
		{ 0 ,-1, -1 , 5 ,-1},
		{-1 , 0, -1 , 4 , 5},
		{-1 ,-1,  0 , 2 ,-1},
		{-1 ,-1, -1 , 0 ,-1},
		{-1 ,-1, -1 ,-1 , 0}
	};

	public static void main(String[] args) {
		for (int i = 0; i < 8; i++) {
			System.out.print(" (" + now_gyo + ", " + now_retu + ")");
			next();
		}
		System.out.println(" (" + now_gyo + ", " + now_retu + ")");
	}

    public static void next() {
        for (int i = now_retu+1; i < v; i++) {
            if (hyo[now_gyo][i] == -1) {
                now_retu = i;
                return;
            }
        }
        for (int i = now_gyo+1; i < v; i++) {
            for (int j = i+1; j < v; j++) {
                if (hyo[i][j] == -1) {
                    now_gyo = i;
                    now_retu = j;
					return;
                }
            }
        }
	}
}

これを次のように変えればよいのではありませんか？

class Main {
	public static int now_gyo = 0;
	public static int now_retu = 1;
	public static int v = 5;

	public static int[][] hyo = new int[][] {
		{ 0 ,-1, -1 , 5 ,-1},
		{-1 , 0, -1 , 4 , 5},
		{-1 ,-1,  0 , 2 ,-1},
		{-1 ,-1, -1 , 0 ,-1},
		{-1 ,-1, -1 ,-1 , 0}
	};

	public static void main(String[] args) {
		for (int i = 0; i < 8; i++) {
			System.out.print(" (" + now_gyo + ", " + now_retu + ")");
			next();
		}
		System.out.println(" (" + now_gyo + ", " + now_retu + ")");
	}

    public static void next() {
		for (int i = now_gyo, j = now_retu; ; )
			if (++j == v) {
				if (++i == v) return;
				j = i;
			}
			else if (hyo[i][j] == -1) {
				now_gyo = i;
				now_retu = j;
				return;
			}
	}
}

追記
for文を無限ループにしないようにしてみました。

    public static void next() {
        for (int i = now_gyo, j = now_retu; i < v; j = ++i)
			while (++j < v)
            	if (hyo[i][j] == -1) {
					now_gyo = i;
					now_retu = j;
					return;
				}
    }
}

Answer 2

2重ループを now_gyo から始め、最初の行の検索の開始列を now_retu+1 にするだけでしたら、外側(iのループ)の初期値を int i = now_gyo; 、内側(jのループ)の初期値を int j = (i==now_gyo?now_retu:i) + 1; とするのは如何でしょうか。