2

以下に投稿されたエラーで誰かが私を助けることができますか? プログラムは実行されますが、寄付の表示が呼び出されるとメインに関するエラーが発生します。これを実行するにはどうすれば修正できますか?構文エラーと関係がありますか? 配列を扱うのは初めてなので、良い習慣についてのフィードバックを歓迎します。ありがとう!

 Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 6
        at donations.processDonations(donations.java:78)
        at donations.main(donations.java:33)

    import java.util.Scanner; //needed for input

    public class donations {
        static double[] cashDonations = new double[6]; // stores the cash donations
                                                        // from 6 sites
        static double[] lbsFood = new double[6]; // stores the pounds of food
                                                    // donated from 6 sites
        static String[] siteName = new String[6]; // stores the names of the 6 sites
        static String bestSiteCash = " "; // stores the site name with the highest
                                            // cash donation
        static String bestSiteFood = " "; // stores the site name with the highest
                                            // food donation
        static double totalCash = 0; // stores the total cash collected for all
                                        // sites
        static double totalFood = 0; // stores the total food pounds collected for
                                        // all sites
        static double maxFood = 0; // store the highest food collection
        static double maxCash = 0; // stores the highest cash collection

        public static void main(String[] args) {

            Scanner input = new Scanner(System.in); // needed for input
            String anotherCourse = "yes"; // variable to control running program
                                            // again
            do {

                getDonations();
                processDonations();
                displayDonations();

                // This code ends the do while to run again
                System.out.print("Enter yes if you want to run again: ");
                anotherCourse = input.next();
                input.nextLine(); // causes skipping issue to fix
                System.out.print("\n\n\n");
            } while (anotherCourse.equalsIgnoreCase("yes"));

        } // end of main

        public static void getDonations() {
            Scanner input = new Scanner(System.in); // needed for input
            // Prompt user for site info
            for (int i = 0; i < 6; i++) {
                System.out.println("Enter site " + (i + 1) + " name: ");
                siteName[i] = input.next();

                System.out.println("Enter cash donation(USD) for " + siteName[i]
                        + ": ");
                cashDonations[i] = input.nextDouble();
                // double [] cashDonations = new double [6]; You have this already
                // at the top
                // int i;
                // for (i = 0 ; i < 6 ; i++)

                // cashDonations[i] = input.nextDouble();

                // double [] lbsFood = new double [6];
                System.out.println("Enter food donation(lbs.) for " + siteName[i]
                        + ": ");
                lbsFood[i] = input.nextDouble();

            }

        }

        public static void processDonations() {
            totalCash = 0;
            totalFood = 0;
            maxCash = cashDonations[0];
            maxFood = lbsFood[0];

            totalCash = cashDonations[1] + cashDonations[2] + cashDonations[3]
                    + cashDonations[4] + cashDonations[5] + cashDonations[6];
            totalFood = lbsFood[1] + lbsFood[1] + lbsFood[2] + lbsFood[3]
                    + lbsFood[4] + lbsFood[5] + lbsFood[6];

        }

        public static void displayDonations() {
            System.out.print("Total Cash Donations are " + totalCash);
            System.out.print("Total Food Donations are " + totalFood);
            System.out.print("Donation totals for " + siteName[1]);
            System.out.print("Total cash: " + cashDonations[1]);
            System.out.print("Food donations " +lbsFood[1]);
            System.out.println();
            System.out.print("Donation totals for " + siteName[2]);
            System.out.print("Total cash: " + cashDonations[2]);
            System.out.print("Food donations " +lbsFood[2]);
            System.out.println();
            System.out.print("Donation totals for " + siteName[3]);
            System.out.print("Total cash: " + cashDonations[3]);
            System.out.print("Food donations " +lbsFood[3]);
            System.out.println();
            System.out.print("Donation totals for " + siteName[4]);
            System.out.print("Total cash: " + cashDonations[4]);
            System.out.print("Food donations " +lbsFood[4]);
            System.out.println();
            System.out.print("Donation totals for " + siteName[5]);
            System.out.print("Total cash: " + cashDonations[5]);
            System.out.print("Food donations " +lbsFood[5]);
            System.out.println();
            System.out.print("Donation totals for " + siteName[6]);
            System.out.print("Total cash: " + cashDonations[6]);
            System.out.print("Food donations " +lbsFood[6]);
            System.out.println();

        }// end of displayDonations()

    }// end of class
4

3 に答える 3

0

よくある「1 つずらした」エラーをしていることに気付きました ¯\_(ツ)_/¯

ここでは、サイズが 6 の 3 つの配列を適切に宣言しています。

    static double[] cashDonations = new double[6]; 

    static double[] lbsFood = new double[6]; 

    static String[] siteName = new String[6];

実際、各配列には 6 つの要素がありますが、最初の要素は 0 と見なされます。

[0]、[1]、[2]、[3]、[4]、[5]

あなたのコードでは、存在しない 7 番目の要素 "[6]" を呼び出します。

   totalCash = cashDonations[1] + cashDonations[2] + cashDonations[3]
                + cashDonations[4] + cashDonations[5] + cashDonations[6];

   totalFood = lbsFood[1] + lbsFood[1] + lbsFood[2] + lbsFood[3]
                + lbsFood[4] + lbsFood[5] + lbsFood[6];

それを修正するには、要素を 0 から 5 にするだけです。

   totalCash = cashDonations[0] + cashDonations[1] + cashDonations[2]
                + cashDonations[3] + cashDonations[4] + cashDonations[5];

   totalFood = lbsFood[0] + lbsFood[1] + lbsFood[2]
                + lbsFood[3] + lbsFood[4] + lbsFood[5];

これは簡単な間違いです。なぜなら、私たちは一生、0 の値はゼロであると教えられてきたからです。

プログラミングを続けて、あきらめないでください!

于 2013-08-06T00:28:58.533 に答える