在模型中放入诸如“ FullName”或“ FormattedPhoneNumber”之类的吸气剂是“一种模式的气味”吗？

我正在开发ASP.NET MVC应用程序，并且已经养成了将看起来有用且方便的吸气剂放入模型/实体类的习惯。

例如：

public class Member
{
    public int Id { get; set; }
    public string FirstName { get; set; }
    public string LastName { get; set; }
    public string PhoneNumber { get; set; }

    public string FullName
    {
        get { return FirstName + " " + LastName; }
    }

    public string FormattedPhoneNumber
    {
        get { return "(" + PhoneNumber.Substring(0, 3) + ") " + PhoneNumber.Substring(3, 3) + "-" + PhoneNumber.Substring(6); }
    }
}

我想知道人们在考虑FullName和FormattedPhoneNumber吸气剂。

它们使在整个应用程序中创建标准化数据格式变得非常容易，而且似乎可以节省大量重复代码，但是可以肯定地说，数据格式是在从模型到视图模型的映射中应该处理的东西。

实际上，我最初是在我进行映射的服务层中应用这些数据格式的，但是，这一直成为一种负担，必须不断编写格式器，然后将其应用到许多不同的地方。例如，在大多数视图中，我都使用“全名”，而不得不model.FullName = MappingUtilities.GetFullName(entity.FirstName, entity.LastName);在各处键入类似的内容似乎比仅仅键入model.FullName = entity.FullName（或者，如果您使用类似AutoMapper的内容，可能根本不键入任何内容）要优雅得多。

因此，在数据格式化方面您会在哪里划清界线。在模型中进行数据格式化是否“还可以”，还是“图案异味”？

注意：我的模型中绝对没有任何HTML。我为此使用HTML帮助器。我严格地说的是格式化或合并数据（尤其是经常使用的数据）。

在您的示例中，我喜欢FullNamegetter（出于您给出的所有原因），但我不喜欢FormattedPhoneNumber getter。其原因是：它可能不是那么容易（一旦你有国际电话号码等），如果你把逻辑中的方法在格式化的电话号码Member，那么你将需要重构（或复制粘贴caugh），一旦你也需要格式化的电话号码Institution，Vendor等等。

编辑：IMO最好有一个PhoneNumber带有Formatted吸气剂的课程。

编写代码时需要考虑的事情：是否正确？可读吗？有效率吗？它可以维护吗？正如@btilly所言，我会争辩说由于特定于文化的格式，它是不可维护的，但是问题似乎更笼统。

使用这样的访问器可以使您的代码更具可读性，并且根据您的使用方式，可能会使代码的其他部分更清晰。我认为这根本没有味道。如果您具有可能要打印的任何类型的字符串的格式化访问器（public string FirstLastName; public string FullName; public string FullNameWithMiddleInitial; public string PhoneNumberWithAreaCode; public string PhoneNumberWithoutAreaCode; public string PhoneNumberWithCountryCode;，等等），它就会开始发出气味。

或者，换句话说，使用模式不会自动使您的代码具有“模式气味”。如果您想获得该属性，则需要滥用它。

打破了单一责任原则。为什么不进行电话号码分类等？

对于您的示例，我认为使用特定格式没有什么大不了的。它是一两个，应用程序的所有部分都使用相同的格式。

当您将相同的数据发送到需要不同格式的多个不同位置时，该决定就会开始崩溃。

如果发生这种情况，我很想将Member课程拉回到：

public class Member
{
    public int Id { get; set; }
    public string FirstName { get; set; }
    public string LastName { get; set; }
    public string PhoneNumber { get; set; }  
}

然后为每个目标执行不同的适配器。例如，假设信息是CSV格式所必需的：

public static class CSVMemberAdapter
{
    public static string ToCSV(this Member mbr)
    {
         return mbr.Id + "," + mbr.LastName + "," + mbr.FirstName, "," mbr.PhoneNumber;
    }
}

始终假定您已经清理了数据，所以字符串中没有逗号等。

适配器不一定必须是扩展方法，但对于这种虚假的情况，它似乎很合适。