c# - 业务对象是否应该能够创建自己的 DTO？

Question

假设我有以下类(class):

class Camera
{
    public Camera(
        double exposure,
        double brightness,
        double contrast,
        RegionOfInterest regionOfInterest)
    {
        this.exposure = exposure;
        this.brightness = brightness;
        this.contrast = contrast;
        this.regionOfInterest = regionOfInterest;
    }

    public void ConfigureAcquisitionFifo(IAcquisitionFifo acquisitionFifo)
    {
        // do stuff to the acquisition FIFO
    }

    readonly double exposure;
    readonly double brightness;
    readonly double contrast;
    readonly RegionOfInterest regionOfInterest;
}

...和一个 DTO，用于跨服务边界 (WCF) 传输相机信息，例如，以便在 WinForms/WPF/Web 应用程序中查看:

using System.Runtime.Serialization;

[DataContract]
public class CameraData
{
    [DataMember]
    public double Exposure { get; set; }

    [DataMember]
    public double Brightness { get; set; }

    [DataMember]
    public double Contrast { get; set; }

    [DataMember]
    public RegionOfInterestData RegionOfInterest { get; set; }
}

现在我可以向 Camera 添加一个方法来公开它的数据:

class Camera
{
    // blah blah

    public CameraData ToData()
    {
        var regionOfInterestData = regionOfInterest.ToData();

        return new CameraData()
        {
            Exposure = exposure,
            Brightness = brightness,
            Contrast = contrast,
            RegionOfInterest = regionOfInterestData
        };
    }
}

或，我可以创建一个方法，该方法需要传入一个特殊的 IReporter，以便相机向其公开其数据。这消除了对 Contracts 层的依赖(Camera 不再需要知道 CameraData):

class Camera
{
    // beep beep I'm a jeep

    public void ExposeToReporter(IReporter reporter)
    {
        reporter.GetCameraInfo(exposure, brightness, contrast, regionOfInterest);
    }
}

那我该怎么做呢？我更喜欢第二种，但它要求 IReporter 有一个 CameraData 字段(由 GetCameraInfo() 更改)，感觉很奇怪。另外，如果有更好的解决方案，请与我分享!我还是一个面向对象的新手。

Answer 1

我通常会说不，他们不应该，因为 DTO 是特定于服务或应用程序的，而域模型是您的“最内层”层，应该没有依赖关系。 DTO 是域模型之外的东西的实现细节，因此，它打破了域模型的抽象以了解它们。

您是否考虑过查看 AutoMapper为了这？您最终会以这种方式编写更少的代码。在这种情况下，我认为您可以简单地逃脱惩罚:

Mapper.CreateMap<RegionOfInterest, RegionOfInterestData>();
Mapper.CreateMap<Camera, CameraData>();

随后:

CameraData cd = Mapper.Map<Camera, CameraData>(camera);

这不仅减少了代码改动，而且将映射代码划分到它自己的“映射层”中 - 您有一个或多个模块来注册这些映射，您可以将这些映射放入任何真正使用 DTO 的程序集中。

当然，您始终可以创建扩展方法来简化实际映射:

public static class CameraExtensions
{
    public static CameraData ToCameraData(this Camera camera)
    {
        return Mapper.Map<Camera, CameraData>(camera);
    }
}

这使得整个事情像编写 camera.ToCameraData() 一样简单，但是没有在领域对象(Camera)之间创建硬依赖>) 和 DTO (CameraData)。您基本上拥有原始版本的所有易用性，但没有耦合。

如果您创建这些依赖项是因为您试图从未公开的私有(private) Camera 数据创建 CameraData 对象，那么我的第一 react 是，这个设计有些不对劲。为什么不在 Camera 对象上公开只读属性？如果您通过 ToData 方法无论如何都让外界访问它们，那么您显然没有隐藏这些信息，您只是让访问变得更加麻烦。

如果您在 3 个月后决定需要一种不同类型的 DTO 怎么办？您不必每次想要支持新用例时都修改以域为中心的 Camera 对象。在我看来，最好在类中放置一些只读公共(public)属性，以便映射器可以访问他们需要的属性。