类中相互依赖的属性

Question

我有以下类可以改变某个查询的结果：

public class UniversalPrincipalQueryOptions
{
    public Boolean IncludeProperties { get; set; }
    public Boolean IncludeOrganizationUnits { get; set; }
    public Boolean IncludeManagers { get; set; }

    public static UniversalPrincipalQueryOptions All
    {
        get
        {
            return new UniversalPrincipalQueryOptions { IncludeOrganizationUnits = true, IncludeProperties = true, IncludeManagers = true };
        }
    }

    public static UniversalPrincipalQueryOptions None
    {
        get
        {
            return new UniversalPrincipalQueryOptions { IncludeProperties = false };
        }
    }

    public UniversalPrincipalQueryOptions()
    {
        this.IncludeProperties = true;
    }
}

问题是，当IncludeProperties 为false 时将IncludeOrganizationUnits 设置为true 在我的情况下没有意义，IncludeManagers 也是如此，这也取决于将IncludeOrganizationUnits 设置为true .

我认为这不是最好的方法。所以我的问题是，我可以使用什么模式来改进这个类并让其他人更容易使用我的类？

更新：

好的，所以我最终切换到使用 Flags 属性装饰的 enum：

[Flags]
public enum UniversalPrincipalQueryOptions
{
    None = 0,
    IncludeProperties = 1,
    IncludeOrganizationUnits = 2 | IncludeProperties,
    IncludeManagers = 4 | IncludeOrganizationUnits
}

Answer 1

简单的答案是验证值而不是纯自动属性：

public Boolean IncludeProperties { get; set; }

private Boolean includeOrganizationUnits;
public Boolean IncludeOrganizationUnits 
{ 
    get { return this.includeOrganizationUnits }
    set
    {
        if(value && !IncludeProperties)
            throw new InvalidOperationException("IncludeProperties must be true when IncludeOrganizationUnits is set to true");
        includeOrganizationUnits = value;
    }
}

这种前置条件也可以使用CodeContracts 进行配置，但请注意performance hit by doing so。

如果用户实际上不需要自己设置这些布尔值，则另一种选择是扩展您的静态列表以包含所有（有效）可能性并将事情设为私有（如 setter 和 ctor）：

public class UniversalPrincipalQueryOptions
{
    public Boolean IncludeProperties { get; private set; }
    public Boolean IncludeOrganizationUnits { get; private set; }
    public Boolean IncludeManagers { get; private set; }

    // UPDATED !!
    public static UniversalPrincipalQueryOptions OrganizationUnitsAndManagers
    {
        get
        {
            return new UniversalPrincipalQueryOptions { IncludeOrganizationUnits = true, IncludeProperties = true, IncludeManagers = true };
        }
    }

    // NEW!!
    public static UniversalPrincipalQueryOptions OrganizationUnits
    {
        get
        {
            return new UniversalPrincipalQueryOptions { IncludeOrganizationUnits = true, IncludeProperties = true};
        }
    }

    public static UniversalPrincipalQueryOptions None
    {
        get
        {
            return new UniversalPrincipalQueryOptions { IncludeProperties = false };
        }
    }

    private UniversalPrincipalQueryOptions()
    {
        this.IncludeProperties = true;
    }
}