Findbugs 有关 Java 中日期对象可变性的问题

Question

这更多的是问题 1 和2。

正如问题中所述，下面的代码

public Date getSomeDate() {
   return someDate;
}

将为您提供 findbug 错误 问题。

建议的解决方案是在 getter 和 setter 中复制 Date 对象，例如

public Date getSomeDate() {
  return new Date(someDate.getTime());
} 

这是一个好的方法还是有其他方法吗？

java中有没有可用的不可变日期库可以解决这个问题？

Answer 1

请注意，

除了调整 getter 和 setter 之外，您还需要注意空值：

public Date getSomeDate() {
  if (this.someDate == null) {
    return null;
  }
  return new Date(this.someDate.getTime());
}

public void setSomeDate(final Date someDate) {
  if (someDate == null) {
    this.someDate = null;
  } else{
  this.someDate = new Date(someDate.getTime());
  }
}

Answer 2

JodaTime 具有不可变的日期。

当然，在 getter 中使用 Date 构造函数是可以的，为什么不呢？

也就是说，仅仅因为 FindBugs 将可变状态标记为潜在错误，并不意味着它本质上值得关注——这取决于类的使用方式。不变性消除了一种类型的错误，您可能需要也可能不需要太关心。

Answer 3

根据您的用例，您可以返回 someDate.getTime() 而不将其包装在 Date 中。

Answer 4

等一下...通过在 getSomeDate 和 setSomeDate 方法中复制对象，我们并没有消除安全风险，因为更改的对象通过 setSomeDate< 返回/code> 和副本保留更改的值。为了解决这种安全问题，有必要删除 setSomeDate ，或者根本不用担心它。