MessageQueue се изхвърля повече от веднъж

Close освобождава всички ресурси на обекта MessageQueue. Вижте документацията тук. Грешката най-вероятно се генерира в CA, защото вижда, че пътят за изпълнение на Close също извиква Dispose.

От документацията:

    public  void ReceiveMessage()
    {
        // Connect to the a on the local computer.
        MessageQueue myQueue = new MessageQueue(".\\myQueue");

        // Set the formatter to indicate body contains an Order.
        myQueue.Formatter = new XmlMessageFormatter(new Type[]
            {typeof(String)});

        try
        {
            // Receive and format the message. 
            Message myMessage1 = myQueue.Receive();
            Message myMessage2 = myQueue.Receive();
        }

        catch (MessageQueueException)
        {
            // Handle sources of any MessageQueueException.
        }

        // Catch other exceptions as necessary.

        finally
        {
            // Free resources.
            myQueue.Close();
        }

        return;
    }

Close очевидно ще освободи ресурсите, но ще позволи на компонента да ги придобие отново, ако все още не са събрани. Може да е по-разумно да отворите обекта MessageQueue, да го използвате и след това да го затворите в рамките на едно и също повикване, вместо да го отваряте за определен период от време и да го затворите по-късно, тъй като кеширането на връзката премахва режийните разходи за отваряне на MessageQueue при повтарящи се повиквания.

*АКТУАЛИЗАЦИЯ* Изглежда, че CA третира CA2202 по различен начин за членски полета в сравнение с предаването на обекта за еднократна употреба към метод, дори ако този метод е частен за класа. Независимо от това, според документацията, трябва да извикате само Close() или Dispose(), но не и двете. Препоръчвам обаче да промените дизайна си, така че да създавате, използвате и след това да затваряте обекта MessageQueue в рамките на обхвата на вашите операции със съобщения, както е показано в примера от примера на документацията по-горе.

да Това може да изхвърли обекта многократно:

Стойността, която this._messageQueue оценява, не се променя след извикване на MsmqHelper.DisposeQueue(this._messageQueue).

Само на локалния параметър (с име messageQueue) беше присвоена стойност null в метода DisposeQueue. По този начин "нулевата охрана" не успява правилно да защити следващите пъти. (Това е така, защото поведението по подразбиране на C# е Call-By-Value: моля, вижте връзката към разберете какво означава това в контекста на „предаване на стойността на препратка към обект“.)

Или вземете ref, или задайте this._messageQueue = null в повикващия.

Ако класът MessageQueue имплементира IDisposable iterface, тогава няма смисъл да използвате изрично метода Dispose и метода Close(), тъй като във всички такива класове методът Close() обикновено не е iterface метод, а по-скоро клас метод. Обикновено в метода Dispose всички правилни имплементации трябва да извикат метода Close() преди освобождаване на управлявани/неуправлявани ресурси.

Отново, като задействате външен статичен помощник, вие нарушавате модела за еднократна употреба. Това не е правилният начин за контролиране на живота на обекта; Не е нужно да бъркате с модела за еднократна употреба, можете просто да го използвате

И вашият код може да бъде опростен по следния начин:

    // 1. Use static class. By the agreement, all helper classes should be static to avoid 
    // IDisposable inheritance, in example
    public static class MsmqHelper//: IDisposable
    {
        //private MessageQueue _messageQueue;

        //public MessageQueueHelper(bool workflowCreated)
        //{
        //    this._messageQueue = MsmqHelper.InitializeQueue();
        //    this._messageQueue.Send(workflowCreated);
        //}

        public static SendMessage(object workflowCreated)
        {
            // 2. If static method in static class does not takes parameters, 
            // I might be better to to implicitly call the constructor?

            // using(MessageQueue msmsq = MsmqHelper.InitializeQueue())

            using(MessageQueue msmsq = new MessageQueue())
            {
                msmq.Send(workflowCreated);
                msmq.Close(); 

                // MsmqHelper.DisposeQueue(msmq);

                // 3. You should explicitly call Close object to immediately release     
                // unmanaged resources, while managed ones will be released 
                // at next GC rounds, as soon as possible
            }
        }
        //private MessageQueue _messageQueue;

        //public void Dispose()
        //{
        //    Dispose(true);
        //    GC.SuppressFinalize(this);
        //}

        //private void Dispose(bool disposing)
        //{
    //    if (disposing == false) { return; }
    //
    //    MsmqHelper.DisposeQueue(this._messageQueue);
    //}

    //public static void DisposeQueue(MessageQueue messageQueue)
    //{
    //    if (messageQueue != null)
    //    {
    //        messageQueue.Close();
    //        messageQueue.Dispose();
    //        messageQueue = null;
    //    }
    //}
}