5

ループを 2 回使用したくないforeachので、LINQ を使用しようとしましたが、残念ながらこれは機能しません。

return _etags.ToDictionary(item => item.Key, async item => await item.Value);

コードを改善するために何をすべきか、またはどのように変換できDictionary<string, Task<string>>ますDictionary<string, string>か?

これが私のコードです：

private static async Task<Dictionary<string, string>> GetETagsAsync(List<string> feedsLink)
{
    var eTags = new Dictionary<string, string>(feedsLink.Count);
    System.Diagnostics.Stopwatch stopWatch = new System.Diagnostics.Stopwatch();
    stopWatch.Start();
    var _eTags = new Dictionary<string, Task<string>>(feedsLink.Count);
    eTags = new Dictionary<string, string>(feedsLink.Count);
    foreach (string feedLink in feedsLink)
    {
        if (Uri.IsWellFormedUriString(feedLink, UriKind.Absolute))
        {
            _eTags.Add(feedLink, GetETagAsync(feedLink));
        }
        else
        {
            throw new FormatException();
        }
        }
        foreach (KeyValuePair<string, Task<string>> eTag in _eTags)
        {
            eTags.Add(eTag.Key, await eTag.Value);
        }
        stopWatch.Stop();
        TimeSpan ts = stopWatch.Elapsed;

        return eTags;
    }
4

1 に答える 1

3

いくつかのタスクを開始した後に例外をスローしたくない場合は、最初に検証することをお勧めします。

    private static async Task<Dictionary<string, string>> GetETagsAsync(List<string> feedsLink)
    {
        ValidateFeedsLinkFormat(feedsLink);

        var _eTags = feedsLink.ToDictionary(f => f, f => GetETagAsync(f));
        await TaskEx.WhenAll(_eTags.Values);

        var eTags = _eTags.ToDictionary(k => k.Key, k => k.Value.Result);

        return eTags;
    }

このコードは、あなたが行ったようにすべてのタスクを開始しますが、すべてのリンクを 1 回待機します。複数のスレッドで Stopwatch のインスタンスを使用したくない場合: MSDNから:「インスタンス メンバーは、スレッド セーフであることが保証されていません。」Stopwatch.GetTimestamp();2 回使用して、結果を比較できます。

于 2013-07-28T13:10:05.203 に答える