“FullName” veya “FormattedPhoneNumber” gibi alıcıları modelinize koymak “kalıp kokusu” mu?

Bir ASP.NET MVC uygulaması üzerinde çalışıyorum ve model / varlık sınıflarıma yararlı ve kullanışlı alıcılar gibi görünen şeyleri koyma alışkanlığına giriyorum.

Örneğin:

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

İnsanların meraklıları FullNameve FormattedPhoneNumbermeraklılarını düşündüklerini merak ediyorum .

Uygulama boyunca standartlaştırılmış veri formatları oluşturmayı çok kolaylaştırıyorlar ve çok sayıda tekrarlanan kod kaydetmiş gibi görünüyorlar, ancak veri formatının modelden görünüm modeline eşlemede ele alınması gereken bir şey olduğu kesinlikle iddia edilebilir.

Aslında, bu veri formatlarını başlangıçta eşlememi yaptığım hizmet katmanımda uyguluyordum, ancak sürekli olarak formatlayıcılar yazmak ve bunları birçok farklı yere uygulamak zorunda kalıyorum. Örneğin, çoğu görünümde "Tam Ad" kullanıyorum ve model.FullName = MappingUtilities.GetFullName(entity.FirstName, entity.LastName);her yere benzer bir şey yazmak zorunda kalmadan sadece yazmaktan çok daha az zarif görünüyordu model.FullName = entity.FullName(ya da AutoMapper gibi bir şey kullanırsanız, hiçbir şey yazmıyorsanız).

Peki, veri biçimlendirmesi söz konusu olduğunda çizgiyi nerede çizersiniz. Modelinizde veri biçimlendirmesi yapmak "iyi" mi yoksa bu bir "desen kokusu" mu?

Not: Modelimde kesinlikle html yok. Bunun için html yardımcıları kullanıyorum. Kesinlikle veri biçimlendirme veya birleştirme (ve özellikle sık kullanılan veriler) hakkında konuşuyorum.

FullNameÖrneğinizde , alıcıyı (verdiğiniz tüm nedenlerden dolayı) seviyorum, ancak FormattedPhoneNumber alıcısını sevmiyorum. Nedeni: muhtemelen olmadığını (eğer uluslararası telefon numaralarını vb bir kez) ile bir yöntemde telefon numaralarını biçimlendirme için mantık yerleştirirseniz kolay Member, büyük ihtimalle gözden geçirmeniz gerekecektir (veya kopyalayıp yapıştırın caugh ) size bir kez için formatlanmış telefon numarası lazım Institution, Vendorvs de.

EDIT: IMO PhoneNumberbir Formattedalıcı ile bir sınıf olması daha iyi olurdu .

Kod yazarken dikkat etmeniz gerekenler: Doğru mu? Okunabilir mi? Verimli mi? Bakımı yapılabilir mi? @Btilly'nin de belirttiği gibi, kültüre özgü biçimlendirme nedeniyle sürdürülemez olduğunu iddia ediyorum, ancak soru bunun daha genel gibi görünüyor.

Bu gibi erişimcilerin kullanılması kodunuzu daha okunaklı hale getirir ve nasıl kullandığınıza bağlı olarak kodunuzun diğer bölümlerini daha temiz hale getirebilir. Bence bu hiç kokmuyor. Yazdırmak isteyebileceğiniz her türlü dize için Biçimlendirme erişiminiz olsaydı kokmaya başlayacaktır ( public string FirstLastName; public string FullName; public string FullNameWithMiddleInitial; public string PhoneNumberWithAreaCode; public string PhoneNumberWithoutAreaCode; public string PhoneNumberWithCountryCode;vb.)

Ya da başka bir deyişle, bir kalıp kullanmak otomatik olarak kodunuzun "kalıp kokusu" olmasını sağlamaz. Bu özelliği kazanmak istiyorsanız kötüye kullanmanız gerekir.

Tek sorumluluk ilkesini ihlal eder. Neden bir telefon numarası sınıfı yapmıyorsunuz?

Örneğin, belirli biçimleri kullanmak için çok büyük bir anlaşma olarak görmüyorum. Bir veya iki ve uygulamanın tüm bölümleri aynı biçimi kullanıyor.

Bu kararın dağılmaya başlayacağı yer, aynı verilerin farklı biçimler gerektiren birkaç farklı yere gittiği zamandır .

Bu olsaydı, Membersınıfı geri çekmeye cazip olurdum :

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

Ve sonra her hedef için farklı adaptörler yapın. Örneğin, bilgilerin CSV biçiminde gerekli olduğunu varsayalım:

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

Her zaman veriyi temizlediğinizi varsayarak dizelerde virgül vb. Olmaz.

Bağdaştırıcının bir uzatma yöntemi olması gerekmez, ancak bu inandırıcı durum için uygun gibi görünüyor.