Ошибка объявления элемента статического класса

Question

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

#include<iostream.h> 
#include<stdlib.h> 

class A { 
public: 
    static int x; //To count number of total objects. incremented in constructor 
    static int myheap; //To count number of heap objects. Incremented in overloaded new 

    void* operator new(size_t t) { 
    A *p; 
    p=(A*)malloc(t); 
    myheap++; 
    return p; 
    } 

    void operator delete(void *p) { 
    free(p); 
    myheap--; 
    } 

    A() { 
    x++; 
    } 

    ~A() { 
    x--; 
    } 
}; 
int A::x=0; 
int A::myheap=0; 

int main() { 
    A *g,*h,*i; 
    A a,c,b,d,e;//Static allocations 5 

    g= new A();//Dynamic allocations 3 
    h= new A(); 
    i= new A(); 

    cout<<"Total"<<A::x<<'\n'; 

    cout<<"Dynamic"; 
    cout<<'\n'<<"HEAP"<<A::myheap; 

    delete g; 
    cout<<'\n'<<"After delete g"<<A::x; 
    cout<<'\n'<<"HEAP"<<A::myheap; 
    delete h; 
    cout<<'\n'<<"After delete h"<<A::x; 
    cout<<'\n'<<"HEAP"<<A::myheap; 
    delete i; 
    cout<<'\n'<<"After delete i"<<A::x; 
    cout<<'\n'<<"HEAP"<<A::myheap; 
} 

Answer 1

Ваш код почти прав, но вы видите ошибки в «myheap», потому что компилятор был смущен более ранними ошибками. Сначала исправьте первую ошибку.

О перегрузке нового оператора, есть больше, чем простой malloc. У меня есть previous example, который может помочь, но это было глобально новым, а не специфичным для класса.

Здесь очищается (это компилируется и работает)

#include <iostream> 
#include <memory> 
#include <new> 
#include <stdlib.h> 

struct A { 
    static int count; 
    static int heap_count; 
    void* operator new(std::size_t t) { 
    void* p = malloc(t); 
    if (!p) throw std::bad_alloc(); 
    heap_count++; 
    return p; 
    } 
    void operator delete(void *p) { 
    free(p); 
    heap_count--; 
    } 
    A() { 
    count++; 
    } 
    ~A() { 
    count--; 
    } 
}; 
int A::count = 0; 
int A::heap_count = 0; 

int main() { 
    using namespace std; 

    A a, b, c, d, e; 
    auto_ptr<A> g (new A), h (new A), i (new A); 

    cout << "Total: " << A::count << '\n'; 
    cout << "Dynamic\nHeap: " << A::heap_count << '\n'; 
    g.release(); 
    cout << "After delete g: " << A::heap_count << '\n'; 
    h.release(); 
    cout << "After delete h: " << A::heap_count << '\n'; 
    i.release(); 
    cout << "After delete i: " << A::heap_count << '\n'; 
    cout << "Heap: " << A::heap_count << '\n'; 

    return 0; 
} 

Answer 2

Это должно быть A::myheap.

Кроме того, ваш оператор new должен вызывать конструктор: Вы правы, вам просто нужно вернуть указатель на вновь выделенный объект.

void * operator new(size_t t) 
{ 
A *p = (A*)malloc(t); 
myheap++; 
return p; 
} 

Answer 3

Вы не местный названный myheap, но у вас есть класс в области видимости статической переменной с именем myheap. Таким образом, вам нужно A::myheap. Но действительно, myheap и x должны быть частными, и вы должны иметь статические getx и статические getmyheap общедоступные методы. И, конечно же, у x должно быть лучшее имя.

Answer 4

Sandeep,

Причина вы получаете дамп, когда новый не возвращался р, потому что ваша функция удаления пытается освободите указатель, который вы передаете.

Поскольку new не возвращал p, значение, отправляемое для удаления(), равно NULL или не инициализировано. Вызов без указателя с NULL или случайным значением из стека приведет к сбою вашей программы.

Бест,

Сэм