0

私のプログラムは、配列から奇数を取得し、配列内で発生する奇数の回数をカウントすることになっています。

たとえば、countOdds({3, 7, 2, 4}) は、配列に 2 つの奇数があるため、2 を返す必要があります。

public int countOdds (int[] values) {
int oddTotal=0;
for(int i=0;i<values.length;i++){
if(i%2!=0){
oddTotal=oddTotal+1;
}
}
return oddTotal;

私は結果としてこれらを取得しています: (ソース: gyazo.com )私の結果

4

4 に答える 4

7

ivalue の代わりにカウンターをチェックしていますvalues[i]

于 2013-05-01T18:54:00.440 に答える
1
public int countOdds (int[] values) {
int oddTotal=0;
if(values == null)
  return oddTotal;
for(int i=0;i<values.length;i++){
    if(values[i]%2!=0){
    oddTotal=oddTotal+1;
  }
}
return oddTotal;
于 2013-05-01T18:55:49.613 に答える
0
public int countOdds (int[] values) {
   int oddTotal=0;
   for(int i=0;i<values.length;i++) {
   if(values[i] % 2 != 0) {
       oddTotal=oddTotal+1;
   }
   return oddTotal;
}
于 2013-05-01T18:55:36.123 に答える
0

2つのこと:

  • 配列内の何もチェックしていません - でそれを行うことができますvalues[i]

  • 配列が null になるとどうなりますか? を取得しNullPointerExceptionます。あなたはそれを確認したいかもしれません:

    if(values == null) {
       return 0;
    }
    
于 2013-05-01T18:55:40.317 に答える