Question

我正在构建一个简单的Guard API来防止传递给函数等的非法参数。

我有以下代码：

public static class Guard
{
    public static GuardArgument<T> Ensure<T>(T value, string argumentName)
    {
        return new GuardArgument<T>(value, argumentName);
    }
}

public class GuardArgument<T>
{
    public GuardArgument(T value, string argumentName)
    {
        Value = value;
        Name = Name;
    }

    public T Value { get; private set; }
    public string Name { get; private set; }
}

// Example extension for validity checks
public static GuardArgument<T> IsNotNull<T>(this GuardArgument<T> guardArgument, string errorMessage)
{
    if (guardArgument.Value == null)
    {
        throw new ArgumentNullException(guardArgument.Name, errorMessage);
    }

    return guardArgument;
}

目前代码可以以类似的方式使用（注意这只是一个愚蠢的例子）：

void DummyMethod(int? someObject) {

    Guard.Ensure(someObject, "someObject")
       .IsNotNull()
       .IsGreaterThan(0)
       .IsLessThan(10);
}

一切正常。我现在想要做的是扩展API，以便以下列方式在检查中包含子属性：

Guard.Ensure(someObject, "someObject")
    .IsNotNull()
    .Property(
        (x => x.ChildProp1, "childProp1")
           .IsNotNull()
           .IsGreaterThan(10)
     )
     .Property(
        (x => x.ChildProp2, "childProp2")
           .IsNotNull()
           .IsLessThan(10)
     );

显然，新的.Property方法需要返回父GuardArgument才能进行链接。此外，子属性需要能够使用现有的检查方法（IsNotNull()等）来避免代码重复。

我无法弄清楚如何构造lambda / Property函数参数或.Property方法应该位于何处 - 即它应该是GuardArgument或其他地方的属性，或者即使在那里是一个更好的API结构。

Answer 1

以下函数允许使用与您想要的语法类似的语法。

public static GuardArgument<T> Property<T, TProp>(this GuardArgument<T> guardArgument, Func<T, TProp> getProperty, string propertyName, Action<GuardArgument<TProp>> validate)
{
    GuardArgument<TProp> propertyGuardArgument = new GuardArgument<TProp>(getProperty(guardArgument.Value), propertyName);

    validate(propertyGuardArgument);

    return guardArgument;
}

该函数为selected属性创建一个新的GuardArgument，然后将其传递给Action参数，以便您根据需要进行验证。

这也允许无限链接属性，尽管我不确定它是否特别易读。

用法：

Guard.Ensure(someObject, "someObject")
     .IsNotNull()
     .Property(x => x.ChildProp1, "childProp1", childProp1 =>
         childProp1.IsNotNull()
                   .IsLessThan(10)
                   .Property(y => y.InnerChildProperty, "innerChildProperty", innerChildProperty =>
                       innerChildProperty.IsNotNull()
                    )
     )
     .Property(x => x.ChildProp2, "childProp2", childProp2 =>
         childProp2.IsNotNull()
                   .IsGreaterThan(10)
     );

Answer 2

我认为将属性检查放在父对象检查的链中没有任何好处。所以我建议为父对象创建一个链，为每个属性创建另一个链。这更具可读性：

  Guard.Ensure(a, "a")
    .IsNotNull("a is null");
  Guard.Ensure(a.p0, "a.p0")
    .IsGreaterThan(10);
  Guard.Ensure(a.p1, "a.p1")
    .IsGreaterThan(5);

Answer 3

我想你在这里重新发明轮子。安装此扩展程序 - Code Contracts，此处docs如何使用它。

除了与您类似的基于代码的断言，即：

 public int[] Bar(){
   Contract.Ensures( Contract.ForAll(0, Contract.Result<int[]>().Length, index => Contract.Result<int[]>()[index] > 0));

 ....
 }

或

 Contract.Requires<ArgumentNullException>( x.Value.NestedObject != null, ”x.Value.NestedObject” );

但是还有用于检查界面的属性和广泛的功能集，良好的前后条件等等。检查它！

如何构建Fluent嵌套Guard API

3 个答案: