今天测试一段新代码的时候,发现一个BUG,原因是同事写的一个类,在对其实例调用execute()方法之前,需要用pushParam()方法把类型为const char *的参数“压”给该对象。
class Foo {
public:
void pushParam( const char *param);
bool execute();
};
public:
void pushParam( const char *param);
bool execute();
};
从类的行为上来说,我理所当然地认为pushParam()会保存一份字符串的副本,原因是通常的STL算法或容器都是这样做,而且这样也更安全,更重要的是,在这里没有任何注释明确要求“这定的字串指针必须长时间(至不在Foo这个对象销毁之前)保持有效“。
而事实上,为了效率,Foo这个类的实现只是保存了字串的指针,而不是个副本,它假定这个参数是长期有效的,正好我这段新代码在这里push了一个临时string对象,问题就产生了......
其实这里没有孰是孰非的问题,问题的关键在于:
- 良好健全的注释很有必要
- 代码安全在绝大多数时候,胜过(所谓的)效率,之所以用“所谓的”,是因为在当今电脑的速度和空间下,保存一个字串的副本的开销,实在是微乎其微。
- 从DELPHI联想到,如果C++有引用计数的string,那么效率和安全都兼顾了。C++实现字串的引用计数相当方便,不同的是DELPHI把这一切隐藏在编译器的背后。