Статическая Переменная Null В Вызове Метода, Но Инициализируется В Программе

имейте в виду, что я только что скопировал ваш код, чтобы попытаться воспроизвести вашу проблему. При запуске этого кода я получаю исключение NullReferenceException при отладке.Write, AnotherClass не был должным образом инициализирован до разрешения вызова.

namespace ConsoleApplication2
{
    class Program
    {
        static void Main(string[] args)
        {
            MefInitilizer.Run();
            Debug.Write(AnotherClass.Test);
        }
    }

    public class AnotherClass
    {
        public static String Test = ContainerSingleton.ContainerInstance;
    }

    public static class MefInitilizer
    {
        public static void Run()
        {
            ContainerSingleton.Initialize("A string");
        }
    }

    public class ContainerSingleton
    {
        private static String compositionContainer;

        public static String ContainerInstance
        {
            get
            {
                if (compositionContainer != null) return compositionContainer;

                var appDomainName = AppDomain.CurrentDomain.FriendlyName;
                throw new Exception("Composition container is null and must be initialized through the ContainerSingleton.Initialize()" + appDomainName);
            }
        }

        public static void Initialize(String container)
        {
            compositionContainer = container;
        }
    }


}

однако, когда я добавляю статические конструкторы ко всем классам со статическими полями, он работает так, как ожидалось:

namespace ConsoleApplication2
{
    class Program
    {
        static void Main(string[] args)
        {
            MefInitilizer.Run();

            Debug.Write(AnotherClass.Test);
        }
    }

    public class AnotherClass
    {
        static AnotherClass()
        {

        }

        public static String Test = ContainerSingleton.ContainerInstance;
    }

    public static class MefInitilizer
    {
        static MefInitilizer()
        {

        }
        public static void Run()
        {

            ContainerSingleton.Initialize("A string");
        }
    }

    public class ContainerSingleton
    {
        static ContainerSingleton()
        {

        }
        private static String compositionContainer;

        public static String ContainerInstance
        {
            get
            {
                if (compositionContainer != null) return compositionContainer;

                var appDomainName = AppDomain.CurrentDomain.FriendlyName;
                throw new Exception("Composition container is null and must be initialized through the ContainerSingleton.Initialize()" + appDomainName);
            }
        }

        public static void Initialize(String container)
        {
            compositionContainer = container;
        }
    }


}

Я бы сказал, что это определенно может быть проблемой BeforeFieldInit.

как я понял, ваш код является плагином для Visual Studio, и основная проблема вашего приложения заключается в том, что ваш класс создается дважды, один раз для обычного AppDomain, и один раз по какой-то другой причине вы не можете действительно узнать.

прежде всего, я вижу здесь потенциал sandboxing из Visual studio-он хочет проверить ваш код в различных наборах прав, чтобы гарантировать, что ваш код не повредит другим частям Visual Studio или работе конечного пользователя. В этом случае ваш код может быть загружен в другой AppDomain, без некоторых прав (вы можете найти хорошая статья в MSDN), так что вы можете понять, почему ваш код вызывается дважды для каждого приложения.

во-вторых, я хочу отметить, что вы неправильно поняли идею статический!--9--> и статический!--10-->:

public static void Initialize(CompositionContainer container)
{
    compositionContainer = container;
}

- это не то же самое как

public static ContainerSingleton()
{
    compositionContainer = container;
}

Итак, я предлагаю вам перенести всю логику инициализации в статический контейнер, что-то вроде это:

public class ContainerSingleton
{
    private static CompositionContainer compositionContainer;

    public static CompositionContainer ContainerInstance
    {
        get 
        {
            if (compositionContainer == null)
            {
                var appDomainName = AppDomain.CurrentDomain.FriendlyName;
                throw new Exception("Composition container is null and must be initialized through the ContainerSingleton.Initialize()" + appDomainName);
            }
            return compositionContainer; 
        }
    }

    public static ContainerSingleton()
    {
        var catalog = new AggregateCatalog();
        //this directory should have all the defaults
        var dirCatalog = new DirectoryCatalog(Path.GetDirectoryName(Assembly.GetExecutingAssembly().Location));
        //add system type plug-ins, too
        catalog.Catalogs.Add(dirCatalog);

        compositionContainer = new CompositionContainer(catalog);
    }
}

второй подход: я хочу отметить, что шаблон, который вы используете для получения синглтона, устарел, попробуйте использовать Lazy<T> класс, что-то вроде этого:

public class ContainerSingleton
{
    private static Lazy<CompositionContainer> compositionContainer;

    public static CompositionContainer ContainerInstance
    {
        get 
        {
            return compositionContainer.Value;
        }
    }

    public static ContainerSingleton()
    {
        compositionContainer = new Lazy<CompositionContainer>(() => Initialize());
    }
    public static void Initialize()
    {
         // Full initialization logic here
    }
}

кроме того, вы должны помнить, что простого добавления пустых статических конструкторов недостаточно - вы должны переместить все назначения на него, поэтому вы должны заменить такой код:

public class AnotherClass
{
    static AnotherClass()
    {

    }

    public static String Test = ContainerSingleton.ContainerInstance;
}

С этим один:

public class AnotherClass
{
    static AnotherClass()
    {
        Test = ContainerSingleton.ContainerInstance;
    }

    public static String Test;
}

обновление:

@Colin вы даже можете использовать [LazyTask type][https://msdn.microsoft.com/en-us/magazine/dn683795.aspx] - просто передайте Func к вашему конструктору, и это будет потокобезопасный подход, см. Больше в статье. То же самое Id на AppDomain ничего не значит-песочница может запускать ваш код через AppDomain.ExecuteAssembly метод (он устарел в 4.5, но все еще может быть возможным вариантом), чтобы увидеть как он ведет себя в различных разрешениях.

может быть, есть еще один метод для этого в .NET 4.5, но не могу найти статью, связанную прямо сейчас.

обновление 2:

как я вижу в вашем коде, вы читаете информацию с диска. Попробуйте добавить Код Доступа Безопасности правило для этого, чтобы увидеть, если ваш код выполняется с ограниченными разрешениями, например:

FileIOPermission f2 = new FileIOPermission(FileIOPermissionAccess.Read, Path.GetDirectoryName(Assembly.GetExecutingAssembly().Location));
//f2.AddPathList(FileIOPermissionAccess.Write | FileIOPermissionAccess.Read, "C:\example\out.txt");
try
{
    f2.Demand();
}
catch (SecurityException s)
{
    Console.WriteLine(s.Message);
}

подробнее о FileIOPermission класс на MSDN.

попробуйте добавить статический конструктор для ContainerSingleton. Я считаю, что это BeforeFieldInit снова поднимает свою уродливую голову.