6

我将 NHibernate 用于我的 C# 项目，因此我有几个模型类。

让我们假设以下示例：

using System;

namespace TestProject.Model
{
    public class Room
    {
        public virtual int Id { get; set; }
        public virtual string UniqueID { get; set; }
        public virtual int RoomID { get; set; }
        public virtual float Area { get; set; }

    }
}

到目前为止，使用 NHibernate 映射这些对象工作正常。现在我想生成一个新的 Room 对象并将其存储在数据库中。为了避免单独设置每个成员，我向模型类添加了一个新的构造函数。在我写的虚拟成员下面：

public RoomProperty()
{

}


public RoomProperty(int pRoomId, int pArea)
{
        UniqueID = Guid.NewGuid().ToString();
        RoomID = pRoomId;
        Area = pArea;
}

用 FxCop 分析我的代码告诉我以下信息：

"ConstructorShouldNotCallVirtualMethodsRule"
This rule warns the developer if any virtual methods are called in the constructor of a non-sealed type. The problem is that if a derived class overrides the method then that method will be called before the derived constructor has had a chance to run. This makes the code quite fragile.

该页面还描述了为什么这是错误的，我也理解。但我不知道如何解决这个问题。

当我删除所有构造函数并添加以下方法时......

public void SetRoomPropertyData(int pRoomId, int pArea)
        {
            UniqueID = Guid.NewGuid().ToString();
            RoomID = pRoomId;
            Area = pArea;

        }

....在调用标准构造函数后设置数据我无法启动我的应用程序，因为 NHibernate 初始化失败。它说：

NHibernate.InvalidProxyTypeException: The following types may not be used as proxies:
VITRIcadHelper.Model.RoomProperty: method SetRoomPropertyData should be 'public/protected virtual' or 'protected internal virtual'

但是将此方法设置为 virtual 与我在构造函数中设置虚拟成员时的错误相同。我怎样才能避免这些错误（违规）？

4

3 回答 3

5

问题在于虚拟集。将值传递给基类构造函数中的虚拟属性将使用覆盖集而不是基集。如果重写集依赖于派生类中的数据，那么你就有麻烦了，因为派生类的构造函数还没有完成。

如果您绝对确定，任何子类都不会在覆盖集中使用其状态的任何数据，那么您可以在基类构造函数中初始化虚拟属性。考虑在文档中添加适当的警告。

如果可能，尝试为每个属性创建支持字段并在基类构造函数中使用它们。

您还可以将属性初始化推迟到派生类。为此，请在派生类的构造函数中调用的基类中创建一个初始化方法。

于 2013-02-20T14:45:09.490 回答
1

我希望以下其中一项工作：

  1. 使属性成为非虚拟的（只要 NHibernate 支持它就首选）。
  2. 从自动实现的属性更改为具有显式支持字段的属性，并在构造函数中设置字段而不是设置属性。
  3. 创建一个静态Create方法，首先构造对象，然后在返回构造的对象之前将值设置为属性。

编辑：从评论中我看到选项#3 不清楚。

public class Room
{
    public virtual int Id { get; set; }
    public virtual string UniqueID { get; set; }
    public virtual int RoomID { get; set; }
    public virtual float Area { get; set; }

    public static Room Create(int roomId, int area)
    {
        Room room = new Room();
        room.UniqueID = Guid.NewGuid().ToString();
        room.RoomID = roomId;
        room.Area = area;
        return room;
    }
}
于 2013-02-20T14:45:33.050 回答
0

恕我直言，好主意是使基类抽象及其构造函数受保护。接下来，继承的类有它们的构造函数私有和 - 对于外部世界 - 统一的静态方法，如“实例”，它首先初始化构造函数，然后 - 以正确的顺序调用整个类方法集，最后 - 返回的实例班级。

于 2018-03-24T11:45:34.877 回答