Question

所以我在使用addActionListeners（）方法添加actionlisteners时遇到了问题，该方法位于某个system.out.printlns之间，因此我可以告诉该方法实际上正在运行。

protected void whoFirst(String first) {
    int currPlayer = 0;
    System.out.println("Hello");
    addActionListeners();
    System.out.println("How are you?");
    if(first == "player1") {
        player1.setVisible(true);
        currPlayer = 1;
    }
    if(first == "player2") {
        player2.setVisible(true);
        currPlayer = 2;
    }
}

添加actionlistener方法我尝试了很多不同的方法，例如让类实现一个actionListener，并使用player1Cards[i].addActionListener(this); ......这不起作用所以我改为：

private void addActionListeners() {
            System.out.println("Number of players = : " + players );
            for(int i = 0; i == player1Cards.length ; i++) {
            if(players == 2) {
                player1Cards[i].addActionListener(e -> cardActions());
                player2Cards[i].addActionListener(e -> cardActions());
            }
            if(players == 3) {
                player1Cards[i].addActionListener(e -> cardActions());
                player2Cards[i].addActionListener(e -> cardActions());
                player3Cards[i].addActionListener(e -> cardActions());
            }
            if(players == 4) {
                player1Cards[i].addActionListener(e -> cardActions());
                player2Cards[i].addActionListener(e -> cardActions());
                player3Cards[i].addActionListener(e -> cardActions());
                player4Cards[i].addActionListener(e -> cardActions());
            }
        }
    }

这是目前的情况，在找到java 8教程后（我使用的是java 8，所以应该没问题？）如果不明显的话，JButton在一个集合中，并且所有玩家的大小都与开始时相同数量的牌相同。这是我的方法，无论哪个玩家先行，都应该调用它...但是它从不在控制台上打印一条线......

private void cardActions() {
    System.out.println("Whats up?");
}

我觉得这应该适用于任何一种情况，但如果有人有任何建议可以帮助那将是太棒了。提前谢谢。

Answer 1

有些事情在你的代码中并不完全正确。

你的for循环不正确：
```
for (int i = 0; i == player1Cards.length; i++)
```
必须是
```
for (int i = 0; i < player1Cards.length; i++)
```
您的for循环可以重写为：
```
{
    int i = 0;
    while (i == player1Cards.length) {
        // code inside for loop
        i++;
    }
}
```
因为player1Cards的长度总是大于0，所以第一个循环的条件i == player1Cards.length为false，导致for循环立即中止。
您正在将字符串与==进行比较。 永远不要这样做！始终使用equals()来比较字符串。这是因为对于对象引用，==比较对象的标识（内存位置）。对于字符串，它是相同的。这就是为什么值为“player1”的字符串并不总是与具有相同值的另一个字符串具有相同的标识。 equals()方法旨在比较被比较对象的值。

作为hinted by Zabuza，this answer on StackOverflow详细说明了==和.equals()之间的区别。

您还应避免变量重复，如player1Cards，player2Cards等。如果你扩展游戏并允许16名玩家怎么办？你必须复制粘贴很多东西。解决该问题的一种方法是为玩家使用数组，例如playerCards[]。此外，您应该阅读有关Java中面向对象的更多信息。它将指导您何时使用类和对象。

为什么我的actionlisteners不工作？

1 个答案: