java中带有逻辑的setter

Question

考虑在 Java 中存储一些文本的 Message 对象。

public class Message {

    private String text;
    private boolean containsDigit;

    public Message() {
        //constructor
    }

    public String getText() {
        return text;
    }

    public void setText(String text) {
        this.text = text;
    }

    public boolean isContainsDigit() {
        return containsDigit;
    }

}

这是一个持久化对象。

我对在构造函数中设置数据没有任何问题，但是可以在创建对象之后设置 Message 的文本字段，并且在设置文本时，此后 containsDigit 字段也应该是可查询的。 显而易见的方法是在 setter 中：

public void setText(String text) {
     // presume existence of method to check for digit
     if(text.containsDigit())
         this.containsDigit = true;

     this.text = text;
}

但这是否会导致由于 setter 方法中的逻辑而导致任何“最佳实践”警报响起？

有人会建议替代实现吗？

编辑

我可能应该补充一点，containsDigit 字段是必需的，因为该对象是持久的，因此可以随后查询 containsDigit 字段。 此外，在使用 Spring/Hibernate 引擎的应用程序中，在重新读取/写入对象时会不断调用此设置器，因此也想知道它的实用性/效率。

Answer 1

您的情况正是使用 setter 和 getter 的原因。如果不允许您在 setter 中使用逻辑，那么您不妨直接访问这些字段！

Answer 2

这可能不是最佳实践，但有时生活比最佳实践更强大。

可能更好的方法是删除 field containsDigit 并将您添加到 setter 的逻辑移动到 isContainsDigit()：

public class Message {
    private final static Pattern d = Pattern.compile("\\d");
    private String text;

    public String getText() {
        return text;
    }

    public void setText(String text) {
        this.text = text;
    }

    public boolean isContainsDigit() {
        return text == null ? false : d.matcher(text).find();
    }

}

Answer 3

实现方法public boolean isContainsDigit()如下：

public boolean isContainsDigit() {
    return getText().containsDigit();
}

这样您就不必让它们保持同步，而不必一次又一次地重新评估它。另一方面，如果您不需要这样做，则永远不要对代码进行性能优化。如果两个线程同时访问 setText() 和 isContainsDigit() 方法，则它们属于竞争条件。如果你想解决这个问题，也许你必须同步它们。

Answer 4

@dre - 真的太奇怪了。除了争论某种设计的优雅或纯粹性之外，Saint 链接的任何链接都没有与您的问题有很大的区别。如果你有一个“setter”，那么重点是能够用一些逻辑来围绕变量的设置，以确保变量得到正确设置，同时考虑到类中其他属性存在的依赖关系，前提是出现这种需要。如果“setter”将是“纯”的，那么摆脱它们并将变量公开。 Robert Martin 的“干净代码”一书中实际上可能有一些内容明确地谈到了 occam 剃刀原则或简约原则。也许这个链接有帮助：http://effectivesoftwaredesign.com/2013/08/05/simplicity-in-software-design-kiss-yagni-and-occams-razor/。最后应该注意的是，完全不建议将属性设为公开，因为返回并将属性设为私有是一场噩梦。我重申-首先选择使您​​的类不可变，并且所有成员属性都是私有的。如果该类的用户尖叫，请他们说明为什么需要对其进行变异。一旦考虑到他们的理由，请创建一个 setter，不要公开变量。这应该作为一组设计规则。 Hibernate 或 JPA 应该与此设计决策无关。

要使具有许多参数的类不可变，请使用构建器模式 - When would you use the Builder Pattern?

Answer 5

我可以说这是“足够好”。但是，如果您想完全遵循面向对象的设计，您将实现类似于事件onTextChange 的东西，只要 set 更改文本就会触发该事件。另一方面，您将为此事件实现处理程序，该处理程序将更新containsDigit 字段。对于您的简单示例，这确实是不必要的，但如果您希望将来在 setText() 中添加新逻辑，您可以考虑这一点。