java - 这段代码为什么不是线程同步的?
天蓬老师
天蓬老师 2017-04-17 16:29:56
[Java讨论组]

写了一个小程序,我觉应该是同步的,但结果不正确。程序是模拟银行账户的流水


public class Account {
    
    private int money = 60000;

    public int getMoney() {
        return money;
    }
    
    public void setMoney(int money) {
        this.money = money;
    }
    
    
}
public class Save implements Runnable{
    
    private Account account;
    
    public Save(Account account) {
        this.account=account;
    }
    @Override
    public void run() {
        
            for(int i=0;i<50000;i++){
                synchronized (account) {  //连续执行一定放入同步快中
                    account.setMoney(account.getMoney()+1);
//                    System.out.println("向账户存了1元,账户余额"+account.getMoney()+"元");
                }


            }

        
    }
}
public class Take implements Runnable{
    
    private Account account;
    
    public Take(Account account) {
        this.account=account;
    }
    @Override
    public void run() {
        
            for(int i=0;i<50000;i++){
                synchronized (account) {
                    if(account.getMoney() > 0){
                        account.setMoney(account.getMoney()-1);
//                        System.out.println("向账户取了1元,账户余额"+account.getMoney()+"元");
                    }else{
//                        System.out.println("账户余额不足");
                    }

                }

            }
        
        
    }
}
public class Main {

    public static void main(String[] args) {
        // TODO Auto-generated method stub
        Account account = new Account();
        
        Save save = new Save(account);
        Take take = new Take(account);
        
        Thread thread1 = new Thread(save);
        Thread thread2 = new Thread(take);
        
        System.out.println("账户余额为"+account.getMoney());
        System.out.println("开始");
        
        thread1.start();
        thread2.start();
        System.out.println("最后账户余额为:"+account.getMoney());
    }
    
}

存钱和取钱的钱数是一样的,所以总数应该还是60000,但结果不是这个数。是不是我Main线程先运行结束了?所以显示出来的“最后账户余额”不正确。

天蓬老师
天蓬老师

欢迎选择我的课程,让我们一起见证您的进步~~

全部回复(4)
怪我咯

根本原因在于,你打印余额时,其实另外两个存钱和取钱线程还没有执行完呢?你一定没注意这一点

PHP中文网

因为GetMoney/SetMoney组合起来不是原子的操作. Get获取的值可能是老的值. 如果只是+1的话, 可以用fetch and add; 如果非要设计成Get/Set, 那么得用compare and swap. 你可以自行Google一下, 这些概念Java都有相应的实现.

PS:
被钓鱼了.

伊谢尔伦

你这种情况下 money 用 AtomicInteger 来代替 Integer,AtomicInteger是线程安全的,而int不是

大家讲道理

主要有两个问题。
1、如@piano 所说的, 打印语句放错地方了,要放在子线程里才对。
2、还有一个更重要的问题,就是for循环要放在同步代码块里面,如下

            synchronized (account) 
            {for(int i=0;i<50000;i++)
               {……
               }
            }

如果把synchronized语句放到for语句里面(如同提问者那样的写法),就会造成每次循环都出现“加锁->修改->解锁”的情况,共50000次,在每次解锁后到再次对account加锁期间,就会出现account被其他线程修改的情况。

热门教程
更多>
最新下载
更多>
网站特效
网站源码
网站素材
前端模板
关于我们 免责申明 举报中心 意见反馈 讲师合作 广告合作 最新更新
php中文网:公益在线php培训,帮助PHP学习者快速成长!
关注服务号 技术交流群
PHP中文网订阅号
每天精选资源文章推送

Copyright 2014-2026 https://www.php.cn/ All Rights Reserved | php.cn | 湘ICP备2023035733号