Skip to content

Возможность назначения имени и поиск пользователей#3

Open
Krotkaya wants to merge 1 commit intomasterfrom
add-search-users
Open

Возможность назначения имени и поиск пользователей#3
Krotkaya wants to merge 1 commit intomasterfrom
add-search-users

Conversation

@Krotkaya
Copy link
Copy Markdown
Owner

Возможность получения username через команду /start и поиск пользователей в базе данных через /searchUser

Возможность получения username через команду /start и поиск пользователей в базе данных через /searchUser

@Override
public Response executeCommand(Request request, Matcher matched, User currentUser) {
try {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше всего этот try внести из каждой команды и расположить в месте, где вы вызываете этот метод

try (Session session = sessionFactory.openSession()) {
String hql = "FROM User u WHERE u.username LIKE :usernamePart";
Query<User> query = session.createQuery(hql, User.class);
query.setParameter("usernamePart", "%" + usernamePart + "%");
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Кажется, что % можно унести внутрь выражения, чтобы много конкатинаций не было

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants