Question

I'm supposed to create a simple multiplication table using GUI and for the most part, it's doing what it is supposed to do. But I just can't figure out the bottom portion of it where if you click on a specific button, a text will say what the two numbers are being multiplied and the answer.

No matter what button I click, it'll only tell me the numbers I input and multiply them together. Here is my code:

import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JOptionPane;

public class MultiplicationTable {
    int clickedCount;
    int rowCount;
    int colCount;

    JFrame frame;
    JPanel buttonPanel, countPanel;
    JLabel countLabel;


    private void createAndShowGui(){

        rowCount = Integer.parseInt(JOptionPane
                .showInputDialog("How many rows do you want your multiplication table to be? "));

        colCount = Integer
                .parseInt(JOptionPane
                        .showInputDialog("How many columns do you want your multiplication table to be? "));



        frame = new JFrame ("Simple Multiplication Table");
        frame.setLayout(new BorderLayout());

        buttonPanel = new JPanel();

        buttonPanel.setLayout(new GridLayout(rowCount, colCount));


        countPanel = new JPanel();      
        countLabel = new JLabel("? x ? = ?");

        for(int rowCounter = 1; rowCounter <= rowCount; rowCounter++)
            for(int colCounter = 1; colCounter <= colCount; colCounter++){

                final JButton j = new JButton(Integer.toString(rowCounter * colCounter));

                j.addActionListener(new ActionListener(){


                    @Override
                    public void actionPerformed(ActionEvent event) {
                        for(int rc = 1; rc <= rowCount; rc++)
                            for(int cc = 1; cc <= colCount; cc++){
                                countLabel.setText(rc + " x " + cc + " = " + String.valueOf(rc * cc));  
                            }   
                    }
                });

                buttonPanel.add(j); 
            }

        frame.add(buttonPanel, BorderLayout.NORTH);
        countPanel.add(countLabel);
        frame.add(countPanel, BorderLayout.SOUTH);

        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.pack();
        frame.setVisible(true);
    }

    public static void main(String[] args){
        MultiplicationTable c = new MultiplicationTable();
        c.createAndShowGui();
    }
}
Était-ce utile?

La solution

Your current ActionListener has no context, it doesn't know what two values were used to generate the value you gave the button.

What you could do is create a ActionListener which takes the values used to produce the button's text, this would then be capable of updating the label accordingly. For example...

public class MultiplerActionListener implements ActionListener {
    private int[] values;
    private JLabel label;
    public MultiplerActionListener(JLabel label, int... valuesToMultiple) {
        values = valuesToMultiple;
        this.label = label;
    }

    @Override
    public void actionPerformed(ActionEvent event) {        
        int answer = 1; 
        StringBuilder sb = new StringBuilder(64);
        for (int value : values) {
            if (sb.length() > 0) {
                sb.append(" x ");
            }
            sb.append(value);
            answer *= value;
        }
        sb.append(" = ");
        sb.append(answer);
        label.setText(sb.toString());
    }
}

Then you would simply need to apply it when you create the buttons

for(int rowCounter = 1; rowCounter <= rowCount; rowCounter++) {
    for(int colCounter = 1; colCounter <= colCount; colCounter++){
        final JButton j = new JButton(Integer.toString(rowCounter * colCounter));
        j.addActionListener(new MultiplerActionListener(countLabel, rowCounter, colCounter));
        buttonPanel.add(j); 
    }
}
Licencié sous: CC-BY-SA avec attribution
Non affilié à StackOverflow
scroll top