我试图将名称,哈希密码,盐和哈希类型插入数据库。唯一改变的是参数的类型。我相信可以更有效地做到这一点。如何避免使用重载?我需要使用泛型吗?谢谢。

插入方法

protected void insert(String name, String secretpassword, String salt, String type)
{
    String sql = "INSERT INTO login(username,password,salt,type) VALUES(?,?,?,?)";

    try (Connection conn = this.connect();
         PreparedStatement pstmt = conn.prepareStatement(sql)) {
        pstmt.setString(1, name);
        pstmt.setString(2, secretpassword);
        pstmt.setString(3, salt);
        pstmt.setString(4, type);
        pstmt.executeUpdate();
        System.out.println("Successful");
    } catch (SQLException e) {
        System.out.println(e.getMessage());
    }
}

protected void insert(String name, byte[] secretpassword, String salt, String type)
{
    String sql = "INSERT INTO login(username,password,salt,type) VALUES(?,?,?,?)";

    try (Connection conn = this.connect();
         PreparedStatement pstmt = conn.prepareStatement(sql)) {
        pstmt.setString(1, name);
        pstmt.setString(2, Arrays.toString(secretpassword));
        pstmt.setString(3, salt);
        pstmt.setString(4, type);
        pstmt.executeUpdate();
        System.out.println("Successful");
    } catch (SQLException e) {
        System.out.println(e.getMessage());
    }
}

最佳答案

我在这里看不到很多问题,因为您总是最终将密码作为字符串插入到login表中。因此,我建议始终使用第一个版本:

protected void insert(String name, String secretpassword, String salt,
    String type);


如果您以byte[]的身份输入密码,只需使用字符串的构造函数生成一个String

byte[] array = ...;
String password = new String(array);


因此,很明显,我建议可能删除第二种方法,因为除了一个参数外,它在数据库级别上的作用完全相同。

保留这两种方法意味着在代码库中进行更多的维护工作,因为如果更改一种方法,则必须在另一种方法中进行相同的逻辑更改。

10-08 19:54