[C++] Errore in esecuzione

[Pacio88]

Devo gestire un array di "risorse". Ciascuna risorsa consiste in una stringa di caratteri e un variabile che indica se quest'ultima è libera o occupata.

Questo è il codice:

#include <iostream>
using namespace std;

enum stato{libera, occupata};


struct risorsa{
	char* str;
	stato st;
};

class Sistema{
	risorsa* ri;
	int lung, disp;
	
	friend ostream& operator<<(ostream&, const Sistema&);
public:
	Sistema(int);
	~Sistema();
	bool aggiungi(const char*);
	bool occupa(const char*);
	bool contiene(const char*);
	bool elimina(const char*);
};

ostream& operator<<(ostream& os, const Sistema& si){
	os << "Capacita' del sistema: " << si.lung << endl
	   << "Risorse presenti nel sistema: " << si.disp << endl;
	if(si.disp>0){
		os << "Elenco risorse libere: ";
		for(int i=0; i<si.disp; i++){
			if(si.ri[i].st==libera)
				os << si.ri[i].str << " ";
		}
		os << "\nElenco risorse occupata: ";
		for(int i=0; i<si.disp; i++){
			if(si.ri[i].st==occupata)
				os << si.ri[i].str << " ";
		}
		os << endl;
	}	
	return os;
}		

Sistema::Sistema(int i){
	lung=i;
	disp=0;
	ri=new risorsa[lung];
}

Sistema::~Sistema(){
	delete[] ri;
}

bool Sistema::aggiungi(const char* str){
	for(int i=0; i<lung; i++){
		if(ri[i].str==str)
			return false;
	}		
	if(lung==disp)
		return false;
	strcpy( ri[disp].str, str);
	ri[disp].st=libera;
	disp++;
	return true;
}		

int main(){
	Sistema si(5);
	cout << si;
	si.aggiungi("R1");
	si.aggiungi("R2");
	si.aggiungi("R3");
	cout << si;
	
	return 0;
}

L'errore è scaturito dalla funzione aggiungi(). Qualcuno sa aiutarmi?

[oregon]

Originariamente inviato da Pacio88
L'errore è scaturito dalla funzione aggiungi().

Perche' non ci dici nulla dell'errore ???

[Pacio88]

Originariamente inviato da oregon
Perche' non ci dici nulla dell'errore ???

Perchè non ne so niente. L'unica cosa che so è per l'appunto che è causata dalla funzione aggiungi(), poichè ho testato funzione per funzione ed è l'unica che quando chiamata provoca la chiusura inaspettata dell'applicazione. Purtroppo non so usare adeguatamente il debugger e l'uniche cose che posso dirvi sull'erore sono queste.

[oregon]

Cioe' non vedi cosa succede? Il sistema non ti da' nessun messaggio?


P.S. In realta' vedo che non allochi mai lo spazio per le stringhe puntate dal puntatore str dei vari elementi di tipo

struct risorsa{
	char* str;
	stato st;
};

Quando, nella funzione Aggiungi, tenti di eseguire la

strcpy

con quel puntatore, hai ovviamente l'errore ...

[Pacio88]

Il sistema mi dice che l'applicazione si è chiusa inaspettatamente. I dettagli dell'errore dovrebbero essere i seguenti:

AppName: senzatitolo1.exe AppVer: 0.0.0.0 ModName: msvcrt.dll
ModVer: 7.0.2600.2180 Offset: 000360b4

[Pacio88]

Grazie, era quello l'errore. Mi ero dimenticando che stavo gestendo quella stringa con un puntatore.