Question

我正在尝试制作一个熄灯游戏，我将其简化为一次仅更换一个磁贴，但我仍然遇到一个错误。我可以点击从黄色变为黑色但我无法点击并发生相反的情况。如果你不熟悉，这就是游戏：http://www.logicgamesonline.com/lightsout/

我的代码：

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

public class LightsOutPanel extends JPanel implements MouseListener {
    private boolean[][] lights;

    public static void main(String[] args) throws Exception {
        JFrame frame = new JFrame();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setTitle("Lights Out!");
        frame.setResizable(false);
        frame.setVisible(true);

        LightsOutPanel panel = new LightsOutPanel();
        if (panel.lights == null) {
            System.out.println("You did not initialize your light array!" +
                               "It's still null...");
            System.exit(-1);
        }
        panel.addMouseListener(panel);
        panel.setPreferredSize(new Dimension(601, 501));
        panel.setMinimumSize(new Dimension(601, 501));

        Container c = frame.getContentPane();
        c.setLayout(new BorderLayout());
        c.add(panel, BorderLayout.CENTER);

        frame.pack();
    }

    public LightsOutPanel() {
        lights = new boolean[5][6];
        for (int i = 0; i < 5; i++) {
            for (int j = 0; j < 6; j++) {
                lights[i][j] = true;
            }
        }

    }

    // unused methods
    public void mouseClicked(MouseEvent e) {}

    public void mouseReleased(MouseEvent e) {}

    public void mouseEntered(MouseEvent e) {}

    public void mouseExited(MouseEvent e) {}

    public void paint(Graphics g) {
        int boxWidth = 600 / 6;
        int boxHeight = 500 / 5;

        int y = 0;
        for (int row = 0; row < 5; row++) {
            int x = 0;
            for (int col = 0; col < 6; col++) {
                if (lights[row][col]==true) {
                    g.setColor(Color.YELLOW);
                } else {
                    g.setColor(Color.BLACK);
                }
                g.fillRect(x, y, boxWidth, boxHeight);

                g.setColor(Color.BLUE);
                g.drawRect(x, y, boxWidth, boxHeight);
                x += boxWidth;
            }
            y += boxHeight;
        }
    }

    // called when the mouse is pressed - determines what row/column the user
    // has clicked
    public void mousePressed(MouseEvent e) {
        int mouseX = e.getX();
        int mouseY = e.getY();

        int panelWidth = getWidth();
        int panelHeight = getHeight();

        int boxWidth = panelWidth / lights[0].length;
        int boxHeight = panelHeight / lights.length;

        int col = mouseX / boxWidth;
        int row = mouseY / boxHeight;

        toggle(row, col);
        repaint();
    }

    // called to "toggle" the selected row and column, as well as the four
    // adjacent lights
    public void toggle(int row, int col) {

        if (row >= 0 && col >= 0 && row < lights.length && 
            col < lights[0].length)
        {
            if (lights[row][col] = true) {
                lights[row][col] = false;
            } else {
                lights[row][col] = true;
            }
        }

    }
}

Answer 1

你的核心＆＃34;问题出在你的toggle方法......

if (lights[row][col] = true) {

=是一项任务，因此您说true分配给lights[row][col]，如果lights[row][col] == true，请执行以下操作......所以它始终是真的。

一种更简单的方法就是......

lights[row][col] = !lights[row][col];

不，如果是，或者其他，很简单。

你还应该看看Painting in AWT and Swing和 Performing Custom Painting。您违反了绘画方法链合同（不会调用super.paint）

而不是覆盖paint，您应该使用paintComponent并且在进行任何自定义绘制之前应该调用super.paintComponent，这将确保Graphics上下文是为你准备好了

使用2d阵列点亮游戏

1 个答案: