我应该在这里使用哪种设计模式

Question

出于学习目的，我正在编写一个基于套接字的客户端/服务器应用程序。我设计了一个自定义协议来确保我可以正确处理我的数据包。今天在检查代码的一些旧部分时，我意识到我创建数据包的方式包含很多冗余代码。

我有不同类型的数据包，例如 ImagePacket、MessagePacket 等。所有类型的创建仅在细微之处有所不同，例如标头和分隔符的创建是相同的。

为了改进这一点，我想出了一个这样的解决方案（简化的）：

abstract class Packet
{
 public Packet(object o)
 {
 MemoryStream memoryStream = new MemoryStream();         

 AddHeader(ref memoryStream);
 AddData(ref memoryStream, obj);
 AddDelimiter(ref memoryStream);
 _packetBytes = memoryStream.ToArray();

 memoryStream.Close();
}
protected abstract void AddData(ref MemoryStream ms, object obj);

AddData 方法作为抽象方法实现并在具体类中重写，而 AddHeader 和 AddDelimiter 是在抽象类 Packet 本身中定义的。

这工作正常，我没有像以前那么多重复的代码，但我不满意 将对象传递给 AddData，因为没有明确说明我不能向 ImagePacket 构造函数提供字符串。

// correct
Packet myMsgPacket = new MessagePacket("hello world"); 
Packet myImagePacket = new ImagePacket(image);
// wrong, but will be compiled :(
Packet myChaosPacket = new ImagePacket("muaha you're doomed");

如果我必须检查传递的数据类型是否正确，我最终会再次得到大量愚蠢的代码。 如何才能既减少重复代码又摆脱上述问题呢？

Answer 1

这听起来像您需要使用工厂模式

Packet MyPacket = MyPacketFactory.CreatePacket(Data)

MyFactory.CreatePacket 然后返回一个 IPacket

更新： 根据下面的评论，我应该'已经更清楚了。您的工厂可以有许多采用不同数据类型的重载 CreatePacket() 方法...

IPacket CreatePacket(Image Data) {}
IPacket CreatePacket(String Data) {}
IPacket CreatePacket(Exception Data) {}

如果您有多种类型的数据包，其中仅包含字符串（例如消息数据包、状态数据包或类似数据包），您可以创建一个表示 需要哪个字符串数据包...

IPacket CreatePacket(String Data, StringPacketTypesEnum PacketType) {}

在数据包工厂内，您可以使用处理所有重复代码的通用函数 - 例如 AddDelimiter() - 这将使您的代码 干

Answer 2

Ruby on Rails 使用验证来解决这个问题。

如果您的 addData() 函数在输入上调用 validate()，您只需在每个子类中重写 validate() 即可。

Answer 3

我会使用返回接口的工厂模式。

除此之外，您不需要使用 ref 作为引用类型。

Answer 4

我认为你目前的解决方案是可以的。您只需将传递给 ImagePacket 构造函数的数据类型缩小为 Image 类型。例如：

class ImagePacket : Packet{
  public ImagePacket(Image img) : base(img){}
}

这将在复制时检查数据是否是图像。当然，如果您确实必须传递 object 类型，则这可能不起作用。