从一个构造函数谈谈的代码的封装性和怎么表现自己的意图

问题背景:

一个机器人,给他下达一个命令,让它按照命令的内容进行相应的移动。最后输出机器人的位置。

题目很简单,我很快写好了一个类

class robot
{
public:
    void SetCmd(string Cmd);
    void SetPos(Point Pos);
    void Action();
private:
    Point Pos;
    string Cmd;
}

有人说,你这样写不好,SetCmd和SetPos这两个方法是多余的,可以直接合并到构造函数中去。

class robot
{
public:
    robot(string Cmd, Point Pos);
    void Action();
private:
    Point Pos;
    string Cmd;
}

他说的有道理,这样函数确实更加简单了,但是这样一定最好吗?不见得。

因为一个机器人在开始被创建的时候不一定会对他输入一个命令,也就是命令不是机器人被创建的必要条件。而且把命令写入构造函数了以后,以后无法对命令进行再次设定,对一个机器人设置两次命令肯定是有的场景。

而位置设置成一个初始值以后,后面机器人只能靠移动来改变位置——除非你的机器人会瞬间移动。而且最重要的是,如果你提供了设置位置的做法,那么别人在开发的时候就有可能去设置机器人的位置,这样就破坏了你当初的设计意图。

因此我把类修改成了下面的写法。我觉得这样既可以有效的封装内部信息,又可以很好的表现我的意图:位置不可以被修改。

class robot
{
public:
    robot(Point Pos);
    void SetCmd(string Cmd);
    void Action();
private:
    Point Pos;
    string Cmd;
}

 

 

posted @ 2012-09-19 23:47  HighFUN  阅读(407)  评论(0编辑  收藏  举报