/** * 编辑会议报名信息 * @param meeting * @return */ @ApiOperation("编辑会议报名信息") @Transactional @PostMapping("/updateMeetingRegistration") public AjaxResult updateMeetingRegistration(@RequestBody MeetingUpdateInfo meeting){ if (StringUtils.isBlank(meeting.getSubscribeId())) { return AjaxResult.error("预约id不能为空!"); } //更新subscribe_meeting表 SubscribeMeeting subscribeMeeting = SubscribeMeeting.builder().subscribeId(meeting.getSubscribeId()) .customName(meeting.getCustomName()) .customIdNumber(meeting.getCustomIdNumber()) .regionName(meeting.getRegionName()) .shopName(meeting.getShopName()) .ticket(meeting.getTicket()) .meetingRemark(meeting.getMeetingRemark()) .build(); subscribeMeetingService.updateById(subscribeMeeting); //更新subscribe_detail表 UpdateWrapper<SubscribeDetail> updateWrapper = new UpdateWrapper<>(); updateWrapper.lambda().set(SubscribeDetail::getCustomPhone,meeting.getCustomPhone()) .set(SubscribeDetail::getCustomType,meeting.getCustomType()) .set(SubscribeDetail::getLeaderName,meeting.getLeaderName()) .eq(SubscribeDetail::getSubscribeId,meeting.getSubscribeId()); subscribeDetailService.update(updateWrapper); return AjaxResult.success(); }优化这段代码
时间: 2024-02-24 21:59:08 浏览: 27
首先,可以考虑将一些重复的代码提取出来,例如获取subscribeId的判断可以提取出来,写成一个方法,减少代码冗余。另外,可以将subscribeMeeting和updateWrapper的操作放到同一个事务中,以保证数据的一致性和完整性。下面是优化后的代码:
```
@ApiOperation("编辑会议报名信息")
@Transactional
@PostMapping("/updateMeetingRegistration")
public AjaxResult updateMeetingRegistration(@RequestBody MeetingUpdateInfo meeting){
String subscribeId = meeting.getSubscribeId();
if (StringUtils.isBlank(subscribeId)) {
return AjaxResult.error("预约id不能为空!");
}
//更新subscribe_meeting表
SubscribeMeeting subscribeMeeting = SubscribeMeeting.builder()
.subscribeId(subscribeId)
.customName(meeting.getCustomName())
.customIdNumber(meeting.getCustomIdNumber())
.regionName(meeting.getRegionName())
.shopName(meeting.getShopName())
.ticket(meeting.getTicket())
.meetingRemark(meeting.getMeetingRemark())
.build();
subscribeMeetingService.updateById(subscribeMeeting);
//更新subscribe_detail表
UpdateWrapper<SubscribeDetail> updateWrapper = new UpdateWrapper<>();
updateWrapper.lambda()
.set(SubscribeDetail::getCustomPhone, meeting.getCustomPhone())
.set(SubscribeDetail::getCustomType, meeting.getCustomType())
.set(SubscribeDetail::getLeaderName, meeting.getLeaderName())
.eq(SubscribeDetail::getSubscribeId, subscribeId);
subscribeDetailService.update(updateWrapper);
return AjaxResult.success();
}
```