1.优化
Showing
17 changed files
with
37 additions
and
32 deletions
... | @@ -20,7 +20,7 @@ import io.swagger.annotations.*; | ... | @@ -20,7 +20,7 @@ import io.swagger.annotations.*; |
20 | */ | 20 | */ |
21 | @Api(tags = "会员地址管理") | 21 | @Api(tags = "会员地址管理") |
22 | @RestController | 22 | @RestController |
23 | @RequestMapping("/ucEngine/memberAddress") | 23 | @RequestMapping("/ucEngine/api/memberAddress") |
24 | @CrossOrigin | 24 | @CrossOrigin |
25 | @Slf4j | 25 | @Slf4j |
26 | public class MemberAddressController { | 26 | public class MemberAddressController { | ... | ... |
1 | package com.topdraw.business.module.member.domain; | 1 | package com.topdraw.business.module.member.domain; |
2 | 2 | ||
3 | 3 | ||
4 | import com.topdraw.business.module.member.service.dto.MemberDTO; | ||
4 | import com.topdraw.util.IdWorker; | 5 | import com.topdraw.util.IdWorker; |
5 | import org.apache.commons.lang3.StringUtils; | 6 | import org.apache.commons.lang3.StringUtils; |
6 | 7 | ... | ... |
... | @@ -20,7 +20,7 @@ import io.swagger.annotations.*; | ... | @@ -20,7 +20,7 @@ import io.swagger.annotations.*; |
20 | */ | 20 | */ |
21 | @Api(tags = "会员属性管理") | 21 | @Api(tags = "会员属性管理") |
22 | @RestController | 22 | @RestController |
23 | @RequestMapping("/ucEngine/memberProfile") | 23 | @RequestMapping("/ucEngine/api/memberProfile") |
24 | @CrossOrigin | 24 | @CrossOrigin |
25 | @Slf4j | 25 | @Slf4j |
26 | public class MemberProfileController { | 26 | public class MemberProfileController { | ... | ... |
... | @@ -2,6 +2,7 @@ package com.topdraw.business.module.member.profile.service.impl; | ... | @@ -2,6 +2,7 @@ package com.topdraw.business.module.member.profile.service.impl; |
2 | 2 | ||
3 | import com.topdraw.aspect.AsyncMqSend; | 3 | import com.topdraw.aspect.AsyncMqSend; |
4 | import com.topdraw.business.module.member.domain.Member; | 4 | import com.topdraw.business.module.member.domain.Member; |
5 | import com.topdraw.business.module.member.domain.MemberBuilder; | ||
5 | import com.topdraw.business.module.member.profile.domain.MemberProfile; | 6 | import com.topdraw.business.module.member.profile.domain.MemberProfile; |
6 | import com.topdraw.business.module.member.profile.domain.MemberProfileBuilder; | 7 | import com.topdraw.business.module.member.profile.domain.MemberProfileBuilder; |
7 | import com.topdraw.business.module.member.service.MemberService; | 8 | import com.topdraw.business.module.member.service.MemberService; | ... | ... |
... | @@ -19,7 +19,7 @@ import io.swagger.annotations.*; | ... | @@ -19,7 +19,7 @@ import io.swagger.annotations.*; |
19 | */ | 19 | */ |
20 | @Api(tags = "相关人员管理") | 20 | @Api(tags = "相关人员管理") |
21 | @RestController | 21 | @RestController |
22 | @RequestMapping("/ucEngine/memberRelatedInfo") | 22 | @RequestMapping("/ucEngine/api/memberRelatedInfo") |
23 | @CrossOrigin | 23 | @CrossOrigin |
24 | @Slf4j | 24 | @Slf4j |
25 | public class MemberRelatedInfoController { | 25 | public class MemberRelatedInfoController { | ... | ... |
... | @@ -26,8 +26,7 @@ import java.util.Objects; | ... | @@ -26,8 +26,7 @@ import java.util.Objects; |
26 | */ | 26 | */ |
27 | @Api(tags = "会员管理") | 27 | @Api(tags = "会员管理") |
28 | @RestController | 28 | @RestController |
29 | //@RequestMapping("/ucEngine/api/member") | 29 | @RequestMapping("/ucEngine/api/member") |
30 | @RequestMapping("/ucEngine/member") | ||
31 | @CrossOrigin | 30 | @CrossOrigin |
32 | @Slf4j | 31 | @Slf4j |
33 | public class MemberController { | 32 | public class MemberController { | ... | ... |
... | @@ -16,7 +16,7 @@ import org.springframework.web.bind.annotation.*; | ... | @@ -16,7 +16,7 @@ import org.springframework.web.bind.annotation.*; |
16 | */ | 16 | */ |
17 | @Api(tags = "会员vip历史管理") | 17 | @Api(tags = "会员vip历史管理") |
18 | @RestController | 18 | @RestController |
19 | @RequestMapping("/ucEngine/memberVipHistory") | 19 | @RequestMapping("/ucEngine/api/memberVipHistory") |
20 | public class MemberVipHistoryController { | 20 | public class MemberVipHistoryController { |
21 | 21 | ||
22 | @Autowired | 22 | @Autowired | ... | ... |
1 | package com.topdraw.business.module.member.viphistory.service.impl; | 1 | package com.topdraw.business.module.member.viphistory.service.impl; |
2 | 2 | ||
3 | 3 | ||
4 | import com.topdraw.aspect.AsyncMqSend; | ||
4 | import com.topdraw.business.module.member.domain.Member; | 5 | import com.topdraw.business.module.member.domain.Member; |
5 | import com.topdraw.business.module.member.service.MemberService; | 6 | import com.topdraw.business.module.member.service.MemberService; |
6 | import com.topdraw.business.module.member.service.dto.MemberDTO; | 7 | import com.topdraw.business.module.member.service.dto.MemberDTO; |
... | @@ -49,6 +50,7 @@ public class MemberVipHistoryServiceImpl implements MemberVipHistoryService { | ... | @@ -49,6 +50,7 @@ public class MemberVipHistoryServiceImpl implements MemberVipHistoryService { |
49 | 50 | ||
50 | @Override | 51 | @Override |
51 | @Transactional(rollbackFor = Exception.class) | 52 | @Transactional(rollbackFor = Exception.class) |
53 | @AsyncMqSend | ||
52 | public void create(MemberVipHistory resources) { | 54 | public void create(MemberVipHistory resources) { |
53 | log.info("MemberVipHistoryServiceImpl ==>> MemberVipHistoryServiceImpl ==>> param ==>> [{}]",resources); | 55 | log.info("MemberVipHistoryServiceImpl ==>> MemberVipHistoryServiceImpl ==>> param ==>> [{}]",resources); |
54 | this.checkMember(resources); | 56 | this.checkMember(resources); | ... | ... |
... | @@ -34,7 +34,7 @@ public interface UserTvService { | ... | @@ -34,7 +34,7 @@ public interface UserTvService { |
34 | * | 34 | * |
35 | * @param resources | 35 | * @param resources |
36 | */ | 36 | */ |
37 | void unbindPriorityMemberCode(UserTv resources); | 37 | UserTvDTO unbindPriorityMemberCode(UserTv resources); |
38 | 38 | ||
39 | /** | 39 | /** |
40 | * | 40 | * | ... | ... |
... | @@ -104,20 +104,21 @@ public class UserTvServiceImpl implements UserTvService { | ... | @@ -104,20 +104,21 @@ public class UserTvServiceImpl implements UserTvService { |
104 | 104 | ||
105 | @Override | 105 | @Override |
106 | @Transactional(rollbackFor = Exception.class) | 106 | @Transactional(rollbackFor = Exception.class) |
107 | public void unbindPriorityMemberCode(UserTv resources) { | 107 | public UserTvDTO unbindPriorityMemberCode(UserTv resources) { |
108 | UserTv UserTv = this.userTvRepository.findById(resources.getId()).orElseGet(UserTv::new); | 108 | UserTv userTv = this.userTvRepository.findById(resources.getId()).orElseGet(UserTv::new); |
109 | ValidationUtil.isNull( UserTv.getId(),"UserTv","id",resources.getId()); | 109 | ValidationUtil.isNull( userTv.getId(),"UserTv","id",resources.getId()); |
110 | UserTv.copy(resources); | 110 | userTv.copy(resources); |
111 | this.userTvRepository.save(UserTv); | 111 | UserTv _userTv = this.userTvRepository.save(userTv); |
112 | return this.userTvMapper.toDto(_userTv); | ||
112 | } | 113 | } |
113 | 114 | ||
114 | @Override | 115 | @Override |
115 | @Transactional(rollbackFor = Exception.class) | 116 | @Transactional(rollbackFor = Exception.class) |
116 | public void delete(Long id) { | 117 | public void delete(Long id) { |
117 | Assert.notNull(id, "The given id must not be null!"); | 118 | Assert.notNull(id, "The given id must not be null!"); |
118 | UserTv UserTv = this.userTvRepository.findById(id).orElseThrow( | 119 | UserTv userTv = this.userTvRepository.findById(id).orElseThrow( |
119 | () -> new EmptyResultDataAccessException(String.format("No %s entity " + "with id %s " + "exists!", UserTv.class, id), 1)); | 120 | () -> new EmptyResultDataAccessException(String.format("No %s entity " + "with id %s " + "exists!", UserTv.class, id), 1)); |
120 | this.userTvRepository.delete(UserTv); | 121 | this.userTvRepository.delete(userTv); |
121 | } | 122 | } |
122 | 123 | ||
123 | @Override | 124 | @Override | ... | ... |
... | @@ -21,13 +21,13 @@ public interface UserWeixinService { | ... | @@ -21,13 +21,13 @@ public interface UserWeixinService { |
21 | * @param resources | 21 | * @param resources |
22 | * @return | 22 | * @return |
23 | */ | 23 | */ |
24 | UserWeixin create(UserWeixin resources); | 24 | UserWeixinDTO create(UserWeixin resources); |
25 | 25 | ||
26 | /** | 26 | /** |
27 | * | 27 | * |
28 | * @param resources | 28 | * @param resources |
29 | */ | 29 | */ |
30 | void update(UserWeixin resources); | 30 | UserWeixinDTO update(UserWeixin resources); |
31 | 31 | ||
32 | /** | 32 | /** |
33 | * | 33 | * | ... | ... |
... | @@ -37,19 +37,20 @@ public class UserWeixinServiceImpl implements UserWeixinService { | ... | @@ -37,19 +37,20 @@ public class UserWeixinServiceImpl implements UserWeixinService { |
37 | 37 | ||
38 | @Override | 38 | @Override |
39 | @Transactional(rollbackFor = Exception.class) | 39 | @Transactional(rollbackFor = Exception.class) |
40 | public UserWeixin create(UserWeixin resources) { | 40 | public UserWeixinDTO create(UserWeixin resources) { |
41 | UserWeixin build = UserWeixinBuilder.build(resources); | 41 | UserWeixin build = UserWeixinBuilder.build(resources); |
42 | this.userWeixinRepository.save(build); | 42 | UserWeixin userWeixin = this.userWeixinRepository.save(build); |
43 | return resources; | 43 | return this.userWeixinMapper.toDto(userWeixin); |
44 | } | 44 | } |
45 | 45 | ||
46 | @Override | 46 | @Override |
47 | @Transactional(rollbackFor = Exception.class) | 47 | @Transactional(rollbackFor = Exception.class) |
48 | public void update(UserWeixin resources) { | 48 | public UserWeixinDTO update(UserWeixin resources) { |
49 | UserWeixin UserWeixin = this.userWeixinRepository.findById(resources.getId()).orElseGet(UserWeixin::new); | 49 | UserWeixin userWeixin = this.userWeixinRepository.findById(resources.getId()).orElseGet(UserWeixin::new); |
50 | ValidationUtil.isNull( UserWeixin.getId(),"UserWeixin","id",resources.getId()); | 50 | ValidationUtil.isNull( userWeixin.getId(),"UserWeixin","id",resources.getId()); |
51 | UserWeixin.copy(resources); | 51 | userWeixin.copy(resources); |
52 | this.userWeixinRepository.save(UserWeixin); | 52 | UserWeixin _userWeixin = this.userWeixinRepository.save(userWeixin); |
53 | return this.userWeixinMapper.toDto(_userWeixin); | ||
53 | } | 54 | } |
54 | 55 | ||
55 | @Override | 56 | @Override | ... | ... |
... | @@ -264,7 +264,6 @@ public class UserOperationController { | ... | @@ -264,7 +264,6 @@ public class UserOperationController { |
264 | String nicknameEncode = Base64Util.encode(nicknameDecode); | 264 | String nicknameEncode = Base64Util.encode(nicknameDecode); |
265 | memberDTO.setNickname(nicknameEncode); | 265 | memberDTO.setNickname(nicknameEncode); |
266 | } | 266 | } |
267 | |||
268 | if (StringUtils.isNotBlank(headimgurl)) { | 267 | if (StringUtils.isNotBlank(headimgurl)) { |
269 | String headimgurlDecode = URLDecoder.decode(headimgurl, "UTF-8"); | 268 | String headimgurlDecode = URLDecoder.decode(headimgurl, "UTF-8"); |
270 | String imageEncode = Base64Util.encode(headimgurlDecode); | 269 | String imageEncode = Base64Util.encode(headimgurlDecode); |
... | @@ -272,7 +271,7 @@ public class UserOperationController { | ... | @@ -272,7 +271,7 @@ public class UserOperationController { |
272 | memberDTO.setAvatarUrl(StringUtils.isNotBlank(image) == true ? image:headimgurlDecode); | 271 | memberDTO.setAvatarUrl(StringUtils.isNotBlank(image) == true ? image:headimgurlDecode); |
273 | } | 272 | } |
274 | }catch (Exception e) { | 273 | }catch (Exception e) { |
275 | log.info("headimgurl , nickname ===>> encode error!"); | 274 | log.info("头像解析失败!!!"); |
276 | e.printStackTrace(); | 275 | e.printStackTrace(); |
277 | } | 276 | } |
278 | 277 | ... | ... |
This diff is collapsed.
Click to expand it.
... | @@ -45,7 +45,6 @@ public class MemberOperationServiceImpl implements MemberOperationService { | ... | @@ -45,7 +45,6 @@ public class MemberOperationServiceImpl implements MemberOperationService { |
45 | @Autowired | 45 | @Autowired |
46 | private ThreadPoolTaskExecutor threadPoolTaskExecutor; | 46 | private ThreadPoolTaskExecutor threadPoolTaskExecutor; |
47 | 47 | ||
48 | @AsyncMqSend | ||
49 | @CachePut(key = "#resources.memberId") | 48 | @CachePut(key = "#resources.memberId") |
50 | @Override | 49 | @Override |
51 | public MemberDTO buyVip(BuyVipBean resources) { | 50 | public MemberDTO buyVip(BuyVipBean resources) { |
... | @@ -101,7 +100,6 @@ public class MemberOperationServiceImpl implements MemberOperationService { | ... | @@ -101,7 +100,6 @@ public class MemberOperationServiceImpl implements MemberOperationService { |
101 | return memberDTO; | 100 | return memberDTO; |
102 | } | 101 | } |
103 | 102 | ||
104 | @AsyncMqSend | ||
105 | @Override | 103 | @Override |
106 | public void createVipHistory(MemberVipHistory memberVipHistory){ | 104 | public void createVipHistory(MemberVipHistory memberVipHistory){ |
107 | this.memberVipHistoryService.create(memberVipHistory); | 105 | this.memberVipHistoryService.create(memberVipHistory); |
... | @@ -166,7 +164,6 @@ public class MemberOperationServiceImpl implements MemberOperationService { | ... | @@ -166,7 +164,6 @@ public class MemberOperationServiceImpl implements MemberOperationService { |
166 | return this.update(member); | 164 | return this.update(member); |
167 | } | 165 | } |
168 | 166 | ||
169 | @AsyncMqSend | ||
170 | @Override | 167 | @Override |
171 | public MemberProfileDTO getMemberProfileAndCheckVip(Long memberId, String appid) { | 168 | public MemberProfileDTO getMemberProfileAndCheckVip(Long memberId, String appid) { |
172 | 169 | ||
... | @@ -205,7 +202,7 @@ public class MemberOperationServiceImpl implements MemberOperationService { | ... | @@ -205,7 +202,7 @@ public class MemberOperationServiceImpl implements MemberOperationService { |
205 | this.threadPoolTaskExecutor.execute(()->{ | 202 | this.threadPoolTaskExecutor.execute(()->{ |
206 | Member member = new Member(); | 203 | Member member = new Member(); |
207 | BeanUtils.copyProperties(memberDTO1,member); | 204 | BeanUtils.copyProperties(memberDTO1,member); |
208 | this.memberService.update(member); | 205 | this.update(member); |
209 | }); | 206 | }); |
210 | 207 | ||
211 | vip = memberDTO1.getVip(); | 208 | vip = memberDTO1.getVip(); | ... | ... |
... | @@ -37,6 +37,7 @@ public class MemberProfileOperationServiceImpl implements MemberProfileOperation | ... | @@ -37,6 +37,7 @@ public class MemberProfileOperationServiceImpl implements MemberProfileOperation |
37 | } | 37 | } |
38 | 38 | ||
39 | @Override | 39 | @Override |
40 | @AsyncMqSend | ||
40 | public MemberProfile create(MemberProfile resources) { | 41 | public MemberProfile create(MemberProfile resources) { |
41 | return this.memberProfileService.create(resources); | 42 | return this.memberProfileService.create(resources); |
42 | } | 43 | } |
... | @@ -83,8 +84,7 @@ public class MemberProfileOperationServiceImpl implements MemberProfileOperation | ... | @@ -83,8 +84,7 @@ public class MemberProfileOperationServiceImpl implements MemberProfileOperation |
83 | String memberCode = resources.getMemberCode(); | 84 | String memberCode = resources.getMemberCode(); |
84 | MemberDTO memberDTO = null; | 85 | MemberDTO memberDTO = null; |
85 | if (StringUtils.isEmpty(memberCode)) { | 86 | if (StringUtils.isEmpty(memberCode)) { |
86 | Long memberId = resources.getMemberId(); | 87 | memberDTO = this.memberOperationService.findByCode(memberCode); |
87 | memberDTO = this.memberOperationService.findById(memberId); | ||
88 | resources.setMemberCode(memberDTO.getCode()); | 88 | resources.setMemberCode(memberDTO.getCode()); |
89 | } | 89 | } |
90 | 90 | ... | ... |
1 | package com.topdraw.business.process.service.impl.member; | 1 | package com.topdraw.business.process.service.impl.member; |
2 | 2 | ||
3 | import com.topdraw.aspect.AsyncMqSend; | ||
3 | import com.topdraw.business.module.member.domain.Member; | 4 | import com.topdraw.business.module.member.domain.Member; |
4 | import com.topdraw.business.module.member.relatedinfo.domain.MemberRelatedInfo; | 5 | import com.topdraw.business.module.member.relatedinfo.domain.MemberRelatedInfo; |
5 | import com.topdraw.business.module.member.relatedinfo.service.MemberRelatedInfoService; | 6 | import com.topdraw.business.module.member.relatedinfo.service.MemberRelatedInfoService; |
... | @@ -29,16 +30,19 @@ public class MemberRelatedInfoOperationServiceImpl implements MemberRelatedInfoO | ... | @@ -29,16 +30,19 @@ public class MemberRelatedInfoOperationServiceImpl implements MemberRelatedInfoO |
29 | } | 30 | } |
30 | 31 | ||
31 | @Override | 32 | @Override |
33 | @AsyncMqSend | ||
32 | public MemberRelatedInfoDTO create(MemberRelatedInfo resources) { | 34 | public MemberRelatedInfoDTO create(MemberRelatedInfo resources) { |
33 | return this.memberRelatedInfoService.create(resources); | 35 | return this.memberRelatedInfoService.create(resources); |
34 | } | 36 | } |
35 | 37 | ||
36 | @Override | 38 | @Override |
39 | @AsyncMqSend | ||
37 | public MemberRelatedInfoDTO update(MemberRelatedInfo resources) { | 40 | public MemberRelatedInfoDTO update(MemberRelatedInfo resources) { |
38 | return this.memberRelatedInfoService.update(resources); | 41 | return this.memberRelatedInfoService.update(resources); |
39 | } | 42 | } |
40 | 43 | ||
41 | @Override | 44 | @Override |
45 | @AsyncMqSend | ||
42 | public void delete(Long id) { | 46 | public void delete(Long id) { |
43 | this.memberRelatedInfoService.delete(id); | 47 | this.memberRelatedInfoService.delete(id); |
44 | } | 48 | } | ... | ... |
-
Please register or sign in to post a comment