保存指针？还是副本？

今天测试一段新代码的时候，发现一个BUG，原因是同事写的一个类，在对其实例调用execute()方法之前，需要用pushParam()方法把类型为const char *的参数“压”给该对象。

class Foo {
public:
    void pushParam(const char *param);
    bool execute();
};

从类的行为上来说，我理所当然地认为pushParam()会保存一份字符串的副本，原因是通常的STL算法或容器都是这样做，而且这样也更安全，更重要的是，在这里没有任何注释明确要求“这定的字串指针必须长时间（至不在Foo这个对象销毁之前）保持有效“。

而事实上，为了效率，Foo这个类的实现只是保存了字串的指针，而不是个副本，它假定这个参数是长期有效的，正好我这段新代码在这里push了一个临时string对象，问题就产生了......

其实这里没有孰是孰非的问题，问题的关键在于：

1. 良好健全的注释很有必要
2. 代码安全在绝大多数时候，胜过（所谓的）效率，之所以用“所谓的”，是因为在当今电脑的速度和空间下，保存一个字串的副本的开销，实在是微乎其微。
3. 从DELPHI联想到，如果C++有引用计数的string，那么效率和安全都兼顾了。C++实现字串的引用计数相当方便，不同的是DELPHI把这一切隐藏在编译器的背后。