i don`t think so, but ok

Author: ynazarets

src/main/java/mate/academy/Main.java

@@ -8,6 +8,7 @@
 import mate.academy.model.CinemaHall;
 import mate.academy.model.Movie;
 import mate.academy.model.MovieSession;
+import mate.academy.model.User;
 import mate.academy.security.AuthenticationService;
 import mate.academy.service.CinemaHallService;
 import mate.academy.service.MovieService;
@@ -65,11 +66,15 @@ public static void main(String[] args) throws RegistrationException, Authenticat
         AuthenticationService authenticationService =
                 (AuthenticationService) injector.getInstance(AuthenticationService.class);
         System.out.println("register method started");
-        authenticationService.register("bob@gmail.com", "123456789qwerty!");
+        User registeredUser = authenticationService.register("bob@gmail.com", "123456789qwerty!");
         System.out.println("register method finished");
+        System.out.println(registeredUser.toString());
+
         System.out.println("-----------------");
+
         System.out.println("login method started");
-        authenticationService.login("bob@gmail.com","123456789qwerty!");
+        User loginedUser = authenticationService.login("bob@gmail.com", "123456789qwerty!");
         System.out.println("login method finished");
+        System.out.println(loginedUser.toString());
     }
 }

src/main/java/mate/academy/dao/impl/UserDaoImpl.java

@@ -34,9 +34,11 @@ public User add(User user) {
     public Optional<User> findByEmail(String email) {
         SessionFactory sessionFactory = HibernateUtil.getSessionFactory();
         try (Session session = sessionFactory.openSession()) {
-            return session.createQuery("from User where u.login = :login", User.class)
-                    .setParameter("login", email)
+            return session.createQuery("from User u where u.login = :email", User.class)
+                    .setParameter("email", email)
                     .uniqueResultOptional();
+        } catch (Exception e) {
+            throw new DataProcessingException("Error finding user by email", e);
         }
     }
 }

src/main/java/mate/academy/model/User.java

@@ -1,5 +1,6 @@
 package mate.academy.model;
 
+import jakarta.persistence.Column;
 import jakarta.persistence.Entity;
 import jakarta.persistence.GeneratedValue;
 import jakarta.persistence.GenerationType;
@@ -12,7 +13,10 @@ public class User {
     @Id
     @GeneratedValue(strategy = GenerationType.IDENTITY)
     private Long id;
+
+    @Column(unique = true)
     private String email;
+
     private String password;
     private byte[] salt;
 

src/main/java/mate/academy/security/AuthenticationServiceImpl.java

@@ -32,16 +32,12 @@ public User register(String email, String password) throws RegistrationException
     @Override
     public User login(String email, String password) throws AuthenticationException {
         Optional<User> userFromDB = userService.findByEmail(email);
-        if (userFromDB.isEmpty()) {
-            throw new AuthenticationException("User with email: "
-                    + email + " does not exist");
-        }
-
         User user = userFromDB.get();
         String hashedPasswordInput = HashUtil.hashPassword(password, user.getSalt());
-        if (user.getPassword().equals(hashedPasswordInput)) {
-            return user;
+
+        if (userFromDB.isEmpty() || !user.getPassword().equals(hashedPasswordInput)) {
+            throw new AuthenticationException("Wrong password, try again.");
         }
-        throw new AuthenticationException("Wrong password, try again.");
+        return user;
     }
 }