• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Dynamiczne alokowanie pamięci, DFS

VPS Starter Arubacloud
0 głosów
312 wizyt
pytanie zadane 11 kwietnia 2016 w C i C++ przez aspoka Mądrala (5,290 p.)

Witam, piszę program który ma przeszukać graf w poszukiwania cykli do których należą poszczególne wierzchołki. 

#include <iostream>
#include <queue>
using namespace std;

int n;
bool *visited;
char **graf;
queue<int>Q;

bool DfsFindCycle(int start, int now)
{
	visited[now]=true;
	for(int i=0; i<n; i++) if((graf[now][i])&&(Q.front()!=now))
	{
		Q.push(now);
		if(i==start)	return true;
		if((!visited[i])&&(DfsFindCycle(start, i)))	return true;
		Q.pop();
	}
	return 0;
}

int main()
{
	int m, v1, v2;
	cin>>n>>m;
	visited = new bool[n];
	graf = new char*[n];
	for(int i=0; i<n; i++)
		graf[n] = new char[n];
		
	for(int i=0; i<n; i++)
	{
		visited=false;
		for(int j=0; j<n; j++) graf[i][j]=0;
	}
		
	for(int i=0; i<n; i++)
	{
		cin>>v1>>v2;
		graf[v1-1][v2-1]=graf[v2-1][v1-1]=1;
	}
	
	for(int i=0; i<n; i++)
	{
		for(int j=0; j<n; j++)
			visited[n]=false;
		Q.push(-1);
		if(DfsFindCycle(i, i))
		{
			cout<<i<<": ";
			while(!Q.empty())
			{
				v1=Q.front();
				Q.pop();
				cout<<v1;
			}
		}
		else	Q.pop();
	}
	
	for(int i=0; i<n; i++) delete [] graf[i];
	delete [] graf;
	delete [] visited;
	return 0;
}

Program po dopaleniu się wczytuje zmienną n i m a następnie kończy prace z błędem 3221225477

Gdy odpalam go na debugerze pokazuje że błąd jest w linijce 35: 

for(int j=0; j<n; j++) graf[i][j]=0;

Jak trochę kombinowałem to zauważyłem, że błąd pojawia się w pierwszej linijce w której próbuje się odwołać do tablicy graf.

Z góry dziękuje za odpowiedź

1 odpowiedź

0 głosów
odpowiedź 11 kwietnia 2016 przez Radfler VIP (101,030 p.)
edycja 11 kwietnia 2016 przez Radfler
for(int i=0; i<n; i++)
{
    visited=false; // 1
    for(int j=0; j<n; j++) graf[i][j]=0; // 2
}

1. To jest wg mnie błąd. Przypisujesz do wskaźnika visited false (czyli 0, nullptr). Tablica alokowana dynamicznie w 27 linijce jest po prostu gubiona (wyciek pamięci). Powinno być raczej:

visited[i] = false;

2. A tu inny błąd: tablica na j-tej pozycji nie istnieje. Spójrz na to:

for(int i=0; i<n; i++)
    graf[n] = new char[n]; // linijka 30

Alokujesz tablicę char na n-tej pozycji tablicy char*. A nie powinieneś przypadkiem robić tego na i-tej pozycji? :)

Pozdrawiam!

Podobne pytania

+2 głosów
3 odpowiedzi 8,696 wizyt
–1 głos
2 odpowiedzi 210 wizyt
0 głosów
1 odpowiedź 1,516 wizyt

92,454 zapytań

141,262 odpowiedzi

319,089 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...