Не могу понять в чем проблема

@JUSTprograming · Регистрация: 29.06.2019

Студворк — интернет-сервис помощи студентам

C++

#include <iostream>
using namespace std;
 
class Apple
{
public:
    int *a;
     Apple (int value)
    {
        this->value=value;
        int *a = new int[value];
        for (int i = 0; i <value; i++)
        {
            a[i]= i;
            cout << a[i];
        }
    }
    void Show()
    {
        cout<<endl << this->value << endl;
        for (int i = 0; i < this->value; i++)
        {
            cout <<a[i] << endl;
        }
    }
    /*int& operator[](int index)
    {
        return a[index];
    }*/
    ~Apple()
    {
        delete[]a;
    }
    
private:
    int value;
};
 
 
int main()
{
    Apple apple(4);
    apple.Show();
    return 0;
}

@~~Pvt~~ · 29.06.2019, 23:59

C++

Apple *apple=new Apple(4);
apple->Show();
delete apple;
return 0;

@L0M · 30.06.2019, 00:09

В строке 11 вы объявляете переменную a, локальную для конструктора, и выделяете память. Далее работаете с этой памятью, адрес которой хранится в этой локальной переменной. При выходе из конструктора локальная переменная умирает, а память остаётся выделенной, но недоступной, поскольку её адрес утерян. В методе Show() и в деструкторе вы обращаетесь к переменной a экземпляра класса, которая неинициализирована. На чём и получаете фейл.

Строка 11 должна выглядеть так:

C++

1	a = new int[value];

Опции темы