使用 ValueObjects 的 DTO 实例化一个实体

Question

我的问题是关于以下情况。有一个艺术家实体

` public class Artist : Entity
{
    public Contact artistContact { get; private set; }

    internal Artist() { }

    public Artist(Guid _artistId, Contact _artistContact): base(_artistId)
    {
        artistContact = new Contact(_artistContact);
    }

    public Artist(Contact _artistContact) : base()
    {
        artistContact = new Contact(_artistContact);

    }

    public void MakeArt(){} // methods and so on
}`

这个艺术家实体有一个值对象联系人，其中包含他的姓名、电子邮件等。

    `public class Contact : ValueObject<Contact>
{
    readonly string name;
    readonly string email;

    public string Name { get { return name; } }
    public string Email { get { return email; } }

    public Contact(string _name, string _email)
    {
        name = _name;
        email = _email;
    }

}`

所以我的问题是：可以吗

1. 创建值对象联系人的 DTO
2. 将 DTO 映射到工厂提供的联系人（ValueObjects 的工厂）
3. 最后调用 ArtistFactory.Create(Contact)

工厂使用 Activator.CreateInstance()，因此使用构造函数（构造函数的参数）创建对象是通过 Activator 完成的。

我正在尝试确保无法在没有联系人的情况下创建 Artist 实体，但不确定这是处理这种情况的正确方法。欢迎就如何处理此问题提出任何建议。

Answer 1

您正在使您的域复杂化并过度设计简单的对象创建。

找到合适的名字

首先，实体不是从天空中创造出来的。也许您可以为您的 ArtistFactory create 方法找到更好的名称。例如register。它确切地说明了那里发生了什么。

一个工厂

其次，您需要询问您的业务，Contact 对象是否存在于Customer 之外是否有任何意义？
I will make assumption, that it can't exist outside of Customer。
那么你的模型是错误的，因为你让人们在艺术家之外创建联系人。 您需要将creation 的Contact 放在ArtistFactory 的内部。

接下来，您应该想，如果电子邮件不是另一个 VO？ 也许您会想在其他地方重复使用它。

不要创建错误的域对象

接下来你不应该允许在你的域层中创建错误的对象。现在，通过您的实现，您确实允许例如使用此类字符串“fdsfsdssfsf”和空名称创建电子邮件。我想这不是你所期望的。 您使用公共构造函数和私有设置器创建 VO。 Setter 内部应该有验证。例如检查您的电子邮件是否正确。

简单的声明。

创建方法声明应使用本机类型。您将从域外部调用 create 方法，因此映射应该尽可能简单。
与其通过提供create (Contact contact) 将有关Contact 的信息推送给客户端，不如在Artist Factory 中包含create (string name, string email)。