主页 / 问题 / 625626
Accepted
Eladei
Asked:2020-02-09 19:51:57 +0000 UTC2020-02-09 19:51:57 +0000 UTC

如何保护在 C# 中引发事件的对象的属性？

  • 772

今天是个好日子。有一个关于 C# 事件的问题。

我们有一个引擎。

引擎有模型参数ModelParams。例如，参数包括阀门数量 countOfClapans。ModelParams 有一个 paramsChanged 事件，表示模型参数发生变化。

在设计过程中，我有一个问题：如何保护模型参数不被 paramsChanged 事件处理程序通过 sender 更改？话虽这么说，我不想让模型参数属性像 {get; private set{}} 禁用在类外编辑属性的能力。

据我了解，您可以制作单独的事件来更改模型的每个参数。但是如果给定的参数很多呢？不要做相同数量的事件处理程序。

总之，我尝试将 ModelParams 类的对象与将订阅模型更改事件的其他对象（在本例中为 Cylinder 类的对象）隔离开来。

下面是一个非常简化的代码：

public class Engine {
    public ModelParams modelParams;
    public List<Cylinder> cylinders = new List<Cylinder>();


    public Engine( ModelParams modelParams ) {
        this.modelParams = modelParams;
    }
}

public class ModelParams {
    private int countOfClapans;
    private string engineType;

    public int ClapansCount {
        get { return countOfClapans; }
        set { countOfClapans = value; }
    }; 

    public int TypeOfEngine {
        get { return engineType; }
        set { engineType= value; }
    }; 


    public event EventArgs<ModelArgs> ModelChanged;

    public void OnModelChange() {
        if(modelChanged != null) {
            ModelArgs args = new ModelArgs();
            args.ClapansCount = this.ClapansCount;
            ModelChanged(this, args);
        }
    }
}

public class ModelArgs{
    private int countOfClapans;
    private string engineType;

    public int ClapansCount {
        get { return countOfClapans; }
        set { countOfClapans= value; }
    };

    public int TypeOfEngine {
        get { return engineType; }
        set { engineType= value; }
    }; 
}
c#

2 个回答

  1. 在 C# 中，具有任何签名的委托都可以充当事件。带有两个参数的 EventHandler 是经典的，到处都在使用。但如果有必要，您可以像这样声明一个事件：

    public delegate void ModelChangeEventHandler(ModelArgs e);
public event ModelChangeEventHandler ModelChanged;
// или так
public event Action<ModelArgs>;
    • 2
  2. Best Answer

    在我看来，正确的做法是对外暴露只读接口。声明一个带有 getters 而没有 setters 的接口IModelParams，并在你event的 'a 定义中使用它。

    如果您还想保护自己免受那些可以恶意回溯到 的人的侵害ModelParams，有几种方法。

    您可以将ModelParams它移动到一个单独的程序集中，将 setter 标记为internal，并将需要访问 setter 的每个人放在同一个程序集中。

    或者您可以编写一个只读包装器，将对象作为私有字段包含，并且只发出 getter（并且在 getter 中从真实对象中获取值）。

    反过来，这些解决方案也不会对反思产生影响。如果有人为了获取私有字段而恶意使用反射，需要先通过架构师来污蔑脑筋。

    而且，如果您真的想保护自己免受此类技术而非管理手段的影响，请将他们的代码上传到一个AppDomain权限较低的单独代码，这样您就可以防止他们使用反射。但我仍然会推荐架构师的客气话、文档和建议的路径：使用您不信任的代码是一项痛苦且吃力不讨好的任务。

    更新：我重新阅读了你的代码，我完全理解我的误解：

    你event的似乎被定义为

    public event EventHandler<ModelArgs> ModelChanged; // а не EventArgs<ModelArgs>

    你实际上传入的是什么event ModelParams，sender它的类型是object什么？如果是这样，则意味着您的客户确实向ModelParams. 这意味着简单地创建一个接口将无济于事，您需要使用第二个选项：创建一个包装类或将类型ModelParams放在一个单独的程序集中。

    • 1

相关问题