使用类中的方法更改数组

Question

我尝试了很多方法来修复这个类，但没有奏效。我想创建一个酒店（数组）并检查并用另一个类中定义的人填充它，但我需要添加、删除、检查空项目并使用酒店类中的方法返回它们的索引。问题是当我在一个方法中更改数组时，我无法返回更改后的值，当我尝试添加其他类成员试图解决它时，我以 indexOutOfboundryException 结束。请您看一下代码并告诉我您是否看到错误，为什么我无法返回更改后的值以及为什么我的索引越界

非常感谢！

package com.company;

public class Hotel1 {

    private int numberOfRooms1;  // Number of rooms
    private Person[]bookingList1=new Person[numberOfRooms1];// booking list with initial length of the number of rooms
    private int uniqueId1;
    private Person person1;
    private int currentIndex;
    private boolean isEmpty=true;


    public int getNumberOfRooms1() {
        return numberOfRooms1;
    }

    private Ticket ticket1;

    public Hotel1(int numberOfRoomss1) {// The constructor has one attribute which is the number of the rooms

        this.numberOfRooms1 = numberOfRoomss1;
    }


    //check in method
    public Boolean isEmpty(){
        for (currentIndex=0;currentIndex<bookingList1.length;currentIndex++){
            if (!bookingList1[currentIndex].equals(null))
                 isEmpty=false;
            else  isEmpty= true;
        }

        return isEmpty;
    }
    public int findEmptyRooms(){
        if (isEmpty)
        {
            for (currentIndex=0;currentIndex<bookingList1.length;currentIndex++){
                return currentIndex;
            }
            return currentIndex;
        }
        return currentIndex;

    }
    public Person checkIn1(Person person){
        if (isEmpty==true){
            return bookingList1[findEmptyRooms()]=person;
        }
        else {
            System.out.println("There is no empty rooms");
            return null;
        }


    }

}

Answer 1

感谢大家的回答，他们帮助我重构了整个代码，让它变得更好，让它工作，以防万一你想看看它是如何变成的，看看`package com.company;

public class Hotel1 {
private int numberOfRooms1;// number of rooms
Person rooms[];



public Hotel1(int numberOfRooms1){    //the constructor
    this.numberOfRooms1=numberOfRooms1;
    Person[]rooms=new Person[numberOfRooms1];
    this.rooms=rooms;
}


//Private method to check if the array has an empty place

private boolean isEmpty(){
    boolean isEmpty =false;
    for (int i=0;i<rooms.length;i++){
        if (rooms[i]==null){
            return true;
        }
    }
    return isEmpty;

}

//Check in
public Person[] checkIn(Person person){
    if (isEmpty()){
    for (int i=0;i<rooms.length;i++){
        if (rooms[i]==null){
            rooms[i]=person;
            return rooms;
        }
        else continue;
    }
    }
    else {
        System.out.println("There is rooms left for "+person.getFirstName()+" Sorry!");
    }
    return rooms;
}}

Answer 2

aarrghh在很少的线条中有这么多错误...

抱歉粗鲁，但请检查您的代码逻辑，我不明白...

例如： 公共 int findEmptyRooms(){ 如果（是空的） { for (currentIndex=0;currentIndex

}

这个函数应该做什么？！

for (currentIndex=0;currentIndex<bookingList1.length;currentIndex++){
        return currentIndex;
    }

总是返回 bookingList1.length + 1 （这就是为什么调用 checkIn1 会引发 indexOutOfboundryException）

另一个例子：

public Boolean isEmpty(){
for (currentIndex=0;currentIndex<bookingList1.length;currentIndex++){
    if (!bookingList1[currentIndex].equals(null))
         isEmpty=false;
    else  isEmpty= true;
}

return isEmpty;

}

完全是错误的，如果数组中有一个元素为 null 后跟非 null 元素，isEmpty 将返回 false

我认为您应该在将问题发布到堆栈溢出之前重新考虑您的所有代码