我的问题是基于 this article .

基本上一个类可以实现一个 Freezable 方法来确保一旦对象进入 Frozen 状态就不能更改任何属性。

我有一个遵循这种设计的界面

public interface IFreezableModel
{
    void Freeze();
    bool IsFrozen{get;}
}

目标是确保在调用 Freeze 方法后，IsFrozen 属性设置为 True，并且对象的属性不能再更改。

为了简化，我将使用一个抽象基类:

public abstract class BaseFreezableModel : IFreezableModel
{
    public void Freeze()
    {
        _isFrozen = true;
    }
    public bool IsFrozen
    { 
       get {return _isFrozen;}            
    }
    protected ThrowIfFrozen()
    {
       if (IsFrozen)
           throw new Exception("Attempted to change a property of a frozen model");
    }
}

这样我就可以上课了

public class MyModel : BaseFreezableModel
{
     private string _myProperty;
     public string MyProperty
     {
        get{return _myProperty;}
        set 
        {
           ThrowIfFrozen();
           _myProperty = value;
        }
     }
 }

这一切都很好也很简单，但是我可以采用哪种策略来确保所有属性都遵循上述模式？ (除了编写 setter 和 getter)

这些是我想出的替代方案:

• 找到一种机制，或许可以使用 emit 将方法注入(inject)到每个属性 setter 中。但我不知道该怎么做，我可能会遇到什么潜在问题(因此需要多长时间)。如果有人知道这一点并能指出我的方向，那就太好了。

• 在构建过程中使用一些模板，以便在编译时间之前插入对 OnCheckFrozen 的调用。这样做的好处是非常容易理解并且可以用于类似的场景。

• 找到一个可以为我做所有这些的框架，但这只是一个极端的情况，因为我不允许在这个项目上使用外部框架。

您会使用什么解决方案来实现这一目标？

最佳答案

您正在进入 Aspect Oriented Programming 的世界这里。您可以使用 PostSharp 在 5 分钟内完成此类功能。 - 但似乎不允许您使用外部框架。因此，您的选择归结为实现您自己的非常简单的 AOP 框架，或者只是硬着头皮向每个属性 setter 添加检查。

就我个人而言，我只会在属性 setter 中编写检查。这可能不像您预期​​的那样痛苦。您可以编写一个 visual studio 代码片段来加快该过程。您还可以编写一个智能单元测试类，该类将使用反射扫描卡住对象的所有属性并尝试设置一个值 - 测试失败如果没有抛出异常..

编辑 响应 VoodooChilds 请求.. 这是一个使用 NUnit 和优秀的 FluentAssertions 库的单元测试类的简单示例。

[TestFixture]
public class PropertiesThrowWhenFrozenTest
{
    [TestCase(typeof(Foo))]
    [TestCase(typeof(Bar))]
    [TestCase(typeof(Baz))]
    public void AllPropertiesThrowWhenFrozen(Type type)
    {
        var target = Activator.CreateInstance(type) as IFreezable;

        target.Freeze();

        foreach(var property in type.GetProperties())
        {
            this.AssertPropertyThrowsWhenChanged(target, property);
        }
    }

    private void AssertPropertyThrowsWhenChanged(object target, PropertyInfo property)
    {
        // In the case of reference types, setting the property to null should be sufficient
        // to test the behaviour...
        object value = null;

        // In the case of value types, just create a default instance...
        if (property.PropertyType.IsValueType)
            value = Activator.CreateInstance(property.PropertyType);

        Action setter = () => property.GetSetMethod().Invoke(target, new object[] { value });

        // ShouldThrow is a handy extension method of the FluentAssetions library...
        setter.ShouldThrow<InvalidOperationException>();
    }
}

此方法使用参数化单元测试来传递被测试的类型，但您同样可以将所有这些代码封装到一个通用基类中(其中 T : IFreezable)并为每个被测试的类型创建扩展类，但是一些测试运行者不喜欢在基类中进行测试。*ahem*Resharper!ahem

EDIT 2 而且，为了好玩，这里有一个 Gherkin 脚本示例，可用于为此类事情创建更灵活的测试 :)

Feature: AllPropertiesThrowWhenFrozen
    In order to make sure I haven't made any oversights in my code
    As a software developer
    I want to be able to assert that all properties of a class throw an exception when the object is frozen

Scenario: Setting the Bar property on the Foo type
  Given I have an instance of the class MyNamespace.MyProject.Foo
    And it is frozen
  When I set the property Bar with a value of 10
  Then a System.InvalidOperationException should be thrown

关于c# - 以编程方式在类的每个属性上插入方法调用，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/7569223/