java - 在 setter 中进行修剪是一种好习惯吗？

Question

我正在做代码审查，我注意到这样的代码:

@Entity
@Table(name = "SOME_TABLE")
public class SomeReportClass {

@Column(name = "REPORT_NUMBER", length = 6, nullable = false)
private String reportNumber;

.....
    public String getReportNumber() {
        return reportNumber;
    }

    public void setReportNumber(String reportNumber) {
        this.reportNumber = StringUtils.trimToNull(reportNumber);
    }

}

每次我看到 setter 内部的修剪时，我都觉得这不是最清晰的解决方案 - 这个问题的一般做法是什么？

Answer 1

如果您知道您总是需要修剪该值，则此方案将避免代码重复，即在你的设置之前你必须总是修剪并担心你错过了修剪的地方。在我看来，将它放在 setter 中是一个很好的做法