Debug Assertion Failed C++ / Dynamiczna alokacja pamięci

Question

Witam !

Mam następujący problem :

 
 

#include<iostream>

using namespace std;

int main() {

    int how_many;

    cout << "How many elements in array? " << endl;

    cin >> how_many;

    
    int *array;

  
    array = new int[how_many];

    for (int i = 0; i < how_many; i++) {

        cout << (int)array << endl;
        array++;
    }

    delete [] array;    

    return 0;
}

Podany kod kompiluje się, jednak po odpaleniu programu wyskakuje następujący komunikat:

https://zapodaj.net/0d64cc1269ee5.png.html

Ps. Podany kod nie jest skończony oczywiście jednak ta część powinna działać (fragment kodu z kursu Pana Mirosława, który już w tej fazie działał Panu Zelentowi).

Answer 1

Witam,

spróbuj sobie zapisać w zmiennej adres pierwszego elementu array. W pętli edytujesz tą wartość i warto aby po jej wyjściu znać ten adres i przesłać go do delete.

Spróbuj tak:

#include<iostream>


using namespace std;

int main() {

    int how_many ;

    cout << "How many elements in array? " << endl;

    cin >> how_many;


    int *array;


    array = new int[how_many];

    int *wskNaPierwszyElement = array;//zapis adresu array
    for (int i = 0; i < how_many; i++)
    {
        cout << (int)array << endl;
        array++;//edytujesz adres array
    }


    array = wskNaPierwszyElement; //z powrotem przypisujesz array wartosc 1 elementu
    delete [] array;


    return 0;
}

 

Comments

Dodatkowo w linii

cout << (int)array << endl;

rzutowanie na 64bitowej platformie może się nie powieść i dostaniesz błąd o utracie precyzji ponieważ typ int nie ma tego samego rozmiaru co wskaźnik więc lepiej użyć typu (long) do rzutowania a jeszcze lepiej (intptr_t) z biblioteki <cstdint> ponieważ long na 64bitowym windowsie ma cały czas 32bity a wskaźnik 64.

Answer 2

delete[] musi dostać to samo co otrzymałeś od new[]. Inkrementujesz wskaźnik, więc do delete[] idzie co innego.

PS: wstaw kod w bloczek kodu (ikonka {...} w edytorze).