最常见的方式就是:
url?arg1=value1&arg2=value2&arg3=value3...
这样的方式最常见最普通也最容易理解,但是在项目中,如果后面带的参数可变而且字段有不同的值或者不同的语义时,这样的方式可维护性和可读性其实并不高,而且代码也容易重复或者冗余。
比如下面的一个对CGI的请求串:
var url = "http://www.tenpay.com/app/v1.0/juhui.cgi?";
var queryString = "method=2&page=index";
if(content a){ //访问的首页的区域A,需要加参数subpage
queryString += "&subpage=a";
}else if(content b){//如果访问了区域B,则参数subpage要变为b
queryString += "&subpage=b";
}
if(spec_method){
//如果查看的时候要按照指定的筛选方法,则还需要加参数spec_method
queryString += "&spec_method=1"
}
这是最常见的拼串逻辑了,这样的代码是没有什么问题,但是写注释很麻烦,而且可读性也不高,字段说明也不明确,如果需要更换一个字段或在原来的基础上更改逻辑都比较麻烦。
我参看了一些同事关于这个问题的解决方案,第一种是将参数用对象的形式存起来,然后再写一个方法在请求时将参数拼起来:
var queryConfig={
"page" : "index",
"method" : 2, //1:按照方法A查看 2:按照方法B查看
"subpage" : -1, //-1:此条件不传递 a:查看contentA b:查看contentB
"spec_method" :-1 //-1:此条件不传递 1:按照销量高低查 2:按照时间查
};
var setQueryConfig = function(){
var _str = "";
for(var o in queryConfig){
if(queryConfig[o] != -1){
_str += o + "=" + queryConfig[o] + "&";
}
}
var _str = _str.substring(0, str.length-1);
return _str;
}
这个方法是挺不错的,优点就是将所有参数一目了然的在对象中全部列出来,注释也可以针对字段更详细,可读性和维护性都得到了提高;但是缺点就是代码有点多,还需要专门加一个方法来组合参数。
另外有一个方法是用数组的方法:
var queryString = [
"method=2", //method字段的注释
"page=index"
];
if(content a){ //访问的首页的区域A,需要加参数subpage
queryString.concat([
"subpage=a", //subpage注释
]);
}else if(content b){//如果访问了区域B,则参数subpage要变为b
queryString.concat([
"subpage=b", //subpage注释
]);
}
if(spec_method){
//如果查看的时候要按照指定的筛选方法,则还需要加参数 queryString.concat([
"spec_method=2", //spec_method注释
]);
}
queryString = queryString.join("&");
这个方法可读性可能比对象的方法差一些,但是维护性也比较高,代码量也比较少。相对来说这个方法我还比较喜欢。
恩恩,要不是昨天的代码评审,这样小的地方的代码优化我是要工作很久之后才能发现的。看来代码评审对自身能力提高是加速剂吖,哈哈。
这两个方法是我暂时发现的,要是后续再发现什么好方法,再补充进来~