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

Java 数组值被覆盖

3 个答案: