返回第一个有效的方法，更优雅的方法吗？

Question

最近，我发现自己编写的方法连续调用其他方法，并根据哪个方法首先返回适当的值来设置一些值。我一直在做的是用一种方法设置值，然后检查该值，如果不好，则检查下一个方法。下面是一个最近的示例：

private void InitContent()
{
    if (!String.IsNullOrEmpty(Request.QueryString["id"]))
    {
        Content = GetContent(Convert.ToInt64(Request.QueryString["id"]));
        ContentMode = ContentFrom.Query;
    }

    if (Content == null && DefaultId != null)
    {
        Content = GetContent(DefaultId);
        ContentMode = ContentFrom.Default;
    }

    if (Content == null) ContentMode = ContentFrom.None;
}

如果 id 不在数据库中，则 GetContent 方法应返回 null。这是一个简短的示例，但您可以想象如果有更多选项，这可能会变得多么笨拙。有更好的方法吗？

Answer 1

空合并运算符可能具有您想要的语义。

q = W() ?? X() ?? Y() ?? Z();

这本质上与以下相同：

if ((temp = W()) == null && (temp = X()) == null && (temp == Y()) == null)
    temp = Z();
q = temp;

也就是说，q 是 W()、X()、Y() 中的第一个非空，或者如果它们全部为空，则为 Z()。

您可以根据需要链接任意数量的链接。

确切的语义并不完全像我所描绘的那样；类型转换规则很棘手。如果您需要确切的详细信息，请参阅规格。

Answer 2

您还可以做一些更偷偷摸摸的事情，大致如下：

private Int64? GetContentIdOrNull(string id)
{
    return string.IsNullOrEmpty(id) ? null : (Int64?)Convert.ToInt64(id);
}

private Int64? GetContentIdOrNull(DefaultIdType id)
{
    return id;
}

private void InitContent()
{
    // Attempt to get content from multiple sources in order of preference

    var contentSources = new Dictionary<ContentFrom, Func<Int64?>> {
        { ContentFrom.Query,   () => GetContentIdOrNull(Request.QueryString["id"]) },
        { ContentFrom.Default, () => GetContentIdOrNull(DefaultId) }
    };

    foreach (var source in contentSources) {
        var id = source.Value();
        if (!id.HasValue) {
            continue;
        }

        Content = GetContent(id.Value);
        ContentMode = source.Key;

        if (Content != null) {
            return;
        }
    }

    // Default
    ContentMode = ContentFrom.None;
}

如果您有更多的来源，这会有所帮助，但代价是增加复杂性。

Answer 3

就我个人而言，我发现当我有很多看似不同的语句时，就该创建一些函数了。

private ContentMode GetContentMode(){
}

private Content GetContent(int id){
}

private Content GetContent(HttpRequest request){
   return GetContent(Convert.ToInt64(request.QueryString["id"]));
}

private void InitContent(){
  ContentMode mode = GetContentMode();
  Content = null;
  switch(mode){
    case ContentMode.Query:
       GetContent(Request);
       break;
    case ContentMode.Default:
       GetContent(DefaultId);
       break;
    case ContentMode.None:
       ... handle none case...
       break;

  }
}

这样，您就可以分离您的意图 - 第一步，确定内容模式。然后，获取内容。

Answer 4

我建议你在这种情况下尝试某种工厂设计模式。您可以通过注册不同的创建者来抽象内容创建过程。此外，您可以根据自己的逻辑为每个创建者添加偏好。此外，我建议您封装与内容相关的所有数据，就像其他人的帖子中的“ContentDefinition”类一样。

一般来说，您需要知道灵活性和效率之间总是需要权衡。有时你的第一个解决方案就足够好了:)

Answer 5

好的，因为我注意到您实际上也想要 ContentFrom 模式有点晚了，所以我已尽力在我的原始答案下方提供您的示例的翻译

---

一般来说，我使用以下内容此类案例的范例。到处搜索并替换您的特定方法 :)

IEnumerable<T> ValueSources()
{
     yield return _value?? _alternative;
     yield return SimpleCalculationFromCache();
     yield return ComplexCalculation();
     yield return PromptUIInputFallback("Please help by entering a value for X:");
}

T EffectiveValue { get { return ValueSources().FirstOrDefault(v => v!=null); } }

请注意，您现在可以如何使 v!=null 任意“有趣”地满足您的目的。

另请注意，当 _value 或 _alternative 设置为“有趣”值时，惰性评估如何确保永远不会完成计算

---

这是我将样本放入此模具中的初步尝试。请注意我如何添加大量管道以确保它实际上编译成独立的 C# exe：

using System.Collections.Generic;
using System.Linq;
using System;
using T=System.String;

namespace X { public class Y
{
    public static void Main(string[]args) 
    {
        var content = Sources().FirstOrDefault(c => c); // trick: uses operator bool()
    }

    internal protected struct Content
    {
        public T Value;
        public ContentFrom Mode;
        //
        public static implicit operator bool(Content specimen) { return specimen.Mode!=ContentFrom.None && null!=specimen.Value; }
    }

    private static IEnumerable<Content> Sources()
    {
        // mock
        var Request = new { QueryString = new [] {"id"}.ToDictionary(a => a) };

        if (!String.IsNullOrEmpty(Request.QueryString["id"]))
            yield return new Content { Value = GetContent(Convert.ToInt64(Request.QueryString["id"])), Mode = ContentFrom.Query };
        if (DefaultId != null)
            yield return new Content { Value = GetContent((long) DefaultId), Mode = ContentFrom.Default };
        yield return new Content();
    }

    public enum ContentFrom { None, Query, Default };
    internal static T GetContent(long id) { return "dummy"; }
    internal static readonly long? DefaultId = 42;

} }

Answer 6

private void InitContent()
{
    Int64? id = !String.IsNullOrEmpty(Request.QueryString["id"])
                ? Convert.ToInt64(Request.QueryString["id"])
                : null;

    if (id != null && (Content = GetContent(id)) != null)
        ContentMode = ContentFrom.Query;
    else if(DefaultId != null && (Content = GetContent(DefaultId)) != null)
        ContentMode = ContentFrom.Default;
    else
        ContentMode = ContentFrom.None;
}