Может ли Unity быть сделано, чтобы не вызывать синхронное исключение Exception все время?

В контейнере инъекции зависимостей Unity есть широко известная проблема, при которой SynchronizedLifetimeManager часто вызывает метод Monitor.Exit, чтобы вызывать исключение SynchronizationLockException, которое затем захватывается и игнорируется. Это проблема для меня, потому что мне нравится отлаживать с помощью Visual Studio, чтобы ломать любое исключенное исключение, поэтому каждый раз, когда мое приложение запускается, я разрываюсь на этом исключении несколько раз без причины.

Как я могу предотвратить выброс этого исключения?

В тех случаях, когда эти проблемы упоминаются в других местах в Интернете, совет обычно включает в себя изменение настроек отладчика, чтобы игнорировать его. Это сродни тому, как идти к доктору и говорить: "Доктор, доктор, моя рука болит, когда я поднимаю его", чтобы сказать: "Ну, перестань поднимать его". Я ищу решение, которое исключает исключение исключения.

Исключение возникает в методе SetValue, потому что он делает предположение, что GetValue будет вызван первым, где вызывается Monitor.Enter. Однако классы LifetimeStrategy и UnityDefaultBehaviorExtension регулярно называют SetValue без вызова GetValue.

Мне не нужно менять исходный код и поддерживать мою собственную версию Unity, поэтому я надеюсь на решение, в котором я могу добавить в контейнер некоторую комбинацию расширений, политик или стратегий, которые гарантируют, что, если диспетчер времени жизни является SynchronizedLifetimeManager, GetValue всегда вызывается перед чем-либо еще.

ОТВЕТЫ

Ответ 1

Я уверен, что существует много способов, которыми код мог бы вызвать SynchronizedLifetimeManager или потомок, такой как ContainerControlledLifetimeManager, но было, в частности, два сценария, которые вызывали у меня проблемы.

Первая была моей собственной ошибкой - я использовал инъекцию конструктора для предоставления ссылки на контейнер, и в этом конструкторе я также добавлял новый экземпляр класса в контейнер для будущего использования. Этот обратный подход повлиял на изменение менеджера жизненного цикла от Transient до ContainerControlled, так что объект Unity, называемый GetValue on, не был тем же объектом, что и SetValue. Выученный урок ничего не делает во время наращивания, что может изменить менеджера жизненного цикла объекта.

Второй сценарий заключался в том, что каждый раз, когда вызывается RegisterInstance, UnityDefaultBehaviorExtension вызывает SetValue, не вызывая сначала GetValue. К счастью, Unity достаточно растяжима, чтобы с достаточной кровавостью вы могли решить эту проблему.

Начните с нового расширения поведения следующим образом:

/// <summary>
/// Replaces <see cref="UnityDefaultBehaviorExtension"/> to eliminate 
/// <see cref="SynchronizationLockException"/> exceptions that would otherwise occur
/// when using <c>RegisterInstance</c>.
/// </summary>
public class UnitySafeBehaviorExtension : UnityDefaultBehaviorExtension
{
    /// <summary>
    /// Adds this extension behavior to the container.
    /// </summary>
    protected override void Initialize()
    {
        Context.RegisteringInstance += PreRegisteringInstance;

        base.Initialize();
    }

    /// <summary>
    /// Handles the <see cref="ExtensionContext.RegisteringInstance"/> event by
    /// ensuring that, if the lifetime manager is a 
    /// <see cref="SynchronizedLifetimeManager"/> that its 
    /// <see cref="SynchronizedLifetimeManager.GetValue"/> method has been called.
    /// </summary>
    /// <param name="sender">The object responsible for raising the event.</param>
    /// <param name="e">A <see cref="RegisterInstanceEventArgs"/> containing the
    /// event data.</param>
    private void PreRegisteringInstance(object sender, RegisterInstanceEventArgs e)
    {
        if (e.LifetimeManager is SynchronizedLifetimeManager)
        {
            e.LifetimeManager.GetValue();
        }
    }
}

Затем вам нужен способ заменить поведение по умолчанию. Unity не имеет способа удалить конкретное расширение, поэтому вам нужно удалить все и снова вставить другие расширения:

public static IUnityContainer InstallCoreExtensions(this IUnityContainer container)
{
    container.RemoveAllExtensions();
    container.AddExtension(new UnityClearBuildPlanStrategies());
    container.AddExtension(new UnitySafeBehaviorExtension());

#pragma warning disable 612,618 // Marked as obsolete, but Unity still uses it internally.
    container.AddExtension(new InjectedMembers());
#pragma warning restore 612,618

    container.AddExtension(new UnityDefaultStrategiesExtension());

    return container;
}

Обратите внимание, что UnityClearBuildPlanStrategies? RemoveAllExtensions очищает все внутренние списки контейнеров политик и стратегий, кроме одного, поэтому мне пришлось использовать другое расширение, чтобы избежать вставки дубликатов при восстановлении стандартных расширений:

/// <summary>
/// Implements a <see cref="UnityContainerExtension"/> that clears the list of 
/// build plan strategies held by the container.
/// </summary>
public class UnityClearBuildPlanStrategies : UnityContainerExtension
{
    protected override void Initialize()
    {
        Context.BuildPlanStrategies.Clear();
    }
}

Теперь вы можете безопасно использовать RegisterInstance, не опасаясь оказаться на грани безумия. Чтобы быть уверенным, здесь несколько тестов:

[TestClass]
public class UnitySafeBehaviorExtensionTests : ITest
{
    private IUnityContainer Container;
    private List<Exception> FirstChanceExceptions;

    [TestInitialize]
    public void TestInitialize()
    {
        Container = new UnityContainer();
        FirstChanceExceptions = new List<Exception>();
        AppDomain.CurrentDomain.FirstChanceException += FirstChanceExceptionRaised;
    }

    [TestCleanup]
    public void TestCleanup()
    {
        AppDomain.CurrentDomain.FirstChanceException -= FirstChanceExceptionRaised;
    }

    private void FirstChanceExceptionRaised(object sender, FirstChanceExceptionEventArgs e)
    {
        FirstChanceExceptions.Add(e.Exception);
    }

    /// <summary>
    /// Tests that the default behavior of <c>UnityContainer</c> leads to a <c>SynchronizationLockException</c>
    /// being throw on <c>RegisterInstance</c>.
    /// </summary>
    [TestMethod]
    public void UnityDefaultBehaviorRaisesExceptionOnRegisterInstance()
    {
        Container.RegisterInstance<ITest>(this);

        Assert.AreEqual(1, FirstChanceExceptions.Count);
        Assert.IsInstanceOfType(FirstChanceExceptions[0], typeof(SynchronizationLockException));
    }

    /// <summary>
    /// Tests that <c>UnitySafeBehaviorExtension</c> protects against <c>SynchronizationLockException</c>s being
    /// thrown during calls to <c>RegisterInstance</c>.
    /// </summary>
    [TestMethod]
    public void SafeBehaviorPreventsExceptionOnRegisterInstance()
    {
        Container.RemoveAllExtensions();
        Container.AddExtension(new UnitySafeBehaviorExtension());
        Container.AddExtension(new InjectedMembers());
        Container.AddExtension(new UnityDefaultStrategiesExtension());

        Container.RegisterInstance<ITest>(this);

        Assert.AreEqual(0, FirstChanceExceptions.Count);
    }
}

public interface ITest { }

Ответ 2

Исправлено в последней версии Unity (2.1.505.2). Получите его через NuGet.

Ответ 3

Ответ на ваш вопрос, к сожалению, нет. Я проследил это с помощью команды dev здесь, в группе шаблонов и практик Microsoft (ранее я был разработчиком до сих пор), и у нас это было как ошибка для EntLib 5.0. Мы провели некоторое расследование и пришли к выводу, что это было вызвано неожиданными взаимодействиями между нашим кодом и отладчиком. Мы рассматривали исправление, но это оказалось более сложным, чем существующий код. В конце концов, это стало приоритетом ниже других вещей и не делало планку на 5.

Извините, у меня нет лучшего ответа для вас. Если это утешает, я нахожу это раздражающим.

Ответ 4

Я использую это короткое решение:

/// <summary>
/// KVV 20110502
/// Fix for bug in Unity throwing a synchronizedlockexception at each register
/// </summary>
class LifeTimeManager : ContainerControlledLifetimeManager
{
    protected override void SynchronizedSetValue(object newValue)
    {
        base.SynchronizedGetValue();
        base.SynchronizedSetValue(newValue);
    }
}

и используйте его следующим образом:

private UnityContainer _container;
...
_container.RegisterInstance(instance, new LifeTimeManager());

проблема заключается в том, что базовый класс ContainerControlledLifetimeManager ожидает, что SynchronizedSetValue будет делать monitor.Enter() через base.GetValue, однако класс ContainerControlledLifetimeManager не может этого сделать (видимо, его разработчикам не было "break at exception", включен?).

С уважением, Koen

Ответ 5

Решение Rory велико - спасибо. Решил проблему, которая меня раздражает каждый день! Я сделал некоторые незначительные изменения в решении Rory, чтобы он обрабатывал все расширения (в моем случае у меня было расширение WPF Prism/Composite).

    public static void ReplaceBehaviourExtensionsWithSafeExtension(IUnityContainer container)
    {
        var extensionsField = container.GetType().GetField("extensions", BindingFlags.Instance | BindingFlags.NonPublic);
        var extensionsList = (List<UnityContainerExtension>)extensionsField.GetValue(container);
        var existingExtensions = extensionsList.ToArray();
        container.RemoveAllExtensions();
        container.AddExtension(new UnitySafeBehaviorExtension());
        foreach (var extension in existingExtensions)
        {
            if (!(extension is UnityDefaultBehaviorExtension))
            {
                container.AddExtension(extension);
            }
        }
    }

Ответ 6

Остерегайтесь одной ошибки в ответе Zubin Appoo: в коде отсутствует UnityClearBuildPlanStrategies.

Правильный фрагмент кода:

FieldInfo extensionsField = container.GetType().GetField("extensions", BindingFlags.Instance | BindingFlags.NonPublic);
List<UnityContainerExtension> extensionsList = (List<UnityContainerExtension>)extensionsField.GetValue(container);
UnityContainerExtension[] existingExtensions = extensionsList.ToArray();
container.RemoveAllExtensions();
container.AddExtension(new UnityClearBuildPlanStrategiesExtension());
container.AddExtension(new UnitySafeBehaviorExtension());

foreach (UnityContainerExtension extension in existingExtensions)
{
   if (!(extension is UnityDefaultBehaviorExtension))
   {
       container.AddExtension(extension);
   }
}

Ответ 7

Unity 2.1 - август 2012 Обновление исправляет ошибку

Чтобы сделать процесс обновления максимально простым для пользователей и избежать необходимости перенаправления привязки к сборке, мы решили только увеличить версию файла сборки, а не версию сборки .NET.

Ответ 8

Это может помочь вам:

  • Перейти к Debug → Исключения...
  • Найдите исключения, которые действительно расстраивают вас как SynchronizationLockException.

Voila.