🎨 规范lock.lock()代码，统一放在try代码块之外

arbing · 谢炳庭 · web-flow · commit 3b8c66a7e155 · 2020-07-17T09:27:19.000+08:00
Co-authored-by: 谢炳庭 &lt;xiebingting@xiongmaocar.com&gt;
diff --git a/weixin-java-miniapp/src/main/java/cn/binarywang/wx/miniapp/api/impl/WxMaJsapiServiceImpl.java b/weixin-java-miniapp/src/main/java/cn/binarywang/wx/miniapp/api/impl/WxMaJsapiServiceImpl.java
@@ -33,8 +33,8 @@ public String getCardApiTicket() throws WxErrorException {
   @Override
   public String getCardApiTicket(boolean forceRefresh) throws WxErrorException {
     Lock lock = this.wxMaService.getWxMaConfig().getCardApiTicketLock();
+    lock.lock();
     try {
-      lock.lock();
       if (forceRefresh) {
         this.wxMaService.getWxMaConfig().expireCardApiTicket();
       }
@@ -60,8 +60,8 @@ public String getJsapiTicket() throws WxErrorException {
   @Override
   public String getJsapiTicket(boolean forceRefresh) throws WxErrorException {
     Lock lock = this.wxMaService.getWxMaConfig().getJsapiTicketLock();
+    lock.lock();
     try {
-      lock.lock();
       if (forceRefresh) {
         this.wxMaService.getWxMaConfig().expireJsapiTicket();
       }
diff --git a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/BaseWxMpServiceImpl.java b/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/BaseWxMpServiceImpl.java
@@ -101,8 +101,8 @@ public String getTicket(TicketType type) throws WxErrorException {
   @Override
   public String getTicket(TicketType type, boolean forceRefresh) throws WxErrorException {
     Lock lock = this.getWxMpConfigStorage().getTicketLock(type);
+    lock.lock();
     try {
-      lock.lock();
       if (forceRefresh) {
         this.getWxMpConfigStorage().expireTicket(type);
       }
diff --git a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/WxMpCardServiceImpl.java b/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/WxMpCardServiceImpl.java
@@ -49,8 +49,8 @@ public String getCardApiTicket() throws WxErrorException {
   public String getCardApiTicket(boolean forceRefresh) throws WxErrorException {
     final TicketType type = TicketType.WX_CARD;
     Lock lock = getWxMpService().getWxMpConfigStorage().getTicketLock(type);
+    lock.lock();
     try {
-      lock.lock();
 
       if (forceRefresh) {
         this.getWxMpService().getWxMpConfigStorage().expireTicket(type);
