Java 数组值被覆盖

Question

我是 Java 的新手，这是一个非常基本的问题。 但是我很难找到解决方案，所以希望有人能给我一些指示。

我正在尝试将值填充到数组“ addedPlayer”中。 但是，每次我运行 AddPlayer() 方法时，它都会再次初始化为零。

我怎样才能以更好的方式构建它？

public class DemoApplication implements CommandLineRunner {
    public static void main(String[] args) {
        SpringApplication.run(DemoApplication.class, args);
    }

    public void AddPlayer() {
        int[] addedPlayer;

        addedPlayer = new int[500];

        System.out.println(" *** Add new player *** ");
        System.out.println("Name:");
        String name = System.console().readLine();
        System.out.println("Age:");
        int age = Integer.parseInt(System.console().readLine());
        System.out.println("JNUM:");
        int jnum = Integer.parseInt(System.console().readLine());

        player p = new player();
        p.SetAge(age);
        p.SetName(name);
        p.SetJnum(jnum);

        System.out.println(addedPlayer[0]);
        for (int j = 0; j < addedPlayer.length; j++) {
            if (addedPlayer[j] != 0) {
            } else {
                addedPlayer[j] = p.GetAge();
                System.out.println(addedPlayer[j]);
                System.out.println(j);
                break;
            }
        }
    }

    public void EditPlayer() {
        //empty
    }

    public void ListPlayer() {
        //empty
    }

    @Override
    public void run(String... args) throws Exception {
        while (true) {
            System.out.println(" *** MENY *** ");
            System.out.println(" 1. Add player ");
            System.out.println(" 2. Edit player ");//ÖKurs
            System.out.println(" 3. List player ");
            System.out.println(" 100. Exit ");
            System.out.println("Ange val");
            int sel = Integer.parseInt(System.console().readLine());

            if (sel == 100) break;
            if (sel == 1) AddPlayer();
            if (sel == 2) EditPlayer();
            if (sel == 3) AddPlayer();
        }
    }
}

Answer 1

int[] addedPlayer; addedPlayer = new int[500]; 它被覆盖，因为您正在创建一个新的本地 var addedPlayer ，然后将所有值设置为 0 （ addedPlayer = new int[500]; ）我假设您希望addedPlayer是全局的，所以不要在本地定义它并且将其设置为 0。

另外， addedPlayer应该是player[]还是只是一个player而不是int[] ？ 另外，你没有在你给我们的代码中关闭 function，所以还有更多遗漏还是你没有关闭它？

Answer 2

你的代码和你的期望完全不同。

阅读本文并告诉我们您是否理解。 如果我是你，我会这样写。 但它未经测试：

public class AddPlayer { // you create a class

    player[] addedPlayer; // array of players
    int index;
    
    public AddPlayer() { // constructor of the class
        addedPlayer = new player[500]; // max 500 players
        index = 0;
    }

    public void addPlayer(player p) {
        if(index < 500) {
            addedPlayer[index] = p; // add at index, 
            index = index + 1; // then increment index for the next added player
        }
    }

    public static void main(String... args) {
        AddPlayer addPlayers = new AddPlayer();
       
        int i = 0;
        while(i < 5) { // will run 5 times, so only 5 players will be added. Change to stop when you will need
            
            // here your read console inputs
            System.out.println(" *** Add new player *** "+ (i+1));
            System.out.println("Name:");
            String name = System.console().readLine();
            System.out.println("Age:");
            int age = Integer.parseInt(System.console().readLine());
            System.out.println("JNUM:");
            int jnum = Integer.parseInt(System.console().readLine());
            
            // initialize the player
            player p = new player();
            p.SetAge(age);
            p.SetName(name);
            p.SetJnum(jnum);
 
            addPlayers.addPlayer(p); // add in the array

            i++;
        }
    }
}

Answer 3

每次运行 AddPlayer() 时，它都会从头开始创建一个新播放器。 如果要将修改保持在最低限度，则必须将其放在方法之外，并使其成为 class 的属性，例如List<int[]> addedPlayers = new ArrayList<int[]>(); 您可以添加这一行 AddPlayer 以将其添加到列表中addedPlayers.add(addedPlayer) 。 否则，如果你想要更简洁的代码，你应该添加更多的类而不是只添加一个主要的 class。 要改进您的代码，您可以查看@g.momo 的答案。