java - 为什么我的 Action 监听器不工作？

Question

所以我在使用 addActionListeners() 方法添加 Action 监听器时遇到了麻烦，该方法位于某些 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

您的代码中有些东西不太正确。

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 循环立即中止。

2. 您正在将字符串与 == 进行比较。 永远不要这样做!始终使用equals()来比较字符串。这是因为对于对象引用，== 会比较对象的身份(内存位置)。对于字符串来说，也是一样的。这就是为什么具有值“player1”的字符串并不总是与具有相同值的另一个字符串具有相同的标识。 equals() 方法旨在比较所比较对象的值。

   如hinted by Zabuza , this answer on StackOverflow详细解释了 == 和 .equals() 之间的区别。

<小时/>

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