Java ActionListener只工作一次，其他人从不工作

我正在尝试做一个简单的项目，根据你的进度随机生成一个“铜”量。每当你得到100铜，你可以把它转换成银，100银可以转换成金。要增加滚动进度，您需要单击一个按钮添加一个、两个、五个或十个，并“支付”一定数量的银和铜

generate按钮和copperToSilver按钮每次都能工作，所以我删除了它的代码并将其压缩。我还从GUI类中删除了显示当前铜/银/金数量的文本字段

问题是addRoll ActionListener只有在我第一次按下它时才起作用，此后无论它是否满足要求，它都不起作用。这个项目还没有完成，但其他已经完成的事情似乎根本不起作用

我删除了我的临时调试，但是当您按下addTwoRoll按钮时，RollProgressPlusTwo方法似乎根本没有运行，它应该这样做

代码如下：

主要类别：

import javax.swing.JFrame;

public class MainClass {    
    public static void main(String args[]) {
        GUI go = new GUI();
        go.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        go.setSize(350,300);
        go.setResizable(true);
        go.setLocationRelativeTo(null);
        go.setVisible(true);
    }
}

GUI类：

import javax.swing.*;
import javax.swing.event.*;
import java.awt.*;
import java.awt.event.*;
import java.util.Random;

public class GUI extends JFrame {
    Button bo = new Button();
    CoinTracker cto = new CoinTracker();
    TextFields tfo = new TextFields();
    RollProgress rpo = new RollProgress();

    public GUI() {
        super("Money dice");
        setLayout(new GridLayout(11,1,3,3));

        add(bo.generate);

        add(tfo.currentCashField);
        add(tfo.goldField);
        add(tfo.silverField);
        add(tfo.copperField);

        add(bo.copperToSilver);
        add(bo.silverToGold);
        add(bo.addRoll);
        add(bo.addTwoRoll);
        add(bo.addFiveRoll);
        add(bo.addTenRoll);

        GenerateHandler ghandler = new GenerateHandler();
        bo.generate.addActionListener(ghandler);

        ConvertToSilverHandler ctshandler = new ConvertToSilverHandler();
        bo.copperToSilver.addActionListener(ctshandler);

        PlusOneHandler pohandler = new PlusOneHandler();
        bo.addRoll.addActionListener(pohandler);

        PlusTwoHandler ptwhandler = new PlusTwoHandler();
        bo.addTwoRoll.addActionListener(ptwhandler);
    }

    private class GenerateHandler implements ActionListener {
        public void actionPerformed(ActionEvent event) {
            Random roll = new Random();
            cto.copper = cto.copper+1+roll.nextInt(rpo.rollProgress);
            tfo.copperField.setText(cto.copper + " copper");
            tfo.silverField.setText(cto.silver + " silver");
            tfo.goldField.setText(cto.gold + " gold");
        }
    }

    private class PlusOneHandler implements ActionListener {
        public void actionPerformed(ActionEvent event) {
            rpo.RollProgressPlusOne();
            tfo.copperField.setText(cto.copper + " copper");
            tfo.silverField.setText(cto.silver + " silver");
            tfo.goldField.setText(cto.gold + " gold");
        }
    }

    private class PlusTwoHandler implements ActionListener {
        public void actionPerformed(ActionEvent event) {
            rpo.RollProgressPlusTwo();
            tfo.copperField.setText(cto.copper + " copper");
            tfo.silverField.setText(cto.silver + " silver");
            tfo.goldField.setText(cto.gold + " gold");
        }
    }

    private class ConvertToSilverHandler implements ActionListener {
        public void actionPerformed(ActionEvent event) {
            cto.toSilver();
            tfo.copperField.setText(cto.copper + " copper");
            tfo.silverField.setText(cto.silver + " silver");
            tfo.goldField.setText(cto.gold + " gold");
        }
    }
}

CoinTracker类别：

import javax.swing.*;
import javax.swing.event.*;
import java.awt.*;
import java.awt.event.*;

public class CoinTracker {
    public int copper = 98;
    public int silver = 1;
    public int gold = 0;

    public void toSilver() {
        if(copper >= 100) {
            copper -= 100;
            silver++;
        }
    }
}

按钮类别：

import javax.swing.*;
import javax.swing.event.*;
import java.awt.*;
import java.awt.event.*;

public class Button extends JFrame {
    public JButton generate;
    public JButton copperToSilver;
    public JButton silverToGold;
    public JButton addRoll;
    public JButton addTwoRoll;
    public JButton addFiveRoll;
    public JButton addTenRoll;

    public Button() {
        generate = new JButton("Generate");
        copperToSilver = new JButton("Convert 100 copper to 1 silver");
        silverToGold = new JButton("Convert 100 silver to 1 gold");
        addRoll = new JButton("Pay 1 silver to add one number to the dice");
        addTwoRoll = new JButton("Pay 2 silver to add two numbers to the dice");
        addFiveRoll = new JButton("Pay 4 silver and 50 copper to add five numbers to the dice");
        addTenRoll = new JButton("Pay 9 silver to add ten numbers to the dice");
    }
}

文本字段类：

import javax.swing.*;
import javax.swing.event.*;
import java.awt.*;
import java.awt.event.*;

public class TextFields {
    public JTextField currentCashField;
    public JTextField copperField;
    public JTextField silverField;
    public JTextField goldField;
    CoinTracker ctotf = new CoinTracker();

    public TextFields() {
        currentCashField = new JTextField("Your current cash is:");
        copperField = new JTextField(ctotf.copper + " copper");
        silverField = new JTextField(ctotf.silver + " silver");
        goldField = new JTextField(ctotf.gold + " gold");   
    }
}

RollProgress类：

import javax.swing.*;
import javax.swing.event.*;
import java.awt.*;
import java.awt.event.*;

public class RollProgress {
    public int rollProgress = 1;
    CoinTracker ctorp = new CoinTracker();

    public void RollProgressPlusOne() { 
        if(ctorp.silver >= 1) {
            rollProgress++;
            ctorp.silver--;
            System.out.println(rollProgress);
        }
    }

    public void RollProgressPlusTwo() {
        if(ctorp.silver >= 2) {
            rollProgress++;
            rollProgress++;
            ctorp.silver--;
            ctorp.silver--;
            System.out.println(rollProgress);
        }
    }
}

---

提前感谢您尝试解决/解决方案。

此代码存在许多结构性问题，但尖锐的问题是您有两个Cointracker实例。处理程序只更新GUI实例，RollProgress没有对该实例的引用。为RollProgress创建构造函数并传递GUI中使用的CoinTracker

CoinTracker ctorp ;//= new CoinTracker();

public  RollProgress(CoinTracker ct){
    this.ctorp = ct;
}

---

处理程序正在以其他方式工作。

发布一篇文章以获得更好的帮助，因为没有人愿意阅读那么多代码。尝试将您的代码缩减为一个较小的示例。。可能有100行，你的输出有什么异常吗？发布Cointracker。此代码似乎工作正常。我将System.out.println放入您执行的每个操作中，它们都工作多次。我无法重现该问题。我复制并粘贴了代码，添加了一些println，似乎一切都正常。你确定你正在重新编译或者遗漏了什么吗？Button和TextFields类没有真正的用途，除非你计划重用GUI的那个部分。Button类更令人困惑，因为它扩展了JFrame，但没有将该扩展用于任何事情（Button是一个令人困惑的名称）。为什么不将按钮和文本字段折叠到GUI中？每一个领域都是公开的，它们都面临着封装。