[反例]超短代码,意义不明

	public String logout(HttpServletRequest request, HttpServletResponse response) throws Exception {
		String ticket = getTicket(request);
		if (ticket == null || ticket.equals("")) {
			return "success";
		}
		setCookie(response, "", -1);
		setSecure(response, "", -1);
		return "success";
	}

这种代码短到吐血
但可看价值真的不高

评论

9 楼 yikaicheng_happy 2010-08-20  

抛出异常的爱 写道

1.只有一种返回
2.setSecure意义不明
3.getTicket如果返回Cookie什么问题都解决了
非要返回string,还不如返回true false.

1:关于一种返回，应该就是到登陆页面去了，没什么不可啊
2:setSecure意义不明,确实不晓得啥意思
3:返回cookie的话，难道cookie保存着用户信息？

8 楼 抛出异常的爱 2010-08-20  

1.只有一种返回
2.setSecure意义不明
3.getTicket如果返回Cookie什么问题都解决了
非要返回string,还不如返回true false.

7 楼 yikaicheng_happy 2010-08-20  

不过就是不晓得，在setSecure和setCookie方法中是否清除了ticket对应的用户信息

6 楼 抛出异常的爱 2010-08-20  

unika_ly12 写道

清除token 

你不对代码有想吐的感觉么?

5 楼 unika_ly12 2010-08-20  

单点登陆清除token

4 楼 抛出异常的爱 2010-08-20  

描述一下?

3 楼 yikaicheng_happy 2010-08-20  

抛出异常的爱 写道

你可以试着读懂
之后你会吐血...

我还真的读懂了

2 楼 抛出异常的爱 2010-08-20  

你可以试着读懂
之后你会吐血...

1 楼 yikaicheng_happy 2010-08-20  

何以见得？