2

我正在重构一个通过构造函数接收可选委托的函数。委托在类中触发事件时运行。如果未传递委托，则使用本地默认函数：

public class Foo
{
    int    _memberVariable;
    readonly Action _onEventOne;
    readonly Action _onEventTwo;

    public Foo(Action onEventOne, Action onEventTwo = null)
    {
        _memberVariable = 0;

        _onEventOne = onEventOne;
        _onEventTwo = onEventTwo ?? DefaultEventTwo;

        _onEventOne();
    }

    private void DefaultEventTwo()
    {
        ++_memberVariable;
    }
}

我正在寻找删除默认值（这是一个公共接口，因此重载会更可取），这是在生产代码中，所以我不想更改接口，除非我必须。

在理想的世界中，我会使用构造函数链接：

public Foo(Action onEventOne) : this(onEventOne, DefaultEventTwo)
{
    //CS0120 An object reference is required for the non-static field, method, or property 'Foo.DefaultEventTwo()
}

（我理解为什么这不起作用，只是举例说明如果这不是构造函数我会使用的那种解决方案）。

因为委托是只读的，所以我不能在共享的初始化类型函数中设置它们。

有没有比仅仅传入 null 然后在主构造函数中捕获它更好的方法来处理这种情况？它感觉不是很优雅，我希望能够在理想情况下将 null Action 捕获为异常（如果外部调用者使用 null 而不是使用重载的构造函数）。我可以从代表中删除只读，但再次感觉不是一个很好的解决方案，因为它们真的是只读的。

任何想法将不胜感激。

4

2 回答 2

1

我可以让它工作的唯一方法是让它变得丑陋（在我自己看来）。

您必须提供一个静态方法，但该静态方法可以使用对此的引用来获取实际方法。

这就是我想出的。

public Foo(Action onEventOne) : this(onEventOne, self => self.DefaultEventTwo)
{
    //CS0120 An object reference is required for the non-static field, method, or property 'Foo.DefaultEventTwo()
}

public Foo(Action onEventOne, Action onEventTwo = null) : this(onEventOne, self => onEventTwo)
{ }

// private constructor, just for the sake of getting it working
private Foo(Action onEventOne, Func<Foo, Action> onEventTwo = null)
{
    _memberVariable = 0;

    _onEventOne = onEventOne;
    _onEventTwo = onEventTwo(this); // <--

    _onEventOne();
}

self => self.DefaultEventTwo是获取动作的静态函数。该函数用于调用以onEventTwo(this)获取this实例的默认事件。

于 2016-12-09T13:16:25.383 回答
1

我错过了什么吗？

public class Foo
{
    int    _memberVariable;
    readonly Action _onEventOne;
    readonly Action _onEventTwo;

    public Foo(Action onEventOne): this(onEventOne, null) { }
    public Foo(Action onEventOne, Action onEventTwo)
    {
        _memberVariable = 0;

        _onEventOne = onEventOne;
        _onEventTwo = onEventTwo ?? DefaultEventTwo;

        _onEventOne();
    }

    private void DefaultEventTwo()
    {
        ++_memberVariable;
    }
}

您所要做的就是删除默认值并创建一个只有一个参数的新构造函数。现在在最详细的构造函数（原始构造函数）中检查提供的值null是否_onEventTwo为DefaultEventTwo.

为了避免任何人使用简化的构造函数，只需 make it internal。

编辑：关于异常处理。将内部构造函数用作“主要”构造函数怎么样？所有其他构造函数都使用指示调用来自何处的参数进行调用：

internal Foo(Action onEventOne): this(onEventOne, null, true) { }
// public API: NULL not allwoed as param
public Foo(Action onEventOne, Action onEventTwo) : this(onEventOne, onEventTwo, false) { }
internal Foo(Action onEventOne, Action onEventTwo, bool internalUse)
{
    _memberVariable = 0;

    _onEventOne = onEventOne;
    if(onEventTwo == null)
    {
        if(!internalUse) throw new ArgumentNullException("onEventTwo");
        else this._onEventTwo = DefaultEventTwo;
    }
    _onEventOne();
}
于 2016-12-09T13:18:27.587 回答