From 0a61a0337d042a9fdb049c460bba296d1f60b9ec Mon Sep 17 00:00:00 2001
From: Binary Wang <binarywang@gmail.com>
Date: Sun, 23 Apr 2023 11:37:48 +0800
Subject: [PATCH] =?UTF-8?q?:art:=20=E4=BC=98=E5=8C=96=E4=BB=A3=E7=A0=81?=
 =?UTF-8?q?=EF=BC=8C=E6=8A=BD=E5=8F=96=E5=B8=B8=E9=87=8F?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../weixin/mp/api/impl/WxMpMenuServiceImpl.java   | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/WxMpMenuServiceImpl.java b/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/WxMpMenuServiceImpl.java
index 5631a44f7..7a3f4ac37 100644
--- a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/WxMpMenuServiceImpl.java
+++ b/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/WxMpMenuServiceImpl.java
@@ -22,6 +22,9 @@ import static me.chanjar.weixin.mp.enums.WxMpApiUrl.Menu.*;
 @Slf4j
 @RequiredArgsConstructor
 public class WxMpMenuServiceImpl implements WxMpMenuService {
+  private static final String MENU_ID = "menuid";
+  private static final String MATCH_RULE = "matchrule";
+
   private final WxMpService wxMpService;
 
   @Override
@@ -38,7 +41,7 @@ public class WxMpMenuServiceImpl implements WxMpMenuService {
     log.debug("创建菜单：{},结果：{}", menuJson, result);
 
     if (menu.getMatchRule() != null) {
-      return GsonParser.parse(result).get("menuid").getAsString();
+      return GsonParser.parse(result).get(MENU_ID).getAsString();
     }
 
     return null;
@@ -48,13 +51,13 @@ public class WxMpMenuServiceImpl implements WxMpMenuService {
   public String menuCreate(String json) throws WxErrorException {
     JsonObject jsonObject = GsonParser.parse(json);
     WxMpApiUrl.Menu url = MENU_CREATE;
-    if (jsonObject.get("matchrule") != null) {
+    if (jsonObject.get(MATCH_RULE) != null) {
       url = MENU_ADDCONDITIONAL;
     }
 
     String result = this.wxMpService.post(url, json);
-    if (jsonObject.get("matchrule") != null) {
-      return GsonParser.parse(result).get("menuid").getAsString();
+    if (jsonObject.get(MATCH_RULE) != null) {
+      return GsonParser.parse(result).get(MENU_ID).getAsString();
     }
 
     return null;
@@ -69,9 +72,9 @@ public class WxMpMenuServiceImpl implements WxMpMenuService {
   @Override
   public void menuDelete(String menuId) throws WxErrorException {
     JsonObject jsonObject = new JsonObject();
-    jsonObject.addProperty("menuid", menuId);
+    jsonObject.addProperty(MENU_ID, menuId);
     String result = this.wxMpService.post(MENU_DELCONDITIONAL, jsonObject.toString());
-    log.debug("根据MeunId({})删除个性化菜单结果：{}", menuId, result);
+    log.debug("根据MenuId({})删除个性化菜单结果：{}", menuId, result);
   }
 
   @Override