Đây có phải là một mô hình mùi của người khác để đặt các getters như là Full Full tên hay hay FormattedPhoneNumber INTERN trong mô hình của bạn không?

Tôi đang làm việc trên một ứng dụng ASP.NET MVC và tôi đã có thói quen đưa những thứ có vẻ như hữu ích và tiện lợi vào các lớp mô hình / thực thể của tôi.

Ví dụ:

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

Tôi đang tự hỏi mọi người nghĩ về FullNamevà FormattedPhoneNumbergetters.

Chúng giúp dễ dàng tạo các định dạng dữ liệu được tiêu chuẩn hóa trong ứng dụng và dường như chúng lưu rất nhiều mã lặp lại, nhưng chắc chắn có thể lập luận rằng định dạng dữ liệu là thứ cần được xử lý trong ánh xạ từ mô hình sang mô hình xem.

Trên thực tế, ban đầu tôi đã áp dụng các định dạng dữ liệu này trong lớp dịch vụ của mình, nơi tôi thực hiện ánh xạ của mình, nhưng nó trở thành gánh nặng khi phải liên tục phải viết các trình định dạng sau đó áp dụng chúng ở nhiều nơi khác nhau. Ví dụ: tôi sử dụng "Tên đầy đủ" trong hầu hết các chế độ xem và phải nhập một cái gì đó giống như model.FullName = MappingUtilities.GetFullName(entity.FirstName, entity.LastName);mọi nơi có vẻ thanh lịch hơn nhiều so với chỉ gõ model.FullName = entity.FullName(hoặc, nếu bạn sử dụng một cái gì đó như AutoMapper, có khả năng không gõ gì cả).

Vì vậy, nơi bạn vẽ đường khi định dạng dữ liệu. Là "ổn" để thực hiện định dạng dữ liệu trong mô hình của bạn hay đó là "mùi mẫu"?

Lưu ý: Tôi chắc chắn không có bất kỳ html nào trong mô hình của mình. Tôi sử dụng trợ giúp html cho việc đó. Tôi nghiêm túc nói về định dạng hoặc kết hợp dữ liệu (và đặc biệt là dữ liệu thường được sử dụng).

Trong ví dụ của bạn, tôi thích FullNamegetter (vì tất cả các lý do bạn đã đưa ra) nhưng tôi không thích getter FormattedPhoneNumber. Lý do là: có thể không dễ dàng như vậy (một khi bạn có số điện thoại quốc tế, v.v.) và nếu bạn đặt logic để tạo số điện thoại theo phương pháp Member, rất có thể bạn sẽ cần phải cấu trúc lại (hoặc sao chép-dán caugh ) một khi bạn cần một số điện thoại hình thành cho Institution, Vendorvv quá.

EDIT: IMO sẽ tốt hơn nếu có một PhoneNumberlớp học với một Formattedgetter.

Những điều bạn cần xem xét khi viết mã: Có đúng không? Có đọc được không? Có hiệu quả không? Có thể duy trì? Tôi sẽ lập luận, như @btilly đã đề cập, rằng nó không thể duy trì được do định dạng văn hóa cụ thể, nhưng câu hỏi dường như khái quát hơn rằng.

Sử dụng các bộ truy cập như thế này làm cho mã của bạn dễ đọc hơn và, tùy thuộc vào cách bạn sử dụng nó, có thể làm cho các phần khác trong mã của bạn sạch hơn nhiều. Theo tôi, điều đó không có mùi gì cả. Nó sẽ bắt đầu có mùi nếu bạn có các bộ truy cập Định dạng cho bất kỳ loại chuỗi nào bạn có thể muốn in ( public string FirstLastName; public string FullName; public string FullNameWithMiddleInitial; public string PhoneNumberWithAreaCode; public string PhoneNumberWithoutAreaCode; public string PhoneNumberWithCountryCode;, v.v.)

Hoặc, nói cách khác, sử dụng một mẫu không tự động làm cho mã của bạn có "mùi mẫu". Bạn cần phải lạm dụng nó nếu bạn muốn kiếm được thuộc tính đó.

Phá vỡ nguyên tắc trách nhiệm duy nhất. Tại sao không tạo một lớp số điện thoại, vv ...?

Ví dụ của bạn, tôi không thấy việc sử dụng các định dạng cụ thể là một vấn đề quá lớn. Đó là một hoặc hai và tất cả các phần của ứng dụng sử dụng cùng một định dạng.

Trường hợp quyết định đó sẽ bắt đầu bị phá vỡ là khi bạn có cùng một dữ liệu đi đến một số nơi khác nhau yêu cầu các định dạng khác nhau .

Nếu điều đó xảy ra, tôi muốn lôi kéo Memberlớp học trở lại:

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

Và sau đó làm các bộ điều hợp khác nhau cho từng mục tiêu. Ví dụ: giả sử thông tin được yêu cầu ở định dạng CSV:

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

Luôn luôn giả định rằng bạn đã vệ sinh dữ liệu để không có dấu phẩy, v.v.

Bộ điều hợp không phải là một phương pháp mở rộng, nhưng đối với trường hợp giả tạo này, nó có vẻ phù hợp.