0

基本的に、ユーザーが 0 を入力するまでユーザー入力を取得してから、配列内の最大値、負の数の数、および正の数の合計を見つける必要があります。問題は、配列を使用する必要があり、ArrayList を使用して他のメソッドを呼び出すことができないため、ArrayList を作成し、その要素を使用して配列を作成すると考えたことです。私はこれをエラーとして受け取り続け、考えられるすべてを試しました。助けてください。

Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 3, Size: 3
    at java.util.ArrayList.RangeCheck(ArrayList.java:547)
    at java.util.ArrayList.get(ArrayList.java:322)
    at Assignment9.assignment9(Assignment9.java:37)
    at Assignment9.assignment9(Assignment9.java:49)
    at Assignment9.assignment9(Assignment9.java:49)
    at Assignment9.assignment9(Assignment9.java:49)
    at Assignment9.main(Assignment9.java:17)

//Class Description: Takes user input and makes it into an array until 0 is entered.
// It then computes the maximum #, amount of negative #s and the sum of the positive #s.
import java.util.*;
import java.text.*;;

public class Assignment9 {

//main method initializes variables then calls method assignment9
public static void main(String[] args)
{
    int count = 0;
    ArrayList<Double> help = new ArrayList<Double>();
    assignment9(help, count);
}//end main

//adds user input to array until 0 is entered; it then calls other methods
public static void assignment9(ArrayList<Double> help, int count)
{
    DecimalFormat fmt = new DecimalFormat("#.##");
    double input;
    double max = 0;
    int negative = 0;
    double sum = 0;
    Scanner scan = new Scanner(System.in);

    input = scan.nextInt();

    if (input == 0)
    {
        double[] numbers = new double[help.size()];
        for(int i = 0; i < numbers.length; i++)
        {
            numbers[i] = help.get(i);
        }
        findMax(numbers, count, max);
        countNegative(numbers, count, negative);
        computeSumPositive(numbers, count, sum);
        System.out.println("The maximum numer is " + fmt.format(max));
        System.out.println("The total number of negative numbers is " + negative);
        System.out.println("The sum of positive numbers is " + fmt.format(sum));
        System.exit(0);
    }else{
            help.add(input);
            count++;
            assignment9(help, count);
    }//end if
}//end assignment9

//compares elements of array to find the max until count = -1
public static double findMax(double[] numbers, int count, double max)
{

    if (count == -1)
    {
        return max;
    }else if(numbers[count] > max){
        max = numbers[count];
        count--;
        findMax(numbers, count, max);
    }//end if
    return max;

}//end findMax

public static int countNegative(double[] numbers, int count, int negative)
{

    if(count == -1)
    {
        return negative;
    }else if(numbers[count] < 0){
        negative++;
        count--;
        countNegative(numbers, count, negative);
    }
    return negative;
}//end countNegative

public static double computeSumPositive(double[] numbers, int count, double sum)
{
     if(count == -1)
     {
         return sum;
     }else{
         sum = sum + numbers[count];
         count++;
         computeSumPositive(numbers, count, sum);
         return sum;
     }
}//end computeSumPositive

}//クラス終了

4

1 に答える 1

3

問題は、数値を渡して、countこの行の上限にアクセスしようとしているときにfindMaxスローしていることです。ArrayIndexOutOfBoundsException

} else if (numbers[count] > max) {

Java の配列はゼロベースであることを思い出してください。また、Java は値渡しを使用するため、結果を の出力に割り当てる必要がありますfindMax。

max = findMax(numbers, count - 1, 0);

論理的にfindMaxは、常に最大数ではなく最後の数を返します。max再帰メソッド内から一時的なものを返す必要があります。

double findMax(double numbers[], int count, int index) {
   // This needs to be size - 1 because the array is 0-indexed.
   // This is our base case
   if (index == count - 1)
      return numbers[index];

   // Call the function recursively on less of the array
   double result = findMax(numbers, count, index + 1);

   // Return the max of (the first element we are examining, the max of the
   // rest of the array)
   if (numbers[index] > result)
      return numbers[index];
   else
      return result;
}

このバージョンは、countインデックスの上限を内部的に処理します。

max = findMax(numbers, count, 0);
于 2013-03-29T21:23:22.710 に答える