Question

c++题目，哪里有问题？怎么改？

下面的程序可以运行，结果为：

书名：中国足球内幕

作者：李承鹏

价格：29 元

书名：傻儿皇帝：阿斗回忆录

作者：王新禧

价格：25 元

运行结果看来也正确。但这个程序有问题，请仔细分析找出问题并修改完善。

#include<iostream>

usingnamespace std;

#include<string>

class Book {

public:

void setBook(constchar*str1,constchar*str2,constfloat num){

int len=strlen(str1);

title=newchar[len+1];

strcpy(title,str1);

len=strlen(str2);

author=newchar[len+1];

strcpy(author,str2);

price=num;

}

Book() {}

Book(constchar*str1,constchar*str2,constfloat num)

{setBook(str1,str2,num);}

~Book(){delete title; delete author;}

void print(){

cout<<"书名："<<title<<endl;

cout<<"作者："<<author<<endl;

cout<<"价格："<<price<<" 元"<<endl;

}

private:

char*title;

char*author;

float price;

};

int main(){

Book obj1("中国足球内幕","李承鹏",29.0),obj2(obj1);

obj1.print();

obj2.setBook("傻儿皇帝：阿斗回忆录","王新禧",25.0);

obj2.print();

return 0;}

Answer 1

1. setBook () 存在内存泄露的可能性，new 之前应先判断 指针是否有效，有效 先 delete 再 new
   

2. ~Book() 未判断指针是否有效，应先判断 有效 才能 执行delete ，如 if(NULL != ptr) delete ptr;

3. Book() 默认构造函数未对成员进行初始化，指针应初始化为NULL，如:

Book()
:title(NULL),author(NULL),price(0.0)
{
}

Answer 2

把string头文件改为string.h再试试