声纳抱怨局部变量的无效赋值

Question

我有以下一段代码在我的程序，我正在运行SonarQube 5的代码质量检查后，它与Maven集成。

然而，Sonar要求删除这个对局部变量“session”的无用赋值。

@RequestMapping(value = "/logoff", method = RequestMethod.GET)
public String getLogoffPage(HttpServletRequest request, HttpServletResponse response) {
    logger.info(" Before Log Offf........ " + request.getSession().getId() );
    HttpSession session =request.getSession(true);
    request.getSession().invalidate();
    myApplication.logout();
    SecurityContextHolder.clearContext();
    session=null;                   
    return "login";
}

Answer 1

此变量是本地变量，因此当到达return语句时，将无法访问它。由于赋值后未读取该变量，因此该变量被视为无效变量。

如果你给一个局部变量赋值而不使用它，这是一个无用的指令，因此应该删除。

将变量设置为null几乎毫无用处，实际上可能会妨碍JVM进行某些优化。

Answer 2

假设问题是“为什么”：

你实际上用会话做什么？没什么。

HttpSession session =request.getSession(true);  // session assigned
request.getSession().invalidate();         // session NOT used
myApplication.logout();
SecurityContextHolder.clearContext();
session=null;                            // session re-assigned

也许你是这个意思？

HttpSession session =request.getSession(true);
session.invalidate();
myApplication.logout();
SecurityContextHolder.clearContext();

顺便说一句，我已经删除了session=null，因为在Java中没有理由这样做（C是另一回事）。

当然，代码可能更干净：

request.getSession().invalidate();
myApplication.logout();
SecurityContextHolder.clearContext();