From 5636a3b962bc19c4e46e67646c74018037ee8792 Mon Sep 17 00:00:00 2001 From: Pu Zhibing <393733352@qq.com> Date: 星期二, 24 九月 2024 13:26:40 +0800 Subject: [PATCH] 修改bug --- UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/specialTrain/server/impl/OrderPrivateCarServiceImpl.java | 63 ++++++++++++-------- UserZYTravel/guns-admin/src/main/resources/application.yml | 3 - UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/taxi/service/impl/PaymentRecordServiceImpl.java | 12 ++- UserZYTravel/guns-admin/src/test/java/com/stylefeng/guns/GunsApplicationTest.java | 48 ++++++++++++++++ UserZYTravel/guns-core/src/main/resources/default-config.properties | 2 UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/system/util/PayMoneyUtil.java | 15 ++-- 6 files changed, 102 insertions(+), 41 deletions(-) diff --git a/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/specialTrain/server/impl/OrderPrivateCarServiceImpl.java b/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/specialTrain/server/impl/OrderPrivateCarServiceImpl.java index b5ba225..0142c10 100644 --- a/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/specialTrain/server/impl/OrderPrivateCarServiceImpl.java +++ b/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/specialTrain/server/impl/OrderPrivateCarServiceImpl.java @@ -33,6 +33,7 @@ import org.springframework.http.HttpHeaders; import org.springframework.http.MediaType; import org.springframework.stereotype.Service; +import org.springframework.transaction.annotation.Transactional; import org.springframework.util.LinkedMultiValueMap; import org.springframework.util.MultiValueMap; import org.springframework.web.client.RestTemplate; @@ -929,6 +930,10 @@ @Override public ResultUtil payPrivateCarOrder(Integer payType, Integer orderId, Integer couponId, Integer type) throws Exception { + String value = redisUtil.getValue(orderId + "_status_1"); + if(ToolUtil.isNotEmpty(value) && "8".equals(value)){ + return ResultUtil.error("订单已完成支付,不允许重复支付", ""); + } OrderPrivateCar orderPrivateCar = this.selectById(orderId); if(orderPrivateCar.getState() == 8 || orderPrivateCar.getState() == 9){ return ResultUtil.error("订单已完成支付,不允许重复支付", ""); @@ -936,8 +941,7 @@ if(orderPrivateCar.getState() != 7){ return ResultUtil.error("订单不在待支付状态,不允许支付", ""); } - String code = orderId + "_1_" + UUIDUtil.getRandomCode(5); - PaymentRecord query3 = paymentRecordService.query(1, null, null, orderId, 1, payType, null); + PaymentRecord query3 = paymentRecordService.query(1, null, null, orderId, 1, null, 1); if(null != query3){ ResultUtil<Map<String, Object>> resultUtil1 = payMoneyUtil.queryWXOrder("", query3.getSerialNumber(), "JSAPI"); if (resultUtil1.getCode() == 200) { @@ -956,12 +960,14 @@ String s = resultUtil1.getData().get("trade_state").toString(); if ("REFUND".equals(s) || "CLOSED".equals(s) || "REVOKED".equals(s) || "PAYERROR".equals(s)) { payMoneyUtil.closeWXOrder(query3.getSerialNumber()); + paymentRecordService.deleteById(query3.getId()); } if ("SUCCESS".equals(s) || "ACCEPT".equals(s)) { return ResultUtil.error("不允许重复支付"); } if ("NOTPAY".equals(s) || "USERPAYING".equals(s)) { payMoneyUtil.closeWXOrder(query3.getSerialNumber()); + paymentRecordService.deleteById(query3.getId()); } } } @@ -980,7 +986,7 @@ UserCouponRecord userCouponRecord = null; if(null != couponId){ userCouponRecord = userCouponRecordService.selectById(couponId); - if(userCouponRecord.getCompanyId() != orderPrivateCar.getCompanyId()){ + if(userCouponRecord.getCompanyId().compareTo(orderPrivateCar.getCompanyId()) != 0){ return ResultUtil.error("优惠券不能用于此订单", ""); } if(userCouponRecord.getState() == 2){ @@ -1027,19 +1033,18 @@ if(ToolUtil.isEmpty(userInfo.getAppletsOpenId())){ return ResultUtil.error("未授权微信,无法完成支付"); } + String code = orderId + "_1_" + UUIDUtil.getRandomCode(5); if(payType == 1){//微信支付 resultUtil = payMoneyUtil.weixinpay("完成订单",orderId +"", code,orderMoney+"","/base/wxPayOrderTaxi","JSAPI",userInfo.getAppletsOpenId()); if(resultUtil.getCode()==200){ - paymentRecordService.saveData(1, null, null, orderId, 1, 1, orderMoney, null, 1, code);//添加预支付数据 + paymentRecordService.saveData(1, orderPrivateCar.getUserId(), 1, orderId, 1, 1, orderMoney, null, 1, code);//添加预支付数据 new Thread(()->{ - int num = 1; - int min = 5000; - int w = 0; - while (num <= 10) { + long time = orderPrivateCar.getInsertTime().getTime(); + //订单创建后15分钟 + while (System.currentTimeMillis() - time <= 900000) { try { - w += min * num; - OrderPrivateCar orderPrivateCar1 = orderPrivateCarMapper.selectById(orderId); - if (orderPrivateCar1.getState() != 7) { + String value1 = redisUtil.getValue(orderId + "_status_1"); + if(ToolUtil.isNotEmpty(value1) && "8".equals(value1)){ break; } ResultUtil<Map<String, Object>> resultUtil2 = payMoneyUtil.queryWXOrder("", code, "JSAPI"); @@ -1055,7 +1060,7 @@ * ACCEPT--已接收,等待扣款 */ String s = resultUtil2.getData().get("trade_state").toString(); - if ("REFUND".equals(s) || "CLOSED".equals(s) || "REVOKED".equals(s) || "PAYERROR".equals(s) || num == 10) { + if ("REFUND".equals(s) || "CLOSED".equals(s) || "REVOKED".equals(s) || "PAYERROR".equals(s)) { break; } if ("SUCCESS".equals(s) || "ACCEPT".equals(s)) { @@ -1064,15 +1069,20 @@ break; } if ("NOTPAY".equals(s) || "USERPAYING".equals(s)) { - Thread.sleep(w); - num++; } - } else { - Thread.sleep(w); - num++; } + Thread.sleep(5000); }catch (Exception e){ e.printStackTrace(); + } + } + String value1 = redisUtil.getValue(orderId + "_status_1"); + if(ToolUtil.isNotEmpty(value1) && "7".equals(value1)){ + //关闭订单,不允许支付 + payMoneyUtil.closeWXOrder(code); + PaymentRecord paymentRecord = paymentRecordService.selectOne(new EntityWrapper<PaymentRecord>().eq("serialNumber", code)); + if(null != paymentRecord){ + paymentRecordService.deleteById(paymentRecord.getId()); } } }).start(); @@ -1084,7 +1094,7 @@ if(payType == 2) {//支付宝支付 resultUtil = payMoneyUtil.alipay("完成订单","完成订单", "",code,orderMoney+"","/base/aliPayOrderTaxi"); if (resultUtil.getCode()==200) { - paymentRecordService.saveData(1, null, null, orderId, 1, 2, orderMoney, null, 1, code);//添加预支付数据 + paymentRecordService.saveData(1, orderPrivateCar.getUserId(), 1, orderId, 1, 2, orderMoney, null, 1, code);//添加预支付数据 } else { resultUtil = ResultUtil.error("支付失败", ""); } @@ -1314,14 +1324,16 @@ } @Override - public void payOrderPrivateCarCallback(Integer id, String order_id, Integer type) throws Exception { - OrderPrivateCar orderPrivateCar = this.selectById(id); - if(orderPrivateCar.getState() != 7){ - return; - } - PaymentRecord query = paymentRecordService.query(1, null, null, Integer.valueOf(id), 1, type, 1); + @Transactional(rollbackFor = Exception.class) + public synchronized void payOrderPrivateCarCallback(Integer id, String order_id, Integer type) throws Exception { + PaymentRecord query = paymentRecordService.query(1, null, null, Integer.valueOf(id), 1, type, null); if(null != query){ + String value = redisUtil.getValue(order_id + "_status_1"); + if(ToolUtil.isNotEmpty(value) && "8".equals(value)){ + return; + } //添加交易明细 + OrderPrivateCar orderPrivateCar = this.selectById(id); transactionDetailsService.saveData(orderPrivateCar.getUserId(), "完成订单", query.getAmount(), 2, 1, 1, 1, query.getOrderId()); orderPrivateCar.setState(8); orderPrivateCar.setPayType(type); @@ -1348,10 +1360,11 @@ userRedPacketRecordService.updateById(userRedPacketRecord); } - query.setState(2); query.setCode(order_id); paymentRecordService.updateById(query); + redisUtil.setStrValue(order_id + "_status_1", "8"); + if(orderPrivateCar.getIsplatPay()==1){ //添加已收入明细 Company company = companyService.selectById(orderPrivateCar.getCompanyId()); diff --git a/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/system/util/PayMoneyUtil.java b/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/system/util/PayMoneyUtil.java index 30f0fc0..eb2213c 100644 --- a/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/system/util/PayMoneyUtil.java +++ b/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/system/util/PayMoneyUtil.java @@ -345,8 +345,9 @@ //将结果xml解析成map body1 = body1.replaceAll("<!\\[CDATA\\[",""); body1 = body1.replaceAll("]]>", ""); + System.err.println("调起支付结果-------------->" + body1); try { - map1 = this.xmlToMap(body1, "UTF-8"); + map1 = PayMoneyUtil.xmlToMap(body1, "UTF-8"); } catch (UnsupportedEncodingException e) { e.printStackTrace(); } catch (DocumentException e) { @@ -366,7 +367,7 @@ map2.put("nonceStr", map1.get("nonce_str")); map2.put("package", "prepay_id=" + prepay_id); map2.put("signType", "MD5"); - map2.put("timeStamp", new Date().getTime() + ""); + map2.put("timeStamp", System.currentTimeMillis() + ""); String s2 = this.weixinSignature(map2); map2.put("prepay_id", prepay_id); @@ -386,7 +387,7 @@ map3.put("package", "Sign=WXPay"); map3.put("partnerid", mchId); map3.put("prepayid", prepay_id); - map3.put("timestamp", new Date().getTime() / 1000); + map3.put("timestamp", System.currentTimeMillis() / 1000); String s1 = this.weixinSignature(map3); map3.put("sign", s1); System.err.println(map3); @@ -394,11 +395,11 @@ } return null; }else{ - System.err.println(map1.get("err_code_des")); + System.err.println("调起支付异常-------------->" + map1.get("err_code_des")); return ResultUtil.error(map1.get("err_code_des")); } }else{//签名错误,请检查后再试1----1602881362 - System.err.println(map1.get("return_msg") + appid + "----" + mchId); + System.err.println("调起支付异常-------------->" + map1.get("return_msg") + appid + "----" + mchId); return ResultUtil.error(map1.get("return_msg"), new JSONObject()); } } @@ -416,7 +417,7 @@ String param = this.getParam(request); param = param.replaceAll("<!\\[CDATA\\[",""); param = param.replaceAll("]]>", ""); - Map<String, String> map = this.xmlToMap(param, "UTF-8"); + Map<String, String> map = PayMoneyUtil.xmlToMap(param, "UTF-8"); String return_code = map.get("return_code"); if("SUCCESS".equals(return_code)){ String result_code = map.get("result_code"); @@ -819,7 +820,7 @@ body1 = body1.replaceAll("<!\\[CDATA\\[",""); body1 = body1.replaceAll("]]>", ""); try { - map1 = this.xmlToMap(body1, "UTF-8"); + map1 = PayMoneyUtil.xmlToMap(body1, "UTF-8"); } catch (UnsupportedEncodingException e) { e.printStackTrace(); } catch (DocumentException e) { diff --git a/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/taxi/service/impl/PaymentRecordServiceImpl.java b/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/taxi/service/impl/PaymentRecordServiceImpl.java index 3948088..6222a56 100644 --- a/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/taxi/service/impl/PaymentRecordServiceImpl.java +++ b/UserZYTravel/guns-admin/src/main/java/com/stylefeng/guns/modular/taxi/service/impl/PaymentRecordServiceImpl.java @@ -1,6 +1,7 @@ package com.stylefeng.guns.modular.taxi.service.impl; import com.baomidou.mybatisplus.service.impl.ServiceImpl; +import com.stylefeng.guns.modular.system.util.RedisUtil; import com.stylefeng.guns.modular.taxi.dao.PaymentRecordMapper; import com.stylefeng.guns.modular.taxi.model.PaymentRecord; import com.stylefeng.guns.modular.taxi.service.IPaymentRecordService; @@ -15,6 +16,9 @@ @Resource private PaymentRecordMapper paymentRecordMapper; + + @Resource + private RedisUtil redisUtil; @@ -31,11 +35,6 @@ @Override public Integer saveData(Integer category, Integer userId, Integer type, Integer orderId, Integer orderType, Integer payType, Double amount, String code, Integer state, String serialNumber) throws Exception { - //先删除现有数据避免重复 - PaymentRecord query = paymentRecordMapper.query(category, userId, type, orderId, orderType, payType, state); - if(null != query){ - this.deleteById(query.getId()); - } PaymentRecord paymentRecord = new PaymentRecord(); paymentRecord.setCategory(category); paymentRecord.setUserId(userId); @@ -49,6 +48,9 @@ paymentRecord.setInsertTime(new Date()); paymentRecord.setSerialNumber(serialNumber); this.insert(paymentRecord); + if(state == 1){ + redisUtil.setStrValue(orderId + "_status_" + orderType, "7", 3600); + } return paymentRecord.getId(); } diff --git a/UserZYTravel/guns-admin/src/main/resources/application.yml b/UserZYTravel/guns-admin/src/main/resources/application.yml index 142a295..8ebc582 100644 --- a/UserZYTravel/guns-admin/src/main/resources/application.yml +++ b/UserZYTravel/guns-admin/src/main/resources/application.yml @@ -31,9 +31,6 @@ mybatis-plus: typeAliasesPackage: com.stylefeng.guns.modular -log: - path: guns-logs - eureka: client: service-url: #注册中心地址 diff --git a/UserZYTravel/guns-admin/src/test/java/com/stylefeng/guns/GunsApplicationTest.java b/UserZYTravel/guns-admin/src/test/java/com/stylefeng/guns/GunsApplicationTest.java new file mode 100644 index 0000000..684fb1d --- /dev/null +++ b/UserZYTravel/guns-admin/src/test/java/com/stylefeng/guns/GunsApplicationTest.java @@ -0,0 +1,48 @@ +//package com.stylefeng.guns; +// +//import com.stylefeng.guns.modular.system.util.PayMoneyUtil; +//import com.stylefeng.guns.modular.system.util.ResultUtil; +//import org.junit.Test; +//import org.junit.runner.RunWith; +//import org.springframework.beans.factory.annotation.Autowired; +//import org.springframework.boot.test.context.SpringBootTest; +//import org.springframework.test.context.junit4.SpringRunner; +// +//import java.util.Map; +// +///** +// * @author zhibing.pu +// * @Date 2024/9/24 9:24 +// */ +//@RunWith(SpringRunner.class) +//@SpringBootTest +//public class GunsApplicationTest { +// +// @Autowired +// private PayMoneyUtil payMoneyUtil; +// +// +// @Test +// public void test(){ +// try { +// ResultUtil<Map<String, Object>> resultUtil1 = payMoneyUtil.queryWXOrder("", "37551_1_74ed0", "JSAPI"); +// if (resultUtil1.getCode() == 200) { +// /** +// * SUCCESS--支付成功 +// * REFUND--转入退款 +// * NOTPAY--未支付 +// * CLOSED--已关闭 +// * REVOKED--已撤销(刷卡支付) +// * USERPAYING--用户支付中 +// * PAYERROR--支付失败(其他原因,如银行返回失败) +// * ACCEPT--已接收,等待扣款 +// */ +// System.err.println(resultUtil1.getData()); +// } +// +//// payMoneyUtil.closeWXOrder("37551_1_74ed0"); +// } catch (Exception e) { +// throw new RuntimeException(e); +// } +// } +//} diff --git a/UserZYTravel/guns-core/src/main/resources/default-config.properties b/UserZYTravel/guns-core/src/main/resources/default-config.properties index 8d458f7..3166d4a 100644 --- a/UserZYTravel/guns-core/src/main/resources/default-config.properties +++ b/UserZYTravel/guns-core/src/main/resources/default-config.properties @@ -22,7 +22,7 @@ # \u5F00\u542F\u7684\u8BDD\uFF0C\u5EF6\u65F6\u52A0\u8F7D\u4E00\u4E2A\u5C5E\u6027\u65F6\u4F1A\u52A0\u8F7D\u8BE5\u5BF9\u8C61\u5168\u90E8\u5C5E\u6027\uFF0C\u5426\u5219\u6309\u9700\u52A0\u8F7D\u5C5E\u6027 mybatis-plus.configuration.multipleResultSetsEnabled=true # \u6253\u5370sql\u8BED\u53E5,\u8C03\u8BD5\u7528 -mybatis-plus.configuration.log-impl=org.apache.ibatis.logging.stdout.StdOutImpl +#mybatis-plus.configuration.log-impl=org.apache.ibatis.logging.stdout.StdOutImpl ########################## beetl\u9ED8\u8BA4\u914D\u7F6E ########################### -- Gitblit v1.7.1