撤销订单API，并重构相关代码，简化开发 #101

Author: binarywang

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/request/WxEntPayQueryRequest.java

@@ -43,6 +43,11 @@ public void setPartnerTradeNo(String partnerTradeNo) {
     this.partnerTradeNo = partnerTradeNo;
   }
 
+  @Override
+  protected void checkConstraints() {
+    //do nothing
+  }
+
   @Override
   public String toString() {
     return ToStringUtils.toSimpleString(this);

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/request/WxEntPayRequest.java

@@ -154,6 +154,11 @@ public class WxEntPayRequest extends WxPayBaseRequest {
   private String spbillCreateIp;
 
 
+  @Override
+  protected void checkConstraints() {
+
+  }
+
   @Override
   public String getAppid() {
     return this.mchAppid;

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/request/WxPayBaseRequest.java

@@ -1,9 +1,14 @@
 package com.github.binarywang.wxpay.bean.request;
 
+import com.github.binarywang.wxpay.config.WxPayConfig;
+import com.github.binarywang.wxpay.util.SignUtils;
 import com.thoughtworks.xstream.XStream;
 import com.thoughtworks.xstream.annotations.XStreamAlias;
+import me.chanjar.weixin.common.exception.WxErrorException;
+import me.chanjar.weixin.common.util.BeanUtils;
 import me.chanjar.weixin.common.util.ToStringUtils;
 import me.chanjar.weixin.common.util.xml.XStreamInitializer;
+import org.apache.commons.lang3.StringUtils;
 
 import java.math.BigDecimal;
 
@@ -23,6 +28,22 @@
  * @author <a href="https://github.com/binarywang">binarywang(Binary Wang)</a>
  */
 public abstract class WxPayBaseRequest {
+  /**
+   * 检查请求参数内容，包括必填参数以及特殊约束
+   */
+  protected void checkFields() throws WxErrorException {
+    //check required fields
+    BeanUtils.checkRequiredFields(this);
+
+    //check other parameters
+    this.checkConstraints();
+  }
+
+  /**
+   * 检查约束情况
+   */
+  protected abstract void checkConstraints();
+
   /**
    * <pre>
    * 公众账号ID
@@ -180,4 +201,41 @@ public String toXML() {
     xstream.processAnnotations(this.getClass());
     return xstream.toXML(this);
   }
+
+  /**
+   * <pre>
+   * 检查参数，并设置签名
+   * 1、检查参数（注意：子类实现需要检查参数的而外功能时，请在调用父类的方法前进行相应判断）
+   * 2、补充系统参数，如果未传入则从配置里读取
+   * 3、生成签名，并设置进去
+   * </pre>
+   * @param config 支付配置对象，用于读取相应系统配置信息
+   */
+  public void checkAndSign(WxPayConfig config) throws WxErrorException {
+    this.checkFields();
+
+    if (StringUtils.isBlank(getAppid())) {
+      this.setAppid(config.getAppId());
+    }
+
+    if (StringUtils.isBlank(getMchId())) {
+      this.setMchId(config.getMchId());
+    }
+
+    if (StringUtils.isBlank(getSubAppId())) {
+      this.setSubAppId(config.getSubAppId());
+    }
+
+    if (StringUtils.isBlank(getSubMchId())) {
+      this. setSubMchId(config.getSubMchId());
+    }
+
+    if (StringUtils.isBlank(getNonceStr())) {
+      this.setNonceStr(String.valueOf(System.currentTimeMillis()));
+    }
+
+    //设置签名字段的值
+    this.setSign(SignUtils.createSign(this, config.getMchKey()));
+  }
+
 }

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/request/WxPayDownloadBillRequest.java

@@ -2,6 +2,10 @@
 
 import com.thoughtworks.xstream.annotations.XStreamAlias;
 import me.chanjar.weixin.common.annotation.Required;
+import org.apache.commons.lang3.ArrayUtils;
+import org.apache.commons.lang3.StringUtils;
+
+import java.util.Arrays;
 
 /**
  * <pre>
@@ -12,6 +16,8 @@
  */
 @XStreamAlias("xml")
 public class WxPayDownloadBillRequest extends WxPayBaseRequest {
+  private static final String[] BILL_TYPE = new String[]{"ALL", "REFUND", "SUCCESS"};
+
   /**
    * <pre>
    * 设备号
@@ -120,4 +126,16 @@ public String getTarType() {
   public void setTarType(String tarType) {
     this.tarType = tarType;
   }
+
+  @Override
+  protected void checkConstraints() {
+    if (StringUtils.isNotBlank(this.getTarType()) && !"GZIP".equals(this.getTarType())) {
+      throw new IllegalArgumentException("tar_type值如果存在，只能为GZIP");
+    }
+
+    if (!ArrayUtils.contains(BILL_TYPE, this.getBillType())) {
+      throw new IllegalArgumentException(String.format("bill_tpye目前必须为%s其中之一,实际值：%s",
+        Arrays.toString(BILL_TYPE), this.getBillType()));
+    }
+  }
 }

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/request/WxPayMicropayRequest.java

@@ -261,6 +261,11 @@ public void setAuthCode(String authCode) {
     this.authCode = authCode;
   }
 
+  @Override
+  protected void checkConstraints() {
+    //do nothing
+  }
+
   public static final class Builder {
     private String signType;
     private String body;

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/request/WxPayOrderCloseRequest.java

@@ -33,4 +33,8 @@ public void setOutTradeNo(String outTradeNo) {
     this.outTradeNo = outTradeNo;
   }
 
+  @Override
+  protected void checkConstraints() {
+
+  }
 }

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/request/WxPayOrderQueryRequest.java

@@ -1,6 +1,7 @@
 package com.github.binarywang.wxpay.bean.request;
 
 import com.thoughtworks.xstream.annotations.XStreamAlias;
+import org.apache.commons.lang3.StringUtils;
 
 /**
  * <pre>
@@ -61,4 +62,12 @@ public String getOutTradeNo() {
   public void setOutTradeNo(String outTradeNo) {
     this.outTradeNo = outTradeNo;
   }
+
+  @Override
+  protected void checkConstraints() {
+    if ((StringUtils.isBlank(transactionId) && StringUtils.isBlank(outTradeNo)) ||
+      (StringUtils.isNotBlank(transactionId) && StringUtils.isNotBlank(outTradeNo))) {
+      throw new IllegalArgumentException("transaction_id 和 out_trade_no 不能同时存在或同时为空，必须二选一");
+    }
+  }
 }

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/request/WxPayOrderReverseRequest.java

@@ -0,0 +1,163 @@
+package com.github.binarywang.wxpay.bean.request;
+
+import com.thoughtworks.xstream.annotations.XStreamAlias;
+import org.apache.commons.lang3.StringUtils;
+
+/**
+ * <pre>
+ * 撤销订单请求类
+ * Created by Binary Wang on 2017-3-23.
+ * @author <a href="https://github.com/binarywang">binarywang(Binary Wang)</a>
+ * </pre>
+ */
+@XStreamAlias("xml")
+public class WxPayOrderReverseRequest extends WxPayBaseRequest {
+
+  /**
+   * <pre>
+   * 微信订单号
+   * transaction_id
+   * String(28)
+   * 1217752501201400000000000000
+   * 微信的订单号，优先使用
+   * </pre>
+   */
+  @XStreamAlias("transaction_id")
+  private String transactionId;
+
+  /**
+   * <pre>
+   * 商户订单号
+   * out_trade_no
+   * String(32)
+   * 1217752501201400000000000000
+   * 商户系统内部的订单号
+   * transaction_id、out_trade_no二选一，如果同时存在优先级：transaction_id> out_trade_no
+   * </pre>
+   */
+  @XStreamAlias("out_trade_no")
+  private String outTradeNo;
+
+  /**
+   * <pre>
+   * 签名类型
+   * sign_type
+   * 否
+   * String(32)
+   * HMAC-SHA256
+   * 签名类型，目前支持HMAC-SHA256和MD5，默认为MD5
+   **/
+  @XStreamAlias("sign_type")
+  private String signType;
+
+  private WxPayOrderReverseRequest(Builder builder) {
+    setTransactionId(builder.transactionId);
+    setAppid(builder.appid);
+    setOutTradeNo(builder.outTradeNo);
+    setMchId(builder.mchId);
+    setSignType(builder.signType);
+    setSubAppId(builder.subAppId);
+    setSubMchId(builder.subMchId);
+    setNonceStr(builder.nonceStr);
+    setSign(builder.sign);
+  }
+
+  public static Builder newBuilder() {
+    return new Builder();
+  }
+
+  public String getTransactionId() {
+    return this.transactionId;
+  }
+
+  public void setTransactionId(String transactionId) {
+    this.transactionId = transactionId;
+  }
+
+  public String getOutTradeNo() {
+    return this.outTradeNo;
+  }
+
+  public void setOutTradeNo(String outTradeNo) {
+    this.outTradeNo = outTradeNo;
+  }
+
+  public String getSignType() {
+    return this.signType;
+  }
+
+  public void setSignType(String signType) {
+    this.signType = signType;
+  }
+
+  @Override
+  protected void checkConstraints() {
+    if (StringUtils.isBlank(transactionId) && StringUtils.isBlank(outTradeNo)) {
+      throw new IllegalArgumentException("transaction_id 和 out_trade_no不能同时为空！");
+    }
+  }
+
+  public static final class Builder {
+    private String transactionId;
+    private String appid;
+    private String outTradeNo;
+    private String mchId;
+    private String signType;
+    private String subAppId;
+    private String subMchId;
+    private String nonceStr;
+    private String sign;
+
+    private Builder() {
+    }
+
+    public Builder transactionId(String transactionId) {
+      this.transactionId = transactionId;
+      return this;
+    }
+
+    public Builder appid(String appid) {
+      this.appid = appid;
+      return this;
+    }
+
+    public Builder outTradeNo(String outTradeNo) {
+      this.outTradeNo = outTradeNo;
+      return this;
+    }
+
+    public Builder mchId(String mchId) {
+      this.mchId = mchId;
+      return this;
+    }
+
+    public Builder signType(String signType) {
+      this.signType = signType;
+      return this;
+    }
+
+    public Builder subAppId(String subAppId) {
+      this.subAppId = subAppId;
+      return this;
+    }
+
+    public Builder subMchId(String subMchId) {
+      this.subMchId = subMchId;
+      return this;
+    }
+
+    public Builder nonceStr(String nonceStr) {
+      this.nonceStr = nonceStr;
+      return this;
+    }
+
+    public Builder sign(String sign) {
+      this.sign = sign;
+      return this;
+    }
+
+    public WxPayOrderReverseRequest build() {
+      return new WxPayOrderReverseRequest(this);
+    }
+  }
+}

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/request/WxPayRedpackQueryRequest.java

@@ -54,4 +54,9 @@ public String getMchBillNo() {
   public void setMchBillNo(String mchBillNo) {
     this.mchBillNo = mchBillNo;
   }
+
+  @Override
+  protected void checkConstraints() {
+
+  }
 }

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/request/WxPayRefundQueryRequest.java

@@ -1,6 +1,7 @@
 package com.github.binarywang.wxpay.bean.request;
 
 import com.thoughtworks.xstream.annotations.XStreamAlias;
+import org.apache.commons.lang3.StringUtils;
 
 /**
  * <pre>
@@ -132,4 +133,15 @@ public String getRefundId() {
   public void setRefundId(String refundId) {
     this.refundId = refundId;
   }
+
+  @Override
+  protected void checkConstraints() {
+    if ((StringUtils.isBlank(transactionId) && StringUtils.isBlank(outTradeNo)
+      && StringUtils.isBlank(outRefundNo) && StringUtils.isBlank(refundId)) ||
+      (StringUtils.isNotBlank(transactionId) && StringUtils.isNotBlank(outTradeNo)
+        && StringUtils.isNotBlank(outRefundNo) && StringUtils.isNotBlank(refundId))) {
+      throw new IllegalArgumentException("transaction_id，out_trade_no，out_refund_no，refund_id 必须四选一");
+    }
+
+  }
 }