2016-11-09 28 views
1

我有下面的代码,它工作正常。我的问题是天气下面的代码是线程安全或不。通过阅读Servlet Filter和Container * Filter概念,我感到困惑。ContainerRequestFilter和ContainerResponseFilter线程安全吗?

如何使此过滤器为线程安全?

@Provider 
public class ResourceLoggingFilter implements ContainerRequestFilter, ContainerResponseFilter { 

@Context 
private ResourceInfo resourceInfo; 

@Context 
private HttpServletRequest servletRequest; 

@Inject 
private java.util.logging.Logger logger; 

public void filter(ContainerRequestContext requestContext, ContainerResponseContext responseContext) throws IOException { 
    String stTime = (String) requestContext.getProperty("StartTime"); 
    logger.log(Level.INFO,"<== Leaving Resource Path: "+ requestContext.getUriInfo().getPath()); 
    logger.log(Level.INFO,"<== Leaving Resource Method: "+ resourceInfo.getResourceMethod().getName()); 
    logger.log(Level.INFO,"<== Leaving Resource class: "+ resourceInfo.getResourceClass().getCanonicalName()); 
    logger.log(Level.INFO,"<== Leaving Session id: "+ servletRequest.getSession().getId()); 
    if (null == stTime || stTime.length() == 0) { 
     logger.log(Level.INFO,"start-time not captured or cleared"); 
     stTime = "0"; 
    } 
    long startTime = Long.parseLong(stTime); 
    long executionTime = System.nanoTime() - startTime; 
    logger.log(Level.INFO,"Total execution time : "+executionTime+" nano seconds." ); 

} 

public void filter(ContainerRequestContext requestContext) throws IOException { 
    requestContext.setProperty("StartTime", String.valueOf(System.nanoTime())); 
    logger.log(Level.INFO,"==> Entering Resource Path: "+ requestContext.getUriInfo().getPath()); 
    logger.log(Level.INFO,"==> Entering Resource Method: "+ resourceInfo.getResourceMethod().getName()); 
    logger.log(Level.INFO,"==> Entering Resource class: "+ resourceInfo.getResourceClass().getCanonicalName()); 
    logger.log(Level.INFO,"==> Entering Session id: "+ servletRequest.getSession().getId()); 

} 

} 

编辑 是好有private volatile String stTime;声明,让stTime可以是线程安全的?

回答

1

它已经是线程安全的。 ResourceInfoHttpServletRequest都是代理(使用线程本地),而java.util.Logger上的方法是线程安全的。

见ASLO:

+0

如果多个响应访问字符串stTime =(字符串)requestContext.getProperty( “开始时间”); ?? – CHowdappaM

+0

它是特定的请求。每个请求都得到它自己的线程。 –

+0

感谢您的快速响应,我在想,filter()在响应中不是线程安全的,而stTime是由另一个响应/线程重写,而第一个线程计算底部的执行时间。 – CHowdappaM