Question

我们今天在生产中发生了重大停电，内存从我们的网络服务器很快就消失了。这可以追溯到Ninject中的缓存机制（我认为它是激活缓存或其他东西 - 不完全确定）。在调查问题之后，我们得出结论，我们在范围回调中有一个循环引用。

class View
{
    Presenter presenter;

    View()
    {
        //service locators are smelly, but webforms forces this uglyness
        this.presenter = ServiceLocator.Get<Func<View, Presenter>>()(this);

        this.presenter.InitUI();
    }
}

class Presenter
{
    CookieContainer cookieContainer;
    View view;

    Presenter(View view, CookieContainer cookieContainer)
    {
        this.view = view;
        this.cookieContainer = cookieContainer;
    }
}

class CookieContainer
{
    HttpRequest request;
    HttpResponse response; 

    CookieContainer()
    {
        this.request = HttpRequest.Current.Request;
        this.response = HttpRequest.Current.Response;
    }
}

Bind<Func<View, Presenter>>().ToMethod(ctx => view => 
        ctx.Kernel.Get<Presenter>(new ConstructorArgument("view", view)));

Bind<Presenter>().ToSelf().InTransientScope();
Bind<CookieContainer>().ToSelf().InRequestScope();

这是导致问题的代码的表示。貌似发生了什么，CookieContainer的范围回调是HttpContext.Current，而CookieContainer也引用了HttpContext.Current。所以Ninject永远不会从其缓存中删除CookieContainer实例，因为CookieContainer实例保持其范围回调对象的活动。当我们将CookieContainer的范围更改为瞬态时，一切正常，正如我们预期的那样。但我仍然不完全确定为什么会发生这种情况，因为看起来这是一个相当传统的做法呢？也许不是......

我也很困惑，因为我认为如果回调对象保持活着状态，那么Ninject就不应该从缓存中移回相同的实例，看到回调仍然存在，所以实例应该看似在范围内？为什么ninject会不断获取CookieContainer的新实例并缓存它们？我想还有其他问题与不正确的对象有关，但这至少只是一个错误，而不是内存泄漏。

我的问题是a）我们是否正确诊断出此错误？ b）是否有建议的方法来避免再次发生这种情况？ c）我可以在代码中加入修复以检查这种类型的循环依赖性（假设我们已经正确诊断了这个）吗？

Answer 1

简单地说，缓存是实例的弱引用范围对象的字典。只要范围是活动的，引用的对象也会保持活动状态。所以，如果你的CookieContainer引用了HttpContext.Current并且在请求范围内，那么这个标准机制永远不会适用于释放它们。

但是在InRequestScope的特殊情况下，OnePerRequestModule实现了另一个释放机制，它将在请求完成后立即释放所有InRequestScoped对象。如果您使用的是Ninject.Web或Ninject.Web.MVC3的最新版本，则它已预先配置。否则，您必须通过在web.config中配置此HTTPModule来显式添加它。

你不理解的另一点是Ninject只要它存在就不会返回同一个对象。例如。在请求范围内，它将为请求返回相同的对象。如果多个请求同时运行，则它们都会获得不同的实例。

由于循环范围回调引用而导致内存泄漏

1 个答案: