Posted in Javascript onFebruary 15, 2012
最常见的方式就是:
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("&");
这个方法可读性可能比对象的方法差一些,但是维护性也比较高,代码量也比较少。相对来说这个方法我还比较喜欢。
恩恩,要不是昨天的代码评审,这样小的地方的代码优化我是要工作很久之后才能发现的。看来代码评审对自身能力提高是加速剂吖,哈哈。
这两个方法是我暂时发现的,要是后续再发现什么好方法,再补充进来~
js URL参数的拼接方法比较
声明:登载此文出于传递更多信息之目的,并不意味着赞同其观点或证实其描述。
Reply on: @reply_date@
@reply_contents@