Question

我正在寻找一种设计代码的正确方法。我有一个按以下方式创建的设备列表：

public void LoadDevices()
{
     using (XmlReader xmlRdr = new XmlTextReader(deviceConfigPath))
         deviceList = (from deviceElem in XDocument.Load(xmlRdr).Element("devices").Elements("device")
                 where (string)deviceElem.Attribute("type") == "mks247"
                 select (SmartCatDeviceBase)new mks247Device(
                    (string)deviceElem.Attribute("ip"),
                    (string)deviceElem.Attribute("name"),
                    (string)deviceElem.Attribute("id"),
                    (bool)deviceElem.Attribute("autoconnect")
                     )).ToList();
}

需要对SmartCatDeviceBase进行类型转换，因为我有更多不同类型的设备（具有相同的基类）进入该列表。

现在问题是“autoconnect”：它要求设备打开异步网络连接，这不应该在构造函数（as Stephen Cleary states here）中完成。

因此我想采用某种类似工厂的东西：

private async Task<SmartCatDeviceBase> Createmks247DeviceAsync(string host, string name, string id, bool autoconnect = true)
    {
        mks247Device dev = new mks247Device(host, name, id); // Now without the autoconnect, that shouldn't be in the constructor.
        // Connect.
        if (autoconnect)
        {
            bool connected = await dev.ConnectAsync();
            // Begin to poll for data.
            dev.BeginPolling();
        }
        return dev;  
    }

所以问题是：我怎样才能使代码有效？因为使用 Createmks247DeviceAsync 而不是 new mks247Device（）在我的LINQ代码中不起作用：

“System.Threading.Tasks.Task”类型无法转换为“SmartCatDeviceBase”。

也无法在select语句中使用await关键字......

或者有没有其他方法正确设计此类代码？在构造函数中设置自动更正标志然后“从外部”连接似乎违反了OOP：当有自动连接选项时，我希望在我将其设置为true时反对自动创建......

提前多多感谢！祝你新年快乐！

Answer 1

将方法一直转换为异步。

使用Linq选择所有任务，然后等待Task.WhenAll连接它们。

public async Task LoadDevicesAsync() {
     using (XmlReader xmlRdr = new XmlTextReader(deviceConfigPath)) {
         var getdeviceListTasks = (from deviceElem in XDocument.Load(xmlRdr).Element("devices").Elements("device")
                 where (string)deviceElem.Attribute("type") == "mks247"
                 select Createmks247DeviceAsync(
                    (string)deviceElem.Attribute("ip"),
                    (string)deviceElem.Attribute("name"),
                    (string)deviceElem.Attribute("id"),
                    (bool)deviceElem.Attribute("autoconnect")
                     ));
        var devices = await Task.WhenAll(getdeviceListTasks);
        deviceList = devices.ToList();
    }
}

C＃异步（工厂）方法＆amp; LINQ

1 个答案: