如何修复此代码中的'Access violation reading location'

Question

运行此代码，我遇到了这样的错误

' 在 Project 1 中的0x778D7FCB (ntdll.dll( 引发异常.exe： 0xC0000005：访问冲突读取位置0x00000014。

此错误发生在此行中

~UnivStudnet() {
delete[]major; // error
    }

#include <iostream>
#include <cstring>
using namespace std;
class Person {
private:
    char * name;
public:
    Person(const char * myname) {
        name = new char[strlen(myname) + 1];
        strcpy_s(name, strlen(name), myname);
    }
    ~Person() {
        delete[]name;
    }
    void WhatYourName() const {
        cout << "My name is " << name << endl;
    }
};
class UnivStudnet : public Person {
private:
    char * major;
public:
    UnivStudnet(const char * myname, const char * const mymajor) :Person(myname) {
        major = new char[strlen(mymajor) + 1];
        strcpy_s(major, strlen(major), mymajor);
    }
    ~UnivStudnet() {
        delete[]major;
    }
    void WhoAreYou() const {
        WhatYourName();
        cout << "My major is " << major << endl;
    }
};
int main(void) {
    UnivStudnet st1("kim", "Mathenatics");
    st1.WhoAreYou();
    UnivStudnet st2("hong", "Physiscs");
    st2.WhoAreYou();
    return 0;
}

如何修复此错误？

Answer 1

两行strcpy_s上有错误。

strcpy_s(name, strlen(name), myname);

应该是

strcpy_s(name, strlen(myname)+1, myname);

同样

strcpy_s(major, strlen(major), mymajor);

应该是

strcpy_s(major, strlen(mymajor)+1, mymajor);

在新分配的 char 数组name和major上调用strlen，这些数组的值不确定，会导致未定义的行为，这是导致崩溃的原因。

Answer 2

您的strcpy_s用法可疑。

    major = new char[strlen(mymajor) + 1];
    strcpy_s(major, strlen(major), mymajor);

要strcpy_s的第二个参数是第一个参数指定的缓冲区的分配大小。 (我刚刚意识到 - 基于另一个答案，在您复制到它之前strlen(major)是未定义的！

您分配的缓冲区足够大以容纳字符串，但随后对 strcpy_s 的调用表明major不够大，无法容纳整个字符串，包括空字符。

更好：

    size_t len = strlen(mymajor) + 1;
    major = new char[len];
    strcpy_s(major, len, mymajor);

对基类name参数也重复上述模式。

Answer 3

你可以

走C++路：

您需要声明：

virtual ~Person()

基类中的析构函数，然后：

class UnivStudnet : public Person {
private:
    std::string major;
public:
    UnivStudnet(const char * myname, const char * const mymajor) :Person(myname), major(mymajor) {
    }
    virtual ~UnivStudnet() {
    }
...

这样，您将实现所需的目标，而无需考虑内存分配/释放。请记住#include <string>标题。

在

课堂上也这样做Person。