我可以重写 Dispose 来创建一个始终调用 SaveChanges 的实体类吗？

发布于 2024-12-04 00:23:13 字数 911 浏览 5 评论 0原文

这是一个相当好的观点，我希望答案是“这从一开始就不是一个热门的想法”——也就是说，如果有人足够好意的话，它有一些我感兴趣的观点。

模型代码：

public partial class MyEntities : ObjectContext
{
    // the idea is if the object is in a using block, this always gets called?
    protected override void Dispose(bool disposing)
    {
        this.SaveChanges();
        base.Dispose(disposing);
    }
}

客户端代码：

using(var model = new MyEntities())
{
   // do something

   // no worry about calling model.SaveChanges()
}

我不确定的问题是：

Dispose 是否是执行此操作的正确位置，因为出于某种原因我在考虑“最终化” - 我总是对 C# 销毁感到困惑。

如果客户端代码中抛出异常，通常会跳过 SaveChanges，这很好，但如果这按照我的想法工作，它总是会调用它。我应该将 try 与空 catch 一起使用吗？

公共分部类 MyEntities : ObjectContext
{
    受保护的覆盖无效处置（布尔处置）
    {
        尝试
        {
           this.SaveChanges();
        }
        抓住 {}
        基础.处置（处置）；
    }
}

原文

This is a fairly fine point, and I expect the answer is "it's not a hot idea to begin with" - that said, it has a points that I'm interested in regardless, if someone is kind enough to indulge.

Model Code:

public partial class MyEntities : ObjectContext
{
    // the idea is if the object is in a using block, this always gets called?
    protected override void Dispose(bool disposing)
    {
        this.SaveChanges();
        base.Dispose(disposing);
    }
}

Client Code:

using(var model = new MyEntities())
{
   // do something

   // no worry about calling model.SaveChanges()
}

The issues I'm uncertain about are:

Is Dispose the right place to do this because I was thinking "Finalize" for some reason - I always get confused on C# destruction.

In the case an exception is thrown in the client code, ordinarily SaveChanges would be skipped and that's good, but if this works how I think, it'll always call it. Should I use try with an empty catch?

public partial class MyEntities : ObjectContext
{
    protected override void Dispose(bool disposing)
    {
        try
        {
           this.SaveChanges();
        }
        catch {}
        base.Dispose(disposing);
    }
}

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

你的笑 2024-12-11 00:23:13

不要这样做。这是一个坏主意。

“Dispose”的目的是尽早礼貌地处置非托管资源，以便其他进程可以使用它。 “Dispose”不应该具有语义——它不应该改变程序的状态或者以某种方式被要求。它应该只做它所说的事情：处置资源。

您应该在终结器中执行此操作吗？ 绝对不是。那就更糟了。终结器可能根本不运行，终结器在另一个线程上运行，即使对象未正确初始化也可以调用终结器，等等。 编写终结器几乎从来都不是正确的事情，而且如果您确实编写了终结器，它也应该只处理资源。不要在终结器中做任何花哨的事情；不要在终结器中做任何花哨的事情；如果你这样做，你几乎肯定会编写出一个危险且不正确且脆弱的程序。

这里坚持的正确原则是：如果出于语义原因需要调用，则强制用户将调用放入代码中。如果他们忘记这样做，他们会在测试中找到答案。 让用户决定将调用放入finally 块中是否正确。不要为他们做出这个决定；你可能决定错误。