java - 通过Java中的方法传递变量

标签 java swing variables methods jbutton

我来这里是为了解决我已经困扰了好几个小时的问题。

在实践中,我希望能够通过矩阵中的方法传递变量“j”和“k”。问题在于传递它们,在不理解原因的情况下,它们在进入 if 时不会被“捕获”。事实上,如果我尝试在 if 之前打印 'r' 和 'c',它们就会匹配。我不明白我哪里错了,因为 if 中的其他东西工作得很好,但是当我尝试在 if 中打印 'r' 和 'c' 时,它们总是变成 0。

按钮创建:

Integer[] x = {10, 70, 130, 190, 250, 310, 370}; Integer[] y = {80, 140, 200, 260, 320, 380};
    for (int i = 0, k = 0, j = 0; i < 42; i++, j++) {
        if (j % 7 == 0 && i != 0) { j = 0; k++; }
        lblCircles[i] = new JLabel(new ImageIcon(this.getClass().getResource("vuoto.png")));
        lblCircles[i].setBounds(x[j], y[k], 50, 50);
        lblCircles[i].setName("vuota");
        lblCircles[i].addMouseListener(new MouseAdapter() {  
            public void mouseClicked(MouseEvent e) {
                for (int i = 0; i < 42; i++) {
                    for (int j = 0; j < 6; j++) {
                        for (int k = 0; k < 7; k++)
                            if (e.getSource() == lblCircles[i] && choose == 1) lblClickedPlayer(lblCircles[i], j, k);
                    }
                }
            }
        });
        frame.getContentPane().add(lblCircles[i]);
    }

方法:

public void lblClickedPlayer(JLabel lbl, int r, int c) {
    if (n == 0 && "vuota".equals(lbl.getName())) {
        n = 1;
        lbl.setIcon(new ImageIcon(this.getClass().getResource("red.png")));
        lbl.setName("piena");
        tbl[r][c] = 1;
        System.out.println("tbl[" + r + "][" + c + "] = " + tbl[r][c]);
    } else if (n == 1 && "vuota".equals(lbl.getName())) {
        n = 0;
        lbl.setIcon(new ImageIcon(this.getClass().getResource("yellow.png")));
        lbl.setName("piena");
        tbl[r][c] = 2;
        System.out.println("tbl[" + r + "][" + c + "] = " + tbl[r][c]);
    }
}

在此先感谢您的帮助。

测试代码:

public class Prova {
public JFrame frame;
public JLabel[] lblCircles = new JLabel[42];
public String hostname;
public Font big = new Font("Comic Sans MS", Font.BOLD, 18);
public Integer[][] tbl = new Integer[6][7];
public int choose = 1, n = 0;

public static void main(String[] args) {
    EventQueue.invokeLater(new Runnable() {
        public void run() {
            try {
                Prova window = new Prova();
                window.frame.setVisible(true);
            } catch (Exception e) { e.printStackTrace(); }
        }
    });
}

public Prova() { initialize(); }

private void initialize() {
    frame = new JFrame();
    baseFrame(frame, 430, 510);

    JPanel panel = new JPanel();
    panel.setLayout(new GridLayout(6, 7));
    panel.setBackground(new Color(41, 41, 41));
    panel.setBounds(10, 80, 410, 350);
    frame.getContentPane().add(panel);

    Integer[] x = {10, 70, 130, 190, 250, 310, 370}; Integer[] y = {80, 140, 200, 260, 320, 380};
    for (int i = 0, k = 0, j = 0; i < 42; i++, j++) {
        if (j % 7 == 0 && i != 0) { j = 0; k++; }
        lblCircles[i] = new JLabel("test");
        lblCircles[i].setBounds(x[j], y[k], 50, 50);
        lblCircles[i].setForeground(Color.RED);
        lblCircles[i].setFont(big);
        lblCircles[i].setName("vuota");
        lblCircles[i].addMouseListener(new MouseAdapter() {  
            public void mouseClicked(MouseEvent e) {
                for (int i = 0; i < 42; i++) {
                    for (int j = 0; j < 6; j++) {
                        for (int k = 0; k < 7; k++) {
                            if (e.getSource() == lblCircles[i] && choose == 1) lblClickedPlayer(lblCircles[i], j, k);
                        }
                    }
                }
            }
        });
        panel.add(lblCircles[i]);
    }
}

public void baseFrame(JFrame baseFrame, int width, int height) {
    baseFrame.getContentPane().setBackground(new Color(41, 41, 41));
    baseFrame.setBounds(100, 100, width, height);
    baseFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    baseFrame.getContentPane().setLayout(null);
    baseFrame.setUndecorated(true);
    baseFrame.setLocationRelativeTo(null);
    baseFrame.setResizable(false);
    baseFrame.setVisible(true);
    baseFrame.setShape(new RoundRectangle2D.Double(0, 0, baseFrame.getWidth(), baseFrame.getHeight(), 30, 30));
}

public void lblClickedPlayer(JLabel lbl, int r, int c) {
    if (n == 0 && "vuota".equals(lbl.getName())) {
        n = 1;
        lbl.setText("ok1");
        lbl.setForeground(Color.GREEN);
        lbl.setName("piena");
        tbl[r][c] = 1;
        System.out.println("tbl[" + r + "][" + c + "] = " + tbl[r][c]);
    } else if (n == 1 && "vuota".equals(lbl.getName())) {
        n = 0;
        lbl.setText("ok2");
        lbl.setForeground(Color.GREEN);
        lbl.setName("piena");
        tbl[r][c] = 2;
        System.out.println("tbl[" + r + "][" + c + "] = " + tbl[r][c]);
    }
}

最佳答案

您不需要那些内部循环,因为 i 就是您所需要的。您可以使用 int division 和 int remainder 轻松地从 i 值计算行和列

    int r = i / COLS;
    int c = i % COLS;

例如

public class Prova {
    private static final int ROWS = 6;
    private static final int COLS = 7;

    // ....

            lblCircles[i].addMouseListener(new MouseAdapter() {
                // better to use mousePressed, not mouseClicked
                public void mousePressed(MouseEvent e) {

                    // no magic numbers such as 42 please. 
                    for (int i = 0; i < lblCircles.length; i++) {
                        if (e.getSource() == lblCircles[i] && choose == 1) {
                            myLabelClicked(lblCircles[i], i);
                        }
                    }
                }
            });

public void myLabelClicked(JLabel label, int i) {
    int r = i / COLS;
    int c = i % COLS;
    if (n == 0 && "vuota".equals(label.getName())) {
        n = 1;
        label.setText("ok1");
        label.setForeground(Color.GREEN);
        label.setName("piena");
        tbl[r][c] = 1;
    } else {
        n = 0;
        label.setText("ok2");
        label.setForeground(Color.GREEN);
        label.setName("piena");
        tbl[r][c] = 2;
    }
    System.out.println("tbl[" + r + "][" + c + "] = " + tbl[r][c]);
}

附带问题:

  • 是的,使用布局管理器比使用 setBounds 好得多,因此您的第二段代码更好
  • 避免使用诸如 42 之类的魔数(Magic Number),而是使用属性和常量。
  • 在我上面的代码中,ROWS = 6 和 COLS = 7

我的 MCVE:

import java.awt.Color;
import java.awt.Font;
import java.awt.GridLayout;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;

import javax.swing.*;

@SuppressWarnings("serial")
public class Prova2 extends JPanel {
    private static final int ROWS = 6;
    private static final int COLS = 7;
    private static final int LBL_EB = 25; // "eb" for empty border
    private static final int PNL_EB = 3;
    public static final Font BIG = new Font("Comic Sans MS", Font.BOLD, 18);
    private static final Color BACKGROUND = new Color(41, 41, 41);
    private static final String VUOTA = "vuota";
    private JLabel[] lblCircles = new JLabel[ROWS * COLS];
    private Integer[][] tbl = new Integer[ROWS][COLS];
    private int choose = 1, n = 0;

    public Prova2() {
        setBorder(BorderFactory.createEmptyBorder(PNL_EB, PNL_EB, PNL_EB, PNL_EB));
        setLayout(new GridLayout(ROWS, COLS));
        setBackground(BACKGROUND);
        for (int i = 0; i < lblCircles.length; i++) {
            lblCircles[i] = new JLabel("test");
            lblCircles[i].setForeground(Color.RED);
            lblCircles[i].setFont(BIG);
            lblCircles[i].setBorder(BorderFactory.createEmptyBorder(LBL_EB, LBL_EB, LBL_EB, LBL_EB));
            lblCircles[i].setName(VUOTA);
            lblCircles[i].addMouseListener(new MouseAdapter() {
                public void mousePressed(MouseEvent e) {
                    for (int i = 0; i < lblCircles.length; i++) {
                        if (e.getSource() == lblCircles[i] && choose == 1) {
                            myLabelClicked(lblCircles[i], i);
                        }
                    }
                }
            });
            add(lblCircles[i]);
        }
    }

    protected void myLabelClicked(JLabel label, int i) {
        int row = i / COLS;
        int col = i % COLS;
        if (!VUOTA.equals(label.getName())) {
            return;
        }
        if (n == 0) {
            n = 1;
            label.setText("ok1");
            tbl[row][col] = 1;
        } else {
            n = 0;
            label.setText("ok2");
            tbl[row][col] = 2;
        }
        label.setForeground(Color.GREEN);
        label.setName("piena");
        System.out.printf("tbl[%d][%d] = %d%n", row, col, tbl[row][col]);
    }

    private static void createAndShowGui() {
        Prova2 mainPanel = new Prova2();

        JFrame frame = new JFrame("Prova 2");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.getContentPane().add(mainPanel);
        frame.pack();
        frame.setLocationRelativeTo(null);
        frame.setVisible(true);
    }

    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> createAndShowGui());
    }
}

关于java - 通过Java中的方法传递变量,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/51329993/

相关文章:

Java/Swing : Desktop. open() 导致 GTK-ERROR

c - C语言中的大整数?

c++ - 无法在动态链接库中找到过程入口点 _ZNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEC1Ev

java - 在 Java 中查找用户主目录的最佳方法是什么?

java - 编辑 java jdk 的替代项时权限被拒绝

java - Junit 测试模拟文件操作

c - 如何计算局部变量和用户定义函数的循环执行次数? C语言

java - ejb有关角色和身份验证的安全性问题

java - 转换对象时 event.getItem() 和comboBox.getSelectedItem() 之间有什么区别

java - JProgressBar 未按预期工作