记录在java中使用返回值为Integer的函数遇到的坑

今天在写leetcode时遇到一道题始终过不了,经过排查,问题出现在下面这个方法:

pop()和peek()的返回值都是Integer类型。我在调试时发现,当pop()和peek()返回值明明相等时,if居然判false,导致我的minStack元素没有成功出栈,后面我换了一个写法,想仔细看看pop()返回值到底是什么,如下:

结果,居然通过了!!!

这两个写法的结果应该一样才对啊,为啥第二种写法的逻辑是正确的?

我编写了一个测试类,如下:

public class TestCall {
    public static void main(String[] args) {
        A a = new A();
        B b = new B();

        if (a.test1() == b.test1()) {
            System.out.println("1=1");
        }

        if (a.test2() == b.test2()) {
            System.out.println("127=127");
        }

        if (a.test3() == b.test3()) {
            System.out.println("12889=12889");
        }
    }
}

class A {
    public Integer test1() {
        return 1;
    }

    public Integer test2() {
        return 127;
    }

    public Integer test3() {
        return 12889;
    }
}

class B {
    public Integer test1() {
        return 1;
    }
    public Integer test2() {
        return 127;
    }

    public Integer test3() {
        return 12889;
    }
}

只有1=1、127=127输出,第三个正是我测试用例失败的那个数,为啥只有前面两个可以成功输出呢?

先说结论,这其实是Integer的特性及java语法导致的问题。

Integer源码注释里面写了如下这段话:

上面说的是Integer会自动缓存-128~127这些数值的对象,而java的"=="号,如果比较的两边都是引用类型,实际比较的其实是两边的对象地址,当值大于128时,Integer会创建新的对象,所以即使两个Integer对象的值相等,”==“比较的结果也会是false,因为它们是两个对象,这就是12889=12889不能输出的原因。1=1、127=127能输出则是因为缓存,"=="两边的Integer是同一个对象。

回到主题,之所以第二种写法能够运行正常,其实是因为java中的自动拆箱机制,这里不深入讨论,以Integer为例子,只要当二元操作符两边有一个操作数为int,那另一个Integer操作数会被自动拆箱为int值,所以第二种写法中的if中比较的是值,不是对象,因此逻辑正确。

如果遇到相同问题,希望这篇文章能帮到你!

  • 7
    点赞
  • 7
    收藏
    觉得还不错? 一键收藏
  • 1
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 1
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值