Це "запах шаблону", щоб у вашу модель включити геттери типу "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 (з усіх наведених вами причин), але мені не подобається Getter FormattedPhoneNumber. Причина полягає в тому, що це, мабуть, не так просто (коли у вас є міжнародні телефонні номери тощо), і якщо ви розмістите логіку формування формату телефонних номерів методом Member, швидше за все, вам знадобиться зробити рефакторинг (або скопіювати вставку обережно ), як тільки ви потрібен відформатований номер телефону Institution, і Vendorт.п. теж.

EDIT: 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;
    }
}

Завжди припускайте, що ви санітували дані, щоб у рядках не було коми тощо.

Адаптер не повинен бути методом розширення, але в цьому випадку, мабуть, він підходить.