如何在类A上实现一个属性，该属性引用类A的子对象的属性

我们有这段代码，经过简化，看起来像这样：

public class Room
{
    public Client Client { get; set; }

    public long ClientId
    {
        get
        {
            return Client == null ? 0 : Client.Id;
        }
    }
}

public class Client 
{
    public long Id { get; set; }
}

现在我们有三个观点。

1）这是很好的代码，因为Client应该始终设置该属性（即不为null），因此该值Client == null将永远不会发生，并且Id值始终0表示一个错误的Id（这是代码编写者的意见；-)）

2）您不能依赖调用方来知道这0是一个错误值，Id并且 Client应始终设置该属性exception，而get当该Client属性恰巧为null 时，则应抛出in

3）当Client应始终设置该属性时，您只需返回Client.Id并NullRef在该Client属性碰巧为null 时让代码引发异常。

以下哪项最正确？还是有第四种可能性？

闻起来像是您应该限制Room班级可以进入的状态数。

您所问的情况Client是null 时，实际上是在暗示Room状态空间太大。

为了简单起见，我不允许Client任何Room实例的属性为null。这意味着其中的代码Room可以安全地假设Client永不为null。

如果将来由于某种原因Client变得null抗拒支持该国家的冲动。这样做会增加维护的复杂性。

而是让代码失败并快速失败。毕竟，这不是受支持的状态。如果应用程序进入这种状态，那么您已经越过了一条无法返回的线。唯一合理的事情就是关闭应用程序。

由于未处理的空引用异常，这种情况可能会发生（自然而然）。

只是一些注意事项：

a）为什么已经有Client实例本身的公共getter的原因，为什么有专门针对ClientId的getter？我不知道为什么long必须在Room签名中将ClientId的信息刻在石头上。

b）关于第二种意见，您可以引入一个常量Invalid_Client_Id。

c）关于意见一和三（这是我的主要观点）：房间必须始终有客人吗？也许这只是语义，但这听起来不对。对于Room和Client以及将它们联系在一起的另一类，可能完全合适。也许是预约，预定，入住？（这取决于您的实际工作。）并且在该类上，您可以强制执行“必须有房间”和“必须有客户”的约束。

我不同意这三种意见。如果Client永远不可能null，那就别把它变成可能null！

1. Client在构造函数中设置的值
2. ArgumentNullException在构造函数中抛出。

因此，您的代码将类似于：

public class Room
{
    private Client theClient;

    public Room(Client client) {
        if(client == null) throw new ArgumentNullException();
        this.theClient = client;
    }

    public Client Client { 
        get { return theClient; }
        set 
        {
            if (value == null) 
                throw new ArgumentNullException();
            theClient = value;
        }
    }

    public long ClientId
    {
        get
        {
            return theClient.Id;
        }
    }
}

public class Client 
{
    public long Id { get; set; }
}

这与您的代码无关，但是最好使用不可变版本的Room，方法是maketheClient readonly，然后如果客户端发生更改，请新建一个房间。除了我回答的其他方面的空安全性之外，这还将提高代码的线程安全性。参见关于可变与不可变的讨论

应该避免第二和第三种选择-除非它们无法控制，否则吸气剂不应向呼叫者发送信号。

您应该确定Client是否可以为null。如果是这样，您应该为调用者提供一种在访问它之前检查它是否为空的方法（例如，bool ClientIsNull属性）。

如果您确定Client永远不能为null，则将其作为构造函数的必需参数，并在传入null的情况下向其抛出异常。

最后，第一个选项还包含代码气味。您应该让客户端处理自己的ID属性。在容器类中仅对所包含的类调用getter的getter进行编码似乎有点过头了。只需将Client公开为属性（否则，您将最终复制Client已经提供的所有内容）。

long clientId = room.Client.Id;

如果Client可以为null，则您至少要对调用方负责：

if (room.Client != null){
    long clientId = room.Client.Id;
    /* other code follows... */
}

如果null客户端属性是受支持的状态，请考虑使用NullObject。

但是很可能这是一个异常状态，因此您应该使其不可能（或者只是不太方便）以null结尾Client：

public Client Client { get; private set; }

public Room(Client client)
{
    Client = client;
}

public void SetClient(Client client)
{
    if (client == null) throw new ArgumentNullException();
    Client = client;
}

如果不支持，则不要在半烘烤溶液上浪费时间。在这种情况下：

return Client == null ? 0 : Client.Id;

您已经在此处“解决”了NullReferenceException，但付出了高昂的代价！这将强制所有的呼叫者Room.ClientId检查0：

var aRoom = new Room(aClient);
var clientId = aRoom.ClientId;
if (clientId == 0) RestartRoombookingWizard();
else ContinueRoombooking(clientid);

奇怪的bug可能会引起其他开发人员（或您自己！）的困扰，而忘记在某个时间点检查“ ErrorCode”返回值。
安全玩游戏，以免失败。允许抛出NullReferenceException并“优雅地”将其捕获到调用堆栈的更高位置，而不是允许调用者将事情弄得更糟。

另一方面，如果您是如此热衷于公开Client和ClientId了解TellDontAsk。如果您对对象提出过多要求，而不是告诉他们您想要实现什么目标，则可能会结束所有事物的结合，从而使以后的变更更加困难。

从现在发布的c＃6.0开始，您应该这样做，

public class Room
{
    public Room(Client client)
    {
        this.Client = client;
    }

    public Client Client { get; }
}

public class Client
{
    public Client(long id)
    {
        this.Id = id;
    }

    public long Id { get; }
}

提高Clientto 的属性Room显然违反封装和DRY原理。

如果您需要访问的Id一个Client，则Room可以执行以下操作：

var clientId = room.Client?.Id;

注意使用空条件运算符，clientId将是一个long?，如果Client是null，clientId将是null，否则clientId将具有的值Client.Id。