更改const char指针的问题

Question

我在更改char指针时遇到一些问题，无法弄清楚我要去哪里

这是我更改说明的功能...

void appointment::changeDescription(const char * s)   //  change an existing description
{

if (desc != NULL)
    strcpy(desc, s);

if (s == NULL)
    return;

}

这是调用更改描述功能的功能。

bool  keyBoardEnterAppointment( schedule & sched)  // return true if successful
// return false if full
{
    if (sched.isFull() == true)
    {
        cout << "Schedule is FULL." << endl;
        return false;
    }
    else
    {
    appointment s;
    int day, month, year;
    char *desc = new char;
    long source;

    cout << "*/Enter Appointment\\* ";
    cout << "Description: "; cin >> desc;
    cout << "Source: "; cin >> source;

    cout << "Month: "; cin >> month;
    cout << "Day: "; cin >> day;
    cout << "Year: "; cin >> year;

    s.changeDescription(desc);
    s.setSource(source);
    s.setDay(day);
    s.setMonth(month);
    s.setYear(year);
    sched.addtoSchedule(s);

    sched.print(cout);

    return true;
}

}

它可以编译并运行，但其描述与默认构造函数描述相同。

Answer 1

如果您使用std::string将描述存储在约会类中，则可以使您自己以及最终使用您的代码的人变得更加轻松。 changeDescription方法将变为：

#include <string>

void appointment::changeDescription(std::string const& s){
    this->desc = s;
}

并将调用代码更改为此：

std::string desc;

然后，几乎所有免费的（导致编程困难的）错误修复导致您面临问题的烦人的内存管理。 一般来说，与使用以空终止的C样式char数组相比，这被认为是更好的惯用c ++代码。

关于代码的另一件事是，您真的应该在尝试复制s之前（而不是之后）检查s是否不为null。