我处理设备网络访问的正确方法是什么

Question

我目前正在开发 blazor 服务器项目，该项目将显示来自 modbus tcp/ip 设备的信息。我有一个名为“DeviceModel”的类，它对 Modbus 设备进行建模。下面显示了一个简化的示例。

public string DeviceName {get;set;}
public string IpAddress {get;set;}
public string Port {get;set;}
public int[] Registers {get;set;}
public string Alarm1 {get;set;}

设备模型类还包含解析寄存器信息的方法。例如，下面的代码片段将检查寄存器数组中某个索引处的值。根据该值，它将 Alarm1 属性设置为 ON 或 OFF。

public void CheckAlarm1(){
int status = Registers[4];
Alarm1 = status == 1 ? "ON" : "OFF";

}

我有另一个名为“NetworkAccess”的类，它处理与设备的 TCP/IP 连接。下面显示了一个简化的示例

// ModbusClient is a package which handles the reading/writing to TCP/IP Modbus 
private ModbusClient _client;
public string IPAddress {get;set;}
public string Port {get;set;}
public DeviceModel Device {get;set;}

public NetworkAccess(DeviceModel dev){
      IPAddress = dev.IPAddress;
      Port = dev.Port;
      _client = new ModbusClient(IPAddress,Port);
      _client.Connect();
}

NetworkAccess 类处理从网络上的设备读取和写入数据。下面是将数据写入 Modbus 设备上的单个寄存器的示例方法。

public void WriteSingleRegister(int address,int dataToAdd){
     _client.WriteSingleRegister(address,dataToAdd);
}

在网页的 Razor 组件中，在 OnInitialized() 方法中，我从数据库中获取包含 DeviceModels 的列表，其中填充了每个设备的 IP 地址、端口和名称等信息。为了将信息读取到设备，我有另一种方法“GetData()”，如下所示。

public async void GetData(){
     foreach(var device in Devices){
          NetworkAccess network = new NetworkAccess(dev);
          var dataUpdate = await network.ReadRegistersAsync(0,20);
          dev.Registers= dataUpdate;
     }
}

我目前的设置工作正常。为了写入设备，我会在 Razor 组件中执行类似的操作，

NetworkAccess network = new NetworkAccess(dev);
network.WriteRegistersAsync(0,new int[] {0,0,0,...}};

但我遇到的问题是我不确定处理我的情况的正确（或最佳）方法。在我看来，如果我的 DeviceModel 类中有用于特定操作（例如“ResetAlarm1”或“ClearRegisters”）的方法，那么对我来说更有意义。这样我就可以这样做，

dev.ResetAlarm1();

而不是在下面的剃刀组件中这样做

NetworkAccess net = new NetworkAccess(dev)
dev.WriteRegister(6,0); // where 6 is the register to write to and 0 is value to write

，我想我的问题是我应该将“NetworkAccess”添加到设备模型中并处理在其中创建连接和读/写设备的操作吗？或者将 NetworkAccess 和 DeviceModel 分开更有意义吗？

我希望这篇文章有意义。这更多的是一个关于设计的问题，而不是一个关于解决问题的问题。虽然我当前的解决方案运行良好，但我想更好地了解这是否是正确的方法，或者我是否偏离了。

感谢您的帮助！

Answer 1

或者将 NetworkAccess 和 DeviceModel 分开更有意义吗？

正如 SOLID 的单一责任原则所说：

单一职责原则（SRP）是一种计算机编程
该原则指出，一个中的每个模块、类或函数
计算机程序应该对其中的一个部分负责
程序的功能，并且它应该封装该部分。所有的
该模块、类或功能的服务应该紧密结合
承担这个责任。

在此处详细了解 SOLID 单一责任原则。

因此，在 Device 类中创建单独的方法 dev.ResetAlarm1() 对我来说更可取。

很难说我的重构代码是否适合你，但我尽力了：

public class Device
{
    public string Name { get; set; }

    public string IpAddress { get; set; }

    public string Port { get; set; }

    public int[] Registers { get; set; }

    public string[] Alarms { get; set; }


    public void CheckAlarm(int registerIndex)
    {
        int status = Registers[registerIndex];
        Alarms[registerIndex] = status == 1 ? "ON" : "OFF";
    }
}