编码风格：从模型类中进行进一步查询是否可以接受？

Question

我有一个模型类Alert与另一个模型类Occurrence有一对多的关系，如下：

@Entity public class Alert extends Model
{
  public String name;

  @OneToMany(cascade=CascadeType.ALL, fetch=FetchType.LAZY, mappedBy="alert")
  public List<Occurrence> occurrences;
}

@Entity public class Occurrence extends Model
{
  @ManyToOne
  public Alert alert;

  @Column(nullable=false)
  public Date alertTime;
}

我有一个视图，它在一个简单的表格中列出了警报，该表格应该有一个标记为“今天发生”的列，并计算今天发生了多少次警报（alertTime）。

我似乎无法找到仅使用 Alert 模型类中的 JPA/Hibernate 注释来执行此操作的方法，并且由于我列出了警报，因此我不知道将计数包含在其中的干净方法每个Alert 对象。

所以现在我想知道是否可以简单地从Alert 模型类本身的内查询警报的出现，如下所示：

@Entity public class Alert extends Model
{
  public String name;

  @OneToMany(cascade=CascadeType.ALL, fetch=FetchType.LAZY, mappedBy="alert")
  public List<Occurrence> occurrences;

  @Transient
  public long getOccurrencesToday()
  {
    return Occurrence.count(
      "alert = ? and alertTime >= ?",
      this, new DateMidnight().toDate());
  }
}

我的问题是：从模型类中查询数据库是否被认为是不好的形式？

第二个问题是：我是不是以错误的方式处理这个问题？有没有更好的方法来实现我错过的最终结果？

Answer 1

在域模型类中进行查询应该是正确的方法，因为这些查询与该模型相关。为此，我认为您的代码没有任何问题。

根据 Play 的理念，领域对象不仅仅是一个数据对象。它应该在里面包含一定的业务逻辑。事实上，游戏并不鼓励纯数据对象。查看更多http://www.playframework.org/documentation/1.2.4/model

Answer 2

虽然您的方法很诱人，但它很危险，因为您实际上是将逻辑放入“getter”(getOc​​currencesToday()) 中，您可能会通过 ${alert.occurrencesToday} 访问它

getter 应该始终没有副作用。

我会把它改成

@Entity public class Alert extends Model
{
  public String name;

  @OneToMany(cascade=CascadeType.ALL, fetch=FetchType.LAZY, mappedBy="alert")
  public List<Occurrence> occurrences;

  @Transient public Long occurrencesToday = null;

  public void populateOccurrencesToday()
  {
    occurrencesToday = Occurrence.count(
      "alert = ? and alertTime >= ?",
      this, new DateMidnight().toDate());
  }
}

然后在您的控制器中，遍历所有警报 您将渲染并执行 populateOccurrencesToday() 方法。 如果您现在将对象存储在缓存中，则其数据将是一致的。