美文网首页
记一次低级bug引起的反思

记一次低级bug引起的反思

作者: 睡着别叫醒我 | 来源:发表于2019-03-17 02:33 被阅读0次
错了

最近,加入了一家跨境电商公司,从发文截止,也已有了17天,在最近开发的一个一元抢购活动中,因为自己的代码问题引发了一个错误,引发了公司亏损。为了惊醒自己不在放在重复的错,特地将这整个过程记了下来。

问题简述

一元抢购活动需要上线后,发现用户购物车内的商品都变成了1沙特币,而老版本的App购买时,是使用购物车的价钱来进行结算的。而App的购物流程都是先添加到购物车再进行下单的,导致用户购买时,所以物品单价都为1沙特币。

抢购Bug

北京时间四点四十六分上线后,在五点四十三分时,支付部门的同事发现了出错的代码,随后开始在群上粘贴出来并且询问是谁改的,这时候才发现代码出错了,而出错的原因,相当低级,把一元秒杀的价格返回没有放在正确的if判断中,导致执行这个方法的返回均为1沙特币,发现之后,随即进行了紧急修复并上线。整个过程,从发现到修复,花费了一个钟的时间。

问题总结

在要发布的当天,没有进行代码Review,跟其他部门的同事在同一分支下进行开发。在业务部门觉得没问题并要求我们发布后,没有进行代码Review,就由其他部门的同事进行发布了;

Git分支合并不合理,在发布前,同事将master分支的代码合并到了开发分支后,又将该分支合并到master中,导致在GitLab查看差异时,不能很形象的观察到出现问题的代码;

最重要的原因,还是自己不够严谨,对代码,特别是对涉及到金额,支付类的,没有足够的细心检查和敬畏之心,给同事和相关组长添加了麻烦,也给公司带来了损失;

反思

工作一年多以来,虽有改进,但一直存在着开发时不能足够细致专注的问题,也没有做好备忘录和开发思路记录的习惯,单纯靠记忆去记住自己修改了哪些;

对代码的不够严谨,在上线时,没有重新仔细的Review过自己的代码,把思路顺着修改代码在过一遍,对代码没有足够的敬畏和严谨;

事情已经发生,损失目前也还没有统计清楚,接下来能做的,只能是正确的面对接下来的问题,并且接受教训,改正自己的这一缺点,而写这篇文章,也是为了将自己的不足暴露出来,推动自我完善和成长。

相关文章

  • 记一次低级bug引起的反思

    最近,加入了一家跨境电商公司,从发文截止,也已有了17天,在最近开发的一个一元抢购活动中,因为自己的代码问题引发了...

  • 记一次由^引起的bug

    前言 后台管理系统使用的UI组件是element-ui,使用的版本是2.8.2。 问题 线上使用人员反馈,最近在使...

  • 记一次线上对外的一次低级BUG!

    今天老板一个需求,里面就去搞了,然后打包发布,很正常的流程。 然后启动tomcat的时候,启动完成突然报错 启动...

  • 记一次友盟引起的BUG

    友盟SDK有一个私有类UMAOCTools,在类方法setupHook中hook了UIViewController...

  • 记一次Maven的profile文件过滤引起的字符编码BUG

    本文记录了一次文件编码差异引起的profile替换占位符失败的bug,及处理思路。记录成文,以便以后反思,或让后来...

  • 05. getWriter()has already been

    记一次bug fixed: getWriter()has already been called for this...

  • 解决bug的思路

    遇到bug不要惊慌,惊慌失措会显得你很低级,冷静分析bug的原因;首先定位bug的位置,然后从代码的层面去分析问题...

  • rpc引起的bug

    今天算是又增强了自己对pomelo RPC的认识,因为一个bug。起初,我们游戏的角色信息是作为一个redis缓存...

  • getUserId引起的bug

    最近项目中在修改用户昵称的时候报了如下bug 这个问题很眼熟, 上次一个项目也报了。 所以这次记录一下。 原因是a...

  • 阶段总结3『腾讯大Bug引起的反思』

    〔1〕主动挖掘新功能,实现群功能最大利用化 提问:如果入群请求中姓名与结营通告中昵称不一致的话怎么办 @马琦琦哥 ...

网友评论

      本文标题:记一次低级bug引起的反思

      本文链接:https://www.haomeiwen.com/subject/hfkymqtx.html