更新实体时,我需要层叠许多更改。 我可以像下面那样做,但是那真是丑陋的恕我直言(已经有太多参数在计数了)。

public class PageRepository : IPageRepository
{
    public void Update(Page page, string oldPath, PageState oldState, IEnumerable<Tag> oldTags /* Maybe even more stuff */)
    {
        using(var t = session.BeginTransaction())
        {
            if(/* path has changed */)
            {
                // move descendant pages
            }
            if(/* state has changed to public */)
            {
                // publish hidden descendant pages
            }
            if(/* state has changed to non-public */)
            {
                // hide public descendant pages
            }
            if(page is ITaggable)
            {
                foreach(var tag in ((ITaggable)page).Tags.Except(oldTags))
                {
                    // increase tag count in ancestor pages
                }
                foreach(var tag in oldTags.Except(((ITaggable)page).Tags))
                {
                    // decrease tag count in ancestor pages
                }
            }

            // ...

            session.Update(page);
            t.Commit();
        }
    }
}

是否有任何模式可以帮助使它更清洁?

我已经考虑过将其拆分为多种方法

pageRepo.BeginTransaction();
pageRepo.Update(page);
if(/* path has changed */)
{
    pageRepo.MoveDescendants(page, oldPath);
}
// ...

pageRepo.Commit();

但我不喜欢手动交易处理。 我得到的另一个想法是

using(var batch = pageRepo.CreateUpdateBatch(page))
{
    if(/* path has changed */)
    {
        batch.MoveDescendants(oldPath);
    }
    // ...
    batch.Commit();
}

===============>>#1 票数:1

这不属于存储库。 级联的更改应由某种工厂或服务类来管理,这些类可协调所有更改。

使您的存储库仅与检索和保存数据有关,而与维护数据完整性无关。 这些业务规则应该放在域代码中,而不是存储库实现中。

  ask by Mika Kolari translate from so

未解决问题?本站智能推荐: