关于#java#的问题：不是已经给线程加过synchronize的锁了为什么运行的结果还会出现安全问题

class showmaker implements Runnable{
    private static  int dk=100;
    Object obj=new Object();
    @Override

    public void run() {

         while(dk>0){
             synchronized (obj){
             try {
                 Thread.sleep(1000);
             } catch (InterruptedException e) {
                 e.printStackTrace();
             }
             System.out.println(Thread.currentThread().getName()+":"+dk);
             dk--;

         }
         }
    }
}

public class scout {
    public static void main(String[] args) {
        showmaker s=new showmaker();
        Thread t1=new Thread(s);
        Thread t2=new Thread(s);
        Thread t3=new Thread(s);
t1.setName("窗口1");t2.setName("窗口2");t3.setName("窗口3");
t1.start();
t2.start();
t3.start();

    }

}

不是已经给线程加过synchronize的锁了为什么运行的结果还会出现安全问题。

我觉得你的那个while(dk>0)应该放在同步块里面吧！

1. synchronize 块里面的代码没有问题，因为输出结果中，没有相同的数字，说明同步块里面的代码没有并行执行的现象。
2. 最终结果出现 -1 的情况，应该是 dk 变量在读取的时候，没有进行保护吧。

    class showmaker implements Runnable{
        private static  int dk=100;
        Object obj=new Object();
        @Override
        public void run() {
            int _dk;
            while (true){
                synchronized (obj){
                    _dk = dk--;
                }
                if (_dk <= 0) {
                    break;
                }
                try {
                    Thread.sleep(1000);
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
                System.out.println(Thread.currentThread().getName() + ":" + _dk);
            };
        }
    }

你的锁的对象不正确。showmaker类生成的三个对象各自有obj属性域。锁加在三个对象的obj上，并不能保证共享成员dk的安全性。
将synchronized (obj){ 改成 synchronized (showmaker.class){应该能达到你想看的效果。
另外，编程时请遵守规范。