Bezdepbonusu7Программирование: статьи и ответы
Меню

Вопрос

Плохо ли иметь вызовы базы данных в классах фильтров?

Автор вопроса: Davoin ShowerhandlesОтветов: 1

У нас есть веб-приложение. Мы хотим предотвратить множественный вход одних и тех же пользователей. По сути, мы хотим, чтобы это произошло так:

  1. Джон входит в приложение как "user1".
  2. Машина Джона немного зависает по какой-то неизвестной причине.
  3. Джон не может ждать. Джон переходит на компьютер2, открывает там браузер и входит в систему как "пользователь1".
  4. Сеанс Джона в браузере на первом компьютере теперь должен быть аннулирован.

для этого мы планируем сохранять в базе данных все sessionId всех вошедших в систему пользователей. Это делается для того, чтобы проверить, не вошел ли другой пользователь с тем же именем на другом компьютере или использовал другой браузер на том же компьютере. Чтобы реализовать это, мы планируем выполнить проверку идентификатора сеанса в классе Filter, который проверяет сеанс. При этом будут вызовы базы данных каждый раз, когда вызывается класс Filter.

Это хорошая практика?

РЕДАКТИРОВАТЬ:

SessionId самого последнего пользователя, вошедшего в систему, всегда будет отслеживаться. Таким образом, в примере, когда Джон входит в систему с помощью браузера на машине 2, идентификатор сеанса, сохраненный в базе данных (с компьютера 1), будет переопределен идентификатором сеанса в браузере на компьютере 2. Теперь, когда браузер на компьютере1 снова используется, он автоматически аннулирует сеанс, поскольку обнаруживает, что идентификатор сеанса для пользователя теперь отличается.

  • Сеанс Джона в браузере на первом компьютере теперь должен быть признан недействительным. Как вы можете быть уверены, что не сделаете недействительным второй компьютер Джона? Manu 01.12.2015
  • Вы хотите, чтобы сеансы были постоянными? Если нет, я не вижу необходимости хранить идентификаторы сеансов в базе данных. Просто держите их в памяти и проверяйте там. Если приложение кластеризовано, вам, возможно, придется предпринять некоторые дополнительные действия, но это все равно должно быть выполнимо таким образом. Thomas 01.12.2015
  • Вы можете использовать локальное хранилище для хранения идентификатора сеанса, и если он использует второй браузер, вы можете добавить проверку, присутствует ли идентификатор сеанса или нет. Rohan Kawade 01.12.2015
  • @Thomas: ты имеешь в виду использовать синглтоны? но мы не должны делать статические классы изменчивыми, верно? Davoin Showerhandles 01.12.2015
  • @Rohan Kawade: ты имеешь в виду использовать файл? Davoin Showerhandles 01.12.2015
  • Это был бы не совсем синглтон. Это было бы больше похоже на кеш, CDI-компонент области приложения и т. д. - сам экземпляр не был бы статичным. Thomas 01.12.2015

javasessionservlet-filters

Ответы

Ответ 1 · Serge Ballesta

Это зависит от того, что вы называете вызовом базы данных. Вполне допустимо, что фильтр использует значение из базы данных. Но прямые вызовы базы данных в фильтре - действительно плохая практика, потому что она не учитывает многоуровневую разработку.

Мой совет заключается в том, что фильтр должен вызывать только сервисный уровень.

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

TL/DR: использование данных, хранящихся в базе данных, в фильтре допустимо, следует избегать вызовов JDBC в фильтре.

Перевод вопроса и ответов сообщества Stack Overflow, лицензия CC BY-SA. Оригинал на Stack Overflow.

Похожие материалы