Странная ошибка конструктора копирования и деструктора

Question

У меня есть класс, и я продолжаю получать некоторые ошибки от деструктора. Это класс:

#pragma once
class Number
{
    int bas;
    char* val;
public:
    Number(const char* value, int base); 
    Number(const Number& x);
    ~Number();
    void SwitchBase(int newBase);
    void Print();
    int  GetDigitsCount();
    int  GetBase(); 
};

Это файл cpp:

#include "Number.h"
#include <iostream>
Number::Number(const char* value, int base)
{
    int a = -1;
    do
    {
        a++;
    } while (value[a] != '\0');
    val = new char[a + 1];
    for (int i = 0; i <= a; i++)
        val[i] = value[i];
    bas = base;
}

Number::Number(const Number& x)
{
    int a = -1;
    bas = x.bas;
    do
    {
        a++;
    } while (x.val[a] != '\0');
    delete[]val;
    val = new char[a + 1];
    int i;
    for (i = 0; i <= a; i++)
        val[i] = x.val[i];
}
Number::~Number()
{
    delete[]val;
}
void Number::Print()
{
    std::cout << "Numarul este: " << val<< std::endl << "Baza este: " << bas<<std::endl;
}
int Number:: GetDigitsCount()
{
    int l = 0;
    do
    {
        l++;
    } while (val[l] != '\0');
    return l;
}

Это главное:

int main()
{
    Number x("123", 10),y("111",10),z("0",10);
    z = y;
    z.Print();
}

Я получаю эту ошибку : Неверный адрес, указанный для RtlValidateHeap (010C0000, 010C8DD8) Если я делаю это изменение в main, он работает правильно, но это не совсем то, что я хочу ...

int main()
{
    Number x("123", 10),y("111",10);
    Number z = y;
    z.Print();
}

Как я могу решить это? Я не могу понять это ...

Answer 1

Это пример того, как код может выглядеть при использовании std :: string:

#include <iostream>
#include <string>

class Number
{
    int bas;
    std::string val;
public:
    Number(std::string, int base); 
    Number(const Number& number);
    Number& operator= (const Number& number);
    ~Number()=default;
    void Print();
    int  GetDigitsCount();
};


Number::Number(std::string value, int base)
{
    val=value;
    bas=base;
}

Number::Number(const Number& number)
{
    val=number.val;
    bas=number.bas;
}

Number& Number::operator= (const Number& number)
{
    val=number.val;
    bas=number.bas;
    return *this;
}

void Number::Print()
{
    std::cout << "Numarul este: " << val<< std::endl << "Baza este: " << bas<<std::endl;
}
int Number:: GetDigitsCount()
{
    return val.size();
}

int main()
{
    Number x("123", 10),y("111",10),z("0",10);
    Number k(y);
    k.Print();
}

Answer 2

В вашем классе Number отсутствует оператор присваивания. Так как вы используете оператор присваивания в main, оператор присваивания по умолчанию вызовет двойное удаление при выходе из main, и это объясняет ошибку.

Также объясняется, почему ошибка исчезает при изменении main на используйте конструктор копирования вместо оператора присваивания.

Вы должны взглянуть на идиому copy и swap , чтобы показать, как легко и эффективно реализовать конструкторы копирования и операторы присваивания.

В качестве альтернативы вы также можете использовать std::string вместо выделения памяти вручную. Это избавит от необходимости писать деструктор, конструктор копирования и оператор присваивания. Это лучшее решение.