2

我有以下财产：

private static Collection<Assembly> _loadedAssemblies = new Collection<Assembly>();
    internal static ReadOnlyCollection<Assembly> LoadedAssemblies
    {
        get { return new ReadOnlyCollection<Assembly>(_loadedAssemblies); }
    }

在我循环的另一堂课中LoadedAssemblies

foreach (Assembly assembly in ResourceLoader.LoadedAssemblies)

在遍历程序集时，底层集合 ( _loadedAssemblies) 有时会发生变化，这会给出一个System.InvalidOperationException. LoadedAssemblies确保安全的首选方法是什么？我无法重现该问题，所以我不能只是尝试。

可以吗？

internal static ReadOnlyCollection<Assembly> LoadedAssemblies
    {
        get { return new ReadOnlyCollection<Assembly>(_loadedAssemblies.ToList()); }
    }

编辑

    public static void Initialize()
    {

        foreach (Assembly assembly in AppDomain.CurrentDomain.GetAssemblies())
        {
            AddAssembly(assembly);
        }
        AppDomain currentDomain = AppDomain.CurrentDomain;
        currentDomain.AssemblyLoad += OnAssemblyLoad;
    }

    private static void OnAssemblyLoad(object sender, AssemblyLoadEventArgs args)
    {
        AddAssembly(args.LoadedAssembly);
    }

    private static void AddAssembly(Assembly assembly)
    {
        AssemblyName assemblyName = new AssemblyName(assembly.FullName);
        string moduleName = assemblyName.Name;

        if (!_doesNotEndWith.Exists(x => moduleName.EndsWith(x, StringComparison.OrdinalIgnoreCase)) &&
            _startsWith.Exists(x => moduleName.StartsWith(x, StringComparison.OrdinalIgnoreCase)))
        {
            if (!_loadedAssemblies.Contains(assembly))
            {
                _loadedAssemblies.Add(assembly);
            }
        }
    }
4

2 回答 2

3

ReadOnlyCollection<T>只是原始集合的包装器，可防止直接修改。但是，如果您在某处公开它，它不会阻止修改底层集合。

由于ReadOnlyCollection<T>.GetEnumerator只是为基础集合返回一个枚举器，因此它具有相同的规则。枚举时不能修改它，这意味着你不能添加或删除项目。

没有看到您的代码就没有简单的解决方法，防止多个线程同时访问底层集合，fe 使用lock. 如果您在其中修改它foreach可能更容易修复，但我们需要查看该代码。

于 2017-08-15T07:58:49.023 回答
3

您在问题中的建议将起作用。那是：

internal static ReadOnlyCollection<Assembly> LoadedAssemblies
{
    get { return new ReadOnlyCollection<Assembly>(_loadedAssemblies.ToList()); }
}

原因是您的问题来自于_loadedAssemblies在您枚举它时被更改。这当然会发生，因为ReadOnlyCollection它只是一个_loadedAssemblies使用基本集合的枚举器的包装器。

如果你这样做了，_loadedAssemblies.ToList()那么这将创建一个新列表，它是原始_loadedAssemblies. 它在创建时将具有所有相同的元素，但永远不会再次更新（因为您甚至没有对新集合的引用，您无法修改它）。这意味着当_loadedAssemblies更新您的新列表时，您ReadOnlyCollection完全没有意识到该更改，因此您的枚举将毫无问题地继续到最后。

于 2017-08-15T08:54:34.093 回答