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 的答案。