效果是：鼠标按下以后，在鼠标移动过的轨迹上，画出一条线，这个线是用很多点组成的
但是我做出来运行以后发现，如果鼠标移动的慢点，画的线还挺好的，如果鼠标移动的快的话，就变成一个个点了
源码在下面，帮忙看下要怎么解决
import java.awt.*;
import java.util.*;
import java.awt.event.*;

public class GraphicsDemo {
        public static void main(String[] args) {
                new FrameFF("draw....");
        }
}

class FrameFF extends Frame {
        ArrayList<Point> points;                //声明容器，作用是存储点对象
        FrameFF(String s) {               
                super(s);
                points = new ArrayList<Point>();       
                setBounds(300,150,500,500);       
                setVisible(true);
               
                //添加鼠标监听
                addMouseMotionListener(new MouseMotionAdapter() {
                        public void mouseDragged(MouseEvent e) {
                                FrameFF f = (FrameFF)e.getSource();
                                Point p = new Point(e.getX(),e.getY());
                                f.addPoint(p);
                                f.repaint();
                        }                       
                });
               
                //窗口关闭
                addWindowListener(new WindowAdapter(){
                        public void windowClosing(WindowEvent e) {
                                System.exit(0);
                        }
                });
        }
       
        public void paint(Graphics g) {
                Color c = g.getColor();       
                for(Iterator<Point> it=points.iterator(); it.hasNext(); ) {
                        Point p1 = (Point)it.next();
                        g.setColor(new Color(123,44,82));
                        g.fillOval(p1.x, p1.y, 4, 4);               

                }
                g.setColor(c);
        }
       
        public void addPoint(Point p){
                points.add(p);
        }
}

问题已经解决

很棒，太谢谢你了！

余明辉 发表于 2012-8-1 22:15
鼠标抬起来的地方有个点，然后鼠标换个地方再按下又是另外一个点，这2个点之间有一条线啊，如果用画图工 ...

刚看到 确实是我疏忽了 可以这样改进： 加入MouseListener 监听鼠标放开
addMouseListener(new MouseAdapter(){
                    public void mouseReleased(MouseEvent e){
                                preX = -1 ;
                            preY = -1 ;
                    }
                });

陈潼 发表于 2012-8-1 22:09
不明白你想表达的是什么意思。这样不就是字鼠标按下后，经过的地方画出么？不是画图工具中的画笔么？
那 ...

鼠标抬起来的地方有个点，然后鼠标换个地方再按下又是另外一个点，这2个点之间有一条线啊，如果用画图工具来画，并不会这样啊

余明辉 发表于 2012-8-1 21:08
感谢你的回答，我运行了下，发现画出来的线都是连着的，这个差别有点大，我想要的效果是，就只在鼠标按下 ...

不明白你想表达的是什么意思。这样不就是字鼠标按下后，经过的地方画出么？不是画图工具中的画笔么？
那是什么？

陈潼 发表于 2012-8-1 09:17
改进后的代码如下：
import java.awt.Color;
import java.awt.Frame;

感谢你的回答，我运行了下，发现画出来的线都是连着的，这个差别有点大，我想要的效果是，就只在鼠标按下去经过的地方才画线条，今天调试了一下你的代码，暂时还没想出解决的方案

改进后的代码如下：
import java.awt.Color;
import java.awt.Frame;
import java.awt.Graphics;
import java.awt.event.MouseEvent;
import java.awt.event.MouseMotionAdapter;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;
import java.awt.image.BufferedImage;


public class GraphicsDemo {
        public static void main(String[] args) {
                new FrameFF("draw....");
        }
}

class FrameFF extends Frame {
                //frame大小
        private static final int AREA_LENGTH = 500 ;
        //记录先前位置。置为-1的原因请见MouseMotionListener
        private int preX = -1 ;
        private int preY = -1 ;
        //用做缓冲。。
        private BufferedImage image = new BufferedImage(AREA_LENGTH,AREA_LENGTH,BufferedImage.TYPE_INT_BGR);
        Graphics g = image.getGraphics();
        FrameFF(String name) {               
                super(name);      
                setBounds(300,150,AREA_LENGTH,AREA_LENGTH);        
                setVisible(true);
                //将底色置为白色，便于观看
                g.fillRect(0,0,AREA_LENGTH,AREA_LENGTH);
                //添加鼠标监听
                addMouseMotionListener(new MouseMotionAdapter() {
                        public void mouseDragged(MouseEvent e) {
                                //均大于0时肯定已经在内部了，且已经点击过一次，记录下了第一次的坐标
                                //所以这时划线已经没有问题了
                                        if(preX >0 && preY > 0 ){
                                                g.setColor(new Color(123,44,82));
                                                g.drawLine(preX, preY, e.getX(), e.getY());
                                        }
                                        preX = e.getX();
                                        preY = e.getY();
                                        //看你原先的代码还需要得到事件源，其实没有必要，因为是Frame
                                        //的内部，所以其属性是可以访问的，直接调用其方法即可
                                repaint();
                        }                        
                });
               
                //窗口关闭
                addWindowListener(new WindowAdapter(){
                        public void windowClosing(WindowEvent e) {
                                System.exit(0);
                        }
                });
        }   
        public void paint(Graphics g) {
                //整体绘制，免去每次集合遍历时间和重新绘制所有点时间。应该会剩下不少时间的，虽然感觉不到，呵呵
                g.drawImage(image,0,0,null);
        }
}
希望可以帮到你  

你好，这个问题我看了下。应该这么来做：
public void paint(Graphics g) {
                Color c = g.getColor();        
                for(Iterator<Point> it=points.iterator(); it.hasNext(); ) {
                        Point p1 = (Point)it.next();
                        g.setColor(new Color(123,44,82));
                        if(prePoint == null){
                                g.fillOval(p1.x, p1.y, 4, 4);
                        }
                        else{
                                g.drawLine(prePoint.x, prePoint.y, p1.x, p1.y);
                        }
                        prePoint = p1 ;

                }
                prePoint = null ;
                g.setColor(c);
        }
解释如下：鼠标拖动时，两次鼠标事件的触发需要一定的时间。而当鼠标拖动过快时，就会看到两点之间的距离--->你出现的这个问题。但是这个距离是很小的，所以通过将两点画上直线完成图形的绘制。由于距离极短，我们是无法分辨的。
关于代码改进意见：由于awt的每次都要绘制全部，所以应该说代码质量不是很高。建议使用BufferedImage类，创建一个和画图区相同大小的image，使用getGraphics得到Graphics，每次调用鼠标时绘制到此Graphics上，然后在组件的paint中一次性绘制，这样就不必使用集合了。  另外我觉得使用Swing中的JPanel还是比较省事的，因为缓冲已经做好了。而且没有闪屏现象。。。