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());

和实例方式 （假设我更改了方法，删除了静态和输入参数）；

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 时会发生什么，你可以使用 fake 实现始终返回 true 或始终返回 false。您不必担心设置数据库连接或确保存在正确的数据或确保测试后正确重置数据库状态，同样重要的是，当不需要执行数据库操作时，测试会快得多沟通。

您可能想要做的介于两种方法之间的另一件事是拥有一个 User 对象来存储用户数据（但无法与数据库或任何此类事物本身进行通信）并将这样的方法放入您的 UserService 中：

User getUser(String username);

此方法将为具有给定用户名的用户返回 User 对象（如果存在）并且 null 否则。您还可以将 checkUsername 实现为

return getUser(username) != null;

Answer 3

好吧，第二种方法更具可扩展性，也更直观（就您以后是否想更改内容而言），所以我推荐它。但第一种方法直接解决了问题，如果“用户”的概念以后不会被扩展（使用新方法），那么任何一种方法都应该没问题。