java - 如何解决这个 "java.lang.IndexOutOfBoundsException"错误？

Question

目前我正在为一个图书馆应用程序编写代码，该应用程序存储手册并允许用户在一段时间内借阅所述手册。

我几乎完成了应用程序，但是我遇到了一个无法解决的令人困惑的问题。

当用户选择从图书馆借阅手册时，系统会要求他们输入一个索引号，该索引号说明要借阅图书馆中的哪本书，每本手册都可以成功借阅，如果输入的索引号不正确，则会显示该索引号带有“错误”消息。

但是，如果用户输入的索引号等于库中存在的手册数量，则应用程序将中断并显示以下错误:

一旦库中存储了 2 本手册，然后用户输入“2”作为索引号，就会发生此错误。

这是我当前正在使用的代码，如果有人可以告诉我导致此错误的原因:

public static void borrowManual(){
    displayManualList();

    //register user's Manual choice.
    ManualChoice = (Console.readInteger(Messages.enterManualIndexMessage, Messages.ManualIndexNotInListMessage, 0, Library.ManualList.size()));

    borrowLoop:
    while(Menu.menuChoice == 3){
        //Check if the Manual to be borrowed is available.
        //ManualChoice = (Console.readInteger(Messages.enterManualIndexMessage, Messages.ManualIndexNotInListMessage, 1, Library.ManualList.size()));

        if ((ManualList.get(ManualChoice).status.equalsIgnoreCase(status1)) && (ManualList.size() >= ManualChoice)){
            //Print the borrowed Manual information and change the Manual status to borrowed.
            ManualList.get(ManualChoice).status = "Borrowed";
            ManualList.get(ManualChoice).borrower = User.userName;
            ManualList.get(ManualChoice).borrowDate = "Today.";
            ManualList.get(ManualChoice).returnDate = "In two weeks.";
            //Add the borrowed Manual to the borrowedManuals arraylist:
            borrowedManuals.add(ManualList.get(ManualChoice));

            System.out.printf("\n==========================================================================\n");
            System.out.printf("\n\nYou have chosen the following Manual:\n\n %s\n\n", ManualList.get(ManualChoice).displayManual());
            System.out.println("Please return the Manual within two weeks!\n");
            System.out.println("\n--------------------------------------------------------------------------");
            System.out.println("\n                             Manual borrowed!\n");
            System.out.println("--------------------------------------------------------------------------\n");
            break borrowLoop;

        }else if(ManualList.get(ManualChoice).status.equalsIgnoreCase(status2) && ManualList.size() >= ManualChoice){
            System.out.println("\n\n--------------------------------------------------------------------------");
            System.out.println("\nError! The manual you wish to borrow is already on loan.");
            System.out.println("\n--------------------------------------------------------------------------\n");
            break borrowLoop;

        }else if(ManualChoice > ManualList.size()-1){
            System.out.println(Messages.noSuchManualMessage);
            break borrowLoop;
        }
    }
    Menu.displayMenu();
}

如果我需要包含更多来自其他类的代码，请告诉我，因为我对编程相对较新。

Answer 1

当ArrayList有2个元素时(即size()为2)，有效索引为0和1 。 2 超出范围。

ManualChoice 必须是 <ManualList.size()。它不能等于它。

你也许应该改变

ManualChoice = (Console.readInteger(Messages.enterManualIndexMessage, Messages.ManualIndexNotInListMessage, 0, Library.ManualList.size()));

至

ManualChoice = (Console.readInteger(Messages.enterManualIndexMessage, Messages.ManualIndexNotInListMessage, 0, Library.ManualList.size() - 1));