配列糸引き数

Question

package javaapplication1; 

/** 
* 
* @author 
*/ 
public class JavaApplication1 { 

    /** 
    * @param args the command line arguments 
    */ 
    public static void main(String[] args) { 
     int[] scores = {1,2,3,4,5,6}; 
     int[] fin = extractAllEvens(scores); 
     for(int i =0; i<fin.length; i++) { 
      System.out.println(fin[i]); 
    } 
    } 
     public static int[] extractAllEvens(int[]scores) { 
      int evenCount = 0; 
      for (int i =0; i<scores.length; i++) { 
       if (scores[i] % 2 ==0) { 
        evenCount++; 
       } 
     } 
     int[] newScores = new int[evenCount]; 
     int j = 0; 
     for(int i = 0; i<scores.length; i++) { 

      if(scores[1] % 2 ==0) { 
      newScores[1] = scores[i]; 
      j++; 
      } 
     } 
    return newScores; 
} 
} 

私は出力2、4にしようとしている、6

しかし、私は、このような0,6,0として結果を得続けます。

私はどこかで変数iとjまたは数字1とおそらくその場所を混乱させたと思います...誰かが正しい方向に私を導くのを助けることができますか？

Answer 1

最後のループでは固定ループインデックスとして1が使用されます。

代わりにiに変更してください。

Answer 2

scores[1] % 2 == 0およびnewScores[1] = scores[i]？

インデックス1（第2要素）のみをnewArrayとしてハードコードしました。

1ではなく、jをインデックスとして使用しているはずです。

Answer 3

変更するには、このようなループのための第二 -

for (int i = 0; i < scores.length; i++) { 
     if (scores[i] % 2 == 0) { 
      newScores[j] = scores[i]; 
      j++; 
     } 
    } 

乾杯！それは

if(scores[i] % 2 ==0) {//← i here 
     newScores[j] = scores[i];//← j here 

O/Pでなければなりません

Answer 4

代わりの

if(scores[1] % 2 ==0) { 
     newScores[1] = scores[i]; 

：2 4 6