代码完成2ed,组成和委托

Question

在这个论坛上看了几个星期后,我觉得是时候做我的第一篇文章.

我目前正在重读Code Complete.我认为这是自上次以来的15年,我发现我仍然无法编写代码;-)

无论如何,在Code Complete中的第138页,你会发现这个编码恐怖的例子.(我删除了一些代码)

class Emplyee {
public: 
 FullName GetName() const;
 Address GetAddress() const;
 PhoneNumber GetWorkPhone() const;
 ...

 bool IsZipCodeValid( Address address);
 ...

private: 
   ...
}

史蒂夫认为不好的是功能松散相关.或者他写道:"检查邮政编码,电话号码或工作分类的员工和例程之间没有逻辑联系"

好的,我完全赞同他.也许类似下面的例子更好.

class ZipCode
{
public:
 bool IsValid() const;
    ...
}

class Address {
public:
   ZipCode GetZipCode() const;
   ...
}

class Employee {
public: 
 Address GetAddress() const;
    ...
}

检查拉链是否有效时,您需要执行类似的操作.

employee.GetAddress().GetZipCode().IsValid();

对于得墨忒耳法而言,这并不好.

因此,如果你想删除三个点中的两个,你需要使用委托和一些像这样的包装函数.

class ZipCode
{
public:
 bool IsValid();
}

class Address {
public:
   ZipCode GetZipCode() const;
   bool IsZipCodeValid() {return GetZipCode()->IsValid());
}

class Employee {
public: 
 FullName GetName() const;
 Address GetAddress() const;
 bool IsZipCodeValid() {return GetAddress()->IsZipCodeValid());
 PhoneNumber GetWorkPhone() const;
}

employee.IsZipCodeValid();

但是,你再次拥有没有逻辑连接的例程.

我个人认为这篇文章中的所有三个例子都很糟糕.这是我没想过的其他方式吗？

Answer 1

您缺少逻辑连接:

class ZipCode
{
public:
 bool IsValid();
}

class Address {
public:
   ZipCode GetZipCode() const;
   bool IsAddressValid();
   bool IsValid() {return GetZipCode()->IsValid() && IsAddressValid());
}

class Employee {
public: 
 FullName GetName() const;
 Address GetAddress() const;
 bool IsEmployeeValid();
 bool IsValid() {return GetAddress()->IseValid() && IsEmployeeValid());
 PhoneNumber GetWorkPhone() const;
}

employee.IsValid();

Answer 2

现在付款与以后付款。

您可以预先编写委托和包装函数（立即付款），然后在稍后更改employee.IsZipCodeValid() 的内部结构即可减少工作量。或者，您可以通过编写隧道到达 IsZipCodeValid

员工.GetAddress().GetZipCode().IsValid();

代码中任何需要它的地方，但是如果您决定以破坏此代码的方式更改类设计，请稍后付费。

你可以选择你的毒药。;)