Ist es ein "Mustergeruch", Getter wie "FullName" oder "FormattedPhoneNumber" in Ihr Modell einzufügen?

Ich arbeite an einer ASP.NET MVC-App und habe mir angewöhnt, in meine Modell- / Entitätsklassen hilfreiche und praktische Informationen einzufügen.

Beispielsweise:

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

Ich frage mich, ob die Leute über das FullNameund die FormattedPhoneNumberGetter nachdenken .

Sie machen es sehr einfach, standardisierte Datenformate in der gesamten App zu erstellen, und sie scheinen viel wiederholten Code zu speichern. Es kann jedoch durchaus argumentiert werden, dass das Datenformat bei der Zuordnung von Modell zu Ansichtsmodell zu berücksichtigen ist.

Eigentlich habe ich diese Datenformate ursprünglich in meiner Service-Schicht angewendet, wo ich mein Mapping durchführe, aber es wurde zu einer Last, ständig Formatierer schreiben zu müssen, um sie dann an vielen verschiedenen Stellen anzuwenden. ZB verwende ich in den meisten Ansichten "Vollständiger Name" und es model.FullName = MappingUtilities.GetFullName(entity.FirstName, entity.LastName);schien viel weniger elegant als nur zu model.FullName = entity.FullNametippen (oder, wenn Sie etwas wie AutoMapper verwenden, möglicherweise gar nichts zu tippen).

Wo ziehen Sie die Linie, wenn es um die Datenformatierung geht? Ist es "okay", Daten in Ihrem Modell zu formatieren, oder ist das ein "Mustergeruch"?

Hinweis: Ich habe definitiv kein HTML in meinem Modell. Ich benutze dafür HTML-Helfer. Ich spreche ausschließlich von der Formatierung oder Kombination von Daten (und insbesondere von Daten, die häufig verwendet werden).

In Ihrem Beispiel gefällt mir der FullNameGetter (aus all den Gründen, die Sie angegeben haben), aber der Getter FormattedPhoneNumber gefällt mir nicht. Der Grund ist: Es ist wahrscheinlich nicht so einfach (wenn Sie internationale Telefonnummern usw. haben) und wenn Sie die Logik für die Formatierung von Telefonnummern in eine Methode von setzen Member, müssen Sie wahrscheinlich umgestalten (oder vorsichtig kopieren und einfügen ), sobald Sie braucht eine formated Telefonnummer Institution, Vendorusw. zu.

EDIT: IMO wäre es besser, eine PhoneNumberKlasse mit einem FormattedGetter zu haben.

Die Dinge, die Sie beim Schreiben von Code beachten müssen: Ist es richtig? Ist es lesbar Ist es effizient? Ist es wartbar? Ich würde, wie @btilly sagte, argumentieren, dass es aufgrund der kulturspezifischen Formatierung nicht wartbar ist, aber die Frage scheint allgemeiner zu sein.

Durch die Verwendung solcher Zugriffsmethoden wird der Code besser lesbar. Je nachdem, wie Sie ihn verwenden, werden andere Teile des Codes möglicherweise wesentlich sauberer. Meiner Meinung nach riecht das überhaupt nicht. Es würde anfangen zu riechen, wenn Sie Formatierungs-Accessoren für irgendeine Art von Zeichenkette hätten, die Sie vielleicht drucken möchten ( public string FirstLastName; public string FullName; public string FullNameWithMiddleInitial; public string PhoneNumberWithAreaCode; public string PhoneNumberWithoutAreaCode; public string PhoneNumberWithCountryCode;usw.)

Oder anders ausgedrückt: Wenn Sie ein Muster verwenden, riecht Ihr Code nicht automatisch nach einem Muster. Sie müssen es missbrauchen, wenn Sie dieses Attribut verdienen möchten.

Verstößt gegen das Prinzip der einheitlichen Verantwortung. Warum nicht eine Rufnummernklasse machen, etc ...?

Für Ihr Beispiel halte ich es nicht für zu umfangreich, bestimmte Formate zu verwenden. Es ist ein oder zwei und alle Teile der Anwendung verwenden das gleiche Format.

Wenn Sie die gleichen Daten an verschiedenen Orten haben, an denen unterschiedliche Formate erforderlich sind , bricht diese Entscheidung zusammen .

In diesem Fall wäre ich versucht, die MemberKlasse auf Folgendes zurückzubringen:

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

Und dann machen Sie verschiedene Adapter für jedes Ziel. Angenommen, die Informationen wurden im CSV-Format benötigt:

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

Immer vorausgesetzt, Sie haben die Daten bereinigt, sodass keine Kommas usw. in den Zeichenfolgen enthalten sind.

Der Adapter muss keine Erweiterungsmethode sein, aber für diesen Fall scheint er zu passen.