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]; 它被覆盖是因为您正在创建一个新的本地变量 addedPlayer，然后将所有值设置为 0 (addedPlayer = new int[500];) 我假设您希望 addedPlayer 是全局的，所以不要t在本地定义，设置为0。

此外，addedPlayer 应该是 player[] 还是只是 player 而不是 int[]？另外，您给我们的代码中没有关闭该功能，所以是缺少更多还是您只是没有关闭它？

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) 中。否则，如果你想要一个更简洁的代码，你应该添加更多的 classes 而不是只添加一个 main class。要改进您的代码，您可以查看@g.momo的回答。

Java 数组值被覆盖

Java array values gets overwritten

java

arrays

structure