发现CS的一个小问题.没有彻底解决.

今天看cs代码,发现了一个小问题,如下:
当前为匿名用户,浏览到一个需要通过身份验证的版面时,页面会转向登陆页,Url是这样的:
/login.aspx?ReturnUrl=/CnForums/forums/AddPost.aspx?ForumID=3  ,  这里都是正常的,考虑这样一种情况,如果当前用户没有注册过站点的会员,这时候点击注册,Url会成为/user/CreateUser.aspx?ReturnUrl=/CnForums/forums/AddPost.aspx?ForumID , 看到错误了吧,ReturnUrl参数被截断了.ForumID后面的内容没有了.正确的应该是ReturnUrl=/CnForums/forums/AddPost.aspx?ForumID=3,分析一下原因.在CommunityServer.Components命名空间中找到SiteUrls类,先看看public virtual string UserRegister的代码,我在代码上加上了注释,所以不再多写.

 1        public virtual string UserRegister 
 2        }

cs中许多地方用到了ReturnUrl,方法都是相同的.上面的代码中有一个方法很重要.ExtractQueryParams(string s).
如果当前路径中已经存在ReturnUrl参数的话,他就会从当前路径中取出ReturnUrl,所以我上面描述的问题应该是在这里发生了问题.先看看他的代码吧,我看了很多评论者总在抱怨别人只帖代码,实质的内容比较少,但我觉得代码才是最有力的论据,当然适当的加上一些说明效果会更好.

 1        protected static NameValueCollection ExtractQueryParams(string url) 
 2        }

这里描述起来有点复杂,没有很多时间写,以后补上来.错误的地方就是在遍历url键值对的时候,

                if (pair.Length > 1)
                    value = pair[1];

这里的判断有问题,它没有考虑当路径中已经存在ReturnUrl参数的时候,很有可能ReturnUrl参数的值里还有1个或2个或更多的=号或?号.所以出现了我前面说的丢失参数的情况/user/CreateUser.aspx?ReturnUrl=/CnForums/forums/AddPost.aspx?ForumID. 不过我还没有找到比较好的解决办法.只是在判断pari.Length之前,先判断了一下if((pair.Length % 2) != 0),代码如下:

                    //url被截断.需要判断当前是否存在ReturnUrl参数,如果存在就不能这样转换.
                    //如果ReturnUrl的值有两个参数,这个方法依然是错的.
                    if((pair.Length % 2) != 0)
                    }

这样的判断方法只能用在cs这种特例中,只能是临时解决问题的方法.哪位有好的解决方法记得通知我一声.
我看了cs2.0好像已经没有这个问题了,不过还没看过cs2.0的代码,有时间再说吧.