一、会议时间
-
二、会议议题
用户端代码评审
运营后端代码评审三、会议主持人
王海泳@水手 (王海泳)
四、会议参会人(请@所有参会人员)
@向诚 (王俊利)@水手 (王海泳)@道长 (乔杰)@冬兵 (张福宁)@未来 (刘纪昌)
五、纪要抄送人
六、会议结论(请写明会议讨论的问题及结论,若问题无结论或未达成共识请标注)
CampusUserServiceImpl99行GlobalTransactional 分布式事务需要增加rollbackFor108行String userid = RpcContext.getServiceContext().getAttachment("userid"); 需要把“userid” 配置成常量181行BeanUtils.copyProperties(userDTO,userSaveOrUpdateRequestDTO); 不建议使用 BeanUtilsfix1:代码行数超过一屏,应该按照逻辑拆分成方法,进行调用;
fix2:入参校验,需要配置拦截,统一放入dto中,区分商户和用户
OrderServiceImpl业务处理中,保证mysql和redis的数据一致性,在某一个发生异常时,需要进行同步回滚或者异步补偿处理
系统中Long类型改成BigDecimal
413行 日期类型处理 采用系统中统一的日期工具类
216行 过多的条件判断,可以抽成方法
259行 集合长度不要使用size(),使用Collections工具类
GoodsApipayTime 类型 使用时间类型,不要使用字符串
模糊查询,sql会全表扫描
112行 异常信息不能使用e.printStackTrace();
fix1:测试环境mq配置和生产环境不一致,需要写2套代码,建议统一配置
fix2:建议增加aop拦截接口响应耗时
fix3:建议增加http请求,rpc请求,DB执行耗时监控
fix4:数据库类型,统一使用20,8精度,java使用BigDecimal七、其他说明或附件