Vaadin ThreadLocal для управления пользователями - PullRequest
0 голосов
/ 31 мая 2018

В настоящее время я занимаюсь разработкой веб-приложения на Java, к которому одновременно могут обращаться несколько пользователей, и поэтому мне необходимо хранить пользовательские данные, чтобы адаптировать приложение к их индивидуальным требованиям (например, к какой компании они принадлежат)так далее).

Есть 2 класса, которые я использую для управления этим.Пользователь, MainSystem, подробно описан ниже:

Пользователь

@Entity
public class User implements Serializable{
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;
    private String firstName;
    private String lastName;
    private String username;
    private String password;
    private String type;
    private String company;
    private String DOB;
    private String email;
    private int PayrollId;

    public User(String firstName, String lastName, String username, String password, String type, String company, String DOB, String email) {
        this.firstName = firstName;
        this. lastName = lastName;
        this.username = username;
        this.password = password;
        this.type = type;
        this.company = company;
        this.DOB = DOB;
        this.email = email;
    }

MainSystem:

public class MainSystem   {

    public UserController userController;
    private UserRepository userRepository;
    private static ThreadLocal<User> loggedInUser = new ThreadLocal<>();
    public DbController dbController;


public MainSystem(){
userController = new UserController(userRepository);
loggedInUser.set(new User());
}

 public Boolean Login(String username, String password) {
        if(userController.checkUser(username,password)){
            User aUser = userController.getUser(username);
            setLoggedInUser(userController.getUser(username));
            VaadinSession.getCurrent().setAttribute("username",loggedInUser.get().getUsername());
            System.out.println("Logged in User:  "+loggedInUser.get().getUsername());
               return true;
        }
        else {
            return false;
        }
        }

        public static void setLoggedInUser(User user){
        loggedInUser.set(user);
        }

    public static User  getLoggedInUser() {
        return loggedInUser.get();
    }

В идеале я хотел бы получить доступ кПеременная ThreadLocal из другого класса, например ViewProfile.View:

public class EditProfileView extends VerticalLayout implements View {

    MainSystem main = new MainSystem();
    NavigatorUI aUI = new NavigatorUI();
    User aUser = main.getLoggedInUser();
    TextField username = new TextField("Username");
    TextField Id = new TextField("Id");
    TextField email = new TextField("Email");
    TextField firstName = new TextField("First name");
    TextField lastName = new TextField("Last name");
    TextField type = new TextField("Type");
    PasswordField oldPassword = new PasswordField("Current Password");
    PasswordField changePassword1 = new PasswordField("New Password");
    PasswordField changePassword2 = new PasswordField("Confirm Password");
    private UserController userController;
    private UserRepository userRepository;

    public EditProfileView() {
        setDefaultComponentAlignment(Alignment.MIDDLE_CENTER);
        userController = new UserController(userRepository);
     setStyleName("backgroundImage");
     setMargin(true);
     setSizeFull();
     addComponent(aUI.getHeader());
     FormLayout content = new FormLayout(generateInfo());
     Panel aPanel = new Panel("Edit User",content);
     aPanel.setWidthUndefined();
    content.setMargin(true);
    aPanel.setStyleName(ValoTheme.PANEL_WELL);
    addComponent(aPanel);
    }
 @Override
    public void enter(ViewChangeListener.ViewChangeEvent event) {
       try {
           aUser = main.getLoggedInUser();
        System.out.println( aUser.getUsername());
        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();}
        catch (Exception e){
           System.out.println(e.getMessage());
        }
    }}

Однако, я нахожу, что он представляет мне «нулевое» значение?Боюсь, я неправильно понял, как работает ThreadLocal.но, по сути, я пытаюсь добиться того, чтобы сохранить относящуюся к экземпляру переменную User.Class в MainSystem для использования другими классами?

Буду признателен за любую помощь.

Мое решение:

Мое решение для этого состояло в том, чтобы сохранить класс User.class в атрибуте VaadinSession следующим образом:

public Boolean Login(String username, String password) {
        if(userController.checkUser(username,password)){
            User aUser = userController.getUser(username);
            VaadinSession.getCurrent().setAttribute("user",aUser);
                          VaadinSession.getCurrent().setAttribute("username",loggedInUser.get().getUsername());
            System.out.println("Logged in User:  "+loggedInUser.get().getUsername());
               return true;
        }
        else {
            return false;
        }
        }

Затем в других классах, если я хочу использовать этот атрибутя нашел это так:

@Override
    public void enter(ViewChangeListener.ViewChangeEvent event) {
       try {
           aUser = (User)VaadinSession.getCurrent().getAttribute("user");
       }
        catch (Exception e){
           System.out.println(e.getMessage());
        }
    }

Ответы [ 2 ]

0 голосов
/ 05 июня 2018

Есть и другой подход.Вместо этого можно создать пользовательский сервис @SessionScoped, а также @Inject или @Autowire, в зависимости от того, используете ли вы CDI или Spring.Поступая таким образом, именно менеджер компонентов будет заботиться о привязке правильной сущности к вашему представлению.

0 голосов
/ 31 мая 2018

Проблема в том, что нет гарантии, что MainSystem.Login() и EditProfileView.enter() произойдут в одном потоке.Каждое действие пользователя обрабатывается как отдельный HTTP-запрос на запуск контейнера сервлета в любом доступном потоке.

Для такого рода функций я бы рекомендовал хранить информацию пользователя в сеансе HTTP.Если у вас нет пользовательских сервлетов или чего-то подобного, вы можете вместо этого иметь поле, содержащее объект пользователя в вашем собственном пользовательском классе UI.Vaadin заботится о том, чтобы UI.getCurrent() всегда возвращал правильное значение во всем коде, который выполняется через Vaadin.

Если вы вместо этого также интегрируетесь с другими функциями сервлета, вы можете вместо этого сохранить пользователя в HttpSession,Общий код сервлета может найти сеанс с помощью метода getSession(), например, в запросах и ответах сервлета.В коде, запущенном Vaadin, вы можете использовать VaadinSession().getCurrent().getSession(), чтобы получить экземпляр WrappedSession, основанный на тех же данных HttpSession.

...