JAVA UI ActionListener 2 连续单击带有 if else 语句的按钮答案

【问题标题】：JAVA UI ActionListener 2 consecutive click of buttons with if else statementsJAVA UI ActionListener 2 连续单击带有 if else 语句的按钮
【发布时间】：2019-02-26 11:14:28
【问题描述】：

谁能告诉我我的代码有什么问题？

我正在尝试制作一个具有 bin、dec 等的 UI。每次单击它时，只有与数字系统相对应的适当数字才能被用户单击或显示在 textField 中，就像用户一样点击 bin 只有 0 和 1 可以显示在 textField 上。

我尝试使用嵌套的 if else 语句，但它似乎不起作用。请帮忙谢谢。

下面是我的代码：

    import java.awt.BorderLayout;
    import java.awt.event.ActionListener;
    import java.awt.event.ActionEvent;
    import javax.swing.JButton;
    import javax.swing.JLabel;
    import javax.swing.SwingConstants;  
    import javax.swing.*;
    import java.awt.*;
    import java.awt.FlowLayout;


public class Trial extends JFrame implements ActionListener
{

private JTextField textField;
private JButton n1, n2, n3, n4, n5, n6, n7, n8, n9, n0, add, subtract, multiply, divide, clear, backspace, bin, dec, oct, hex;
private JPanel panel, panel1, p;


public Trial(String str)
{

    super(str);
    setSize(300, 200);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setLocationRelativeTo(null);
    setResizable(false);


    JPanel panel = new JPanel();
    panel.setLayout(new GridLayout(5, 4));

    panel.add(n1 = new JButton("1"));
    panel.add(n2 = new JButton("2"));
    panel.add(n3 = new JButton("3"));
    panel.add(backspace = new JButton("C"));
    panel.add(n4 = new JButton("4"));
    panel.add(n5 = new JButton("5"));
    panel.add(n6 = new JButton("6"));
    panel.add(bin = new JButton("BIN"));
    panel.add(n7 = new JButton("7"));
    panel.add(n8 = new JButton("8"));
    panel.add(n9 = new JButton("9"));
    panel.add(dec = new JButton("DEC"));
    panel.add(add = new JButton("+"));
    panel.add(n0 = new JButton("0"));
    panel.add(subtract = new JButton("-"));
    panel.add(oct = new JButton("OCT"));
    panel.add(multiply = new JButton("*"));
    panel.add(clear = new JButton("AC"));
    panel.add(divide = new JButton("/"));
    panel.add(hex = new JButton("HEX"));



    JPanel panel1 = new JPanel();
    panel1.setLayout(new FlowLayout());
    panel1.add(textField = new JTextField(20));


    panel.setBounds(20,20,300,150);
    panel1.setBounds(0,0,300,20);
    add(panel, BorderLayout.SOUTH);
    add(panel1, BorderLayout.NORTH);


    n1.addActionListener(this);
    n2.addActionListener(this);
    n3.addActionListener(this);
    n4.addActionListener(this);
    n5.addActionListener(this);
    n6.addActionListener(this);
    n7.addActionListener(this);
    n8.addActionListener(this);
    n9.addActionListener(this);
    n0.addActionListener(this);
    add.addActionListener(this);
    subtract.addActionListener(this);
    multiply.addActionListener(this);
    divide.addActionListener(this);
    clear.addActionListener(this);
    backspace.addActionListener(this);
    bin.addActionListener(this);
    dec.addActionListener(this);
    oct.addActionListener(this);
    hex.addActionListener(this);


    setVisible(true);

}

public void actionPerformed(ActionEvent event)
{
    if(event.getSource() == bin){
        if(event.getSource() == n1){
            textField.setText(textField.getText() + "1");
        }
        else if(event.getSource() == n0){
            textField.setText(textField.getText() + "0");
        }
        else if(event.getSource() == add){
            textField.setText(textField.getText() + " + ");
        }
        else if(event.getSource() == subtract){
            textField.setText(textField.getText() + " - ");
        }
        else if(event.getSource() == multiply){
            textField.setText(textField.getText() + " * ");
        }
        else if(event.getSource() == divide){
            textField.setText(textField.getText() + " / ");
        }
        else if(event.getSource() == clear){
            textField.setText("");
        }
        else{
            textField.setText("");
        }
    }

    else if(event.getSource() == dec){
        if(event.getSource() == n1){
            textField.setText(textField.getText() + "1");
        }
        else if(event.getSource() == n2){
            textField.setText(textField.getText() + "2");
        }
        else if(event.getSource() == n3){
            textField.setText(textField.getText() + "3");
        }
        else if(event.getSource() == n4){
            textField.setText(textField.getText() + "4");
        }
        else if(event.getSource() == n5){
            textField.setText(textField.getText() + "5");
        }
        else if(event.getSource() == n6){
            textField.setText(textField.getText() + "6");
        }
        else if(event.getSource() == n7){
            textField.setText(textField.getText() + "7");
        }
        else if(event.getSource() == n8){
            textField.setText(textField.getText() + "8");
        }
        else if(event.getSource() == n9){
            textField.setText(textField.getText() + "9");
        }
        else if(event.getSource() == n0){
            textField.setText(textField.getText() + "0");
        }
        else if(event.getSource() == add){
            textField.setText(textField.getText() + " + ");
        }
        else if(event.getSource() == subtract){
            textField.setText(textField.getText() + " - ");
        }
        else if(event.getSource() == multiply){
            textField.setText(textField.getText() + " * ");
        }
        else if(event.getSource() == divide){
            textField.setText(textField.getText() + " / ");
        }
        else if(event.getSource() == clear){
            textField.setText("");
        }
    }

}




public static void main(String [] args)
{
    new Trial("Le Frame");
}

}

上述代码的问题是它在文本字段中根本不显示任何字符。

下面是程序的 sn-p： ![1]: (https://i.stack.imgur.com/R2Len.png)

【问题讨论】：

提示：pass = 1; if(pass == 1){ 代码的意义为 0；在检查之前为该变量分配一个值。它永远是 1。
请提供minimal reproducible example，以便我们为您提供解决方案。您在此处发布的代码片段看起来肯定是错误的，但如果不查看您的完整代码，我们将无法提供解决方案。
我支持@SergiyMedvynskyy 的MCVE 电话。话虽如此，最好将编号系统（十六进制、十进制、二进制等）之间的选择放入单选按钮的按钮组中。在数字按钮的动作监听器中，代码可以检查选择了哪个编号系统按钮组按钮。
@SergiyMedvynskyy 你好！感谢您的回复。我编辑了此处显示的代码，还添加了一张图片。
@AndrewThompson 我确实考虑过单选按钮，但我的主要问题是在我的代码中我使用 getSource 来了解用户单击了哪个数字系统，然后继续执行另一个 if 语句，其中它位于在文本字段上显示数字和操作，为此我需要使用另一个 getSource 但我知道一个事件只有一个源。我被困在这里了。

标签： java swing user-interface jbutton actionlistener

【解决方案1】：

当event.getSource() == bin 为真时，嵌套的 if 语句都不能为真： if(event.getSource() == n1){

此外，这个 if 将始终为真，因为赋值语句 pass = 1; if(pass == 1){

【讨论】：

感谢您的回复。我已经尝试过不使用 pass = 1 和 if(pass ==1)。但我认为问题在于，正如您可能已经提到的那样，一个事件只能有一个与我试图做的事情相矛盾的来源。您可能知道任何替代方案吗？请帮忙。谢谢你

【解决方案2】：

更新后尝试重新绘制或重新验证：

textField.repaint();

或者可能：

textField.revalidate();

查看：https://docs.oracle.com/javase/tutorial/uiswing/painting/problems.html

【讨论】：

您好！我有点明白现在的确切问题是什么。我认为这是关于 getSource 的事情，因为我认为你只能有一个来源。你能提供一些替代品吗？谢谢你

【解决方案3】：

因此，您需要一个单独的变量来保存当前视图类型（二进制/八进制/十进制/十六进制）

在您的代码中应该如下所示：

private static final int BINARY = 2; // probably use enum
private static final int OCTAL = 8; // probably use enum
private static final int DECIMAL = 10; // probably use enum
private JTextField textField;
private JButton n1, n2, n3, n4, n5, n6, n7, n8, n9, n0, add, subtract, multiply, divide, clear, backspace, bin, dec, oct, hex;
private JPanel panel, panel1, p;
private int system = DECIMAL;

// initialization code

public void actionPerformed(ActionEvent event)
{
    if(event.getSource() == bin){
        system = BINARY;
    } else if (event.getSource() == dec) {
        system = DECIMAL;
    } // same for other systems.

    if (system == BINARY) {
        if(event.getSource() == n1){
            textField.setText(textField.getText() + "1");
        }
        else if(event.getSource() == n0){
            textField.setText(textField.getText() + "0");
        }
        else if(event.getSource() == add){
            textField.setText(textField.getText() + " + ");
        }
        else if(event.getSource() == subtract){
            textField.setText(textField.getText() + " - ");
        }
        else if(event.getSource() == multiply){
            textField.setText(textField.getText() + " * ");
        }
        else if(event.getSource() == divide){
            textField.setText(textField.getText() + " / ");
        }
        else if(event.getSource() == clear){
            textField.setText("");
        }
    } else if (system == DECIMAL) {
        if(event.getSource() == n1){
            textField.setText(textField.getText() + "1");
        }
        else if(event.getSource() == n2){
            textField.setText(textField.getText() + "2");
        }
        else if(event.getSource() == n3){
            textField.setText(textField.getText() + "3");
        }
        else if(event.getSource() == n4){
            textField.setText(textField.getText() + "4");
        }
        else if(event.getSource() == n5){
            textField.setText(textField.getText() + "5");
        }
        else if(event.getSource() == n6){
            textField.setText(textField.getText() + "6");
        }
        else if(event.getSource() == n7){
            textField.setText(textField.getText() + "7");
        }
        else if(event.getSource() == n8){
            textField.setText(textField.getText() + "8");
        }
        else if(event.getSource() == n9){
            textField.setText(textField.getText() + "9");
        }
        else if(event.getSource() == n0){
            textField.setText(textField.getText() + "0");
        }
        else if(event.getSource() == add){
            textField.setText(textField.getText() + " + ");
        }
        else if(event.getSource() == subtract){
            textField.setText(textField.getText() + " - ");
        }
        else if(event.getSource() == multiply){
            textField.setText(textField.getText() + " * ");
        }
        else if(event.getSource() == divide){
            textField.setText(textField.getText() + " / ");
        }
        else if(event.getSource() == clear){
            textField.setText("");
        }
    } // same for other systems
}

请注意：最好为任何按钮提供单独的处理程序。例如，您可以使用 lambdas 来做到这一点。在这种情况下，您不需要巨大的 if-else-if 块

【讨论】：