Java OO静态实用工具方法帮助！

Question

我最近在User类中做了一个方法，看起来像这样；

  public static boolean checkUN(String username) {
  boolean check = false;
  ResultSet rs;
  String dbQuery;
  SQLController db = new SQLController();
  db.setUp();
  dbQuery = "SELECT * FROM User WHERE User_name ='" + username + "'";
  try {
   db.setUp();
   rs = db.readRequest(dbQuery);
   if (rs.next()) {
    check = true;
   }
  } catch (Exception e) {
   e.printStackTrace();
  }
  db.terminate();
  return check;
 }

我打算在用户能够继续下一步注册之前对其进行验证检查。

当我把它拿给老师看时，她说它会好的，因为我把它当作一种实用的方法。然而，她后来改变了主意，说我应该将其更改为一个实例方法，然后创建一个新的User对象来进行验证。

哪种方式更有效？

user.checkUsername(jTextUN.getText());

和实例化的方式(假设我改变了方法，去掉了static和输入参数)；

User user = new User();
user.setUsername(jTextUN.getText());
user.checkUsername();

干杯!

Answer 1

我可能会创建某种UserValidator类，并创建该类的一个实例来包含您的方法。

Answer 2

首先，我不喜欢这样的解决方案，即创建一个新的User对象，然后对其调用checkUsername()，原因如下：

• 如果不存在具有给定用户名的有效用户，则不应该有任何表示该用户的User对象...因为他们并不存在。这并不是一个硬性的规则，因为有时你可能需要一个User对象来表示你将要创建的用户或者类似的东西，但是在这里我发现这是不自然的。
• 它违背了对数据对象能做什么的期望。创建一个new User()，然后让checkUsername()根据提供给用户的用户名返回不同的值，这将是令人惊讶的。

现在，我知道你们正在上课，所以这可能超出了你们所学的范围，但更进一步：

对我来说，这两种解决方案似乎都不是很好，因为这两种解决方案都将使用它们的任何代码紧密地耦合到数据库中，使得代码很难测试。

对此的一般解决方案是包装代码，这将使使用它的组件难以在接口中进行测试，如下所示：

public interface UserService {
  boolean checkUsername(String username);
  ...
}

然后，您可以创建一个与数据库对话的UserService实现，需要使用该代码的类可以在其构造函数中注入该代码的实现：

public class UserServiceClient {
  private final UserService userService;

  public UserServiceClient(UserService userService) {
    this.userService = userService;
  }

  ...
}

这就是依赖注入的原理。除了使您的代码更灵活之外，它还允许您提供用于测试的假UserService实现。如果您想测试当checkUsername返回true时某个类中发生了什么，以及当它返回false时会发生什么，那么您可以使用总是返回true或始终返回false的伪实现。您不必担心设置数据库连接或确保存在正确的数据，或者确保在测试后正确地重置数据库状态，同样重要的是，当测试不需要进行数据库通信时，它的速度要快得多。

介于这两种方法之间的另一件事是创建一个存储用户数据的User对象(但不能与数据库或其他类似的东西通信)，并在UserService中放置一个类似下面这样的方法

User getUser(String username);

此方法将返回具有给定用户名的用户的User对象，否则返回null。您也可以将checkUsername实现为

return getUser(username) != null;

Answer 3

好的，第二种方法更具扩展性和直观性(如果你想在以后改变一些东西)，所以我推荐它。但第一种方法直接解决了问题，如果“用户”的概念不能在以后扩展(使用新方法)，那么任何一种都可以。