C＃アクション<T>、パラメータとしての関数を渡す

アクションとして条件を別のメソッドに渡したいとします。「ComputerPriceGenerator」の最初の行は動作しますが、配列work（第2ライン）を作る方法？..私はアドバイスを探しています任意のアイデアC＃アクション<T>、パラメータとしての関数を渡す

は...、CalculateAllPriceはまだ

public void ComputerPriceGenerator() 
{ 
    //Below line Works 
    PriceMachine.CalculatePrice(cart.Computers[0],() => ComputerConverter(cart.Computers[0]));   
    //How to make this work, i don't want to loop it??? 
    PriceMachine.CalculateAllPrice(cart.Computers,() => ComputerConverter(??)); 
} 

public void ComputerConverter(Computer comp) 
{ 
    if (comp.Memory <= 2) 
     comp.Discount = 10; 
}

出典

2011-01-31 kayak

あなたのコードは、LINQを使用するのに最適な候補者であったであろう、しかしそれは副作用がいっぱいです。私はリファクタリングを検討したいと思います。 –

CalculatePriceとCalculateAllPriceの署名を表示せずに、あなたを助ける方法を知るのは難しいです。 –

@Jon Skeet：「CalculateAllPrice」のシグネチャの内容に関する部分的なアドバイスを探しているようでした。しかし、私は間違っている可能性があります。 –

あなたCalculatePrice方法 - 両方の方法はAction<Computer>を取る必要があります。

public static void CalculatePrice(Computer computer, Action<Computer> action) 
public static void CalcuateAllPrices(IEnumerable<Computer> computers, 
            Action<Computer> action)

をし、このようにそれらを呼び出す：だから私はこのようなメソッドを持っているでしょう

PriceMachine.CalculatePrice(cart.Computers[0], ComputerConverter); 
PriceMachine.CalculateAllPrice(cart.Computers, ComputerConverter);

出典

2011-02-01 00:13:38

それは多くのおかげで働いた – kayak

ジョン、私は未解決の質問があります（この質問はこの1つに関連していない、私はこれを削除します..）、あなたが何か提案があればお知らせください "http://stackoverflow.com/questions/4444972/iterate-a-linq-expression-and-db-into-db-parameter " – kayak

@kayak：そこの答えは私の言うこととほとんど同じです。あなたはほとんど確かにその道を下りたくない。 –

に設計されていません配列のすべての要素にメソッドを適用したいので、それを反復処理することはありません。

次のようなPriceMachine.CalculateAllPrice定義することができます。cart.Computersを通じてCalculateAllPrice反復処理を行い、匿名関数にそれぞれ1を渡すその後

public void CalculateAllPrice(IEnumerable<Computer> data, Action<Computer> action) 
{ 
    foreach(Computer c in data) 
    action(c); 
}

出典

2011-01-31 23:52:01 Femaref

私は同意しますが、メインメソッド「PriceMachine.CalculateAllPrice（cart.Computers、（）=> ComputerConverter（??））;」から、 – kayak

PriceMachine.CalculateAllPrice(cart.Computers, (Computer x) => ComputerConverter(x));

を。がちょうどAction、IMO取るべきではない

出典

2011-01-31 23:54:57

ビットオーバーキルと思われます。 '（cart.Computers、（Computer x）=> ComputerConverter（x））'は '（cart.Computers、ComputerConverter）'と同じです。 – Juliet

@Juliet：もちろん、私は彼が投稿したラムダスタイルを保っていた。 –

C＃アクション<T>、パラメータとしての関数を渡す

答えて

関連する問題