首页 文章详情

2022年第一个线上问题,被领导骂惨了!

Java技术江湖 | 77 2022-01-15 21:08 0 0 0
UniSMS (合一短信)

hi 大家好,我是田哥

全职做面试辅导技术支持的田哥。

昨天,一位朋友说他们生产环境出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来比较(有点开发经验的朋友都知道),但刚出道的朋友犯着错也是可以理解的,也希望还不太清楚的朋友,往后别再给自己挖类似的坑了。

吃一堑长一智!

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

39岁阿里P9失业了,总资产1.5亿


100行代码透彻解析RPC原理


西安一码通到底是通过后端下发图片的吗?新笑话:CPU转速过高导致内存溢出?


关注公众号【Java技术江湖】后回复“PDF”即可领取200+页的《Java工程师面试指南》

强烈推荐,几乎涵盖所有Java工程师必知必会的知识点,不管是复习还是面试,都很实用。



good-icon 0
favorite-icon 0
收藏
回复数量: 0
    暂无评论~~
    Ctrl+Enter