Question

很多时候我运行它，有一个无限循环问题，但我根本无法找出它是什么/在哪里。其他时候，它工作正常。在无限错误运行期间，程序甚至没有运行初始部分，我告诉程序显示arrayList。我有一个名为dailyPlanetStreet的arrayList，其中包含Goodguy Badguy Person或NormalGuy类的对象。在我的删除机制下面，如果坏人的一边有一个普通人/人，我删除那个人/ normalguy。如果双方都有一个普通人/人，我会随机删除其中一个。此外，每次我删除一个，我随机将另一个人/普通人从每日星球街转移到另一个称为安全土地的arraylist。

例如，无限错误运行可能如下所示：

请帮我找到问题

import java.util.ArrayList;
public class GothamLikeAdventureTown 
{
    static int y=5;
    static int x=3;

    static ArrayList<Person> dailyPlanetStreet=new ArrayList<Person>();
    static ArrayList<Person> safetyLand=new ArrayList<Person>();
    static ArrayList<Person> unfortunatelyFatallyWoundedPeople=new ArrayList<Person>();

    public static void rescue() {
        int goodpos=0;
        int randompos=(int)(Math.random()*dailyPlanetStreet.size());
        for (int counter1=0; counter1<dailyPlanetStreet.size(); counter1++){
            if (dailyPlanetStreet.get(counter1).name()!="Kent Clark") {goodpos=counter1; }
        }
        dailyPlanetStreet.add(randompos,dailyPlanetStreet.remove(goodpos));

    }



    public static void tosafety() {
        System.out.println("Got here!");
        int outer=12;
        while (outer!=12){
        int randompos=(int)(Math.random()*dailyPlanetStreet.size());
        if ((dailyPlanetStreet.get(randompos).name()!="Clark Kent") &&  (dailyPlanetStreet.get(randompos).name()!="James Kalvin")) 
        {
            System.out.println("Got into the loop!");
            safetyLand.add(dailyPlanetStreet.remove(randompos));
            outer=12;
        } }
    }
    public static void  remove(int deletepos) {
        unfortunatelyFatallyWoundedPeople.add(dailyPlanetStreet.remove(deletepos)); 
    }

    public static void display(){
        System.out.println();
        System.out.println();
        for (int z=0; z<dailyPlanetStreet.size(); z++)
        {
            if (dailyPlanetStreet.get(z) instanceof BadGuy) System.out.println("badguy");
            else if (dailyPlanetStreet.get(z) instanceof GoodGuy) System.out.println("goodguy");
            else if (dailyPlanetStreet.get(z) instanceof NormalGuy) System.out.println("normal guy");
            else if (dailyPlanetStreet.get(z) instanceof Person) System.out.println("person");
        } 
    }

    public static void main(String args[])
        {
        String[] firstname= {"a ", "b", "c", "d","e","f","g","h","i","j"};
        String[] lastname= {"k ", "l", "m", "n","o","p","q","r","s","t"};
        while (y==x){
            y=(int)(Math.random()*10);
            x=(int)(Math.random()*10);

        }



        for (int z=0; z<10; z++)
        {
            if (z==y)
                {
                    GoodGuy superMan=new GoodGuy( "This looks like a job for superman!"   , "A lot of them"   ,  true , 10   , "Superman"  , "Clark Kent"  , "plumber" , 100  ,   10,10);
                    dailyPlanetStreet.add(superMan);                    }

            else if (z==x)
                {
                    BadGuy superMan=new BadGuy( "haha"   , "Darkness"   ,  true , 10   , "Batman"  , "James Kalvin"  , "farmer" , 100  ,   10,10);
                    dailyPlanetStreet.add(superMan);
                                                                }   
            else  if (((int)(Math.random()*2)+1)==1)
                 {
                    Person superMan=new Person( firstname[(int)(Math.random()*10)]+lastname[(int)(Math.random()*10)], "teacher", 100  , 10,10);
                    dailyPlanetStreet.add(superMan); 
                 }  
            else 
                {
                    NormalGuy superMan=new NormalGuy( firstname[(int)(Math.random()*10)]+lastname[(int)(Math.random()*10)], "teacher", 100  , 10,10);
                    dailyPlanetStreet.add(superMan); 
                }
        }

        display();


        while (dailyPlanetStreet.size()>2){
        for (int counter=0; counter<dailyPlanetStreet.size(); counter++){
            if (dailyPlanetStreet.get(counter).name()=="James Kalvin") 
            {
                if ((counter==0) && (dailyPlanetStreet.get(1).name()=="Clark Kent")) {}
                else if ((counter==0) && (dailyPlanetStreet.get(1).name()!="Clark Kent")) {System.out.println("yo"); remove(counter+1);display();rescue();} 
                else if ((counter==dailyPlanetStreet.size()-1) && (dailyPlanetStreet.get(dailyPlanetStreet.size()-2).name()=="Clark Kent")) {System.out.println("yo"); tosafety();rescue();display();}
                else if ((counter==dailyPlanetStreet.size()-1) && (dailyPlanetStreet.get(dailyPlanetStreet.size()-2).name()!="Clark Kent")) {System.out.println("yo");remove(counter-1);tosafety();rescue();display();}
                else if  (dailyPlanetStreet.get(counter+1).name()=="Clark Kent") {System.out.println("yo");remove(counter-1);tosafety();rescue();display();}
                else if (dailyPlanetStreet.get(counter-1).name()=="Clark Kent") {System.out.println("yo");remove(counter+1); tosafety();rescue();display();}
                else if ((dailyPlanetStreet.get(counter-1).name()!="Clark Kent")&& (dailyPlanetStreet.get(counter+1).name()!="Clark Kent")) 
                {
                    if (((int)(Math.random()*2))==1) {System.out.println("yo");remove(counter+1); tosafety();rescue();display();}
                    else {System.out.println("yo");remove(counter-1);  tosafety();rescue();display();}
                } 

                }
            }
        System.out.println(dailyPlanetStreet.size());
        }


        System.out.println("hi");

        }   

}

Answer 1

您将String与==进行比较而不是.equals，这是一个常见的错误，不起作用。

例如在你的while循环中：

if (dailyPlanetStreet.get(counter).name()=="James Kalvin")

应更改为：

if (dailyPlanetStreet.get(counter).name().equals("James Kalvin"))

以及比较字符串的所有其他事件。

修改

关于无限循环的问题：

你的while循环运行直到.size()小于2.但是在你的while循环中，只有当.name()中的dailyPlanetStreet值之一为"James Kalvin"时才会执行某些操作。从我看到的那些值是随机生成/选择的。因此，如果你碰巧没有任何詹姆斯·卡尔文斯，那么你将永远在那里循环。

为了能够详细说明为什么它会永远循环，我将不得不看到你正在使用的BadGuy的构造函数。

编辑2

在查看下面的评论之后，我看到while循环中的所有if else if子句都以一个将String与==进行比较的条件开始。因此，没有一个会回归真实，也不会做任何事情。请将您的所有....name()=="Something"更改为....name().equals("Something")。

无限循环错误

1 个答案: