【问题标题】:Copy Constructor Issue C++: "0xC0000005: Access violation writing location 0x00000000."复制构造函数问题 C++:“0xC0000005:访问冲突写入位置 0x00000000。”
【发布时间】:2016-02-08 05:47:03
【问题描述】:

我在让我的复制构造函数在我正在处理的BigInt 类中工作时遇到了一些麻烦。

在复制构造函数的BigIntVector.cpp 中,行:

    for (int i = 0; i < vectorSize; i++) {
        vectorArray[i] = orig.vectorArray[i];
    }

导致异常0xC0000005: Access violation writing location 0x00000000. 任何帮助找出原因将不胜感激。谢谢你。

BigInt.cpp:

// copy constructor
BigInt::BigInt(BigInt const& orig) 
 : isPositive(orig.isPositive)
, base(orig.base)
, skip(orig.skip) 
{
    this->bigIntVector = new BigIntVector(*orig.bigIntVector);
}

// constructor where operand is a long
BigInt::BigInt(long num) {
    base = 10;

    long sizeOfLong = 0; //holds size of num
    long tempNum = num;

    //get size of num
    if (tempNum == 0) {
        sizeOfLong = 1;
    }
    while (tempNum != 0)
    {
        tempNum /= 10;
        ++sizeOfLong;
    }

    //resize vector to match size of long
    this->bigIntVector = new BigIntVector(sizeOfLong);

    //cout << "sizeVec: " << bigIntVector.getSize() << endl;

    if (num < 0) {
        isPositive = false;
        num *= -1;
    }
    else {
        isPositive = true;
    }
    long pushedNum;
    //cout << "num: " << num << endl;
    for (int i = sizeOfLong - 1; i >= 0; --i) {
        pushedNum = (long)(num%base);
        bigIntVector->setElementAt(i, pushedNum);
        num /= base;
    }
}

// destructor
BigInt::~BigInt() {
    delete[] this->bigIntVector;
}

// binary addition
BigInt BigInt::operator+(BigInt const& other) const {

    BigInt temp(*this);
    return temp += other;
}

//more code...

我的BigInt.h

class BigInt {
private:
    BigIntVector *bigIntVector;
    bool isPositive;
    int base;
    unsigned int skip;

public:
    BigInt(BigInt const& orig);
    BigInt(long num);
    ~BigInt();
    BigInt operator+(BigInt const& other) const;
    BigInt operator+() const;
    BigInt operator++();
    BigInt operator++(int dummy);
    BigInt operator+=(BigInt const& other);
    BigInt BigInt::operator-(BigInt const& other) const;
    BigInt BigInt::operator-=(BigInt const& other);
    bool operator==(BigInt const& other) const;
    friend std::ostream & operator<<(std::ostream& os, BigInt& num);
};
inline BigInt operator+(long num, BigInt const& val) {
    return val + num;
}
inline bool operator==(long num, BigInt const& val) {
    return val == num;
}

我的BigIntVector 是一个自定义的Vector 类。与 STL 向量相比。

BigIntVector.cpp中的构造函数:

// copy constructor
BigIntVector::BigIntVector(BigIntVector const& orig)
    : vectorSize(orig.vectorSize)
{
    for (int i = 0; i < vectorSize; i++) {
        vectorArray[i] = orig.vectorArray[i];
    }
}

BigIntVector::~BigIntVector() {
    delete[] vectorArray;
}

//default constructor
BigIntVector::BigIntVector()
{
    vectorSize = 1;

    //vectorArray = (long *)malloc(10 * sizeof(long));
    vectorArray = new long[vectorSize];
    for (long i = 0; i < vectorSize; i++) {
        vectorArray[i] = 0;
    }
}

//constructor that initializes a custom size for vector
BigIntVector::BigIntVector(long initialSize)
{
    vectorSize = initialSize;

    //vectorArray = (long *)malloc(initialSize*sizeof(long));
    vectorArray = new long[vectorSize];
    for (long i = 0; i < initialSize; i++) {
        vectorArray[i] = 0;
    }
}

//more code

main.cpp:

int main(void) {

    // object with explicit constructor from long
    BigInt num1(12);

    cout << "num1 (12): " << num1 << endl;

    // object with implicit constructor from long
    BigInt num2 = 19;

    cout << "num2 (19): " << num2 << endl;

    // binary addition BigInt+BigInt
    BigInt num3 = num1 + num2;

    cout << "num3 (31): " << num3 << endl;

    BigInt numA = 99999;

    cout << "numA (99999): " << numA << endl;

    BigInt numB = 99999;

    cout << "numB (99999): " << numB << endl;

    BigInt numC = numA + numB;

    cout << "numC (199998): " << numC << endl;

    return EXIT_SUCCESS;
}

【问题讨论】:

  • 您正在传递对 orig 的引用。你应该使用'orig.bigIntVector'

标签: c++ pointers constructor copy-constructor biginteger


【解决方案1】:

您正在传递对 orig 的引用。你应该使用'orig.bigIntVector'

BigInt::BigInt(BigInt const& orig) 
 : isPositive(orig.isPositive)
, base(orig.base)
, skip(orig.skip) 
{
    this->bigIntVector = new BigIntVector(*(orig.bigIntVector));
}

should use *(orig.bigIntVector)

【讨论】:

  • this->bigIntVector = new BigIntVector*(orig.bigIntVector);返回一个错误,因为它们不能被分配给彼此。
  • this->bigIntVector = new BigIntVector(*(orig.bigIntVector));调用 BigIntVector 的复制构造函数
  • 哦,好吧,这行得通。但是,我现在看到内存错误(上面列出的相同消息)位于 for 循环中 BigIntVector 的复制构造函数中。我用新的错误位置更新了 OP。
  • 哪一行?你能通过吗?
  • 即使您的原始问题不同并且我的回答有效,但您再次更改了问题..如果您的程序正常工作,请接受我的回答
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2013-10-11
  • 1970-01-01
相关资源
最近更新 更多