Javaメソッドで無用な戻りを回避するにはどうすればよいですか？

理論的には、return2つのforループにネストされたステートメントに常に到達する状況にあります。

コンパイラーはこれに同意せずreturn、forループ外のステートメントを要求します。私の現在の理解を超えているこの方法を最適化するエレガントな方法を知りたいのですが、私が試みたbreakの実装はどれもうまくいかないようです。

添付されているのは、ランダムな整数を生成し、intパラメータとしてメソッドに渡された範囲内で生成された2番目のランダムな整数が見つかるまで繰り返される反復を返す割り当てからのメソッドです。

private static int oneRun(int range) {
    int[] rInt = new int[range+1]; // Stores the past sequence of ints.
    rInt[0] = generator.nextInt(range); // Inital random number.

    for (int count = 1; count <= range; count++) { // Run until return.
        rInt[count] = generator.nextInt(range); // Add randint to current iteration.
        for (int i = 0; i < count; i++) { // Check for past occurence and return if found.
            if (rInt[i] == rInt[count]) {
                return count;
            }
        }
    }
    return 0; // Never reached
}

コンパイラーのヒューリスティックでは、最後のを省略できませんreturn。到達しないと確信している場合はthrow、状況を明確にするためにa に置き換えます。

private static int oneRun(int range) {
    int[] rInt = new int[range+1]; // Stores the past sequence of ints.
    rInt[0] = generator.nextInt(range); // Inital random number.

    for (int count = 1; count <= range; count++) {
        ...
    }

    throw new AssertionError("unreachable code reached");
}

以下のよう@BoristheSpiderが指摘よろしい二作ることができreturn文が意味的に到達不能です：

private static int oneRun(int range) {
    int[] rInt = new int[range+1]; // Stores the past sequence of ints.
    int count = 0;

    while (true) {
        rInt[count] = generator.nextInt(range); // Add randint to current iteration.
        for (int i = 0; i < count; i++) { // Check for past occurence and return if found.
            if (rInt[i] == rInt[count]) {
                return count;
            }
        }
        count++;
    }
}

コンパイルして正常に実行します。そして、もしあなたがこれを手に入れたら、ArrayIndexOutOfBoundsException明示的に何かを投げる必要なしに、実装が意味的に間違っていることを知るでしょう。

2つのforループから抜け出すことについて質問したので、ラベルを使用してそれを行うことができます（以下の例を参照）。

private static int oneRun(int range) {
    int returnValue=-1;

    int[] rInt = new int[range+1]; // Stores the past sequence of ints.
    rInt[0] = generator.nextInt(range); // Inital random number.

    OUTER: for (int count = 1; count <= range; count++) { // Run until return.
        rInt[count] = generator.nextInt(range); // Add randint to current iteration.   
        for (int i = 0; i < count; i++) { // Check for past occurence and return if found.
            if (rInt[i] == rInt[count]) {
                returnValue = count;
                break OUTER;
            }
        }
    }
    return returnValue;
}

一方、アサートは優れた高速ソリューションです。一般に、この種の問題は、コードが複雑すぎることを意味します。私があなたのコードを見ているとき、配列に以前の数値を保持させたくないのは明らかです。あなたはSet：

Set<Integer> previous = new HashSet<Integer>();

int randomInt = generator.nextInt(range);
previous.add(randomInt);

for (int count = 1; count <= range; count++) {
    randomInt = generator.nextInt(range);
    if (previous.contains(randomInt)) {
       break;
    }

    previous.add(randomInt);
}

return previous.size();

ここで、返されるのは実際にはセットのサイズです。コードの複雑さが2次から線形に減少し、すぐに読みやすくなりました。

これで、そのcountインデックスも必要ないことに気付くことができます。

Set<Integer> previous = new HashSet<Integer>();

int randomInt = generator.nextInt(range);

while (!previous.contains(randomInt)) {          
    previous.add(randomInt);      
    randomInt = generator.nextInt(range);
}

return previous.size();

戻り値は外部ループの変数に基づいているので、外部ループの条件を単にに変更してcount < range、関数の最後でこの最後の値（先ほど省略したもの）を返すことができます。

private static int oneRun(int range) {
    ...

    for (int count = 1; count < range; count++) {
        ...
    }
    return range;
}

この方法では、決して到達しないコードを導入する必要はありません。

"result"などの一時変数を使用して、内部の戻り値を削除します。forループをwhileループに適切な条件で変更します。私にとっては、関数の最後のステートメントとしてリターンを1つだけ持つほうが常にエレガントです。

多分これはあなたがあなたのコードを書き直すべきであることを示しています。例えば：

1. 整数0 ..範囲1の配列を作成します。すべての値を0に設定します。
2. ループを実行します。ループで、乱数を生成します。リストをそのインデックスで見て、値が1かどうかを確認します。1の場合は、ループを抜けます。それ以外の場合は、そのインデックスの値を1に設定します
3. リスト内の1の数を数え、その値を返します。

returnステートメントがあり、ループ内にループがあるメソッドには、常にループの外側にreturnステートメントが必要です。ループ外のこのステートメントに到達することはありません。このような場合、不必要なreturnステートメントを回避するために、メソッドの最初、つまりそれぞれのループの前と外側で、それぞれの型の変数（この場合は整数）を定義できます。ループ内の目的の結果に到達したら、それぞれの値をこの事前定義された変数に帰することができ、それをループ外のreturnステートメントに使用できます。

rInt [i]がrInt [count]と等しい場合にメソッドが最初の結果を返すようにするため、rInt [i]がrInt [count]と等しい場合にメソッドが最後の結果を返すため、上記の変数のみを実装するだけでは不十分です。1つのオプションは、目的の結果が得られたときに呼び出される2つの「breakステートメント」を実装することです。したがって、メソッドは次のようになります。

private static int oneRun(int range) {

        int finalResult = 0; // the above-mentioned variable
        int[] rInt = new int[range + 1];
        rInt[0] = generator.nextInt(range);

        for (int count = 1; count <= range; count++) {
            rInt[count] = generator.nextInt(range);
            for (int i = 0; i < count; i++) {
                if (rInt[i] == rInt[count]) {
                    finalResult = count;
                    break; // this breaks the inside loop
                }
            }
            if (finalResult == count) {
                break; // this breaks the outside loop
            }
        }
        return finalResult;
    }

到達できないステートメントが発生した場合、例外をスローする必要があることに同意します。同じメソッドがこれをより読みやすい方法で実行できる方法を示したかっただけです（Java 8ストリームが必要です）。

private static int oneRun(int range) {
    int[] rInt = new int[range + 1];
    return IntStream
        .rangeClosed(0, range)
        .peek(i -> rInt[i] = generator.nextInt(range))
        .filter(i -> IntStream.range(0, i).anyMatch(j -> rInt[i] == rInt[j]))
        .findFirst()
        .orElseThrow(() -> new RuntimeException("Shouldn't be reached!"));
}

private static int oneRun(int range) {
    int result = -1; // use this to store your result
    int[] rInt = new int[range+1]; // Stores the past sequence of ints.
    rInt[0] = generator.nextInt(range); // Inital random number.

    for (int count = 1; count <= range && result == -1; count++) { // Run until result found.
        rInt[count] = generator.nextInt(range); // Add randint to current iteration.   
        for (int i = 0; i < count && result == -1; i++) { // Check for past occurence and leave after result found.
            if (rInt[i] == rInt[count]) {
                result = count;
            }
        }
    }
    return result; // return your result
}