我遇到了这个奇怪的问题，但我似乎无法解决这个问题。 我有一个按钮，上面写着＆＃34;新游戏＆＃34;。我玩了一次游戏，然后按下＃34;新游戏＆＃34;回到initGame（），我遇到了一个问题，因为我不能右键点击上一个游戏的地雷。我可以左键单击没有问题，因此我迷路了。
（p.s。如果你想建议我如何缩短/简化我的代码，也可以随意这样做！）
请建议，这是我的代码的一部分（其他一切正常）：
private void initGame() {
minesFlagged=10;
tfFlag.setText(minesFlagged + "");
CellMouseListener listener = new CellMouseListener();
// Reset cells, mines, and flags
for (int row = 0; row < ROWS; row++) {
for (int col = 0; col < COLS; col++) {
// Set all cells to un-revealed
btnCells[row][col].setEnabled(true); // enable button
btnCells[row][col].setForeground(FGCOLOR_NOT_REVEALED);
btnCells[row][col].setBackground(BGCOLOR_NOT_REVEALED);
btnCells[row][col].setFont(FONT_NUMBERS);
btnCells[row][col].setText(""); // display blank
mines[row][col] = false; // clear all the mines
flags[row][col] = false; // clear all the flags
zeroesVisit[row][col] = false;
btnCells[row][col].setIcon(null);
btnCells[row][col].addMouseListener(listener);
}
}
for (int row = 0; row <= ROWS; row++) {
for (int col = 0; col <= COLS; col++) {
sol[row][col]=-1;
}
}
// Set the number of mines and the mines' location
numMines = 10; //rmb the array starts from zero and ends at index 9
for(int i=0;i<numMines;i++){
Random rand = new Random();
//get random position for next mine
int row = rand.nextInt(ROWS);
int col = rand.nextInt(COLS);
while(mines[row][col]||mines[0][0]){//if this position is a mine
//we get new position
row = rand.nextInt(ROWS);
col = rand.nextInt(COLS);
}
mines[row][col]=true;
}
}
private void findSol(int rowSelected,int colSelected){
for (int rr = rowSelected - 1; rr<= rowSelected + 1; rr++){//finding how many surrounding mines
for (int cc = colSelected - 1; cc <= colSelected + 1; cc++){
if(rr>=0 && cc>=0 && rr<ROWS && cc<COLS){
if (mines[rr][cc]){
sol[rowSelected][colSelected]++;
}
}
}
}
}
private class CellMouseListener extends MouseAdapter {
@Override
public void mouseClicked(MouseEvent e) {
// Determine the (row, col) of the JButton that triggered the event
int rowSelected = -1,colSelected = -1;
// Get the source object that fired the Event
JButton source = (JButton)e.getSource();
// Scan all rows and columns, and match with the source object
boolean found = false;
boolean done = true;
for (int row = 0; row < ROWS && !found; ++row) {
for (int col = 0; col < COLS && !found; ++col) {
if (source == btnCells[row][col]) {
rowSelected = row;
colSelected = col;
found = true; // break both inner/outer loops
}
}
}
// Left-click to reveal a cell; Right-click to plant/remove the flag.
if (e.getButton() == MouseEvent.BUTTON1 && (!flags[rowSelected][colSelected])) { // Left-button clicked
//If you hit a mine, game over - OKAY
//Otherwise, reveal the cell and display the number of surrounding mines - okay
if (mines[rowSelected][colSelected]){
btnCells[rowSelected][colSelected].setBackground(BGCOLOR_REVEALED);
imgMines = new ImageIcon(getClass().getResource("Bombicon.png"));
btnCells[rowSelected][colSelected].setIcon(imgMines);
JOptionPane.showMessageDialog(null, "Game Over! Try to complete the game~");
}else{
btnCells[rowSelected][colSelected].setBackground(FGCOLOR_REVEALED); //though foreground is text color
btnCells[rowSelected][colSelected].setIcon(null);
sol[rowSelected][colSelected]=0;
findSol(rowSelected,colSelected);
if(sol[rowSelected][colSelected]>0){
btnCells[rowSelected][colSelected].setText(""+sol[rowSelected][colSelected]);
btnCells[rowSelected][colSelected].removeMouseListener(this);
colorCell(rowSelected,colSelected);
}
else if(sol[rowSelected][colSelected]==0){
do{
done = true;
for(int r=0;r<ROWS;r++){//scanning through whole board
for(int c=0;c<COLS;c++){
if(sol[r][c]==0 && !zeroesVisit[r][c]){
zeroesVisit[r][c]=true;
for(int row=r-1;row<=r+1;row++){//open the 8 surrounding cells
for(int col=c-1;col<=c+1;col++){
if(row>=0 && col>=0 && row<ROWS && col<COLS){//make sure not out of bounds
btnCells[row][col].setBackground(FGCOLOR_REVEALED);
btnCells[row][col].setIcon(null);
sol[row][col]=0;
findSol(row,col);
btnCells[row][col].removeMouseListener(this);
if(sol[row][col]>0){
btnCells[row][col].setText(""+sol[row][col]);
colorCell(row,col);
}
else if(sol[row][col]==0){
btnCells[row][col].setText("");
done=false;//to prompt looping again
}
}
}
}
}
}
}
}while(!done);//scan again to find all/any the zero cells
}
}
}else if (e.getButton() == MouseEvent.BUTTON3) { // right-button clicked
//If the location is flagged, remove the flag
// Otherwise, plant a flag. - COMPLETED
if(flags[rowSelected][colSelected]){
flags[rowSelected][colSelected]=false;
btnCells[rowSelected][colSelected].setBackground(BGCOLOR_NOT_REVEALED);
btnCells[rowSelected][colSelected].setIcon(null);
minesFlagged++;
tfFlag.setText(minesFlagged + "");
}else{
flags[rowSelected][colSelected]=true;
btnCells[rowSelected][colSelected].setBackground(FGCOLOR_NOT_REVEALED);
imgFlags = new ImageIcon(getClass().getResource("whiteflag.png"));
btnCells[rowSelected][colSelected].setIcon(imgFlags);
minesFlagged--;
tfFlag.setText(minesFlagged + "");
}
}
//Check if the player has won, after revealing this cell - COMPLETED
int completeCount=0,flagCount=0;
for (int row = 0; row < ROWS; ++row) {
for (int col = 0; col < COLS; ++col) {
if(!mines[row][col] && btnCells[row][col].getBackground()==FGCOLOR_REVEALED){
completeCount++;
if((ROWS*COLS-numMines) == completeCount){
JOptionPane.showMessageDialog(null, "CONGRATULATIONS!");
}
}
}
}
}
}
答案 0 :(得分：1)
每次游戏初始化时，您似乎都在调用addMouseListener。这不会取代之前游戏中现有的MouseListener，因此您现在在按钮上有一个额外的侦听器，您没有调用removeMouseListener。如果某些按钮没有触发事件，我会先调查一下。
最初只在首次创建Frame时添加一次侦听器会更好。然后向mouseClicked事件方法添加逻辑，以忽略对按钮的点击，这些按钮被停用＆＃34;而不是一遍又一遍地删除和添加监听器。
如果你必须采用这种方法，我建议在initGame方法中添加监听器之前，从具有它们的任何按钮中删除所有MouseListener（可以使用方法getMouseListeners找到它们）。
正如ArielB所述，最佳做法是将您的模型（地雷的位置和瓷砖的状态等）分开到远离您的UI代码的另一个类。