最常见的方式就是:
复制代码 代码如下:
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(&);
这个方法可读性可能比对象的方法差一些,但是维护性也比较高,代码量也比较少。相对来说这个方法我还比较喜欢。
恩恩,要不是昨天的代码评审,这样小的地方的代码优化我是要工作很久之后才能发现的。看来代码评审对自身能力提高是加速剂吖,哈哈。
这两个方法是我暂时发现的,要是后续再发现什么好方法,再补充进来~