Question

我正在使用NHibernate作为我的C＃pojects，因此我有几个模型类。

让我们假设以下示例：

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.

This page也描述了为什么这是错误的，我也理解它。但我不知道如何解决这个问题。

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

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将与我在构造函数中设置虚拟成员时的错误相同。我怎样才能避免这些错误（违规）？

Answer 1

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

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

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

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

Answer 2

我希望以下其中一项能够奏效：

使属性非虚拟（只要NHibernate支持它就是首选）。
从自动实现的属性更改为具有显式支持字段的属性，并在构造函数中设置字段，而不是设置属性。
创建一个静态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;
    }
}

Answer 3

恕我直言，好主意是制作基类 - 抽象及其构造函数 - 受保护。接下来，继承的类的构造函数 private 和 - 对于外部世界 - 统一静态方法，如＆＃34; Instance ＆＃34;，首先，初始化构造函数，然后 - 以正确的顺序调用整个类方法，最后 - 返回类的实例。

替代在C＃中调用虚方法

3 个答案: