リファクタリング、どの親関数が非静的

Question

コンテキストをせずに2つの2の静的関数を崩壊します

は、我々はマイクロサービスに移動するためにコードをrefectoringしています。 複数の製品（A、B、CおよびA、B、Cの共通コードはモノリシックで サービス）。現在、共通のコード用の新しいサンドボックスを作成しています。

問題：

User.java 
    Class User { 
    public **static** void init(){ 
     List<Items> users=Items.getItemsList(); 
    } 


     } 


Items.java   
     Class Items { 
      public **static** Items getItemsList(){ 
       //many static functions and dependancy 

       return items; 
      } 
      } 

だからここに、両方の機能は静的であり、私は新しいサンドボックスではないItems.javaにのみUser.java を移動したいです。どのように私はこの依存を分解することができます。 と私はそれ以外の場合はコンパイルされません、サンドボックスはItems`もサンドボックス内に存在する必要があります」は、jarファイルを生成し、独立したプロジェクトを意味と仮定

Answer 1

User.init（）は非静的にすることはできません。

しかし、ItemsからIItems（恐ろしい名前を許して）などのインターフェイスを抽出できます。

public interface IItems { 
    // methods... 
} 

サンドボックスに含まれています。

そのように、工場のためのインタフェースの作成：また、サンドボックスに含まれ

public interface IItemsFactory { 
    List<IItem> create(); 
} 

を。

醜い部分は、静的としてUser.init()を維持しています。ハッキーなIoCパターンを使用して、IItemsFactoryの実装をUserに設定します。工場も静的でなければならない。

public class User { 
    private static volatile IItemsFactory factory; 

    public static setFactory(IItemsFactory factory) { 
     User.factory = factory; 
    } 

    public static void init() { 
     List<IItems> users = factory.getItemsList(); 
    } 
} 

A、B、およびCプロジェクトがIItemFactoryの実装を提供し、User.init()を呼び出す前に、それを設定するための責任がある：だからUserは次のようになります。

これはハーフベークされており、これらの静的メソッドは、次のリファクタリングの反復処理中に離れる必要があります。引き続きIoCパターンを使用しますが、工場をUserコンストラクタの一部として注入します。

public class User { 
    private IItemsFactory factory; 

    public User(IItemsFactory factory) { 
     this.factory = factory; 
    } 

    public void init() { 
     List<IItems> users = factory.getItemsList(); 
    } 
}