错了
最近,加入了一家跨境电商公司，从发文截止，也已有了17天，在最近开发的一个一元抢购活动中，因为自己的代码问题引发了一个错误，引发了公司亏损。为了惊醒自己不在放在重复的错，特地将这整个过程记了下来。
问题简述
一元抢购活动需要上线后，发现用户购物车内的商品都变成了1沙特币，而老版本的App购买时，是使用购物车的价钱来进行结算的。而App的购物流程都是先添加到购物车再进行下单的，导致用户购买时，所以物品单价都为1沙特币。
抢购Bug
北京时间四点四十六分上线后，在五点四十三分时，支付部门的同事发现了出错的代码，随后开始在群上粘贴出来并且询问是谁改的，这时候才发现代码出错了，而出错的原因，相当低级，把一元秒杀的价格返回没有放在正确的if判断中，导致执行这个方法的返回均为1沙特币，发现之后，随即进行了紧急修复并上线。整个过程，从发现到修复，花费了一个钟的时间。
问题总结
在要发布的当天，没有进行代码Review，跟其他部门的同事在同一分支下进行开发。在业务部门觉得没问题并要求我们发布后，没有进行代码Review，就由其他部门的同事进行发布了；
Git分支合并不合理，在发布前，同事将master分支的代码合并到了开发分支后，又将该分支合并到master中，导致在GitLab查看差异时，不能很形象的观察到出现问题的代码；
最重要的原因，还是自己不够严谨，对代码，特别是对涉及到金额，支付类的，没有足够的细心检查和敬畏之心，给同事和相关组长添加了麻烦，也给公司带来了损失；
反思
工作一年多以来，虽有改进，但一直存在着开发时不能足够细致专注的问题，也没有做好备忘录和开发思路记录的习惯，单纯靠记忆去记住自己修改了哪些；
对代码的不够严谨，在上线时，没有重新仔细的Review过自己的代码，把思路顺着修改代码在过一遍，对代码没有足够的敬畏和严谨；
事情已经发生，损失目前也还没有统计清楚，接下来能做的，只能是正确的面对接下来的问题，并且接受教训，改正自己的这一缺点，而写这篇文章，也是为了将自己的不足暴露出来，推动自我完善和成长。
网友评论