2022年第一个线上问题，被领导骂惨了！-技术圈

昨天，一位朋友说他们生产环境出bug了，哎，年底了，今年的年终奖估计凉凉了！

这位朋友刚刚出道不久，出了问题，第一时间找的我。

他把出问题的代码发我了，叫我帮他review这段代码。

for (ProductOrderItem orderItem : orderItemList) {
    if (orderItem.getProductOrderItem_user().getUser_id() != user.getUser_id()) {
       logger.warn("用户订单项与用户不匹配，回到购物车页");
       return "redirect:/cart";
    }
    if (orderItem.getProductOrderItem_order() != null) {
        logger.warn("用户订单项不属于购物车，回到购物车页");
        return "redirect:/cart";
    }
}

看这段代码，不是很简单吗？

我只能盲猜了

是不是两个user_id不一样，数据有问题导致的？
你这user_id是Integer类型？
或者是orderItem.getProductOrderItem_order()不等null？

我让他debug看看，过一会，他说奇了怪了，并甩给我一个截图：

看到最后这张截图，就不用想了，在判断两个user_id不相等的地方，我猜测IDEA编辑器已经提示他了。两边都是Integer，属于int的包装类，也就是两个Integer对象，你用！=去比较，这不是扯淡么？

况且，人家IDEA编辑器会这么提示的：

都明确提示使用equals()来做对象比较，假装没见，这锅，没人替你背了！

使用IDEA编辑器开发的朋友，田哥建议你多看看IDEA给你的代码提示

问题来源

暂且不讨论他这代码写的很让人不爽，怎么有的驼峰，有的下划线？难道你们项目就没有管这个吗？让你瞎写，想干啥就干啥。我也问了，这位小伙伴是在一家小公司，在搞一个电商项目，由于急着上线，所以也没顾忌到那么多。

在此，也希望大家养成一个良好习惯，不要再这么编码，对团队负责也是对自己负责！

这位小伙伴出的问题，根本原因在于使用!=没有使用equals()导致的。

故事还没结算

就算使用！=，在本地测试和测试环境，也都没问题！

自己本地测试，用户数量就几十个，测试环境也估计只有几十个，但是线上就不止几十个了，于是就这个bug就顺利暴露出来了！

最后，其实测试的小伙伴这会也要跟着背锅！

请看下面这段代码，将输出什么结果？

public class Test {
    public static void main(String[] args) {
        Integer aa=100;
        Integer bb=100;
        System.out.println(aa==bb);
        Integer cc=128;
        Integer dd=128;
        System.out.println(cc==dd);
        Integer ee=129;
        Integer ff=129;
        System.out.println(ee==ff);
    }
}

这又聊到了Integer的知识盲区，因为Integer对-128~127做了缓存。

关于Integer的相关内容，之前已跟大家分享给了。

吊打面试官系列：说说Integer缓存范围

于是，上面的那段代码输出如下：

true
false
false

后记

我们在对象比较的时候，一定记得使用equals来比较（有点开发经验的朋友都知道），但刚出道的朋友犯着错也是可以理解的，也希望还不太清楚的朋友，往后别再给自己挖类似的坑了。

吃一堑长一智！

也希望这位朋友也别灰心，抽时间多补补基础。

有道无术，术可成；有术无道，止于术

欢迎大家关注Java之道公众号

好文章，我在看❤️