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

Posted 2022-02-05 Hollis Chuang

tags:

篇首语：本文由小常识网(cha138.com)小编为大家整理，主要介绍了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做了缓存。