Dlaczego pamięć "zwalnia się dwa razy"

Question

Witam, przeglądając kurs c++ MZ na YT, a konkretnie odcinek o wskaźnikach w pierwszym programie
( kod poniżej )  Visual Studio wyrzuca mi tutaj error, konkretnie mówi, że znalazł breakpoint.

Doszedłem do tego, że problem pojawia się przy zwalnianiu pamięci ( kiedy usunę linijke delete [] tablica; problem nie występuje ). W teorii pamięć została zwolniona już wcześniej, ale wobec tego gdzie?

#include <iostream>
#include <conio.h>

using namespace std;

int main()
{
	int ile;

	cout << "Ile chcesz miec miejsc w tablicy?: ";
	cin >> ile;

	int *tablica;

	tablica = new int[ile];


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

		cout << (int)tablica << endl;
		tablica++;


	}


	delete [] tablica;

	_getch();
}

 

Answer 1

Ty  musiałeś coś kliknąć. :P Breakpoint ustawia programista w celu zatrzymania przebiegu programu przy debugowaniu.

A tak przy okazji - pamięć alokowana dynamicznie nie zwalnia się sama. ;)

Comments

Chyba to nie on bo ja na jego kodzie też dostaje:

a.out(13390,0x7fff7a69b300) malloc: *** error for object 0x7f9a4bd00010: pointer being freed was not allocated

*** set a breakpoint in malloc_error_break to debug

---

W takim razie włączam IDE i sprawdzam osobiście. :)

---

Już wiem, co jest grane. :) Podajesz do operatora delete[] wskaźnik, który został zwiększony i w konsekwencji wskazuje poza tablicę. Należy cofnąć ten wskaźnik na początek tablicy i wtedy wysłać go do delete[], np. tak:

    tablica -= ile;
    delete [] tablica;

 

---

Ale czy w tym momencie usuwam wskaźnik, czy tylko zarezerwowaną pamięć?

---

Wskaźnik zostaje i możesz go nadal używać, oczywiście jeśli przypiszesz do niego jakąś sensowną wartość. Zwalniasz tylko zaalokowane zasoby. Spójrz na ten przykład:

#include <iostream>

using namespace std;

int main()
{
    int ile;

    cout << "Ile chcesz miec miejsc w tablicy?: ";
    cin >> ile;

    int *tablica;

    tablica = new int[ile];


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

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

    tablica -= ile;
    delete [] tablica;
    // ponowne użycie wksaźnika...
    tablica = new int[15];
    for(unsigned i = 0; i < 15; ++i)
    {
        tablica[i] = i * 2;
    }

    for(unsigned i = 0; i < 15; ++i)
    {
        cout<<tablica[i]<<endl;
    }
    delete[] tablica; // tym razem poprawnie zwalnia pamięć, bo nie modyfikowaliśmy adresu wskaźnika
    return 0;
}