查询用户

Question

寻找上进行身份验证时查询用户的一些反馈时JPA潜在的性能问题。我在编写下面的代码时首先想到的是通过查询数据库中的所有用户名并检查提供的用户名是否属于用户列表来获取用户名。当用户表增长时，这种类型的检查会成为性能问题吗？

EntityManager entityManager = factory.createEntityManager(); 
    /*Create a data structure to hold a list of users in our database*/ 
    List<String> allUsernames = new ArrayList<String>(); 

    allUsernames = entityManager.createQuery("SELECT user.username FROM Users user").getResultList(); 

    /*Loop through each user in our available usernames checking if the username passed exists*/ 
    for (String user : allUsernames) { 

     if (user.equals(username)) { 
      System.out.println("Found real user\n"); 
      userFoundFLAG = 1; // set the flag equal to 1 when this user is found 
     } 

    } 

Answer 1

为什么不按名称查询用户？如果返回非零列表，则用户存在，否则不存在。更好的是，为什么不直接阅读整个用户对象呢？可以想象，您将需要它来获得额外的认证/授权逻辑？

final TypedQuery<String> query = entityManager 
     .createQuery("SELECT u.username FROM Users u WHERE u.username = :username", String.class); 
query.setParameter("username", "BillyBob"); 
if (query.getResultList().size() > 0) { 
    // ... 
} 

或者：

final TypedQuery<User> query = entityManager 
     .createQuery("SELECT u FROM Users u WHERE u.username = :username"); 
query.setParameter("username", "BillyBob"); 
final User user = query.getFirstResult(); 

Answer 2

是的，这将是一个性能问题，成长为更大的问题，随着用户数量的增加。绝对不要这样做。

你应该这样做：

try { 
    User user = (User)entityManager.createQuery("SELECT user FROM Users user where user.username = :username") 
     .setParameter ("username", username); 
     .getSingleResult(); 
    userFoundFLAG = 1; 
} catch (NoResultException e) { 
    //There is no user with this username - do something 
} catch (NonUniqueResultException e) { 
    //There is more than one user with this username - do something 
}