Подтвердить что ты не робот

Инициализаторы объектов в блоке использования генерируют предупреждение анализа кода CA2000

Если я использую инициализаторы объектов в используемом блоке, я получаю предупреждение "Анализ кода" о том, чтобы не утилизировать объект должным образом:

CA2000: Microsoft.Reliability: В методе 'ReCaptcha.CreateReCaptcha(этот HtmlHelper, string, string)' объект '< > g__initLocal0' не расположен по всем путям исключений. Вызовите System.IDisposable.Dispose на объекте '< > g__initLocal0', прежде чем все ссылки на него будут недоступны.

Вот код:


    using (var control = new ReCaptchaControl()
    {
        ID = id,
        Theme = theme,
        SkipRecaptcha = false
    })
    {
        // Do something here
    }

Если я не использую инициализаторы объектов, Code Analysis удовлетворен:


    using (var control = new ReCaptchaControl())
    {
        control.ID = id;
        control.Theme = theme;
        control.SkipRecaptcha = false; 

        // Do something here
    }

В чем разница между этими двумя блоками кода? Я думал, что они приведут к тому же ИЛ. Или это ошибка в анализе кода?

4b9b3361

ОТВЕТЫ

Ответ 1

Нет, есть разница.

Инициализатор объекта присваивает переменной только после того, как все свойства были установлены. Другими словами, это:

Foo x = new Foo { Bar = "Baz" };

эквивалентно:

Foo tmp = new Foo();
tmp.Bar = "Baz";
Foo x = tmp;

Это означает, что если один из определителей свойств выбрал исключение в вашем случае, объект не будет удален.

EDIT: Как я думал... попробуйте следующее:

using System;

public class ThrowingDisposable : IDisposable
{
    public string Name { get; set; }

    public string Bang { set { throw new Exception(); } }

    public ThrowingDisposable()
    {
        Console.WriteLine("Creating");
    }

    public void Dispose()
    {
        Console.WriteLine("Disposing {0}", Name);
    }
}

class Test
{
    static void Main()
    {
        PropertiesInUsingBlock();
        WithObjectInitializer();
    }

    static void PropertiesInUsingBlock()
    {
        try
        {
            using (var x = new ThrowingDisposable())
            {
                x.Name = "In using block";
                x.Bang = "Ouch";
            }
        }
        catch (Exception)
        {
            Console.WriteLine("Caught exception");
        }
    }

    static void WithObjectInitializer()
    {
        try
        {
            using (var x = new ThrowingDisposable
            {
                Name = "Object initializer",
                Bang = "Ouch"
            })
            {
                // Nothing
            }
        }
        catch (Exception)
        {
            Console.WriteLine("Caught exception");
        }
    }
}

Вывод:

Creating
Disposing In using block
Caught exception
Creating
Caught exception

Обратите внимание на то, что нет строки "Инициализация объекта удаления".