Question

我有一个便于发现HID设备的类，当检测到设备时，event被引发并随后被另一个类检测到，该类将最终负责创建{object 1}}代表HID设备。 creation class使用新创建的event对象引发自己的HID。

考虑到这一点，我有几个设计查询：

1）   我已经对＆＃34; 最佳实践＆＃34; 进行了一些研究，关于在运行时创建未知数量或类型的对象Abstract Factory设计模式经常使用在结果中。 Abstract Factory设计模式是否适合我所拥有的场景，或者我还应该做些什么呢？

2）   HidFinder类引发一个事件以通知那些感兴趣的人（主要是HidCreator类）已发现设备。然后，HidCreator类会引发一个包含新创建的HID设备的事件。这似乎是正确的方法，但是，无论哪种方式确认都会受到赞赏。

下面是相关代码的一个愚蠢的例子。

public class HidFinder
{
    public event EventHandler<HidFoundArgs> HidFoundHandler;

    private void DeviceAdded(object sender, EventArrivedEventArgs e)
    {
        OnHidFoundHandler(new HidFoundArgs());
    }

    protected virtual void OnHidFoundHandler(HidFoundArgs e)
    {
        EventHandler<HidFoundArgs> handler = this.HidFoundHandler;
        if (handler != null)
        {
            handler(this, e);
        }
    }
}

public class HidCreator
{
    private readonly HidFinder hidFinder;

    public event EventHandler<IHidDevice> HidDeviceCreatedHandler;

    public HidCreator(HidFinder hidFinder)
    {
        this.hidFinder = hidFinder;
        this.hidFinder.HidFoundHandler += HidFinderOnHidFoundHandler;
    }

    private void HidFinderOnHidFoundHandler(object sender, HidFoundArgs hidFoundArgs)
    {
        // Create a new HID
        var newHidDevice = Factory.CreateMethod();
        OnHidDeviceCreatedHandler(newHidDevice);
    }

    protected virtual void OnHidDeviceCreatedHandler(IHidDevice e)
    {
        EventHandler<IHidDevice> handler = this.HidDeviceCreatedHandler;
        if (handler != null)
        {
            handler(this, e);
        }
    }
}

Answer 1

它通常看起来很好，但我会做两处修改：

Factory似乎是一个全局对象，最好使用依赖注入，例如更好的单元测试。

更改Factory.CreateMethod以使用参数，因为我们不知道将创建IHidDevice的具体实施方式，以及我们是否需要HidFoundArgs

更改后的代码：

public class HidCreator
{
    private readonly HidFinder hidFinder;
    private readonly IHidDeviceFactory factory;

    public event EventHandler<IHidDevice> HidDeviceCreatedHandler;

    public HidCreator(IHidDeviceFactory factory, HidFinder hidFinder)
    {
        this.factory = factory;
        this.hidFinder = hidFinder;
        this.hidFinder.HidFoundHandler += HidFinderOnHidFoundHandler;
    }

    private void HidFinderOnHidFoundHandler(object sender, HidFoundArgs hidFoundArgs)
    {
        // Create a new HID
        var newHidDevice = factory.Create(HidFoundArgs.ToCreationParameters(hidFoundArgs));
        OnHidDeviceCreatedHandler(newHidDevice);
    }

    protected virtual void OnHidDeviceCreatedHandler(IHidDevice e)
    {
        EventHandler<IHidDevice> handler = this.HidDeviceCreatedHandler;
        if (handler != null)
        {
            handler(this, e);
        }
    }
}

public interface IHidDeviceFactory
{
    IHidDevice Create(HidCreationParameters parameters);
    ...
}

public class HidCreationParameters
{
   ...
}

public class HidFoundArgs
{
    public static HidCreationParameters ToCreationParameters(HidFoundArgs args)
    {
        ...
    }
}

运行时对象创建的设计模式

1 个答案: