刚毕业的我上来就犯错了····

27 阅读1分钟

今天在工作中犯了一个错误,原因是我要修改内部留言和外部留言为留言,相当于把两个字段改成一个字段。于是我开始读代码,发现内部留言和外部留言都是request.getParameter传过来的,我一看这就是前端传过来的啊,并且还要删除一块留言的逻辑,所以直接把内部留言(appInAdivce)和外部留言(appOutAdvice)都删除了改成了留言(leaveMessage),因为controller中调用service方法中需要传这两个参数,于是我直接把传参改了,进入service方法中发现还有很多调用公司其他系统的接口也需要传这两个参数,于是我全都改了,并且测试了一下。嗯~确实没有什么问题,于是开心的提了MR,我的leader给评审了一下,给我一顿说,我在service中改的方法中可能会影响其他业务,我没有排查出来。。。哎,实在太难受了。于是我要重新写这个需求,发现把这两个参数合并为一个参数必然会改service的传参,就算让我现在重写我也会像上一次那么写一遍。。但是leader说的也对,我改的时候根本没有意识到改传参对其他业务的影响。