先貼出來(lái)代碼
public void ProcessRequest(HttpContext context)
{
context.Response.ContentType = "text/plain";
string jsonStr = "";
var oReturnValue = new ReturnValue("9999", "無(wú)參數(shù)");
if (context.Request.Form.AllKeys.Length > 0)
{
try
{
var oAlipayRefundApply = new RefundApplyDTO
{
OrderNo = context.Request.Form["on"],
OrderAmount = context.Request.Form["oa"],
Remark = context.Request.Form["or"]
};
var dt = new DataTable();
var listOrderNo = new List<string>();
var dicMoney = new Dictionary<string, string>();
var dicRemark = new Dictionary<string, string>();
oReturnValue = new AlipayPaymentBLL.AlipayRefundRecord().BuilderArray(listOrderNo, dicMoney, dicRemark, oAlipayRefundApply);//解析傳入?yún)?shù)信息
if (oReturnValue.Code == "0000")
{
oReturnValue = new AlipayPaymentBLL.AlipayRefundRecord().DoRefund(oAlipayRefundApply, dt, listOrderNo, dicMoney, dicRemark);
if (oReturnValue.Code == "0000")
{
oReturnValue = BuilderRequest(dt);
}
}
}
catch (Exception ex)
{
CommonUtils.LogCommon.instance.writePay(this, "支付寶退款請(qǐng)求組建異常：" + ex);
oReturnValue = new ReturnValue("9999", "請(qǐng)求發(fā)生異常");
}
}
jsonStr = JsonConvert.SerializeObject(oReturnValue);
context.Response.Write(jsonStr);
}
代碼分析
以上是一個(gè)一般處理程序(.ashx文件)的類(lèi)文件里的主方法ProcessRequest的方法體。
不考慮邏輯蚂四，僅從代碼簡(jiǎn)潔的角度來(lái)看光戈，如上代碼段存在如下問(wèn)題：
- 變量/對(duì)象oReturnValue被重復(fù)賦值并使用
- 層層判斷嵌套
- ReturnValue的構(gòu)造器是兩個(gè)string參數(shù)。我要說(shuō)的是第一個(gè)參數(shù)code遂赠，在這段代碼里久妆，code取值只有0000和9999這兩種值，分別代表成功和失敗跷睦。這時(shí)筷弦，不如直接改為true/false更直觀
如下是ReturnValue類(lèi)的定義：
public class ReturnValue
{
/// <summary>
/// 錯(cuò)誤碼
/// </summary>
private string _code;
/// <summary>
/// 錯(cuò)誤信息
/// </summary>
private string _msg;
public ReturnValue(string code, string msg)
{
_code = code;
_msg = msg;
}
public string Code { get { return _code; } }
public string MsgStr { get { return _msg; } }
}
代碼重構(gòu)
從如下幾個(gè)角度進(jìn)行重構(gòu)：
- 封裝處理方法，主方法里統(tǒng)一做try..catch的異常捕獲處理
- 引入自定義異常抑诸， 當(dāng)判斷失敗時(shí)烂琴，返回自定義異常。這樣可以去掉很多if的嵌套蜕乡。
- 修改接口返回值類(lèi)型
重構(gòu)后的代碼如下：
HttpRequest _ctxRequest;
public void ProcessRequest(HttpContext context)
{
context.Response.ContentType = "text/plain";
_ctxRequest = context.Request;
ResponseModel oReturnValue = null;
try
{
oReturnValue = ProcessRequestAndReturnFormHtml();
CommonUtils.LogCommon.instance.writePay(this, "支付寶退款請(qǐng)求處理完成");
}
catch (Exception ex)
{
if (ex is ResponseErrorException)
{
oReturnValue = new ResponseModel(false, ex.Message);
CommonUtils.LogCommon.instance.writePay(this, "支付寶退款請(qǐng)求處理失敿楸痢：" + ex.Message);
}
else
{
CommonUtils.LogCommon.instance.writePay(this, "支付寶退款請(qǐng)求處理出現(xiàn)異常：" + ex.ToString());
oReturnValue = new ResponseModel(false, "請(qǐng)求發(fā)生異常");
}
}
string jsonStr = JsonConvert.SerializeObject(oReturnValue);
context.Response.Write(jsonStr);
}
private ResponseModel ProcessRequestAndReturnFormHtml()
{
var oReturnValue = new ReturnValue("9999", "無(wú)參數(shù)");
if (_ctxRequest.Form.AllKeys.Length > 0)
{
throw new ResponseErrorException("無(wú)參數(shù)");
}
var oAlipayRefundApply = new RefundApplyDTO
{
OrderNo = _ctxRequest.Form["on"],
OrderAmount = _ctxRequest.Form["oa"],
Remark = _ctxRequest.Form["or"]
};
var dt = new DataTable();
var listOrderNo = new List<string>();
var dicMoney = new Dictionary<string, string>();
var dicRemark = new Dictionary<string, string>();
var alipayPaymentBLL = new AlipayPaymentBLL.AlipayRefundRecord();
alipayPaymentBLL.BuilderArray(listOrderNo, dicMoney, dicRemark, oAlipayRefundApply);//解析傳入?yún)?shù)信息
alipayPaymentBLL.DoRefund(oAlipayRefundApply, dt, listOrderNo, dicMoney, dicRemark);// 發(fā)起退款，并獲得結(jié)果做相應(yīng)處理
string formHtml = BuildRequestFormHtml(dt);// 創(chuàng)建支付寶請(qǐng)求（Form表單）
return new ResponseModel(true, formHtml);
}
可見(jiàn)层玲，代碼清晰了很多健盒。主要的方式是引入了自定義異常ResponseErrorException，使得方法只管返回理想情況下應(yīng)該返回的參數(shù)類(lèi)型称簿，而現(xiàn)實(shí)很骨感，所以惰帽，當(dāng)不滿足判斷條件時(shí)憨降，就通過(guò)拋出自定義異常的方式來(lái)實(shí)現(xiàn)，同時(shí)也沒(méi)有破壞方法的結(jié)構(gòu)该酗。 另外授药，我將異常捕獲統(tǒng)一放到了主方法ProcessRequest里士嚎，也使得代碼結(jié)構(gòu)清晰，少了那些if的判斷悔叽，是不是很漂亮莱衩？
同時(shí)，涉及到的改動(dòng)還有相關(guān)的上下游的引用：
1. 前端js在對(duì)響應(yīng)的判斷要由判斷0000/9999改為true/false
var url = "ExcuteRefund.ashx";
$.ajax({
url: url,
type: "POST",
dataType: "json",
data: { "on": arr[0], "oa": arr[1], "or": arr[2] },
success: function (result) {
if (result !== "") {
var jsonData = eval(result);
if (jsonData.Code === "0000") {
$(".table_MidGl").html("");
$(".table_MidGl").append(jsonData.MsgStr);
} else {
alert(jsonData.MsgStr);
window.location.reload();
return false;
}
2. 所引用的方法娇澎，AlipayRefundRecord.BuilderArray()笨蚁、AlipayRefundRecord.DoRefund()和本類(lèi)里的BuildRequestFormHtml()，如下是BuildRequestFormHtml()的改動(dòng)
private string BuildRequestFormHtml(DataTable dt)
{
......
var sortedDicPara = obll.GetRefundParameters(CommonModel.PayCenterConfig.PayPartner.PayPartnerEnums.AlipayApp, dt, ref sInputCharset, ref sLinkString, out sMsg);
if (!sortedDicPara.Any())
{
throw new ResponseErrorException(sMsg);
}
//拼接url'?'后條件
string payUrlValueStr = string.Format("{0}&sign={1}&sign_type={2}",
sLinkString,
sortedDicPara["sign"],
sortedDicPara["sign_type"]
);
//發(fā)起退款URL
string url = new CommonModel.PayCenterConfig.ServerURLConfig(CommonModel.PayCenterConfig.PayPartner.PayPartnerEnums.AlipayApp).REFUND_URL;
CommonUtils.LogCommon.instance.writePay(this, "進(jìn)入支付寶退款趟庄，發(fā)起消息： " + url + payUrlValueStr);
string formHtml = WebCommon.BuildRequest(sortedDicPara, "post", "確認(rèn)", url, sInputCharset);
CommonUtils.LogCommon.instance.writePay(this, "進(jìn)入支付寶退款括细，發(fā)起消息(form表單)： " + formHtml);
return formHtml;
}