Discussion :

[Seabirds]

Salut à toutes et à tous !

Honte à moi, je ne comprends pas ce qui se passe dans la boucle for suivante : la boucle n'effectue que la première itération (le reste, mystère) et la suite du code s'exécute normalement.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
         unsigned int time = 1;
         unsigned id = 0;
         int x =1;
 
         for(int y = 1; y < 4; y++){
                auto coords = make_pair(x, y);
		std::cout << "in loop :" << environment(id, coords, time) << std::endl;
	}
 
	std::cout << "in main : " << environment(id, std::make_pair(x, 1), time) << std::endl;
	std::cout << "in main : " << environment(id, std::make_pair(x, 2), time) << std::endl;
	std::cout << "in main : " << environment(id, std::make_pair(x, 3), time) << std::endl;

Qu'est ce qui peut faire que l'itération "manuelle" réussisse quand la boucle for plante (sans vraiment planter d'ailleurs...) ?

environnement est un objet qui implémente l'opérateur () : il trouve un jeu de donnée dans une map et délègue la lecture au jeu de donnée en question qui implémente l'opérateur () suivant (implémentation suspicieuse : forcément, visez qui l'a écrit ) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
	value_type operator()(coord_type p, time_type t) const {
		GDALRasterBand  *poBand = resource->GetRasterBand( int(t) );
		if(!poBand)
			throw std::string("unable to access band");
 
		float *val;
		int x = p.first;
		int y = p.second;
		poBand->RasterIO( GF_Read, x, y, 1, 1, val, 1, 1, GDT_Float32, 0, 0 );
		if(!val)
			throw std::string("unable to read data");
 
		return double(*val);
	}

J'essaie de mon mieux, mais oui c'est crado, l'utilisation de l'API de GDAL ne m'aidant malheureusement pas !
Merci !

[Médinoc]

Que fait la fonction environment()?
Edit: Le message a été édité.

[Seabirds]

environnement est un foncteur qui va chercher dans une map d'ID un jeu de donnée qui utilise l'API d'une bibliothèque spatiale pour renvoyer la valeur de l'environnement aux coordonnées (x,y) au temps t Je modifie le code pour que ce soit plus lisible. As-tu besoin du code des fonctions ?

[Médinoc]

Ta bibliothèque, est-elle possiblement asynchrone?
Peux-tu poster exactement la sortie?

[Seabirds]

Asynchrone... Je vais googliser.
Par contre, quelques bidouilles m'ammènent à penser qu'il y a modification d'un élément de la paire là où on n'en veut pas. Voyez plutôt :

Code :

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

1
2
3
4
5
6
7
	for(int j = 1; j < 4;){
		int temp = j;
		std::cout << "temp before : " << temp << std::endl;
		std::cout << "value =" << environment(0, std::make_pair(1, temp), 1) << std::endl;
		std::cout << "temp after :" << temp << std::endl;
		j = j+1;
	}

renvoie :

Code :

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

1
2
3
4
5
6
7
8
9
temp before : 1
value =99
temp after :1120272384
temp before : 2
value =101
temp after :1120534528
temp before : 3
value =-inf
temp after :-8388608

Donc y'a bien un endroit où la paire de coordonnées est modifiée n'est-ce pas ? J'imagine que c'est à l'intérieur de la fonction RasterIO. Ce qui m'étonne c'est que la paire de coordonnées est déclarée constante...

[foetus]

Pourtant d'après la documentation de std::make_pair, les paramètres sont passés par valeur en C++98

À moins que tu utilises le C++11, qui, dans ce cas, "déplace, transfert - move" les 2 paramètres

[Médinoc]

Et en plus, make_pair n'est pas censé contenir une référence vers les éléments qu'on passe à make_pair()!

Tu peux essayer en spécifiant explicitement makepair<int, int>(), mais ça me parait vraiment bizarre qu'il y ait même besoin de le faire...

D'autant plus que ton operator() ne fait que lire les valeurs de la paire...

Edit: Doublé par foetus