2022年第一个线上问题,被领导骂惨了!
Hollis
共 1781字,需浏览 4分钟
· 2022-01-14
昨天,一位朋友说他们生产环境出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编辑器
会这么提示的:使用IDEA编辑器开发的朋友,田哥建议你多看看IDEA给你的代码提示
问题来源
在此,也希望大家养成一个良好习惯,不要再这么编码,对团队负责也是对自己负责!
!=
没有使用equals()导致的。故事还没结算
!=
,在本地测试和测试环境,也都没问题!最后,其实测试的小伙伴这会也要跟着背锅!
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);
}
}
-128~127
做了缓存。true
false
false
后记
吃一堑长一智!
有道无术,术可成;有术无道,止于术
欢迎大家关注Java之道公众号
好文章,我在看❤️
评论
字节员工:35岁以后被裁员的,后来都走了哪条路?现在2-2,要不要利用最后一年拼命上个岸。
架构师大咖
架构师大咖,打造有价值的架构师交流平台。分享架构师干货、教程、课程、资讯。架构师大咖,每日推送。
公众号该公众号已被封禁在当今竞争激烈的职场环境中,年龄并不总是一个决定性
源码共读
0
你只是卡住了,你并没有被击垮
一旦思维僵化了,那就很难跟上这个真实世界的快节奏,更不可能自发地去发现自身问题,进而打破自己。思维僵化,会导致我们无法“活在当下,开放和接纳,并去做自己觉得重要的事情”。觉察自己思维僵化的特征,是改变的第一步。思维僵化导致了你的选择都是错误的。思维方式的不同,才是人跟人之间的不同。有的人遇到挫折了,
小Q聊产品
1
被开除了,因为用了Docker。。
将Python客栈设为“星标⭐”第一时间收到最新资讯来源丨经授权转自 古时的风筝(ID:gushidefengzheng)作者丨风筝前几天网上冲浪,看到一位技术管理者发了一个帖子。大意就是:“因为团队里一个程序员跟他反应说开发机内存不够了,请求换一台内存大一点的机器。这位管理者感到很诧异,为什么别人
Python客栈
1
字节面试:如何解决MQ消息积压问题?
面试题大全:www.javacn.siteMQ(Message Queue)消息积压问题指的是在消息队列中累积了大量未处理的消息,导致消息队列中的消息积压严重,超出系统处理能力,影响系统性能和稳定性的现象。1.消息积压是哪个环节的问题?MQ 执行有三大阶段:消息生产阶段。消息存储阶段。消息消费阶段。
Java中文社群
0
中科曙光董事长李国杰院士被证监会立案调查:涉嫌违法炒股
最近,中科曙光(603019.SH)的董事长李国杰成为舆论焦点。据2024年4月19日晚的公告显示,李国杰因涉嫌短线交易本公司股票而被证监会立案调查。李国杰,一位80岁的中国工程院院士,其职业生涯几乎涵盖了中国计算机科技领域的所有重要发展阶段。1943年5月出生的李国杰在湖南省邵阳市长大,早年毕业于
Python涨薪研究所
0
惹到雷军,智己惨了
作为上汽集团品牌向上的“双翼战略”,智己汽车与飞凡汽车的价格接连下探,被迫与小米汽车同台竞争。可在热度与销量上,三者相形见绌。2023年,智己和飞凡销量累计约6万台;2024年3月28日,小米汽车上市27分钟大定破5万。老实巴交的车二代眼馋营销高手小米的热度,于是做了一场照猫画虎的“致敬”。不成想却
亿欧网
0
42岁日本童颜女神二度离婚后,被曝与年下11岁帅哥演员热恋中!
女優の安達祐実の熱愛報道が話題だ。お相手は、俳優の鈴木勝大。09年、『JUNONスーパーボーイコンテスト』準グランプリで芸能界入りした鈴木は、12年にスーパー戦隊シリーズ『特命戦隊ゴーバスターズ』(テレビ朝日系)で主演を務め、その後はドラマや映画などで活躍する若手の人気俳優だ。安達と鈴木は昨年夏の舞
python教程
0
JetBrains 如何看待自己的软件在中国被频繁破解?
往期热门文章:1、程序员因Bug被要求归还4万多年终奖,网友:不还!2、一套万能通用的异步处理方案3、微服务全做错了!谷歌提出新方法,成本直接降9倍!4、IntelliJ IDEA 2024 首个大版本发布,好用到爆!5、Web 后端的一生之敌:分页器文章来源:【公众号:猿大侠】在知乎看到一个话题“
Java后端技术
0