有一个小需求,给前端出了一个接口,这个接口只需要调用另外一个sdk就可以了,但是写完后,前端反馈调用这个接口的时间非常久,基本上都在1000ms以上了。
try {
CounterServices counterServices = new CounterServices()
CounterBean res = counterServices.counter(ShareConstant.BUSINESSID,ShareConstant.RESULTTYPE,0,contentId);
return res.getLvalue();
} catch (NumberFormatException e) {
return 0L;
} catch (Exception e) {
log.error("share count calculate error!article={}",contentId);
log.error(e.getMessage());
throw new ServiceException(SystemCode.FAILURE);
}
在try之前和进入try之后打了log之后发现是进入try…catch的耗时比较久,接口调用就几十毫秒,以为是trycatch的问题。网上查了许多资料,说trycatch会增加耗时,但是不应该增加这么多。这次学聪明了,在new实例的时候打了一个log,发现new实例花费比较久。
再看发现,这个sdk会链接这个服务的redis,而且不是长连接,我这么写代码会导致每次调用服务的时候都会去创建这个实例。
改成静态属性就好了,只需要创建一次,如果实例意外被销毁了,再创建一次就可以了。
private static CounterServices counterServices = new CounterServices();
@Override
public Long getShareCount(String contentId) {
if (counterServices == null){
log.error("bean instance error, re-new instance.");
counterServices = new CounterServices();
}
try {
CounterBean res = counterServices.counter(ShareConstant.BUSINESSID,ShareConstant.RESULTTYPE,0,contentId);
return res.getLvalue();
} catch (NumberFormatException e) {
return 0L;
} catch (Exception e) {
log.error("share count calculate error!article={}",contentId);
log.error(e.getMessage());
throw new ServiceException(SystemCode.FAILURE);
}
}