BigDecimal在Debug模式下诡异的行为

前言

同事在一次偶然的Debug过程中,发现了一件不可思议的事情,程序在Run和Debug模式下运行结果居然不一致!

问题复现

测试代码很简单,堪比Hello world。

public class Main {

    public static void main(String[] args) {
        BigDecimal a = new BigDecimal("123.455");
        System.out.println("结果是 -> " + a);
    }
}

这段程序会输出什么?
脑编译一波,输出结果结果是 -> 123.455
IDEA中Run模式跑一把儿,验证一下

image.png

一切看起来很合理,不过,我对BigDecimal解析字符串的过程有点兴趣,在构造函数上打个断点跟踪调试一下


image.png

现在以Debug模式启动,一路next后,看看控制台输出


image.png

诡异的事情发生了,结果居然是0。


image.png

问题研究

现在再加两行代码看看

public class Main {

    public static void main(String[] args) {
        BigDecimal a = new BigDecimal("123.455");
        System.out.println("结果是 -> " + a);
        BigDecimal b = a.add(BigDecimal.TEN);
        System.out.println("加10,结果是 -> " + b);
    }
}

Run模式启动


image.png

Debug模式启动


image.png

问题似乎变得更诡异了。。。

一波冷静思考(瞎折腾)后,貌似找对了方向。

接下来尝试对修改后程序Debug模式下的调试结果进行解释:

首先,我们来整理一下思路:

  • 控制台打印结果时,调用的是BigDecimal.toString()方法;
  • 打印出a错误的结果0
  • a + 10之后,将结果赋予b,打印出正确的结果133.45

这样看来,问题极有可能出在BigDecimal.toString()方法中!
OK,看看这段代码。

@Override
public String toString() {
    String sc = stringCache;
    if (sc == null)
        stringCache = sc = layoutChars(true);
    return sc;
}

很明显,这里做了一个缓存,真正的字符串生成在layoutChars(),进去看一下,过程还是有些复杂的,加一个缓存减少计算量的确是一个明智的设计。
那么这个诡异的现象就很好解释了,肯定是stringCache这个作为缓存的成员变量被污染了。
下面开始搜索stringCache变量被引用的位置,发现只有toString()方法这两处,那也就是说,debug模式下,过早的调用了toString()方法。因为我们把断点打在了构造函数上,这时BigDecimal对象还没有构造完成,调试器在展示对象值的时候,通过特殊方式调用了toString()方法,污染了stringCache缓存,也就导致了后续调用toString()方法时一直拿到的都是这个脏缓存。

image.png

PS:我把断点打在toString()方法里,发现并没有拦住,IDEA给了一句这样的提示:

image.png

扩展思考

toString()方法中使用缓存的我还是第一次见,平时我们写POJO时,IDE生成的代码都是直接拼接,为什么不也生成一个带缓存的方法?

问题的关键点是BigDecimal对象是不可变对象
不可变对象也就意味着对象一旦构造完成后,不可再改变。如果需要修改,原来的对象不变,生成一个新对象。

乍一看起来,这似乎是一个非常奢侈的举动,会浪费大量的内存。但带来的好处却也是巨大的:

  • 在Java中,所有的对象都是引用类型,在修改一个对象时,会影响该对象的所有持有者,导致一些难以排查的问题。
  • 不可变对象对函数式编程和并发编程是极其友好的,对象不可变意味着不用加锁,提高了性能,并降低了开发难度。
  • 不可变对象可以使用缓存技术提高性能,例如JVM针对Integer、String等类型会维护常量池。

我们再来分析一下BigDecimal.toString()方法使用的缓存设计。

  • 首先,BigDecimal是一个不可变对象,Java标准要求对象必须在构造完成后才能调用方法,因此在程序正常Run过程中,是没有任何问题的,但是debug过程采用的特殊方式提前调用了toString()方法,其实是破坏了这种约定,才导致的诡异现象发生。
  • 其次,这是一个懒加载缓存,只有在首次调用BigDecimal.toString()时,才触发stringCache的更新,在实际的使用中,我们并不总会将BigDecimal对象按字符串输出,这样也就不会触发stringCache的计算,降低性能开销。

Date类的设计一直饱受争议,其可变性就被认为是一个设计的失误。

比如如下代码

public class Person {
    private Date birth;

    public Date getBirth() {
        return birth;
    }

    public void setBirth(Date birth) {
        this.birth = birth;
    }

    public static void main(String[] args) {
        Person person = new Person();
        person.setBirth(new Date(2000 - 1900, 0, 1));
        System.out.println(person.getBirth());

        Date date = person.getBirth();
        date.setMonth(1);
        System.out.println(person.getBirth());
    }
}

这段代码的输出是

Sat Jan 01 00:00:00 CST 2000
Tue Feb 01 00:00:00 CST 2000

想象一下,如果person是从数据库里读取出来的一条记录,我们仅仅是想得到person生日那年的一个2月份时间,却在不经意间改变了person本身的birth属性,在某些ORM框架中,即使没有显示调用update操作,对person的修改也会保存到数据库,这种问题带来的影响是十分难以排查的。
但是如果Date是不可变类型,对于每一次的修改实际是生成了一个新对象,那么上述问题将不复存在。基于此,Java 8全新的Time API都是不可变的。

最后,我们看一下LocalDate.toString()源码

@Override
public String toString() {
    int yearValue = year;
    int monthValue = month;
    int dayValue = day;
    int absYear = Math.abs(yearValue);
    StringBuilder buf = new StringBuilder(10);
    if (absYear < 1000) {
        if (yearValue < 0) {
            buf.append(yearValue - 10000).deleteCharAt(1);
        } else {
            buf.append(yearValue + 10000).deleteCharAt(0);
        }
    } else {
        if (yearValue > 9999) {
            buf.append('+');
        }
        buf.append(yearValue);
    }
    return buf.append(monthValue < 10 ? "-0" : "-")
            .append(monthValue)
            .append(dayValue < 10 ? "-0" : "-")
            .append(dayValue)
            .toString();
}

这里貌似没有使用类似BigDecimal的缓存,不知道设计人员是不是也考虑到了BigDecimal在debug下可能会产生一些问题,亦或者是感觉缓存带来的性能优化是寥寥无几的。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 202,980评论 5 476
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 85,178评论 2 380
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 149,868评论 0 336
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,498评论 1 273
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,492评论 5 364
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,521评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 37,910评论 3 395
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,569评论 0 256
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 40,793评论 1 296
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,559评论 2 319
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,639评论 1 329
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,342评论 4 318
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 38,931评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,904评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,144评论 1 259
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 42,833评论 2 349
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,350评论 2 342

推荐阅读更多精彩内容

  • 周一晚上(2018.1.29)第一次清晰的看到宝宝自己搬凳子踩着凳子去拿桌上的苹果。就这样一天天长大咯。 ...
    美好彤行阅读 151评论 0 0
  • 很小的时候,每年春夏相交,碰上连雨天,村里的河套就会发一次大水,这个时候不用奔走相告,村里的小孩子们就都会齐齐奔向...
    山蓝水浅阅读 395评论 0 0
  • 七夕雨 是谁 在七夕的清晨 落下相思的泪 惹得尚未相会的牛郎织女 也不禁泪雨纷飞 这从天而落的情丝 打湿了地上有情...
    5780933168ec阅读 304评论 4 3