java编的单机五子棋，来看一下为什么有这种错误

Question

源码如下：
import java.awt.*;
import java.awt.event.*;
import java.applet.Applet;
import java.awt.Color;
public class GoBang extends Applet implements ActionListener, MouseListener,
  MouseMotionListener, ItemListener {
 int color = 0;// 旗子的颜色标识 0:白子 1:黑子 boolean isStart = false;// 游戏开始标志 int bodyArray[][] = new int[16][16]; // 设置棋盘棋子状态 0 无子 1 白子 2 黑子 Button b1 = new Button("游戏开始"); Button b2 = new Button("重置游戏"); Label lblWin = new Label(" "); Checkbox ckbHB[] = new Checkbox[2]; CheckboxGroup ckgHB = new CheckboxGroup();    @Override
 public void init() {
  setLayout(null);  addMouseListener(this);
  add(b1);
  b1.setBounds(20, 320, 80, 30);
  b1.addActionListener(this);
  add(b2);
  b2.setBounds(120, 320, 80, 30);
  b2.addActionListener(this);
  ckbHB[0] = new Checkbox("白子先", ckgHB, false);
  ckbHB[0].setBounds(320, 20, 60, 30);
  ckbHB[1] = new Checkbox("黑子先", ckgHB, false);
  ckbHB[1].setBounds(380, 20, 60, 30);
  add(ckbHB[0]);
  add(ckbHB[1]);
  ckbHB[0].addItemListener(this);
  ckbHB[1].addItemListener(this);
  add(lblWin);
  lblWin.setBounds(330, 130, 80, 30);  gameInit();
  this.resize(new Dimension(450,350));
 } public void itemStateChanged(ItemEvent e) {
  if (ckbHB[0].getState()) // 选择黑子先还是白子先
  {
   color = 0;
  } else {
   color= 1;
  }
 } public void actionPerformed(ActionEvent e) {
  if (e.getSource() == b1) {
   gameStart();
  } else {
   reStart();
  }
 } public void mousePressed(MouseEvent e) {
 } public void mouseClicked(MouseEvent e) {
  int x1, y1;
  x1 = e.getX();
  y1 = e.getY();
  if (e.getX() < 20 || e.getX() > 300 || e.getY() < 20 || e.getY() > 300) {
   return;
  }  if (x1 % 20 > 10) {
   x1 += 20;
  }  if (y1 % 20 > 10) {
   y1 += 20;
  }  x1 = x1 / 20 * 20;
  y1 = y1 / 20 * 20;
  setDown(x1, y1); } public void mouseEntered(MouseEvent e) {
 } public void mouseExited(MouseEvent e) {
 } public void mouseReleased(MouseEvent e) {
 } public void mouseDragged(MouseEvent e) {
 } public void mouseMoved(MouseEvent e) {
 }    @Override
 public void paint(Graphics g) {
  g.setColor(Color.pink);
  g.fill3DRect(10, 10, 300, 300, true);
  g.setColor(Color.black);
  for (int i = 1; i < 16; i++) {
   g.drawLine(20, 20 * i, 300, 20 * i);
   g.drawLine(20 * i, 20, 20 * i, 300);
  }
 } public void setDown(int x, int y) // 落子
 {
  if (!isStart) // 判断游戏未开始
  {
   return;
  }  if (bodyArray[x / 20][y / 20] != 0) {
   return;
  }
  Graphics g = getGraphics();  if (color == 1)// 判断黑子还是白子
  {
   g.setColor(Color.black);
   color = 0;
  } else {
   g.setColor(Color.white);
   color = 1;
  }  g.fillOval(x - 10, y - 10, 20, 20);  bodyArray[x / 20][y / 20] = color +1;  if (gameWin1(x / 20, y / 20)) // 判断输赢
  {
   lblWin.setText(startColor(color) + "赢了!");
   isStart = false;
  }  if (gameWin2(x / 20, y / 20)) // 判断输赢
  {
   lblWin.setText(startColor(color) + "赢了!");
   isStart = false;
  }  if (gameWin3(x / 20, y / 20)) // 判断输赢
  {
   lblWin.setText(startColor(color) + "赢了!");
   isStart = false;
  }  if (gameWin4(x / 20, y / 20)) // 判断输赢
  {
   lblWin.setText(startColor(color) + "赢了!");
   isStart = false;
  }
 } public String startColor(int x) {
  if (x == 0) {
   return "黑子";
  } else {
   return "白子";
  }
 } public void gameStart() // 游戏开始
 {
  isStart = true;
  enableGame(false);
  b2.setEnabled(true);
 } public void gameInit() // 游戏开始初始化
 {
  isStart = false;
  enableGame(true);
  b2.setEnabled(false);
  ckbHB[0].setState(true);  for (int i = 0; i < 16; i++) {
   for (int j = 0; j < 16; j++) {
    bodyArray[i][j] = 0;
   }
  }
  lblWin.setText("");
 } public void reStart() // 游戏重新开始
 {
  repaint();
  gameInit();
 } public void enableGame(boolean e) // 设置组件状态
 {
  b1.setEnabled(e);
  b2.setEnabled(e);
  ckbHB[0].setEnabled(e);
  ckbHB[1].setEnabled(e);
 } public boolean gameWin1(int x, int y) // 判断输赢 横
 {
  int x1, y1, t = 1;
  x1 = x;
  y1 = y;  for (int i = 1; i < 5; i++) {
   if (x1 > 15||x1<1) {
    break;
   }
   if (bodyArray[x1 + i][y1] == bodyArray[x][y]) {
    t += 1;
   } else {
    break;
   }  }  for (int i = 1; i < 5; i++) {
   if (x1 < 1||x1>15) {
    break;
   }   if (bodyArray[x1 - i][y1] == bodyArray[x][y]) {
    t += 1;
   } else {
    break;
   }
  }  if (t > 4) {
   return true;
  } else {
   return false;
  }
 } public boolean gameWin2(int x, int y) // 判断输赢 竖
 {
  int x1, y1, t = 1;
  x1 = x;
  y1 = y;  for (int i = 1; i < 5; i++) {
   if (x1 > 14||x1<1) {
    break;
   }
   if (bodyArray[x1][y1 + i] == bodyArray[x][y]) {
    t += 1;
   } else {
    break;
   }  }  for (int i = 1; i < 5; i++) {
   if (x1 < 1||x1>14) {
    break;
   }   if (bodyArray[x1][y1 - i] == bodyArray[x][y]) {
    t += 1;
   } else {
    break;
   }
  }  if (t > 4) {
   return true;
  } else {
   return false;
  }
 } public boolean gameWin3(int x, int y) // 判断输赢 左斜
 {
  int x1, y1, t = 1;
  x1 = x;
  y1 = y;  for (int i = 1; i < 5; i++) {
   if (x1 > 14||x1<1) {
    break;
   }
   if (bodyArray[x1 + i][y1 - i] == bodyArray[x][y]) {
    t += 1;
   } else {
    break;
   }  }  for (int i = 1; i < 5; i++) {
   if (x1 < 1||x1>14) {
    break;
   }   if (bodyArray[x1 - i][y1 + i] == bodyArray[x][y]) {
    t += 1;
   } else {
    break;
   }
  }  if (t > 4) {
   return true;
  } else {
   return false;
  }
 } public boolean gameWin4(int x, int y) // 判断输赢 左斜
 {
  int x1, y1, t = 1;
  x1 = x;
  y1 = y;  for (int i = 1; i < 5; i++) {
   if (x1 > 14||x1<1) {
    break;
   }
   if (bodyArray[x1 + i][y1 + i] == bodyArray[x][y]) {
    t += 1;
   } else {
    break;
   }  }  for (int i = 1; i < 5; i++) {
   if (x1 < 1||x1>14) {
    break;
   }   if (bodyArray[x1 - i][y1 - i] == bodyArray[x][y]) {
    t += 1;
   } else {
    break;
   }
  }  if (t > 4) {
   return true;
  } else {
   return false;
  }
 }
}

Answer 1

public boolean gameWin1(int x, int y) // 判断输赢 横
中 你判断是这样写的：
  if (x<1 || x>15) {
  break;
  }
这个应该是对的（1-15）竖条另外3个函数gameWin2、gameWin3、gameWin4
中判断你这样写的
  if (x1 > 14||x1<1) {
  break;
  }
这样的话只判断了（1-14）竖条
最右边竖排的都给你排除了
在右边落子肯定赢不了嘛

Answer 2

数组越界吧。
把判断输赢那几段程序中，bodyArray的下标中有加i的那几个if条件加上些限定，让x1+i（或y1+i）不能大于15.或者把bodyArray的大小设成17应该也行。我再试试。
public boolean gameWin1(int x, int y) // 判断输赢 横
 {
  int x1, y1, t = 1;
  x1 = x;
  y1 = y;  for (int i = 1; i < 5; i++) {
  if (x1 > 15||x1<1) { 
  break;
  }
  if (x1 + i <= 15 && 
  bodyArray[x1 + i][y1] == bodyArray[x][y]) {
  t += 1;
  } else {
  break;
  }  }  for (int i = 1; i < 5; i++) {
  if (x1 < 1||x1 > 15) { 
  break;
  }  if (bodyArray[x1 - i][y1] == bodyArray[x][y]) {
  t += 1;
  } else {
  break;
  }
  }
  
  System.out.println(x1+ "," + y1);  if (t > 4) {
  return true;
  } else {
  return false;
  }
 } public boolean gameWin2(int x, int y) // 判断输赢 竖
 {
  int x1, y1, t = 1;
  x1 = x;
  y1 = y;  for (int i = 1; i < 5; i++) {
  if (x1 > 15||x1<1) {
  break;
  }
  if (y1 + i <=15 && 
  bodyArray[x1][y1 + i] == bodyArray[x][y]) {
  t += 1;
  } else {
  break;
  }  }  for (int i = 1; i < 5; i++) {
  if (x1 < 1||x1>15) {
  break;
  }  if (bodyArray[x1][y1 - i] == bodyArray[x][y]) {
  t += 1;
  } else {
  break;
  }
  }  if (t > 4) {
  return true;
  } else {
  return false;
  }
 } public boolean gameWin3(int x, int y) // 判断输赢 右斜
 {
  int x1, y1, t = 1;
  x1 = x;
  y1 = y;  for (int i = 1; i < 5; i++) {
  if (x1 > 14||x1<1) {
  break;
  }
  if (x1 + i<=15  && 
  bodyArray[x1 + i][y1 - i] == bodyArray[x][y]) {
  t += 1;
  } else {
  break;
  }  }  for (int i = 1; i < 5; i++) {
  if (x1 < 1||x1>14) {
  break;
  }  if (bodyArray[x1 - i][y1 + i] == bodyArray[x][y]) {
  t += 1;
  } else {
  break;
  }
  }  if (t > 4) {
  return true;
  } else {
  return false;
  }
 } public boolean gameWin4(int x, int y) // 判断输赢 左斜
 {
  int x1, y1, t = 1;
  x1 = x;
  y1 = y;  for (int i = 1; i < 5; i++) {
  if (x1 > 14||x1<1) {
  break;
  }
  if (x1 + i<=15  && y1 + i<=15  && 
  bodyArray[x1 + i][y1 + i] == bodyArray[x][y]) {
  t += 1;
  } else {
  break;
  }  }  for (int i = 1; i < 5; i++) {
  if (x1 < 1||x1>14) {
  break;
  }  if (bodyArray[x1 - i][y1 - i] == bodyArray[x][y]) {
  t += 1;
  } else {
  break;
  }
  }  if (t > 4) {
  return true;
  } else {
  return false;
  }
 }
}

Answer 3

上面太麻烦了，把bodyArray的行和列都设成17，这样等于在棋盘四周裹上一层永远为0的壳，当碰到这个“壳”时，就break。
还要把所有的14改成15，不怕越界。
一点见解，不知合适不。呵呵。

Answer 4

其实也没什么，就是x1你用的是不动的，而上面这些for循环中的中止判断都没有加偏移。其他小问题也有。
整个判断输赢的四个方法调整如下： public boolean gameWin1(int x, int y) // 判断输赢 横
{
int count = 1;
for (int i = 1; i < 5; i++) { //正向
if (x+i > 15 ) break; //到边缘了
if (bodyArray[x+i][y] == bodyArray[x][y]) count++;
else break;
}
for (int i = 1; i < 5; i++) { //逆向
if (x-i < 0 ) break;
if (bodyArray[x-i][y] == bodyArray[x][y]) count++;
else break;
}
if (count > 4) {
return true;
} else {
return false;
}
} public boolean gameWin2(int x, int y) // 判断输赢 竖
{
int count = 1;
for (int i = 1; i < 5; i++) {
if (y+i > 15 ) break;
if (bodyArray[x][y+i] == bodyArray[x][y]) {
count += 1;
} else {
break;
}
}
for (int i = 1; i < 5; i++) {
if (y-i < 0) break;
if (bodyArray[x][y - i] == bodyArray[x][y]) {
count += 1;
} else {
break;
}
}
if (count > 4) {
return true;
} else {
return false;
}
} public boolean gameWin3(int x, int y) // 判断输赢 左斜
{
int t = 1;
for (int i = 1; i < 5; i++) {
if (x+i > 15 || y-i < 0) break;
if (bodyArray[x + i][y - i] == bodyArray[x][y]) {
t += 1;
} else {
break;
}
}
for (int i = 1; i < 5; i++) {
if (x-i < 0 || y+i > 15) break;
if (bodyArray[x - i][y + i] == bodyArray[x][y]) {
t += 1;
} else {
break;
}
}
if (t > 4) {
return true;
} else {
return false;
}
} public boolean gameWin4(int x, int y) // 判断输赢 左斜
{
int t = 1;
for (int i = 1; i < 5; i++) {
if (x+i > 15 || y+i > 15) break;
if (bodyArray[x + i][y + i] == bodyArray[x][y]) {
t += 1;
} else {
break;
}
}
for (int i = 1; i < 5; i++) {
if (x-i < 0 || y-i < 0) break;
if (bodyArray[x - i][y - i] == bodyArray[x][y]) {
t += 1;
} else {
break;
}
}
if (t > 4) {
return true;
} else {
return false;
}
}