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

[birthcat]

源码如下：
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;
}
}
}

[birthcat]

当在右边落子的时候，怎么有五个了，还是不能赢棋呢（在四个子中间落子）

[birthcat]

[bf234511171]

这种错是那种错啊！

[冥月天]

好像报错了。你再试试，

[zhouzh2006]

看不到图片

[sunyiz]

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）竖条
最右边竖排的都给你排除了
在右边落子肯定赢不了嘛

[birthcat]

谢谢了，我再试试啊

[tianyaofande]

哈哈，我最近也有写过一个五子棋的源码哦！

[birthcat]

大家帮我看看这个程序啥，急急也，我真的找不到什么错误，感谢大家的帮忙！！！

[tomorroweve]

数组越界吧。
把判断输赢那几段程序中，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;
}
}
}

[tomorroweve]

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

[magong]

[Quote=引用楼主 birthcat 的回复:]
if (x1 > 15||x1<1) {
[/Quote]
其实也没什么，就是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;
		}
	}