最近审核团队成员代码，发现确实有很多不规范的地方，其中问题之一就是将request、response、session等参数到传入到sevice层处理，在sevice中通过request对象获取到参数后再进行对应的业务逻辑处理，虽然功能上实现了，但这是一个很不好的编码习惯。尤其是在正式的项目，为方便以后维护绝不能这样做。
首先这有点破坏分层的架构，本来用了MVC架构，controller层该处理的事不应该传递到sevice层，否则加重了代码依赖，失去了分层的意义。
其次，这样做不适合做单元测试. 如果你想对Service层的方法做单元测试. 但是其参数是HttpServletRequet, 此时你必须用Mock的Request对象或者自己生成Request stub片段来完成.。再次，这样做很不利于将来扩展，如果Service的客户端不仅仅是Web的Controller, 假如还有一个走其他协议的RPC框架, 它就没法构造(或者说完全没有必要)HttpServletRequet。