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的相关内容,之前已跟大家分享给了。4
吊打面试官系列:说说Integer缓存范围
于是,上面的那段代码输出如下:
true
false
false
后记
我们在对象比较的时候,一定记得使用equals来比较(有点开发经验的朋友都知道),但刚出道的朋友犯着错也是可以理解的,也希望还不太清楚的朋友,往后别再给自己挖类似的坑了。
吃一堑长一智!
也希望这位朋友也别灰心,抽时间多补补基础。
福利
我之前搞过一本《面试小抄》后台回复77
即可获取.
再多说几句
很多人以为自己背背八股文,改改简历,然后就直接去面试了。其实,真正用心准备面试的人不是这样的。既然决定面试,那就还是得花点心思在上面。
还有就是面试中,遇到问题不会怎么办?面试后面如何反问面试官?......
不打无准备的仗
自己没有准备好,就开始出去面试,会导致哪些问题?
- 机会浪费
- 浪费时间
- 丧失自信
浪费机会
:很多面试机会可能是朋友介绍的,或者是自己意向的公司,但是你没准备好就去面试,很有可能这种机会就
浪费了。
浪费时间
:很多人都是靠着面试过几家公司后,才找到面试的感觉。比如说:面试三家才找到面试的感觉,就相当于你请假三天。就算按照每个公司半天,最少也得花一天半时间。
丧失自信
:由于自己没有准备好,面试中受挫也是在所难免的,被面试官怼的怀疑人生,怀疑自己是否适合干这一行。
另外,大家都知道,田哥现在是全职做面试辅导和技术分享。
10年码农,为你面试保驾护航!
面试辅导
主要内容:
- 提供面试资料,精简面试八股文,核心知识总结。
- 车轮战
模拟面试
,(面试-->总结问题-->再面试-->再总结-->再面试...找个最优状态
) - 美化简历,很多人的简历那真是烂得一塌糊涂,投出去必定沉默大海。
如果工作后,遇到什么技术问题、职场问题都随时可以找我一起探讨。