Błąd podczas inkrementowania wskaźnika

Question

Kiedy próbowałem odtworzyć program z kursu https://www.youtube.com/watch?v=0DQl74alJzw&list=PLOYHgt8dIdoxx0Y5wzs7CFpmBzb40PaDo&index=11, program włącza się, ale wyskakuje błąd, który bym program tam nie robił to inny błąd, ale dotyczy tego samego elementu - wskaznik++;.

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

using namespace std;

int ile;
clock_t start, stop;
double czas;

int main()
{
	cout << "ile liczb w tablicy: ";
	cin >> ile;

	int* tablica;
	tablica = new int[ile];

	start = clock();

	for (int i = 0; i < ile; i++)
	{
		tablica[i] = i;
		tablica[i] += 50;
	}

	stop = clock();
	
	czas = (double)(stop - start)/CLOCKS_PER_SEC;
	cout << "czas zapisu (bez wskaznika): " << czas << endl;

	delete[]tablica;

	int* wskaznik = tablica;
	tablica = new int[ile];

	start = clock();
	for (int i = 0; i < ile; i++)
	{
		*wskaznik = i;
		*wskaznik += 50;
		wskaznik++;
	}

	delete[]tablica;

	_getch();
	return 0;
}

Dodam że programuje w visual studio.

Comments

Podaj treść błędu.

Answer 1

W linii 33 usuwasz "tablica", a linijkę dalej zapamiętujesz wartość tej zmiennej we "wskaznik" i potem używasz tego w drugiej pętli. Może się zdarzyć, że druga allokacja zajmie miejsce pierwszej, ale nie ma gwarancji.

Comments

Jego sugestia jest jak najbardziej prawdziwa. OP zwolnił pamięć a następnie próbuje do niej uzyskać dostęp (UB). Dlatego program mu crashuje. Powinien zamienić linijki 35 i 36 miejscami (a najlepiej usunąć ten program bo nie ma sensu).

---

@VBService,
Nie każdy kod zamieszczony w internecie jest bezbłędny. Zobacz komentarze pod filmem.

---

Dobra, dzięki nie zauważyłem tego wcześniej

---

@Oscar, Tylko nie wiem dlaczego nie działa mi w takim razie ten program 

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

using namespace std;

int ile;

int main()
{
	cout << "Ile liczb 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();
	return 0;
}

A jak dodam kolejny wskaźnik, na którym działam w pętli, to program działa bez błędu

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

using namespace std;

int ile;

int main()
{
	cout << "Ile liczb w tablicy: ";
	cin >> ile;

	int* tablica;
	tablica = new int[ile];
	int* wskaznik = tablica;

	for (int i = 0; i < ile; i++)
	{
		cout << (int)wskaznik << endl;
		wskaznik++;
	}

	delete[] tablica;

	_getch();
	return 0;
}

 

---

Operatorowi delete[] trzeba dostarczyć adres, który zwrócił new[]

Ty inkrementujesz wskaźnik w pętli a potem go dostarczasz delete[] przez co program próbuje zdealokować niepoprawny obszar pamięci co kończy się wysypaniem programu.

---

Dziękuje bardzo, chyba w końcu ogarnąłem o co chodzi