From 345f41b1535d8ec7ead8b312259e70fbce028ff2 Mon Sep 17 00:00:00 2001 From: YunaiV Date: Mon, 29 Jul 2024 19:18:23 +0800 Subject: [PATCH] =?UTF-8?q?=E3=80=90=E4=BB=A3=E7=A0=81=E8=AF=84=E5=AE=A1?= =?UTF-8?q?=E3=80=91SYSTEM=EF=BC=9A=E8=AE=A2=E9=98=85=E6=B6=88=E6=81=AF?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../yudao/module/pay/enums/MessageTemplateConstants.java | 1 + .../module/pay/message/subscribe/SubscribeMessageClient.java | 1 + .../pay/service/wallet/PayWalletRechargeServiceImpl.java | 2 ++ .../yudao/module/system/api/social/SocialClientApi.java | 1 + .../yudao/module/system/api/social/SocialClientApiImpl.java | 3 +++ 5 files changed, 8 insertions(+) diff --git a/yudao-module-pay/yudao-module-pay-api/src/main/java/cn/iocoder/yudao/module/pay/enums/MessageTemplateConstants.java b/yudao-module-pay/yudao-module-pay-api/src/main/java/cn/iocoder/yudao/module/pay/enums/MessageTemplateConstants.java index ca4c948af..197b08ad5 100644 --- a/yudao-module-pay/yudao-module-pay-api/src/main/java/cn/iocoder/yudao/module/pay/enums/MessageTemplateConstants.java +++ b/yudao-module-pay/yudao-module-pay-api/src/main/java/cn/iocoder/yudao/module/pay/enums/MessageTemplateConstants.java @@ -11,6 +11,7 @@ public interface MessageTemplateConstants { String PAY_WALLET_CHANGE = "充值成功通知"; + // TODO @puhui999：这种建议不枚举，直接写~嘿嘿。 /** * 充值成功通知模版参数 * diff --git a/yudao-module-pay/yudao-module-pay-biz/src/main/java/cn/iocoder/yudao/module/pay/message/subscribe/SubscribeMessageClient.java b/yudao-module-pay/yudao-module-pay-biz/src/main/java/cn/iocoder/yudao/module/pay/message/subscribe/SubscribeMessageClient.java index cdb42a519..3ef26156d 100644 --- a/yudao-module-pay/yudao-module-pay-biz/src/main/java/cn/iocoder/yudao/module/pay/message/subscribe/SubscribeMessageClient.java +++ b/yudao-module-pay/yudao-module-pay-biz/src/main/java/cn/iocoder/yudao/module/pay/message/subscribe/SubscribeMessageClient.java @@ -11,6 +11,7 @@ import java.util.Map; import static cn.iocoder.yudao.module.pay.enums.MessageTemplateConstants.PAY_WALLET_CHANGE; +// TODO @puhui999：建议可以先直接调用，不要新建一个 client。 /** * 订阅消息 * diff --git a/yudao-module-pay/yudao-module-pay-biz/src/main/java/cn/iocoder/yudao/module/pay/service/wallet/PayWalletRechargeServiceImpl.java b/yudao-module-pay/yudao-module-pay-biz/src/main/java/cn/iocoder/yudao/module/pay/service/wallet/PayWalletRechargeServiceImpl.java index d801aa262..40050ac01 100644 --- a/yudao-module-pay/yudao-module-pay-biz/src/main/java/cn/iocoder/yudao/module/pay/service/wallet/PayWalletRechargeServiceImpl.java +++ b/yudao-module-pay/yudao-module-pay-biz/src/main/java/cn/iocoder/yudao/module/pay/service/wallet/PayWalletRechargeServiceImpl.java @@ -139,8 +139,10 @@ public class PayWalletRechargeServiceImpl implements PayWalletRechargeService { sendPayWalletChangeMessage(payOrderId, walletRecharge); } + // TODO @puhui999：发送，使用异步发送；@Async private void sendPayWalletChangeMessage(Long payOrderId, PayWalletRechargeDO walletRecharge) { PayWalletDO wallet = payWalletService.getWallet(walletRecharge.getWalletId()); + // TODO @puhui999：可以使用 MapUtil.builder()；另外，不应该是并发 hashmap 哈 Map messages = MapUtil.newConcurrentHashMap(4); messages.put(MessageTemplateConstants.PayWalletChangeTemplateParams.NO, String.valueOf(payOrderId)); messages.put(MessageTemplateConstants.PayWalletChangeTemplateParams.PRICE, diff --git a/yudao-module-system/yudao-module-system-api/src/main/java/cn/iocoder/yudao/module/system/api/social/SocialClientApi.java b/yudao-module-system/yudao-module-system-api/src/main/java/cn/iocoder/yudao/module/system/api/social/SocialClientApi.java index e91b136d1..72534768a 100644 --- a/yudao-module-system/yudao-module-system-api/src/main/java/cn/iocoder/yudao/module/system/api/social/SocialClientApi.java +++ b/yudao-module-system/yudao-module-system-api/src/main/java/cn/iocoder/yudao/module/system/api/social/SocialClientApi.java @@ -57,6 +57,7 @@ public interface SocialClientApi { */ List getSubscribeTemplateList(Integer userType); + // TODO @puhui999：sendSubscribeMessage 两个方法，可以融合成一个么？ /** * 发送微信小程序订阅消息 * diff --git a/yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/api/social/SocialClientApiImpl.java b/yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/api/social/SocialClientApiImpl.java index 2885f19eb..fe72d9706 100644 --- a/yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/api/social/SocialClientApiImpl.java +++ b/yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/api/social/SocialClientApiImpl.java @@ -75,6 +75,7 @@ public class SocialClientApiImpl implements SocialClientApi { public void sendSubscribeMessage(String templateTitle, Map messages, Integer userType, Long userId, Integer socialType, String path) { + // TODO @puhui999：建议是，先不拆小方法。因为逻辑的复杂度其实不高哈。合在一个方法里，因为咱写了 1.1 1.2 2. 这样的逻辑，也能一下子看懂。 // 1.1 获得订阅模版 SocialWxSubscribeTemplateRespDTO template = getTemplate(templateTitle, userType); if (template == null) { @@ -104,6 +105,7 @@ public class SocialClientApiImpl implements SocialClientApi { .setTemplateId(template.getId()).setToUser(openId).setPage(path); } + // TODO @puhui999：建议下沉到 service 实现。 /** * 获得小程序订阅消息模版 * @@ -120,6 +122,7 @@ public class SocialClientApiImpl implements SocialClientApi { return CollectionUtil.findOne(templateList, item -> ObjUtil.equal(item.getTitle(), templateTitle)); } + // TODO @puhui999：建议下沉到 service 实现。 /** * 获得用户 openId *