diff --git a/mallinkAdmin/src/main/java/com/iformall/controller/CouponInjectController.java b/mallinkAdmin/src/main/java/com/iformall/controller/CouponInjectController.java index 3cbf0d339..b8143d028 100644 --- a/mallinkAdmin/src/main/java/com/iformall/controller/CouponInjectController.java +++ b/mallinkAdmin/src/main/java/com/iformall/controller/CouponInjectController.java @@ -91,7 +91,7 @@ public class CouponInjectController extends BaseController { CouponInject couponInject = couponInjectService.getById(id); if (couponInject != null) { List tagids = JSON.parseArray(couponInject.getTags(), Long.class); - if (tagids.size() > 0) { + if (!tagids.isEmpty()) { couponInject.setWxChooseTagVo(wxCUserTagsService.findChooseTag(getTenantId(), tagids)); } } diff --git a/mallinkAdmin/src/main/java/com/iformall/controller/WxCUserBasicInfoController.java b/mallinkAdmin/src/main/java/com/iformall/controller/WxCUserBasicInfoController.java index 3ffc258a8..0642188b2 100644 --- a/mallinkAdmin/src/main/java/com/iformall/controller/WxCUserBasicInfoController.java +++ b/mallinkAdmin/src/main/java/com/iformall/controller/WxCUserBasicInfoController.java @@ -143,7 +143,7 @@ public class WxCUserBasicInfoController extends BaseController { WxCUserTags uTag = wxCUserTagsService.getById(info.getTagId()); if (StringUtils.isNotBlank(uTag.getTags())) { List ids = JSONObject.parseArray(uTag.getTags(), Long.class); - if (ids.size() > 0) { + if (!ids.isEmpty()) { WxTags wxTags = new WxTags(); wxTags.setIds(ids); PageInfo page = wxTagsService.listAsPage(wxTags, 1, 5000); diff --git a/mallinkService/src/main/java/com/iformall/service/impl/WxCampaignServiceImpl.java b/mallinkService/src/main/java/com/iformall/service/impl/WxCampaignServiceImpl.java index 0a167fe54..f63223967 100644 --- a/mallinkService/src/main/java/com/iformall/service/impl/WxCampaignServiceImpl.java +++ b/mallinkService/src/main/java/com/iformall/service/impl/WxCampaignServiceImpl.java @@ -62,10 +62,7 @@ public class WxCampaignServiceImpl implements WxCampaignService { } record = wxCampaignMapper.selectByPrimaryKey(record); if (record.getStatus() == 0) { - List ids= JSONArray.parseArray(record.getCouponIds(),String.class); - if (ids.size() > 0) { - addOrUpdateBatch(ids, record); - } + addOrUpdateBatch(JSONArray.parseArray(record.getCouponIds(),String.class), record); } else { delBatch(record); }