C# Properties - Đặt câu hỏi

Question

Tôi đang làm việc trong C# và tôi bắt đầu chơi với các thuộc tính. Một điều tôi không biết là cách tốt nhất/nơi để đặt logic cho các bộ truy cập thiết lập các thuộc tính lớp và cách xử lý các lỗi.

Ví dụ, nói rằng tôi có điều này (cơ bản) lớp:

class Person 
{ 
    private int _Age = 18; 

    public Person() 
    { 

    } 

    public int Age 
    { 
     get 
     { 
      return _Age; 
     } 
     set 
     { 
      _Age = value; 
     } 
    } 
} 

Bây giờ nói rằng tôi có một yêu cầu về quyền sở hữu Age, 0 < Tuổi < 100. Tôi nên đặt logic cho việc này?

Tôi có nên đặt nó trong tài sản không?

public int Age 
    { 
     get 
     { 
      return _Age; 
     } 
     set 
     { 
      if (value < 0 || value > 99) 
       // handle error 
      else 
       _Age = Convert.ToInt32(value); 
     } 
    } 

hoặc thông qua lớp học đang tạo đối tượng Person?

static void Main(string[] args) 
{ 
    Person him = new Person(); 
    int NewAge = -10; 

    if (NewAge < 0 || NewAge > 100) 
     // handle error 
    else 
     him.Age = NewAge; 
} 

Bây giờ nếu có vấn đề gì với NewAge (nó không đáp ứng được ràng buộc của tôi)? Tôi có nên tạo một ngoại lệ tùy chỉnh và ném nó không? Tôi có nên in một thông báo cho biết nguồn cung cấp có hợp lệ không?

Tôi đã thực hiện một số Googling và tôi không thể tìm thấy bất kỳ điều gì có câu trả lời đầy đủ cho câu hỏi của tôi. Tôi cần một cuốn sách: -/

Answer 1

Sử dụng trình thiết lập thuộc tính, có lý do đó (thêm chức năng vào trường).

Nếu vượt quá giá trị phạm vi, bạn có thể ném ArgumentOutOfRangeException hoặc chỉ đặt giá trị tối thiểu (hoặc tối đa), nhưng điều này tùy thuộc vào yêu cầu quy trình của bạn.

Answer 2

Đặt vào tài sản. Đó là một trong những mục đích chính của tài sản!

Hãy xem xét rằng bạn sẽ đặt trách nhiệm ở mức thấp nhất có thể. Ví dụ bạn đưa ra không yêu cầu bất kỳ thông tin nào khác ngoài thông số giá trị để đưa ra quyết định. Nó thậm chí không phụ thuộc vào các thành viên khác của cùng một lớp. Không có lý do gì để phần còn lại của lớp biết được hiệu lực hoạt động của thuộc tính Age như thế nào, và chắc chắn không có lý do gì cho bất kỳ đoạn mã nào khác biết nó.

Answer 3

Bạn sẽ muốn đặt logic trong setter và ném một ngoại lệ nếu nó không đáp ứng yêu cầu. Tuy nhiên, bạn cũng sẽ muốn tạo một phương thức tĩnh IsValidAge hoặc một cái gì đó để các lớp tạo ra một Person có thể kiểm tra giá trị độ tuổi thay vì chỉ nhìn thấy nếu nó ném một ngoại lệ. Ngoài ra, bạn có thể có các thuộc tính MinAge và MaxAge để mã gọi có thể kiểm tra xem độ tuổi mà chúng sắp đặt có nằm giữa nó hay không.

Không tạo loại ngoại lệ tùy chỉnh, sử dụng ArgumentOutOfRangeException hoặc một cái gì đó tương tự.

Answer 4

Bạn có thể muốn xem giao diện IDataErrorInfo.

Bằng cách triển khai giao diện này trong lớp học của mình, bạn sẽ mở lớp này sang các cơ chế khác có thể hưởng lợi từ thông tin lỗi bổ sung.

Answer 5

Sử dụng trình thiết lập và sử dụng ArgumentOutOfRangeException, như những người khác đã nói.Đôi khi bạn sẽ gọi một phương thức khác, thường là một cái gì đó như OnAgeChanged (int age), nơi bạn có thể thực hiện xác thực (có thể là một cuộc gọi đến phương thức khác để bạn có thể sử dụng nó ở bất kỳ đâu) và gọi trình xử lý sự kiện nếu kết nối, có thể có logic khác để áp dụng tùy thuộc vào cách bạn đang sử dụng đối tượng của bạn. Điều đó có thể không cần thiết trong kịch bản của bạn, nhưng nó khá phổ biến để làm điều đó cho các thuộc tính. Đặc biệt nếu bạn định cập nhật biểu mẫu - biểu mẫu sẽ móc nối vào sự kiện AgeChanged để tự cập nhật.

Answer 6

Tôi sẽ chơi chủ trương của quỷ và đưa ra câu trả lời tại sao bạn KHÔNG muốn đặt nó trong setter. Có rất nhiều trường hợp khi bạn muốn có thể đặt giá trị của thuộc tính Độ tuổi của bạn và sau đó hỏi xem đối tượng đó có hợp lệ theo nghĩa toàn diện hay không.

Ví dụ, giữ tài sản của bạn đơn giản:

public int Age { get; set; } 

Sau đó, khi một giá trị không hợp lệ được thông qua tại bạn có thể có một số chức năng IsValid mà chỉ nếu đối tượng trong câu hỏi là ok. Điều này có thể cực kỳ hữu ích vì bạn có thể thực hiện xác nhận phức tạp hơn ngoài hạn chế Độ tuổi đơn giản.

bool IsValid() 
{ 
    if (Age < 0 || Age > 99) 
    return false; 
} 

Đối với một cái gì đó đơn giản như thế này không có rất nhiều lợi ích, nhưng xem xét cũng là bạn có thể sử dụng trong lớp kiên trì của bạn, do đó bạn có thể đảm bảo rằng bất kỳ đối tượng đó là KHÔNG hợp lệ sẽ không bao giờ được tiếp tục tồn. Trong những trường hợp đó bạn không nhất thiết muốn ném một ngoại lệ.

Cũng xem xét việc này:

DateTime StartDate { get; set; } 
DateTime EndDate { get; set;} 

bool IsValid() 
{ 
    return StartDate > EndDate  
} 

Đây chỉ là giả, nhưng bạn sẽ có được quan điểm của tôi. Đây là điều mà bạn không thể làm bên trong một setter, hoặc ít nhất, không theo cách có thể duy trì được.

Answer 7

tôi muốn thực hiện nó như thế này:

public int Age 
{ 
    get 
    { 
     return _Age; 
    } 
    set 
    { 
     if (IsValidAge(value)) 
      _Age = value; 
     else 
      throw new ArgumentOutOfRangeException("value", string.Format("value should be between {0} and {1} inclusive.", MinAge, MaxAge)); 
    } 
} 

private bool IsValidAge(int age) 
{ 
    return (age >= MinAge && age <= MaxAge); 
} 

Một số điều cần lưu ý:

• Không thay đổi giá trị của họ thay vì ném một ngoại lệ, đây là hành vi bất ngờ.
• Khuôn khổ .NET sẽ ném đối số * ngoại lệ trong các trình cài đặt, vì vậy, tôi khuyên bạn nên thực hiện theo cách này. Trong trường hợp này ArgumentOutOfRangeException là hoàn hảo, IMO.
• Khi đề cập đến đối số trong thông báo ngoại lệ và tài liệu xml, tiêu chuẩn là gọi đối số "giá trị", không phải là tên của thuộc tính của bạn.
• Tôi muốn giới thiệu MinAge và MaxAge là các khối riêng tư trong lớp học của bạn, không rơi vào bẫy thông báo lỗi mã hóa có ranh giới phạm vi trong chúng, không có gì tệ hơn là được nói "5 không hợp lệ, vui lòng nhập một số 1 và 10 "khi ai đó thay đổi thông số kỹ thuật sau này nhưng quên cập nhật chuỗi.

Answer 8

Nếu bạn đang thực hiện thiết kế thừa (nghĩa là trạng thái của đối tượng là hợp lệ tại tất cả các điểm trong suốt thời gian của đối tượng), thì bạn có thể đặt và kiểm tra điều này trong hàm tạo. Trong thiết kế thành phần, bạn muốn kiểm tra điều này trong phương thức sử dụng thuộc tính, ví dụ:

class Person 
{ 
    public int calculateTimeToExpiration() 
    { 
    if (Age < 0 || Age > 100) 
    //throw 
    } 
} 

Chính khoảng trống là mã máy khách và không phải là nơi tốt cho logic nghiệp vụ.

Answer 9

Cảm ơn câu trả lời! Trong trường hợp bạn tò mò ở đây là những gì tôi thay đổi ví dụ nhỏ của tôi (mặc dù nó là chi tiết hơn bây giờ) để (dựa trên các ý kiến).

class Person 
{ 
    private string _FirstName = "Joe"; 
    private string _LastName = "Smith"; 
    private int _Age = 18; 

    private const int MinAge = 1; 
    private const int MaxAge = 99; 

    public Person() 
    { 

    } 

    public Person(string FirstName, string LastName, int Age) 
    { 
     this.FirstName = FirstName; 
     this.LastName = LastName; 
     this.Age = Age; 
    } 

    public string FirstName 
    { 
     get 
     { 
      return _FirstName; 
     } 
     set 
     { 
      _FirstName = value; 
     } 
    } 

    public string LastName 
    { 
     get 
     { 
      return _LastName; 
     } 
     set 
     { 
      _LastName = value; 
     } 
    } 

    public int Age 
    { 
     get 
     { 
      return _Age; 
     } 
     set 
     { 
      if (IsValidAge(value)) 
       throw new ArgumentOutOfRangeException("value","Please enter a positive age less than 100."); 
      else 
       _Age = value; 
     } 
    } 

    private bool IsValidAge(int age) 
    { 
     return (age < MinAge || age > MaxAge); 
    } 

    public override string ToString() 
    { 
     if (Age == 1) 
      return "My name is " + FirstName + " " + LastName + " and I am " + Age + " year old."; 
     else 
      return "My name is " + FirstName + " " + LastName + " and I am " + Age + " years old."; 
    } 
} 

static void Main(string[] args) 
    { 
     Person him, her; 

     try 
     { 
      him = new Person("Joe Bob", "McGruff", 1); 
      Console.WriteLine(him); 
     } 
     catch (ArgumentOutOfRangeException range) 
     { 
      Console.WriteLine(range.Message); 
     } 

     try 
     { 
      her = new Person(); 
      her.Age = -5; 
      Console.WriteLine(her); 
     } 
     catch (ArgumentOutOfRangeException range) 
     { 
      Console.Write(range.Message); 
     } 

     Console.ReadKey(); 
    } 

Answer 10

câu trả lời của bạn cập nhật là tốt nhưng bạn nên dọn dẹp một phần nhỏ trong setter của bạn/xác nhận

public int Age 
    { 
     get 
     { 
      return _Age; 
     } 
     set 
     { 
      if (!IsValidAge(value)) 
       throw new ArgumentOutOfRangeException("Age","Please enter a positive age less than 100."); 

      _Age = value; 
     } 
    } 

    private bool IsValidAge(int age) 
    { 
     return (age > MinAge && age < MaxAge); 
    }