如果我们在实体类的getter和setter中添加逻辑，这是一个糟糕的设计吗

Question

JAVA 8

我有一门POJO课程：

class User {
    private String name;
    private String password;

    //... Getters / Setters ...
}

我将使用它作为实体类。

在密码的getter/setter中，我想添加解密/加密逻辑。

public String getPassword() {
    return EncryptionFactory.decryption(password);
}

public void setPassword(String password) {
    this.password = EncryptionFactory.encryption(password);
}

加密工厂（EncryptionFactory）是一个对字符串进行加密/解密的实用程序类。

根据一般的Java编码准则，如果我添加逻辑来更改密码，它是破坏设计还是糟糕的设计？

在使用它的时候，我从我的教授那里得到了糟糕的设计反馈。

Answer 1

通常setter伴随着getter（也称为访问器），它返回私有成员变量的值

由于getter是用来访问私有字段值的，因此它违反了最不令人惊讶的原则：可能会引发异常，EncrpytionFactory的impl可能会在某个时候失效，等等。。。当开发人员希望只访问一个值时。

你是否认为这个设计很糟糕取决于你的设计标准有多严格。确定的最佳方法是查看缺点：

• 无法获取加密密码
• 强制使用String作为纯文本密码来存储密码（因为setPassword会自动加密）
• 在包含get/setPassword
的类型中引入 EncrpytionFactory的依赖项

因此，在您的特定情况下，这是一个糟糕的设计。

尽管在某些情况下，一些开发人员可能会首选它。例如，如果您有一个Person#getAge（）：int，但使用了一个Date属性来管理此人的年龄：

class Person {
    private Date dateOfBirth;

    public int getAge() {
        Date now = ...; //grab date/time right now
        int age = ...; //calculate age using dateOfBirth and now

        return age;
    }

有些人会争辩说，计算应该是解耦的，或者应该命名为calculateAge。

来自wiki文章：

相反，访问器允许从内部变量合成有用的数据表示，同时保持它们的结构封装并隐藏在外部模块之外。货币getAmount访问器可以从数字变量构建字符串，其小数位数由隐藏的货币参数定义。

你真的想解密密码吗，比如，有什么理由吗？假设你想维护安全，有更安全的方法来维护密码。

无论何时，只要将输入的密码与存储的密码进行比较，只需加密输入的密码并将结果与存储的密码进行比较即可。

如果有人想“恢复”他们的密码，以明文形式向用户发送他们的密码将存在安全风险。这就是为什么大型企业要求您在忘记密码时重置密码的原因。

你真的需要在每次有人调用getPassword（）时解密吗？如果有人想要encrpyted值呢？

您应该将解密与访问分开。查看您已经拥有的加密工厂：

User user = ...;
String encryptedPassword = user.getPassword();
String decryptedPassword = EncryptionFactory.decryption(encryptedPassword);

应该只有很少几次您的密码被解密（有些人可能会认为它不应该存在于内存中解密）。关键是，它应该是完全可选的。

密码应设置为已加密：

String encryptedPassword = EncryptionFactory.encrpytion(...);

User user = ...;
user.setPassword(encryptedPassword);

它应该以加密形式访问。您永远不应该强制这样的安全风险（解密密码），使其可选。

最后，如果必须公开解密的getPassword备选方案，则应将其重命名为decryptPassword（）：

public String decryptPassword() {
    return EncrpytionFactory.decryption(getPassword());
}

尽管这在用户（或您使用的任何类型）和加密工厂（EncryptionFactory）之间引入了不必要的耦合。

您应该更深入地了解安全方面。您不应该将解密后的密码保存为字符串，如果只有用户知道密码，这是最安全的。