配列要素の減算と比較に関するロジック

Question

私は次の質問のコードを書きましたが、出力が期待どおりではありません。自分のコードに何か問題があるかどうかはわかりません。ロジックはうまくいくようです。私のコードに何か問題があるかどうか誰にでも見えますか？

昇順にソートされたスコアの配列が与えられている場合、配列に{3,4,5}や{3,5,5など}のように互いに2つずつ異なる3つの隣接スコアが含まれている場合はtrueを返します。 }。次のように

私のソースコードは次のとおりです。

public boolean scoresClump(int[] scores) { 

    boolean result = false; 

    for(int i=0; i<scores.length-2; i++){ 
     if((scores[i+1]-scores[i])<=2 && (scores[i+2]-scores[i+1])<=2){ 
      result = true; 
      break; 
     } 
    } 

    return result; 
} 

Thisは、質問へのリンクです。

Answer 1

ここに行きます。以前の投稿では、隣接していない値間の差分を考慮していませんでした。

public boolean scoresClump(int[] scores) { 
    boolean result = false; 

    for(int i=0; i<scores.length-2; i++){ 
     if((scores[i+2]-scores[i])<=2){ 
      result = true; 
      break; 
     } 
    } 

    return result; 
} 

Answer 2

お客様のforループはfor(int i=0; i<scores.length-2; i++){でなければなりません。そうでない場合はArrayOutOfBoundsExceptionとなります。

また、お客様のbreakはであり、falseではありません。現在、条件に一致しない最初の3つの要素の後にループが存在します。彼らはすでにMath.abs

Answer 3

ファーストを行う必要はありません昇順にソートされている場合は、エラーや予期しない動作を得た場合

また、あなたはそれが（例えば、予想と実際の出力）であるかを説明すべきです。

あなたのコードで少なくとも1つのエラーが発生する可能性があります。ループスコア[i + 2]でアクセスできます。私が長さ1になると、あなたは長さ+1にアクセスします。最後の要素はインデックスの長さ-1を持つため、length + 0にアクセスすることもエラーです。あなたは完全なコードを示していませんでしたが、スローされた例外のために結果が呼び出し関数にfalseとして解釈されると仮定します。

さらに、2つのチェックは必要ありません。次の繰り返しで次のチェックを実行するので、1つのチェックで十分です。

Answer 4

はこれを試してみてください。

public boolean scoresClump(int[] scores) { 
boolean result=false; 
for(int i=0; i<scores.length-2; i++){ 
     if(Math.abs(scores[i]-scores[i+1])<=2 && 
     Math.abs(scores[i+1]-scores[i+2])<=2 && 
     Math.abs(scores[i]-scores[i+2])<=2){ 
      result=true; 
     } 
} 
return result; 
}