Discussion :

[Atharendil]

Bonjour,

mon problème est le suivant : j'aimerais parcourir une list avec un iterator, non const car j'appelle des méthodes des objets contenus dans la collection, et tout se passe très bien jusqu'à l'exécution où, au 2e parcours de la collection (car j'itère à l'intérieur d'une autre boucle), le programme plante et j'obtiens un zouli message d'erreur :

Debug assertion failed !
...
Expression : list iterator not dereferencable
...

Voici mon code :

d'abord où j'initialise la list :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
list<SjSl_Balloon> ballons;
SjSl_Timer timer;
 
srand(time(NULL));
for(unsigned int x=0; x<dim[0]; x++){
	for(unsigned int y=0; y<dim[1]; y++){
		for(unsigned int z=0; z<dim[2]; z++){
			int pos[3]={x, y, z};
			for(unsigned int nbBallon=0; nbBallon<SjSl_Random::GetInt(0, 3); nbBallon++){
				SjSl_Balloon tmp(timer, pos);
				ballons.push_back(tmp);
			}
		}
	}
}

et ensuite où je l'utilise :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
list<SjSl_Balloon>::iterator itB;
list<SjSl_WeatherStation>::iterator itS;
list<SjSl_WeatherStation> stations=earth.GetStations();
while(true){
	for(unsigned int d=0; d<31; d++){
		for(unsigned int h=0; h<24; h++){
			for(unsigned int mn=0; mn<60; mn++){
				for(unsigned int s=0; s<60; s++){
					int currentTime[4]={d, h, mn, };
					timer.SetTime(currentTime);
                                                                _sleep(10);
				}
			}
			itB=ballons.begin();
			for(; itB!=ballons.end(); itB++){
				(*itB).Move(earth);
				cout<<*itB<<endl;
			}
		}
		itS=stations.begin();
		for(; itS!=stations.end(); itS++){
			(*itS).BulletinMeteo();
			cout<<"Bulletins météo rangés dans le répertoire ulletins"<<endl;
		}
	}
}

Je n'arrive vraiment pas à comprendre d'où le problème peut venir car tout se passe au premier de la list ballons. En ce qui concerne la liste stations, je n'ai pas encore pu tester puisque le programme n'arrive pas jusque là...

[Pacorabanix]

Est ce que tes méthodes modifient ton conteneur ? Car dans ce cas les itérateurs deviennent invalides, et ne peuvent plus être déréférencés.

PS : utilises ++it au lieur de it++ dans le for, ça ne change pas grand chose mais évite de faire une copie inutile.

[Atharendil]

En fait, dès lors que le conteneur ballons est initialisé et remplit (donc dans la première boucle), il n'est plus modifié par aucune méthode.

Je vais essayer en mettant ++it.

Merci pour ton aide !

Edit : testé avec ++it au lieu de it++ et... ben rien ne change à l'exécution Toujours ce foutu problème.

[yan]

Bonjour,
à quoi ressemble le createur par recopie de SjSl_Balloon ??
et

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

int currentTime[4]={d, h, mn, };

il ne manque pas quelque chose?

que fait la fonction "(*itB).Move(earth);" ?

[Pacorabanix]

Je vais essayer en mettant ++it.

Merci pour ton aide !

Edit : testé avec ++it au lieu de it++ et... ben rien ne change à l'exécution Toujours ce foutu problème.

Juste par rapport à ça : ce n'était absolument pas censé réglé le problème, juste une petite habitude à prendre qui ne mange pas de pain pour optimiser très légèrement ton programme.

[Atharendil]

Ok paco tout conseil est bon à prendre, j'essaierais d'y penser

Mongaulois-> En fait, pour le tableau, c'est juste un pb de recopie dans le forum, le compilo aurait pas aimé sinon
Il manquait ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

int currentTime[4]={d, h, mn, s};

Voilà le constructeur par recopie de SjSl_Balloon :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
SjSl_Balloon::SjSl_Balloon(const SjSl_Balloon& balloon):
timer(balloon.timer), hygrometer(balloon.hygrometer), windGauge(balloon.windGauge), thermometer(balloon.thermometer), humidityValue(balloon.humidityValue), windValue(balloon.windValue), temperatureValue(balloon.temperatureValue), balloonNumber(balloon.balloonNumber)
{
	for(int i=0;i<3;i++)
	{
		position[i] = balloon.position[i];
	}
 
	for(int i=0;i<3;i++)
	{
		windDirection[i] = 0;
	}
	numberOfBalloon++;
}

Et en ce qui concerne la fonction Move, elle modifie la plupart des attributs d'un SjSl_Balloon et en particulier met à jour la position puis fait appel à des méthodes de earth (qui est un SjSl_Atmosphere).

[yan]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

windDirection[i] = 0;

? bizarre pour une recopie...

Sinon ça a l'air ok.

As tu essayé sans appeler Move ?? (y aurait il pas des division par windDirection , des pointeurs ou un parcoure mémoire dans cette fonction?)

Pour bien comprend :
Le premier parcoure de la list ne pose pas de problème,
mais le second oui. C'est bien cela?

Tu est sous visual?