Java HttpServlet，重写方法服务，最佳实践

Question

For logging purposes, I override the service method from HttpServlet like this:

@Override
protected void service(HttpServletRequest req, HttpServletResponse resp)
{
    logger.debug("enter servlet");
    logger.debug("Request Header: {}", MyHttpTools.requestHeaderToSting(req);

    super.service(req, resp);

    logger.debug("leaving servlet");
    logger.debug("Response Header: {}", MyHttpTools.responseHeaderToSting(resp);
}

I found some places in the web, where authors say "don't override service".

What do you think of my approach?

Answer 1

您的方法没有问题，但是您可以“更干净”的方式（在我看来），例如，通过实现（并注册） javax.servlet.Filter或使用AspectJ或其他更常用的方法。

Answer 2

Nothing inherently wrong with your approach.

The authors are saying that don't override service() for business code, override the doPost() , doGet() etc. methods instead.

Answer 3

Your approach is correct. You can do logging with aspectJ too.

Answer 4

The default service() method in an HTTP servlet routes the request to another method based on the HTTP transfer method (POST, GET, and so on). For example, HTTP POST requests are routed to the doPost() method, HTTP GET requests are routed to the doGet() method, and so on. This enables the servlet to perform different request data processing depending on the transfer method. Since the routing takes place in service(), there is no need to generally override service() in an HTTP servlet . Instead, override doGet(), doPost(), and so on, depending on the expected request type.

For detail plz go through this article : Sun Java System Web Server 6.1 SP6 Programmer's Guide to Web Applications