经常见到的一个的面试题。。

ilex

【题：】编写类String的构造函数、析构函数和赋值函数，已知类String的原型为
class String
{
　public:
　　String(constchar*str = NULL); // 普通构造函数
　　String(const String &other); // 拷贝构造函数
　　~ String(void); // 析构函数
　　String & operator =(const String &other); // 赋值函数
　private:
　　char*m_data; // 用于保存字符串
};
　解答：
//普通构造函数
String::String(constchar*str)
{
　if(str==NULL)
　{
　　m_data =newchar[1]; // 得分点：对空字符串自动申请存放结束标志'\0'的空
　　//加分点：对m_data加NULL 判断
　　*m_data ='\0';
　}
　else
　{
　　int length = strlen(str);
　　m_data =newchar[length+1]; // 若能加 NULL 判断则更好
　　strcpy(m_data, str);
　}
}
// String的析构函数
String::~String(void)
{
　delete [] m_data; // 或deletem_data;
}
//拷贝构造函数
String::String(const String &other) 　　　// 得分点：输入参数为const型
{
　int length = strlen(other.m_data);
　m_data =newchar[length+1]; 　　　　//加分点：对m_data加NULL 判断
　strcpy(m_data, other.m_data);
}
//赋值函数
String & String:perator =(const String &other) // 得分点：输入参数为const型
{
　if(this==&other) 　　//得分点：检查自赋值
　　return*this;
　delete [] m_data; 　　　　//得分点：释放原有的内存资源
　int length = strlen( other.m_data );
　m_data =newchar[length+1]; 　//加分点：对m_data加NULL 判断
　strcpy( m_data, other.m_data );
　return*this; 　　　　　　　　//得分点：返回本对象的引用
}


最近在整理一份面试题库，抄来抄去抄到这个题了，其实是有错误（编辑一下，不是错误是有点不够完善，如楼下的所说的，new没有判断是否为NULL）的，但发现网上到处都是这个样子，那些转博客的人貌似也就是转转罢了，都不细纠代码。

bruceteen

你总得说说错在什么地方吧？！

hellioncu

好像出自林锐的那本书

cokeboL

回帖攒积分

mirnshi

1. int length = strlen(str);
   
2. 　　m_data =newchar[length+1]; // 若能加 NULL 判断则更好
   
3. 　　strcpy(m_data, str);

复制代码

这是为了凑代码行数？

另外，作为基础类，诸如内存异常是必须处理的吧？

mightofcode

哪有错 没看出来

bruceteen

mightofcode 发表于 2013-07-16 18:28
哪有错 没看出来

我也没看出来，作为示例代码这完全够了，加上完善的细枝末节代码反而会影响自己所想表达的代码。
但楼主草鸡毛，没说出他认为不正确的地方。我猜可能指的是其注释，比如“加NULL 判断”等

要我说这段代码中最烂的就是 检查字符串指针值是否为NULL 的行为了
String::String( const char* str )
{
　if(str==NULL)

总有一些人鼓吹要检查str是否NULL，还觉得自己这种行为是严谨细致的表现，其实是个半吊子。
想想C中的strcpy，检查参数为NULL了吗？想想C++中的std::string，检查参数为NULL了吗？保证 错误的行为得到错误的结果 才是正常的做法。

hellioncu

bruceteen 发表于 2013-07-17 09:05
我也没看出来，作为示例代码这完全够了，加上完善的细枝末节代码反而会影响自己所想表达的代码。
但楼主 ...

可能是说缺省new失败时抛异常，当然VC6不是

zhaohongjian000

一般的程序，new失败了直接跪了就好，没内存了还怎么搞？

另外如8楼所说，正常编译器new失败要抛异常的，判断是否为NULL有啥用。

jackarain

if(str==NULL)

总有一些人鼓吹要检查str是否NULL，还觉得自己这种行为是严谨细致的表现，其实是个半吊子。

说的太对了, 因为有些东西, 不是半吊子能掌握的, 半吊子就是半吊子, 写出的代码, 一眼就可以看出半吊子! 还面试人呢!

我已经N次拿memcpy/strcpy跟很多人讲这个问题, 可是, 天朝程序员基本上领味不到我在说什么, 偏偏一意孤行的, 检查这, 检查
那, 最后就是一团"面条代码"而不自知, 这种"面条代码", 无论如何乱来, 怎么调用都保证不会出错! 这就是那些人追求的稳定性!兼容性!