463 Star 7.3K Fork 3.1K

众邦科技 / CRMEB_Java电商系统

 / 详情

建议重构SmsServiceImpl类及其相关的代码

进行中
缺陷
创建于  
2021-07-14 10:22

建议1:
统一使用:onePassUtil.postFrom(String url, MultiValueMap<String, Object> param, Map<String, String> header);
可删除内部方法:
private JSONObject checkResult(String result)
private JSONObject postFrom(String url, MultiValueMap<String, Object> param, Map<String, String> header)

建议2:
短信发送服务和短信管理服务高度耦合,建议从接口SmsService提取公共短信发送接口,与一号通管理相关的接口区分开。
现在我有一个需求就是直接使用其他短信服务商的服务,不使用一号通的接口发送短信,只能自己封装一套接口。

建议3:
建议合并SmsServiceImpl类的方法
private Boolean commonSendSms(SendSmsVo sendSmsVo)和public Boolean sendCode(SendSmsVo sendSmsVo)
两个方法区别就是一个调用sendSmsVo.getParam和sendSmsVo.getContent。
我认为可以使用可以增加判空逻辑,选择字段赋值可以合并,例如:
Map<String, Object> map = (Map<String, Object>) JSONObject.parseObject(StringUtils.isEmpty(sendSmsVo.getParam())?sendSmsVo.getContent():sendSmsVo.getParam());

评论 (2)

anzangun 创建了任务
anzangun 修改了描述
展开全部操作日志

优秀,感谢大佬

大粽子 任务状态待办的 修改为进行中
聆听 任务类型任务 修改为缺陷
聆听 负责人设置为大粽子

登录 后才可以发表评论

状态
负责人
项目
里程碑
Pull Requests
关联的 Pull Requests 被合并后可能会关闭此 issue
分支
开始日期   -   截止日期
-
置顶选项
优先级
预计工期 (小时)
参与者(3)
105979 stivepeim 1607409551 855495 killking 1587379819
Java
1
https://gitee.com/ZhongBangKeJi/crmeb_java.git
git@gitee.com:ZhongBangKeJi/crmeb_java.git
ZhongBangKeJi
crmeb_java
CRMEB_Java电商系统

搜索帮助

344bd9b3 5694891 D2dac590 5694891