-4

我今天早些时候曾问过有关该程序的问题，并且能够完成我需要完成的大部分工作，但似乎人们不再关注它了， 这是它的链接。

这是我现在拥有的：

import java.util.*;
import java.text.*;

public class Lab4 {
public static void main(String[] args){
    Scanner s= new Scanner(System.in);
    String input;
    int students;
    int correctAnswers=0;

    char [] answerKey= { 'B' , 'D' , 'A' , 'A' , 'C' , 'A' , 'B' , 'A' , 'C' , 'D' , 'B' , 'A' };
    char [] userAnswers = new char[answerKey.length];

    DecimalFormat df = new DecimalFormat("#0.0");

    System.out.print("how many students are in your class?");
    input = s.nextLine();
    students=Integer.parseInt(input);

    String [] name = new String[students];

    int j=1;
    while(students>=j)
    {
        System.out.print("Enter name of student" + j + ": ");
        name[j] = s.nextLine();

        System.out.print("Enter quiz score answers");
        userAnswers[answerKey.length] = s.next().charAt(0);

        for (int i = 0; i < userAnswers.length; ++i)
        {
            if(userAnswers[i]==answerKey[i]);
            correctAnswers++;
        }

        System.out.print((df.format(correctAnswers/answerKey.length)) + "%");
    j++;

    }

}

    }

但是在输入用户的答案后，我不断收到此错误：

线程“main”中的异常 java.lang.ArrayIndexOutOfBoundsException: 12 在 Lab4.main(Lab4.java:29)

我不确定这意味着什么或如何解决它。

4

2 回答 2

3

这意味着您的数组索引可能超过数组中的元素数。从您的代码中，您似乎展示了一个Off-by-one error。请注意，Java 数组是从零开始的，即数组索引以 0 开始并以array.length - 1.

（注意：未经测试的代码，我已经Scanner好几个月没有使用了......）

改变

int j=1;
while(students>=j)

到

int j = 0;
while (students > j)

而且，这条线

userAnswers[answerKey.length] = s.next().charAt(0);

这是一个逻辑错误。不仅是根据@Creakazoid 的答案写入越界，即使它是固定的，您也会将所有答案写入数组的最后一个元素，这意味着您将所有学生的答案作为用户的最后一个字符输入。

这应该是

for (int i = 0; i < answerKey.length; ++i) {
    userAnswers[i] = s.next().charAt(0);
}

编辑：看起来您需要阅读一行充满答案的输入。因此，阅读整行，然后将行分成字符。（未经测试）

String line = s.nextLine();
for (int i = 0; i < answerKey.length; ++i) {
    userAnswers[i] = line.charAt(i);
}

此外，

if(userAnswers[i]==answerKey[i]);

注意到行尾的分号了吗？您正在编写一个空语句（由分号组成）并且correctAnswers++;无论此条件是否为真都会运行

将其更改为

if (userAnswers[i] == answerKey[i])

你可能需要改变

System.out.print("Enter name of student" + j + ": ");

到

System.out.print("Enter name of student" + (j + 1) + ": ");

这样输出不受影响，虽然

实际上，您的 while 循环可以替换为 for 循环 - 它更易于阅读：

for (int j = 0; j < students; ++j) {
    // .. your code
}
于 2013-02-27T00:26:29.323 回答
0 
userAnswers[answerKey.length] = s.next().charAt(0);

数组索引是从零开始的，所以最后一个可寻址索引是answerKey.length - 1.

于 2013-02-27T00:30:23.533 回答