Vaadin中ThreadLocal存储用户实例返回null问题及解决方案

看起来你已经搞定了核心问题，但咱们先拆解下为什么之前用ThreadLocal会返回null，再给你补充点实用的优化建议，让你的代码更健壮～

为什么你的ThreadLocal方案失效？ #

你在EditProfileView里做了这个操作：

MainSystem main = new MainSystem();

问题就出在这儿——虽然loggedInUser是MainSystem里的静态ThreadLocal变量，但你在MainSystem的构造函数里还执行了：

loggedInUser.set(new User());

这会把当前线程的ThreadLocal值直接重置成一个空的User对象！就算之前登录时在别的地方设置过有效值，新建MainSystem实例时也会被覆盖，所以后续调用getLoggedInUser()自然拿到的是null。

另外，Vaadin的请求模型里，不同的用户请求可能由不同的线程处理，ThreadLocal是线程隔离的，但它没法跨线程、跨请求保存用户会话信息——这本来就不是它的设计场景，Vaadin的VaadinSession才是专门用来管理用户会话数据的工具。

你的VaadinSession解决方案完全正确 #

改用VaadinSession存储登录用户是Vaadin应用里的标准做法，这里可以给你小优化下代码：

登录逻辑优化 #

public Boolean Login(String username, String password) {
    if(userController.checkUser(username,password)){
        User aUser = userController.getUser(username);
        VaadinSession.getCurrent().setAttribute("user", aUser);
        // 直接用aUser的属性，不用再依赖ThreadLocal了
        VaadinSession.getCurrent().setAttribute("username", aUser.getUsername());
        System.out.println("Logged in User: " + aUser.getUsername());
        return true;
    } else {
        return false;
    }
}

View中获取用户的逻辑优化 #

你在enter方法里的写法没问题，不过可以加个非空判断，避免空指针：

@Override
public void enter(ViewChangeListener.ViewChangeEvent event) {
    try {
        aUser = (User) VaadinSession.getCurrent().getAttribute("user");
        if(aUser != null) {
            Id.setValue(aUser.getId().toString());
            username.setValue(aUser.getUsername());
            firstName.setValue(aUser.getFirstName());
            lastName.setValue(aUser.getLastName());
            type.setValue(aUser.getType());
            email.setValue(aUser.getEmail());
            aUI.setUserMenu();
            aUI.refreshPayroll();
        } else {
            // 处理未登录的情况，比如跳转到登录页
        }
    } catch (Exception e){
        System.out.println(e.getMessage());
    }
}

额外实用建议 #

1. 封装会话工具类：避免重复的强制类型转换，让代码更简洁：

public class SessionHelper {
    // 获取当前登录用户
    public static User getCurrentUser() {
        return (User) VaadinSession.getCurrent().getAttribute("user");
    }

    // 设置当前登录用户
    public static void setCurrentUser(User user) {
        VaadinSession.getCurrent().setAttribute("user", user);
    }

    // 判断用户是否登录
    public static boolean isLoggedIn() {
        return getCurrentUser() != null;
    }
}

之后在任何View里，直接调用SessionHelper.getCurrentUser()就能拿到用户，不用每次都写强制转换的代码。

2. 登出时清理会话：用户登出时记得移除会话里的用户属性，避免残留数据：

public void logout() {
    SessionHelper.setCurrentUser(null);
    // 跳转到登录页面的逻辑，比如：
    // UI.getCurrent().getNavigator().navigateTo("login");
}

内容的提问来源于stack exchange，提问作者James Palfrey