Question

java多线程同步小程序，请高手看看问题出在哪

这个程序一个线程画圆，一个线程画矩形，我用了synchronized这个关键字想实现两个线程同步运行，可是结果两个线程一起运行了，我是新手，请帮忙看看，哪儿有问题，如何改是好，谢谢了！

程序如下：

import java.awt.*;
import javax.swing.*;

class MyThread extends Thread
{
Container fp;
int flag = 0;

public MyThread(Container fp, int flag)
{
this.fp = fp;
this.flag = flag;
}

public synchronized void draw()
{
int x,y,w,h,r;
Color c;
//Graphics2D类扩展 Graphics类，提供对几何形状、坐标转换、颜色管理和文本布局更为复杂的控制
Graphics2D g = (Graphics2D)fp.getGraphics(); //调用 getGraphics()来创建Graphics
w = fp.getWidth(); //取出画面大小
h = fp.getHeight();
x = (int)(Math.random()*w); //计算随即位置
y = (int)(Math.random()*h);
r = (int)(Math.random()*(w > h ? w : h)/3) + 10;
c = new Color((int)(Math.random()*256),(int)(Math.random()*256),(int)(Math.random()*256));
g.setColor(c);
if(flag == 0)
g.fillOval(x, y, r, r); //画圆
else
g.fillRect(x, y, r, r); //画正方形
try
{
Thread.sleep(1000);
}
catch(InterruptedException e){}
}

public void run()
{
while(true)
{
draw();
}
}
}

public class RandomDraw extends JFrame
{
public RandomDraw()
{
this.getContentPane().setBackground(Color.white); //获得内容面板并设置背景颜色
this.setTitle("随即画图");
this.setBounds(400, 300, 400, 300);
this.setVisible(true);
this.setDefaultCloseOperation(EXIT_ON_CLOSE);
}

public static void main(String[] args)
{
RandomDraw mainFrame = new RandomDraw();
MyThread th1 = new MyThread(mainFrame.getContentPane(), 0);
MyThread th2 = new MyThread(mainFrame.getContentPane(), 1);
th1.start();
th2.start();
}
}

请Java能人帮忙看看，困了好久了

Answer 1

你这程序就是同步运行吧 不知道你说的是什么意思 其实程序是在同步运行 画完一个再画另外一个 只不过切换的速度很快 而且sleep的时间是一样的 所以你看不出来是同步运行的 吧等待代码改为Thread.sleep((int)(Math.random()*10000));再看一下
同步这个概念应该是在判断与执行之间存在线程安全问题才产生的 你这个程序用不用同步根本没有区别的
而且你用extends Thread的类时不可以用synchronized同步方法，因为当你创建新的线程时会new新对象 而同步锁就不一样了

Answer 2

mark
个人觉得你使用sleep(1000)并不能把线程锁死啊，等时间到了，线程自己就获取到锁执行了

Answer 3

程序没有任何问题，是你的电脑太快了，看上去像一起画的。我给正方形加上3秒延时，效果就很明显了

import java.awt.*;
import javax.swing.*;

class MyThread extends Thread {
Container fp;

int flag = 0;

public MyThread(Container fp, int flag) {
this.fp = fp;
this.flag = flag;
}

public synchronized void draw() {
int x, y, w, h, r;
Color c;
// Graphics2D类扩展 Graphics类，提供对几何形状、坐标转换、颜色管理和文本布局更为复杂的控制
Graphics2D g = (Graphics2D) fp.getGraphics(); // 调用
// getGraphics()来创建Graphics
w = fp.getWidth(); // 取出画面大小
h = fp.getHeight();
x = (int) (Math.random() * w); // 计算随即位置
y = (int) (Math.random() * h);
r = (int) (Math.random() * (w > h ? w : h) / 3) + 10;
c = new Color((int) (Math.random() * 256), (int) (Math.random() * 256),
(int) (Math.random() * 256));
g.setColor(c);
if (flag == 0) {
g.fillOval(x, y, r, r); // 画圆
} else {
try {
Thread.sleep(3000);
} catch (InterruptedException e) {
}

g.fillRect(x, y, r, r); // 画正方形
}
try {
Thread.sleep(1000);
} catch (InterruptedException e) {
}
}

public void run() {
while (true) {
draw();
}
}
}

public class RandomDraw extends JFrame {
public RandomDraw() {
this.getContentPane().setBackground(Color.white); // 获得内容面板并设置背景颜色
this.setTitle("随即画图");
this.setBounds(400, 300, 400, 300);
this.setVisible(true);
this.setDefaultCloseOperation(EXIT_ON_CLOSE);
}

public static void main(String[] args) {
RandomDraw mainFrame = new RandomDraw();
MyThread th1 = new MyThread(mainFrame.getContentPane(), 0);
MyThread th2 = new MyThread(mainFrame.getContentPane(), 1);
th1.start();
th2.start();
}
}