Является ли это «запахом шаблона» для размещения в вашей модели геттеров типа «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 для этого. Я строго говорю о форматировании или объединении данных (и особенно данных, которые часто используются).

В вашем примере мне нравится метод получения FullName(по всем причинам, которые вы указали), но мне не нравится метод получения FormattedPhoneNumber. Причина в том, что это, вероятно, не так просто (если у вас есть международные телефонные номера и т. Д.), И если вы разместите логику для форматирования телефонных номеров в методе Member, скорее всего, вам потребуется рефакторинг (или копирование-вставка caugh ), как только вы нужен форматированный номер телефона Institutionи Vendorт. д.

РЕДАКТИРОВАТЬ: ИМО было бы лучше иметь 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;
    }
}

Всегда предполагая, что вы санировали данные, чтобы в строках не было запятых и т. Д.

Адаптер не должен быть методом расширения, но для этого вымышленного случая он подходит.