有没有办法重写这个查询更多“LINQ-ier”?

问题描述 投票:1回答:1

以下内容将/分隔的URL路径解析为键值对的字典:

    private Dictionary<string, string> ParsePathParameters(string path)
    {
        var parameters = new Dictionary<string, string>();
        if (string.IsNullOrEmpty(path))
        {
            return parameters;
        }

        var pathSegments = path.Split(new[] { '/' }, StringSplitOptions.RemoveEmptyEntries);

        for (var i = pathSegments.Length - 1; i >= pathSegments.Length % 2; i -= 2)
        {
            parameters.Add(pathSegments[i - 1], pathSegments[i]);
        }

        return parameters;
    }

输入格式为[/preamble][/key1/value1][/key2/value2]...[/keyN/valueN],例如,给定输入“/ foo / 1 / bar / Thing”或“/ slug / foo / 1 / bar / Thing”,输出将为:

Dictionary<string, string>
{
    { "foo", "1" },
    { "bar", "Thing" },
}

这段代码很好;简单,不言自明,快速。但是,因为我喜欢挑战,所以我决定在LINQ中重写它:

    private Dictionary<string, string> ParsePathParameters(string path)
    {
        if (string.IsNullOrEmpty(path))
        {
            return new Dictionary<string, string>();
        }

        var pathSegments = path.Split(new[] { '/' }, StringSplitOptions.RemoveEmptyEntries);
        var skip = pathSegments.Length % 2;

        return pathSegments.Skip(skip)
            .Where((_, i) => i % 2 == 0)
            .Select((_, i) => i * 2)
            .ToDictionary(i => pathSegments[i + skip], i => pathSegments[i + skip + 1]);
    }

这可行,但它肯定不会感觉最佳,可能是因为它也不像使用LINQ实现这一点的“正确”方式。任何人都可以建议是否有可能以更“类似LINQ”的方式编写此代码,如果是这样,请给我一些关于这方面的指示?

c# linq
1个回答
5
投票

我会像这样写:

private Dictionary<string, string> ParsePathParameters(string path)
{
    return GetSegmentPairs().ToDictionary(x => x.k, x => x.v);
    IEnumerable<(string k, string v)> GetSegmentPairs()
    {
        var segments = path?.Split(new[] { '/' }, StringSplitOptions.RemoveEmptyEntries)
                ?? new string[0];
        for (int i = 0, l = segments.Length; i < l; i += 2)
            yield return (segments[i+0], segments[i+1]);
    }
}

不要低估本地功能和发电机的功率。当您需要创建难以编写为直接linq查询的序列时,生成器很棒。然后可以在linq查询中使用这些生成器。对于这种特殊情况,它甚至可能不需要一个相当简单的查询,但对于更复杂的查询,它是非常宝贵的。但这是一种你应该学会更频繁地使用的模式。

如果使用C#8,我会养成在适当的时候使用span / memory和slice的习惯。

private Dictionary<string, string> ParsePathParameters(string path)
{
    return GetSegments().ToDictionary(x => x.Span[0], x => x.Span[1]);
    IEnumerable<System.Memory<string>> GetSegments()
    {
        var segments = path?.Split(new[] { '/' }, StringSplitOptions.RemoveEmptyEntries) ?? new string[0];
        for (int i = 0, l = segments.Length; i < l; i += 2)
            yield return segments[^i..i+1];
    }
}

否则,如果您正在使用MoreLINQ,您可以使用Pairwise()TakeEvery()方法来有效地执行与上面的GetSegmentPairs()方法相同的操作。

private Dictionary<string, string> ParsePathParameters(string path) =>
    (path?.Split(new[] { '/' }, StringSplitOptions.RemoveEmptyEntries)
            ?? Enumerable.Empty<string>())
        .Pairwise(ValueTuple.Create)
        .TakeEvery(2) // pairwise produces overlapping pairs so take every other
        .ToDictionary(x => x[0], x => x[1]);
© www.soinside.com 2019 - 2024. All rights reserved.