如何在获取中联合条件语句；在 C# 中设置日期时间的属性？

Question

我在玩课程和约会时间。我想检查一个人的年龄是否超过 16 岁或不使用 get,set 属性。这是课程Test

class Test
    {
        private DateTime _Dob;
        public DateTime Dob
        {
            get { return _Dob; }
            set
            {
                if (DateTime.Now.Year - 16 < _Dob.Year)
                {
                    Console.WriteLine("error");
                }
                else
                {
                _Dob = value; }
            }
        }
        public Test(DateTime dob)
        {
            _Dob = dob;
        }

这里是主要的

Test dob = new Test(new DateTime(2007,12,01));
            dob.Dob();
            Console.ReadLine();

它在 main 中说 dob.Dob(); 不是一种方法。我该如何解决这个问题？

Answer 1

这里发生了几件事和许多思想流派。

1. 如果您需要进行验证，最好不要在属性设置器中执行此操作。有一些很好的理由说明你不想在 setter 中做任何繁重的工作，如下所示
   • 因为属性“看起来”是字段，所以并不总是很明显它们可以抛出（设计使然）异常；而对于方法，程序员被训练来预期和调查异常是否是调用该方法的预期结果。
   • 许多 .NET 基础架构都使用 Getter，例如序列化程序和数据绑定（例如在 WinForms 和 WPF 中） - 在此类上下文中处理异常可能会很快成为问题。
   • 当您观察或检查对象时，调试器会自动评估属性获取器。这里的异常可能会令人困惑并减慢您的调试工作。出于同样的原因，在属性中执行其他昂贵的操作（如访问数据库）也是不可取的。
   • 属性通常用于链式约定：obj.PropA.AnotherProp.YetAnother - 使用这种语法，决定在哪里注入异常捕获语句变得有问题。
2. 如果你要验证，可能值得扔ArgumentOutOfRangeException，如果你这样做，那么最好在一个方法中并将你的属性设置为private集，所以肮脏的小手指不要'不跳过验证

所以我的强迫症修改了你的课程Full Demo here

public class Test
{
    // make it a private setter so people don't try to set the property
    // without validation
    public DateTime DateOfBirth
    {
        get;
        private set;
    }

    // validation is probably best done outside of a property
    public void SetDateOfBirth(DateTime dateOfBirth)
    {
        if (dateOfBirth.Year > DateTime.Now.Year - 16)
        {
            throw new ArgumentOutOfRangeException("dateOfBirth", "Date of birth year cannot be after : " + (DateTime.Now.Year - 16));
        }

        DateOfBirth = dateOfBirth;
    }

    public Test()
    {
    }

    // if you really want one stage construction,
    // lets call the common DoB validation
    public Test(DateTime dateOfBirth)
    {
        SetDateOfBirth(dateOfBirth);
    }
}

使用示例

try
{
    Console.WriteLine("Testing invalid constrcutor");
    var test = new Test(new DateTime(2007, 12, 01));
}
catch (ArgumentOutOfRangeException e)
{
    Console.WriteLine(e.Message);
}

---

进一步阅读

Property Design

Answer 2

为了解决您的代码未编译的原因：您正在调用Dob，就好像它是一种方法一样。不是；这是一个财产。你的属性有一个 getter 和一个 setter；它们在这些情况下被执行：

 Test test = new Test(new DateTime(...));
 DateTime fetched = test.Dob;  // Executes the getter
 test.Dob = new DateTime(...); // Executes the setter

这就是您的代码无法编译的方面。

一旦修复（例如，通过使用 DateTime dateOfBirth = test.Dob;，您仍然不会验证任何内容，因为这段代码：

public Test(DateTime dob)
{
    _Dob = dob;
}

这是直接分配给字段；它没有使用 setter - 你绕过了所有的验证。

接下来，您的验证使用_Dob.Year 而不是value，这意味着它正在验证之前设置的 属性值是否正确。

接下来，验证本身是不正确的，因为它不仅仅是检查年份部分而不考虑月份和日期。如果我的出生日期是 2002 年 4 月 22 日（我希望！），那么昨天我不会是 16 岁，但今天我会......你的代码不会发现差异。在下面的代码中，我假设您的“超过 16 岁”约束实际上是“至少 16 岁”。

最后，在控制台中打印错误消息并不是报告错误的惯用方式。使用异常报告错误。 （在某些情况下，您希望对多个项目分别执行验证并收集所有无效方面，但这超出了本问题的范围。）

我个人认为在属性设置器中验证很好，假设您需要它完全可用 - 另一种设计是将Dob 属性设置为只读，并在构造时验证。这将回避关于属性设置器是否应该抛出的整个讨论。但是，假设您确实想要一个属性设置器，下面是完整的代码，可以满足您的预期：

using System;

class Test
{
    private DateTime dateOfBirth;

    public DateTime DateOfBirth
    {
        get => dateOfBirth;
        set
        {
            // Alternative: if (DateTime.Now.AddYears(-16) > value)
            // They behave differently around leap years. Note that
            // currently this is sensitive to the system time zone
            // and the system clock.
            if (value.AddYears(16).Date > DateTime.Today)
            {
                throw new ArgumentOutOfRangeException(
                    nameof(value), "User must be at least 16 years old");
            }
            dateOfBirth = value;
        }
    }

    public Test(DateTime dateOfBirth)
    {
        DateOfBirth = dateOfBirth;
    }
}

class Program
{
    static void Main(string[] args)
    {
        // Note: executed on April 22nd 2018
        Test test = new Test(new DateTime(1970, 1, 1));
        Console.WriteLine(test.DateOfBirth);
        test.DateOfBirth = new DateTime(2002, 4, 22); // Fine: 16th birthday
        Console.WriteLine(test.DateOfBirth);
        test.DateOfBirth = new DateTime(2002, 4, 23);
    }
}

最后一点：直接使用系统时钟（和系统时区）会使您的代码难以测试。我喜欢将时钟视为依赖项，具有一个带有系统时钟实现和一个假时钟实现的接口（例如IClock）。然后，您需要确定您感兴趣的时区，因为这会影响用户的年龄。 （例如，我可能 16 岁在纽约，但不在洛杉矶。）不过，这些是相当不同的问题。

Answer 3

当你使用 get 时；放;您不想将 Dob 作为函数调用，但就像访问类的普通属性一样。

Test dob = new Test(new DateTime(2007,12,01));
var myDob = dob.Dob;
// myDob is what is return from the get in your get function

Answer 4

你没有使用你的 setter。

像这样在你的构造函数中使用它：

public Test(DateTime dob)
{
    this.Dob = dob;
}

或者从外部调用setter：

dob.Dob = new DateTime(2007,12,01);

Answer 5

Dob 是类Test 的属性，而不是方法。

您可以访问Dob 喜欢：

var test = new Test(new DateTime(2010, 01, 01));
var dob = test.Dob;

但是，您已经注意到，当您调用 Test 构造函数时，您的 set 逻辑并未启动。我相信这是因为您的年龄检查不正确。当您调用构造函数时，set 逻辑未正确执行检查的原因是您正在比较 _Dob.Year 在它有值之前。您想使用 value 关键字来获取尝试设置的值。

参见下面的示例（使用固定的年龄检查）：

public DateTime Dob
{
    get
    {
        return _Dob;
    }
    set
    {
        if (GetAge(value) < 16)
        {
            Console.WriteLine("You ain't 16!");
        }
        else
        {
            _Dob = value;
        }
    }
}

private int GetAge(DateTime dob)
{
    DateTime today = DateTime.Today;
    int age = today.Year - dob.Year;
    if (dob > today.AddYears(-age))
        age--;

   return age;
}