c# - ObjectDisposedException を適切に回避する

Question

ObjectDisposedException約 50% の確率でがスローされるという問題に遭遇しました。try以下の( 内の) 内のコードfinallyは、例外を引き起こしています。これを処理する方法がわかりません。以下に示すように例外を食べることもできますが、例外が発生することなくオブジェクトをチェックして閉じる方法はありますか?

    public static FindResponse Discover(FindCriteria findCriteria, 
                                        DiscoveryEndpoint discoveryEndpoint = null)
    {
        DiscoveryClient discoveryClient = null;

        try
        {
            if (discoveryEndpoint == null) { 
                 discoveryEndpoint = new UdpDiscoveryEndpoint(); 
            }

            discoveryClient = new DiscoveryClient(discoveryEndpoint);

            return discoveryClient.Find(findCriteria);
        }
        finally
        {
            try
            {
                if (discoveryClient != null)
                {
                    discoveryClient.Close();
                }
            }
            catch (ObjectDisposedException)
            {
                // Eat it.
            }
        }
    }

score 2 · Accepted Answer

どうですか

public static FindResponse Discover(FindCriteria findCriteria, DiscoveryEndpoint discoveryEndpoint = null)
{
    if (discoveryEndpoint == null) 
      discoveryEndpoint = new UdpDiscoveryEndpoint();

    using (var client = new DiscoveryClient(discoveryEndpoint))
    {
        return client.Find(findCriteria);
    }
}

アップデート

例外DiscoveryClient.Dispose()をスローするようです。OPの元のアプローチは、唯一の受け入れられる答えのようです。

score -1 · Accepted Answer

また、IDisposable オブジェクトには「using」を使用することをお勧めします。DiscoveryClient が IDisposable であることを考慮すると、

public static FindResponse Discover(FindCriteria findCriteria, DiscoveryEndpoint discoveryEndpoint = null)
    {

        FindResponse response = null;
        try
        {
            if (discoveryEndpoint == null) { discoveryEndpoint = new UdpDiscoveryEndpoint(); }

            using (DiscoveryClient discoveryClient = new DiscoveryClient(discoveryEndpoint))
            {
                response = discoveryClient.Find(findCriteria);
                discoveryClient.Close();
            }
        }
        finally
        {   
            // other finalizing works, like clearing lists, dictionaries etc.
        }
        return response;
    }

c# - ObjectDisposedException を適切に回避する

3 に答える 3

Related

Reference