c# - 同様の機能のリファクタリングパターン

Question

私はほぼ同じパターンの約20の関数を持っています。サイトの配列で実行し、Siteオブジェクトを使用してSiteOperationを作成し、いくつかの操作を実行します（この場合、1つのパラメーターを使用しますが、それ以上ない場合もあります）

int wantedBandwidthInLBps = 2048 / 8;
foreach (Sites site in _sitesManagement.GetAll())
{
 SiteOperation siteOperation = new SiteOperation(site);
 siteOperation.LimitBandwidth(wantedBandwidthInLBps);
}

foreach (Sites site in _sitesManagement.GetAll())
{
 SiteOperation siteOperation = new SiteOperation(site);
 siteOperation.KillJames();
}

foreach (Sites site in _sitesManagement.GetAll())
{
 SiteOperation siteOperation = new SiteOperation(site);
 siteOperation.FlyToMoon(2012, new TaskIdentifier(10,20));
}

このコードをラップすることは可能ですか？それで、すべてのサイトでこのアクションを実行する_sitesManagementなどのアクションを送信しますか？

score 4 · Accepted Answer

これを試して：

void PerformOperationOnAllSites(Action<SiteOperation> doIt) {
    foreach (Sites site in _sitesManagement.GetAll()) {
        SiteOperation siteOperation = new SiteOperation(site);
        doIt(siteOperation);
    }
}

...

_sitesManagement.PerformOperationOnAllSites(op => op.LimitBandwidth(wantedBandwidthInLBps));
_sitesManagement.PerformOperationOnAllSites(op => op.KillJames());
_sitesManagement.PerformOperationOnAllSites(op => op.FlyToMoon(2012, new TaskIdentifier(10,20));

score 1 · Accepted Answer

public void DoSiteOperationActions(Action<SiteOperation> toDo)
{
        foreach (Sites site in _sitesManagement.GetAll())
        {
           SiteOperation mySiteOperation = new SiteOperation(site);
           toDo(mySiteOperation);
        }
}

そして、あなたはそれを次のように呼ぶことができます...

DoSiteOperationActions(so => so.FlyToMoon(2012, new TaskIdentifier(10,20)));

構文が少しずれている可能性があります。ここにはIDEがありませんが、詳細については http://msdn.microsoft.com/en-us/library/018hxwa8.aspxを参照してください。

score 0 · Accepted Answer

    private static void InvokeMethod(Action<SiteOperation> action)
    {
        foreach (SiteOperation siteOperation in _sitesManagement.GetAll()
                               .Select(site=>new SiteOperation(site))
        {
            action(siteOperation);
        }
    }
..

    public static void KillJames()
    {
        InvokeMethod(so => so.KillJames());
    }

    public static void LimitBandwidth(int wantedBandwidthInLBps)
    {
        InvokeMethod(so => so.LimitBandwidth(wantedBandwidthInLBps));
    }

    public static void FlyToMoon(int year=2012, TaskIdentifier ti=new TaskIdentifier(10,20))
    {
        InvokeMethod(so => so.FlyToMoon(year, ti));
    }

score 0 · Accepted Answer

これは、テンプレートメソッドパターンの適切な候補のように思えます。

http://en.wikipedia.org/wiki/Template_method_pattern

c# - 同様の機能のリファクタリングパターン

4 に答える 4

Related

Reference