この場合、循環参照をリファクタリングまたは修正する方法はありますか？ Javaの

Question

が、私はこの

public class MyAttibute extends MyObject 
{ 
    private MyClass definedOnClass;//point to its parentClass 

} 

よう コードこのようなMyClassのと呼ばれる親クラス、

MyAttributeクラスで

public class MyClass extends CompositeObject{ 
    protected Map<String,MyAttibute> attributes = new  

    HashMap<String,MyAttribute> 
    ..... 

} 

、 コードを読んでいるこれは、円形のreference.Whichは、トラブル時にあなたを作る実際にされました深いserlizationと等しくする。そしてそれは良いデザインではないかもしれません。それを避ける方法は？また、それを修正した後も、その属性からparentClassを簡単に見つけることができます。

P.S.私は別の2つのクラスのデザインを参照してください

public class Transaction{ 
    private ChangeManager parentManager; 
    .... 
    public Transaction(ChangeManager parentManager) 
} 

public class ChangeManager { 
    //record transaction when commit 
    private List<Transaction> transactions = new ArrayList<Transaction>(); 
    Transaction currentTransaction; 
    .... 
} 

あなたはこの種のデザインが良いと思いますか？どうして？ これらのクラスが定義しているドメインがわかるように、かなり一般的です。 誰もそれについていくつかの洞察を共有できますか？ TransactionにChangeManagerが認識され、MyAttributesにそのプロパティのMyClassが認識されるようにすることは有害でしょうか？コメントは大歓迎です。短所と長所

Answer 1

親として論理的に独立した子としてMyAttibuteを置くので、attr1.equals（attr2）はそれぞれの親を伴わない（シリアル化と同じ;ストリームには含めない）、definedOnClassプロパティ

か、MyClassの

public class MyAttibute extends MyObject 
{ 
    private NgcClass definedOnClass;//point to its parentClass 

    public boolean equals(Object o){ 
     if(o instanceof MyAttibute){ 
      MyAttibute other = (MyAttibute)o; 

      if(!this.definedOnClass.equals(other.definedOnClass)) 
       return false;//when not from the same parent they are never the same 

      return this.equals2(other); 
     } 
     return false; 
    } 

    //this one should then be called from MyClass 
    public boolean equals2(MyAttibute o){ 
     //check equality without worrying about definedOnClass 
    } 
} 

Javaのobjectstreamsのデフォルトの直列化は、私が正しくあなたを理解していれば、あなたの親クラスが参照を持っている循環参照