Java ArrayList:检查id是否在列表中

Question

我在用Java解决这个基本问题时遇到了困难。我有一个5个动物笼子的数组，每个笼子都有一个ID。当我尝试验证一个笼子里是否已经有一个动物时，检查只对前两个笼子有效，后面的都是错误的。

这是主要的：

    Cat picasso = new Cat ("Cat", "Picasso");
    Cat athena = new Cat("Cat", "Athena");
    Dog thunder = new Dog ("Dog", "Thunder");
    Dog buffy = new Dog("Dog", "Buffy");

    Pets store = new Pets("Super store");

    store.addToCage(buffy, 1);
    store.addToCage(athena, 1);

这会抛出一条错误消息："cage is full“

但这个不是：

    store.addToCage(buffy, 1);
    store.addToCage(athena, 2);
    store.addToCage(thunder, 2);

它打印消息(笼子已满)，但它仍然将动物添加到笼子2中。任何超过2个笼子的都是有问题的:例如，它打印第三个笼子两次。

以下是类Pets中的addToCage方法：

     class Pets {

        ArrayList<Cage> cages = new ArrayList<Cage>();

     public void addToCage(Animal animal, int cageId){
       int nbrCages = cages.size();
       Cage tmp, cage;

       if(nbrCages == 0){
           cage = new Cage(cageId);
           cage.setAnimal(animal);
           this.cages.add(cage);
       }else{

            for(int i=0; i < nbrCages; i++){
                tmp = cages.get(i);
                if(tmp.getId() == cageId){

                System.out.println("cage is full");
                }else{
                    cage = new Cage(cageId);
                    cage.setAnimal(animal);
                    this.cages.add(cage);

这是打印结果的方法：

    public String displayResults(){
        String results="";
        int nbrCages = cages.size();

        if(nbrCages > 0){
           Cage tmp;
           for(int i=0; i < nbrCages; i++){
             tmp = cages.get(i);
             results += tmp.showCages();
         }

       }else{
       results += "store is empty"; 
    }

       return results;
   }

Answer 1

您的问题是else语句。你应该像这样做一个验证：

public void addToCage(Animal animal, int cageId) {
    int nbrCages = cages.size();
    Cage tmp, cage;

    if (nbrCages == 0) {
        cage = new Cage(cageId);
        cage.setAnimal(animal);
        this.cages.add(cage);
    } else {

        boolean cageInUse = false;

        for (int i = 0; i < nbrCages; i++) {
            tmp = cages.get(i);
            if (tmp.getId() == cageId) {
                System.out.println("cage is full");
                cageInUse = true;
                break;
            }
        }

        if (!cageInUse) {
            cage = new Cage(cageId);
            cage.setAnimal(animal);
            this.cages.add(cage);
        }
    }
}

这不是最好的解决方案，但它解决了您的问题。

Answer 2

这就是你正在检查的问题，笼子是空的还是不是nbrCages == 0，而不是笼子有没有空间。

如果animal显示tmp.getId() == cageId，则只需打印cage is full，将其更改为animal is present in cage

public void addToCage(Animal animal, int cageId){
   int nbrCages = cages.size();
   Cage tmp, cage;

   if(nbrCages < 2){     // First check the cage have space or not

    for(int i=0; i < nbrCages; i++){
            tmp = cages.get(i);
            if(tmp.getId() == cageId){     // Now check animal present in cage or not, if present don't add

            System.out.println(" Animal already present in cage ");
            }else{                          // If not present the add the animal to cage
                cage = new Cage(cageId);
                cage.setAnimal(animal);
                this.cages.add(cage);     
   }else{    // If not print cage is full

        System.out.println(" cage is full on animals ");
    }