È un "odore di modello" inserire getter come "FullName" o "FormattedPhoneNumber" nel tuo modello?

Sto lavorando su un'app ASP.NET MVC e ho preso l'abitudine di inserire quelli che sembrano getter utili e convenienti nelle mie classi di modello / entità.

Per esempio:

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); }
    }
}

Mi chiedo che la gente pensi al FullNamee ai FormattedPhoneNumbergetter.

Semplificano la creazione di formati di dati standardizzati in tutta l'app e sembrano salvare un sacco di codice ripetuto, ma si potrebbe sicuramente sostenere che il formato dei dati è qualcosa che dovrebbe essere gestito nella mappatura da modello a modello di vista.

In effetti, stavo originariamente applicando questi formati di dati nel mio livello di servizio in cui eseguo la mia mappatura, ma stava diventando un peso dover costantemente scrivere i formattatori e applicarli in molti luoghi diversi. Ad esempio, utilizzo "Nome completo" nella maggior parte delle visualizzazioni e dover digitare qualcosa di simile in model.FullName = MappingUtilities.GetFullName(entity.FirstName, entity.LastName);tutto il luogo mi è sembrato molto meno elegante della semplice digitazione model.FullName = entity.FullName(o, se si utilizza qualcosa come AutoMapper, potenzialmente non si digita affatto).

Quindi, dove si disegna la linea quando si tratta di formattazione dei dati. È "ok" fare la formattazione dei dati nel tuo modello o è un "odore di pattern"?

Nota: sicuramente non ho HTML nel mio modello. Per questo uso gli helper HTML. Sto parlando rigorosamente di formattazione o combinazione di dati (e in particolare i dati utilizzati di frequente).

Nel tuo esempio, mi piace il FullNamegetter (per tutti i motivi che hai indicato) ma non mi piace il getter FormattedPhoneNumber. Il motivo è: probabilmente non è così facile (una volta che hai numeri di telefono internazionali ecc.) E se metti la logica per formulare i numeri di telefono in un metodo di Member, è probabile che dovrai refactoring (o copia e incolla caugh ) una volta che è necessario un numero di telefono formattato per Institution, Vendorecc.

EDIT: IMO sarebbe meglio avere una PhoneNumberclasse con un Formattedgetter.

Le cose che devi considerare quando scrivi il codice: è corretto? È leggibile? È efficiente? È mantenibile? Direi che, come accennato da @btilly, non è gestibile a causa della formattazione specifica della cultura, ma la domanda sembra essere più generale.

L'uso di accessori come questi rende il tuo codice più leggibile e, a seconda di come lo usi, potrebbe rendere più pulite altre parti del codice. Secondo me, questo non ha alcun odore. Comincerebbe a puzzare se avessi gli accessori di formattazione per qualsiasi tipo di stringa che potresti voler stampare ( public string FirstLastName; public string FullName; public string FullNameWithMiddleInitial; public string PhoneNumberWithAreaCode; public string PhoneNumberWithoutAreaCode; public string PhoneNumberWithCountryCode;, ecc.)

Oppure, per dirla in altro modo, l'uso di un pattern non rende automaticamente il tuo codice "odore di pattern". Devi abusarne se vuoi guadagnare quell'attributo.

Rompe il principio della singola responsabilità. Perché non fare una classe di numero di telefono, ecc ...?

Per il tuo esempio, non lo vedo come un affare troppo grande per utilizzare formati specifici. È una o due e tutte le parti dell'applicazione utilizzano lo stesso formato.

Il punto in cui tale decisione inizia a essere scomposta è quando si hanno gli stessi dati che vanno in luoghi diversi che richiedono formati diversi .

Se ciò accadesse, sarei tentato di Memberriportare la classe a:

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

E quindi eseguire adattatori diversi per ciascun target. Ad esempio, supponiamo che le informazioni fossero richieste in formato CSV:

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

Supponendo sempre di aver disinfettato i dati in modo che non ci siano virgole ecc. Nelle stringhe.

L'adattatore non deve essere un metodo di estensione, ma per questo caso fittizio sembra adattarsi.