利用自定义异常来重构代码（▄︻┻┳═一不了解自定义异常者勿看）
##先贴出来代码
项目中有个页面，用来支付回调。窗体里无任何html元素。服务端程序Page_Load如下，即获取到请求的订单，校验订单是否有效，然后持久化订单数据，并回写处理成功或失败的标识。
1 /// 2 /// 由PayAndRefund/RefundForm.aspx请求过来，处理业务系统的退款请求 3 /// 4 public partial class RefundForm : System.Web.UI.Page 5 { 6 private AlipayPaymentBLL.AlipayBLL obll = new AlipayPaymentBLL.AlipayBLL(); 7 private AlipayPaymentBLL.AlipayRefundRecord alipayRefundBll = new AlipayPaymentBLL.AlipayRefundRecord(); 8 protected void Page_Load(object sender, EventArgs e) 9 {10 // 记录日志和相关标记11 CommonUtils.LogCommon.instance.writePay(this, "===================================================================================");12 string sMsg = UrlInfo.Initial(Request).GetMsg();13 CommonUtils.LogCommon.instance.writePay(this, sMsg);14 15 ReturnValue result;16 try17 {18 string reqStr = WebCommon.ReadStreamReqStr(Request, System.Text.Encoding.UTF8);//获得流 序列化19 var refundApply = jsonConvert.DeserializeObject(reqStr);20 if (refundApply == null)21 {22 result = new ReturnValue("9999", "无通知参数");23 }24 else25 {26 var dt = new DataTable();27 var listOrderNo = new List();28 var dicMoney = new Dictionary();29 var dicRemark = new Dictionary();30 result = alipayRefundBll.BuilderArray(listOrderNo, dicMoney, dicRemark, refundApply);//解析传入参数信息31 if (result.Code == "0000")32 {33 result = new AlipayPaymentBLL.AlipayRefundRecord().CheckRefundRecordOrder(listOrderNo, refundApply);34 if (result.Code == "0000")35 {36 result = alipayRefundBll.DoRefund(refundApply, dt, listOrderNo, dicMoney, dicRemark);37 if (result.Code == "0000")38 {39 dt.Columns.Remove("trade_no");40 string[] sTableColumName = CommonModel.CommonFun.GetColumNameOfDataTable(dt);41 // 将退款数据批量insert到dbo.T_AlipayRefundRecord42 // 支付宝退款是有密的，这里只保存。 在单独的支付宝退款页做跳转到支付宝输入密码进行退款操作43 bool bResult = alipayRefundBll.BulkCopy(dt, sTableColumName);44 result = bResult ? new ReturnValue("0000", "支付宝退款请求支付中心已收到！") : new ReturnValue("9999", "退款信息入库失败！");45 }46 }47 }48 }49 }50 catch (Exception ex)51 {52 CommonUtils.LogCommon.instance.writePay(this, "支付宝退款发起异常==>" + ex);53 result = new ReturnValue("9999", "支付宝退款发起异常");54 }55 56 string jsonStr = JsonConvert.SerializeObject(result);57 CommonUtils.LogCommon.instance.writePay(this, "支付宝退款返回值==>" + jsonStr);58 Response.Write(jsonStr);59 }60
##代码分析
不考虑逻辑，仅从代码简洁的角度来看，如上代码段存在如下问题，使得代码有了坏味道（bad smell）：
* 对象oReturnValue被重复赋值并使用
* 层层嵌套, if和try...catch使得代码嵌套了好多层##代码重构
从如下几个角度进行重构：
* 引入自定义异常， 当判断失败时，返回自定义异常。这样可以去掉很多if的嵌套。
* 要对捕获到的自定义异常做处理
重构后：
1 /// 2 /// 由PayAndRefund/RefundForm.aspx请求过来，处理业务系统的退款请求 3 /// 4 public partial class RefundForm : System.Web.UI.Page 5 { 6 private AlipayPaymentBLL.AlipayBLL obll = new AlipayPaymentBLL.AlipayBLL(); 7 private AlipayPaymentBLL.AlipayRefundRecord alipayRefundBll = new AlipayPaymentBLL.AlipayRefundRecord(); 8 protected void Page_Load(object sender, EventArgs e) 9 {10 // 记录日志和相关标记11 CommonUtils.LogCommon.instance.writePay(this, "===================================================================================");12 string sMsg = UrlInfo.Initial(Request).GetMsg();13 CommonUtils.LogCommon.instance.writePay(this, sMsg);14 15 ReturnValue result;16 try17 {18 string reqStr = WebCommon.ReadStreamReqStr(Request, System.Text.Encoding.UTF8);//获得流 序列化19 var refundApply = JsonConvert.DeserializeObject(reqStr);20 if (refundApply == null)21 {22 throw new ResponseErrorException("无通知参数");23 }24 25 if (alipayRefundBll.RefundOrderExists(refundApply.OrderNo))26 {27 throw new ResponseErrorException("支付宝退款请求订单号:" + refundApply.OrderNo + "已提交支付中心,请不要重复提交！");28 }29 30 var dt = alipayRefundBll.DoRefund1(refundApply);31 dt.Columns.Remove("trade_no");32 // 将退款数据批量insert到dbo.T_AlipayRefundRecord33 // 支付宝退款是有密的，这里只保存。 在单独的支付宝退款页做跳转到支付宝输入密码进行退款操作34 bool saveSuccess = alipayRefundBll.BulkCopy(dt);35 result = saveSuccess ? new ReturnValue("0000", "支付宝退款请求支付中心已收到！") : new ReturnValue("9999", "退款信息入库失败！");36 }37 catch (Exception ex)38 {39 if (ex is ResponseErrorException)40 {41 result = new ReturnValue("9999", ex.Message);42 }43 else44 {45 CommonUtils.LogCommon.instance.writePay(this, "支付宝退款发起异常==>" + ex.ToString());46 result = new ReturnValue("9999", "支付宝退款发起异常");47 }48 }49 50 string jsonStr = JsonConvert.SerializeObject(result);51 CommonUtils.LogCommon.instance.writePay(this, "支付宝退款返回值==>" + jsonStr);52 Response.Write(jsonStr);53 }54
可见，代码清晰了很多。主要的方式是引入了自定义异常ResponseErrorException，使得方法只管返回理想情况下应该返回的参数类型，而现实很骨感，所以，当不满足判断条件时，就通过抛出自定义异常的方式来实现，同时也没有破坏方法的结构。 另外，我将异常捕获统一放到了主方法ProcessRequest里，也使得代码结构清晰，少了那些if的判断，是不是很漂亮？
public class ResponseErrorException : System.Exception{ // // 摘要: // 使用指定的错误消息初始化 ResponseErrorException 类的新实例。 // // 参数: // message: // 描述错误的消息。 public ResponseErrorException(string message) : base(message) { } // // 摘要: 使用指定的错误消息初始化 ResponseErrorException 类的新实例 // // 参数: // format: // 复合格式字符串。 // // args: // 一个对象数组，其中包含零个或多个要设置格式的对象。 // // 异常: // System.ArgumentNullException: // format 或 args 为 null。 // // System.FormatException: // format 无效。- 或 -格式项的索引小于零或大于等于 args 数组的长度。 public ResponseErrorException(string format, params object[] args) : base(FormatString(format, args)) { } static string FormatString(string format, params object[] args) { try { string message = string.Format(format, args); return message; } catch (FormatException ex) { //LogHelper.Write("执行string.Format时(内容：\"{0}\",参数个数：{1})出现格式异常:{2}", format, args.Length, ex.Message); return string.Empty; } }}
版权声明：本文内容由网络用户投稿，版权归原作者所有，本站不拥有其著作权，亦不承担相应法律责任。如果您发现本站中有涉嫌抄袭或描述失实的内容，请联系我们jiasou666@gmail.com 处理，核实后本网站将在24小时内删除侵权内容。
暂时没有评论，来抢沙发吧~