我遇到的一个BUG(一)
作天一个群里的朋友发了一段难以调试的代码,由于好奇心再加上那么一点点的上进心,我决定自己试试.虽然用了大半天的时
间,但却学到了看一天书都学不到的东西.我想这就是BUG的魅力吧!
首先,程序所要实现的功能是利用鼠标画线,由于不能用repaint()重绘所以看似简单的程序并没有想象的那么容易.
代码如下:
import java.awt.*;
import java.awt.event.*;
import java.util.*;
public class DrawLine extends Frame {
ArrayList al=new ArrayList();
Iterator it=al.iterator();
public DrawLine() {
addWindowListener(new WindowAdapter() {
public void windowClosing(WindowEvent e) {
Window w;
w=e.getWindow();
w.setVisible(false);
w.dispose();
System.exit(0);
}
});
addMouseListener(new MouseAdapter() {
int x1=0,y1=0,x2=0,y2=0;
public void mousePressed(MouseEvent e) {
x1=e.getX();
y1=e.getY();
}
public void mouseReleased(MouseEvent e) {
x2=e.getX();
y2=e.getY();
Graphics g=DrawLine.this.getGraphics();
g.setColor(Color.green);
g.setFont(new Font("sadfa",Font.BOLD+Font.ITALIC,50));
g.drawString("woaini", x2, y2);
g.drawLine(x1, y1, x2, y2);
al.add(new Point1(x1,y1,x2,y2));
}
});
}
public void paint(Graphics g) {
while(it.hasNext()) {
try {
Point1 pt=(Point1)it.next();
pt.drawme(g);
}
catch(Exception e){}
}
}
public static void main(String[] args) {
DrawLine dl=new DrawLine();
dl.setSize(500,600 );
dl.setLocation(30, 40);
dl.setVisible(true);
}
}
public class Point1 {
int x1,x2,y1,y2;
public Point1(int x1,int y1,int x2,int y2) {
this.x1=x1;
this.x2=x2;
this.y1=y1;
this.y2=y2;
}
public void drawme(Graphics g) {
g.drawLine(x1, y1, x2, y2);
}
}
遇到的问题可以总结为:1.绘图后无法关闭窗口,产生"堵塞线程"且不抛出异常.2,最小化后还原窗口后图形被清除.
第一个问题总结如下:
1.捕获所有异常要用try{}catch(Exception e){e.printStackTrace();}首先,e.printStackTrace()是必须的,无法再控制台上
看到异常.其次,try子句中不能包含方法,即只能再方法中捕获异常.
2.throw Exception是否也可以捕获所有异常以及如何捕获,现在还不知道,以后添加.
3.关键问题,抛出异常如下:
java.util.ConcurrentModificationException
at java.util.AbstractList$Itr.checkForComodification(Unknown Source)
at java.util.AbstractList$Itr.next(Unknown Source)
at DrawLine.paint(DrawLine.java:55)
at sun.awt.RepaintArea.paintComponent(Unknown Source)
at sun.awt.RepaintArea.paint(Unknown Source)
at sun.awt.windows.WComponentPeer.handleEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForHierarchy(Unknown Source)
通过查阅JDK,解释如下:
public class ConcurrentModificationExceptionextends RuntimeException当方法检测到对象的并发修改,但不允许这
种修改时,抛出此异常。
例如,某个线程在 Collection 上进行迭代时,通常不允许另一个线性修改该 Collection。通常在这些情况下,迭代的结果是不确
定的。如果检测到这种行为,一些迭代器实现(包括 JRE 提供的所有通用 collection 实现)可能选择抛出此异常。执行该操作的
迭代器称为快速失败 迭代器,因为迭代器很快就完全失败,而不会冒着在将来某个时间任意发生不确定行为的风险。
注意,此异常不会始终指出对象已经由不同 线程并发修改。如果单线程发出违反对象协定的方法调用序列,则该对象可能抛出此异
常。例如,如果线程使用快速失败迭代器在 collection 上迭代时直接修改该 collection,则迭代器将抛出此异常。
其实我并不明白这是什么意思,我的解释是通过al.add(new Point1(x1,y1,x2,y2))在主线程上修改迭代器.而while
(it.hasNext())在作图线程中进行迭代所以会造成异常.所义将Iterator it=al.iterator();移进paint()函数中竟然可以解决第一
个问题.但是在百度上搜到一个高手是这么说的:
"Iterator 是工作在一个独立的线程中,并且拥有一个 mutex 锁。 Iterator 被创建之后会建立一个指向原来对象的单链索
引表,当原来的对象数量发生变化时,这个索引表的内容不会同步改变,所以当索引指针往后移动的时候就找不到要迭代的对象,
所以按照 fail-fast 原则 Iterator 会马上抛出 java.util.ConcurrentModificationException 异常。
所以 Iterator 在工作的时候是不允许被迭代的对象被改变的。但你可以使用 Iterator 本身的方法 remove() 来删除对象,
Iterator.remove() 方法会在删除当前迭代对象的同时维护索引的一致性。
有意思的是如果你的 Collection / Map 对象实际只有一个元素的时候, ConcurrentModificationException 异常并不会被抛出。
这也就是为什么在 javadoc 里面指出: it would be wrong to write a program that depended on this exception for its
correctness: ConcurrentModificationException should be used only to detect bugs.修改后: it.remove();"
不过我现在是要添加而不是删除,所以我想最好用另外一种思路来重新考虑这个程序.
第二个问题总结如下:
简单的说就是把g.setColor(Color.green);g.setFont(new Font("sadfa",Font.BOLD+Font.ITALIC,50));g.drawString
("woaini", x2, y2);加到drawne()中去,具体原理是什么,我想还是跟线程有关,关于作图线程的问题我没有深入涉及过,有待今后解
决吧.
与其说是总结还不如说是记录,有的问题等解决后我会重新编辑这篇文章.如果哪位高手碰巧看到,还请留言指教一二!