非同期メソッド呼び出しを使用してif条件のセットをDictionaryに変換するにはどうすればよいですか？

Question

私はすべてのGenerateAsyncは異なる型のオブジェクトを生成し、この

if (SectionContainedWithin(args, RegisterSection.StudentPersonalData)) 
    schoolRegister.StudentPersonalData = await _sectionGeneratorsProvider.StudentPersonalDataGenerator.GenerateAsync(args); 

if (SectionContainedWithin(args, RegisterSection.StudentAttendances)) 
    schoolRegister.StudentAttendances = await _sectionGeneratorsProvider.StudentMonthAttendancesGenerator.GenerateAsync(args); 

if (SectionContainedWithin(args, RegisterSection.Grades)) 
    schoolRegister.Grades = await _sectionGeneratorsProvider.GradesGenerator.GenerateAsync(args); 

// More generating here ... 

のように見えるいくつかのメソッド呼び出しを持っています。

public interface IGenerator<TResult, in TArgs> 
{ 
    Task<TResult> GenerateAsync(TArgs args); 
} 

にはどうすればいいので、私はアクションと条件のリストを定義することができますし、それらを反復処理、それらのif Sを書き換えることができます。 のような何か：

var sections = new Dictionary<bool, Func<Task>>() 
{ 
    { 
     SectionContainedWithin(args, RegisterSection.StudentPersonalData), 
     () => schoolRegister.StudentPersonalData = await _sectionGeneratorsProvider.StudentPersonalDataGenerator.GenerateAsync(args); 
    }, 

    // More generating here ... 
} 

foreach(var item in sections) 
{ 
    if(item.Key) 
    { 
     await item.Value(); 
    } 
} 

SOLUTION：

おかげで、私はDictionary<bool, Func<Task>>を作成するためのアイデアを落とし、それがより理にかなっているのでIReadOnlyDictionary<RegisterSection, Func<RegisterXml, RegisterGenerationArgs, Task>>でそれを置き換える@ピーター・dunihoの答えに。 複数のキーで、2つの真偽のみではありません。

は、だから私は私が正しくコード例を理解していれば、辞書は本当にこの仕事のための適切なツールではありません

private IReadOnlyDictionary<RegisterSection, Func<RegisterXml, RegisterGenerationArgs, Task>> CreateSectionActionsDictionary() 
{ 
    return new Dictionary<RegisterSection, Func<RegisterXml, RegisterGenerationArgs, Task>> 
    { 
     { RegisterSection.RegisterCover, async(reg, args) => reg.Cover = await _sectionGenerators.RegisterCoverGenerator.GenerateAsync(args) }, 
     { RegisterSection.StudentPersonalData, async(reg, args) => reg.StudentPersonalData = await _sectionGenerators.StudentPersonalDataGenerator.GenerateAsync(args)}, 

     // Add more generating here ... 
    }; 
} 

private async Task GenerateSectionsAsync(RegisterGenerationArgs args, RegisterXml reg) 
{ 
    foreach (var sectionAction in SectionActions) 
     if (SectionContainedWithin(args, sectionAction.Key)) 
      await sectionAction.Value(reg, args); 

} 

Answer 1

になってしまった厥。あなたは値のペアを格納するためだけに使用しているようです。すでに知っているキー値を別の値にマップすることができる辞書の主要な機能を使用していません。

SectionContainedWithin()呼び出しを評価するために、argsの値が必要なので、提案しているコード例は機能しません。あなたはargsが有効であると辞書を初期化するために使用することができ、あなたはそれはあなたが唯一の辞書に2つのエントリを持つことができますを意味しますキータイプboolを作る問題を抱えているだろう文脈で辞書を宣言しようとしていた場合でも、多くの場合、SectionContainedWithin()メソッドが返されたすべての組み合わせを実際に処理する方法はありません。true。あなたがやっていることを明らかに示している良いMinimal, Complete, and Verifiable code exampleなし

、それはあなたが必要な正確に何を確実に知ることは不可能です。そして、あなたが何かを行うことができます

struct SectionGenerator<TArgs> 
{ 
    public readonly RegisterSection RegisterSection; 
    public readonly Func<TArgs, Task> Generate; 

    public SectionGenerator(RegisterSection registerSection, Func<TArgs, Task> generate) 
    { 
     RegisterSection = registerSection; 
     Generate = generate; 
    } 
} 

SectionGenerator<TArgs>[] generators = 
{ 
    new SectionGenerator<TArgs>(RegisterSection.StudentPersonalData, 
     async args => schoolRegister.StudentPersonalData = await _sectionGeneratorsProvider.StudentPersonalDataGenerator.GenerateAsync(args); 
    // etc. 
} 

：すべてのこれらの非同期操作が並行して進行中であるために、それは合理的だと仮定すると、

foreach (SectionGenerator<TArgs> generator in generators) 
{ 
    if (SectionContainedWithin(args, generator.RegisterSection)) 
    { 
     await generator.Generate(args); 
    } 
} 

を、あなたもこのような何かを行うことができますが、それはこのようになります。 ：

await Task.WhenAll(generators 
    .Where(g => SectionContainedWithin(args, g.RegisterSection)) 
    .Select(g => g.Generate(args));