サブクラスのコンストラクタ例外は親クラスのインスタンスを残します

Question

大量のコードを読み込むことは残念です。それは問題を示す最も簡単な方法です。

using System; 
using System.Collections.Generic; 

namespace P1 
{ 
    class A 
    { 
     static Dictionary<int, A> a = new Dictionary<int, A>(); 
     static int i = 0; 

     int id; 
     public A() 
     { 
      id = ++i; 
      a[id] = this; 
     } 

     public static int Count() { return a.Count; } 
    } 

    class B : A 
    { 
     public B() 
     { 
      throw new Exception(); 
     } 
    } 

    class Program 
    { 
     static void Main(string[] args) 
     { 
      try 
      { 
       var b = new B(); 
      } 
      catch 
      { 
       // What should be here ???? 
      } 

      Console.WriteLine(A.Count()); //prints 1 - not good 
      Console.ReadKey(); 
     } 
    } 
} 

誰でもサブクラスのコンストラクタが失敗した場合のクリーンアップロジックを提案できますか？

Answer 1

コンストラクターが失敗した後でインスタンス参照にアクセスできないため、クリーンアップロジックをBのコンストラクターに配置する必要があります。ここで

は、あなたがこれを行うことができます方法の例です：例外がスローされる前に

class A 
{ 
    static Dictionary<int, A> a = new Dictionary<int, A>(); 
    static int i = 0; 

    int id; 
    public A() 
    { 
     id = ++i; 
     a[id] = this; 
    } 

    protected void Destroy() 
    { 
     a.Remove(id); 
     i--; 
    } 

    public static int Count() { return a.Count; } 
} 

class B : A 
{ 
    public B() 
    { 
     try 
     { 
      throw new Exception(); 
     } 
     catch (Exception) 
     { 
      Destroy(); 
      throw; 
     } 
    } 
} 

Answer 2

あなたの基底クラスのコンストラクタが呼び出されているので、オブジェクトがすでにaで作成され、それのスポットに割り当てられています。コンストラクタによってオブジェクトが作成されないようにするには、デフォルトのコンストラクタを使用する代わりに、新しいAをインスタンス化する静的メソッドを検討するとよいでしょう。オブジェクトのインスタンス化が失敗すると（Bで）、オブジェクトが辞書に追加される前に例外がスローされます。