Discussion :

[poukill]

Bonjour à tous, je me demandais si ce code là était correct? peut-on renvoyer une paire d'éléments comme ceci? La variable ne va t-elle pas mourir à la fin de la fonction (et donc le programma qui l'appelle va se planter lamentablement).
Faut-il détruire la paire?

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45

 
template <typename T> std::pair<float, float> pointInterpolation(std::pair <int, int>& H, 

std::vector< std::pair <int, int> >& depart, const cimg_library::CImg<T>& u, constfloat coeff_mul) {
std::pair<
float, float> H_v; //vecteur déplacement pour le point à remplir

int a = depart.size();

/* Allocation dynamique de mémoire */
std::vector< std::pair <int, int> > pointschoisis;

// calcul des points nécessaires pour l'interpolation
nProchePoints(H, depart, NBPOINTSCHOISIS, pointschoisis);
 
int x,y; // point de controle regardé
float d; // distance entre le point courant et le point de controle
float D=0; // distance totale
float v_x,v_y; // vecteur déplacement du point de controle

 
 
 

for (int i=0; i<NBPOINTSCHOISIS; i++ ) {
x = (pointschoisis.begin() +i)->first;
y = (pointschoisis.begin() +i)->second;
v_x = u(x,y,0);
v_y = u(x,y,1);
d = dist(H, pointschoisis[i]);
D+=1/d; 
 
 
// pour la normalisation
H_v.first += v_x / d;
H_v.second += v_y / d;

 
 
}
H_v.first = H_v.first/D;
H_v.second = H_v.second/D;
return H_v;

 
 
}

Merci à vous !

[Matthieu Brucher]

C'est le principe du passage par valeur, la variable sera recopiée.

[poukill]

C'est le principe du passage par valeur, la variable sera recopiée.

C'est mieux ou pas que de renvoyer un float* ?
Vu que dans ce cas, il n'y a pas de copie....

[Matthieu Brucher]

Sauf que pour un float*, tu as de l'allocation dynamique, alors avant que tu constates qu'effectivement c'est critique au niveau temps, utilise le passage par valeur.

[poukill]

Mais là aussi il y a de l'allocation dynamique non?
Sinon, ça ne marcherait pas.

Exemple

Code :

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

1
2
3
4
5
 
int *maFonction (....) {
     int H[5]={0,1,2,3,4};
     return(H);
}

plante car la variable est détruite avant même d'être renvoyée, non?

[Pragmateek]

Mais là aussi il y a de l'allocation dynamique non?
Sinon, ça ne marcherait pas.

Exemple

Code :

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

1
2
3
4
5
 
int *maFonction (....) {
     int H[5]={0,1,2,3,4};
     return(H);
}

plante car la variable est détruite avant même d'être renvoyée, non?

Pour que ça marche il faut allouer dynamiquement H.

[Laurent Gomila]

Mais là aussi il y a de l'allocation dynamique non?

Non. Tu en vois où ?

Exemple

Dans ton exemple tu renvoies un pointeur sur le tableau, pas une copie de celui-ci (ce qui n'est d'ailleurs pas réalisable directement pour ce genre de tableaux).

[Laurent Gomila]

Il faut arrêter de voir la recopie comme un démon tout droit sorti de l'enfer ; au contraire c'est bien souvent la meilleure solution.

Ca posera très rarement problème, encore plus pour une simple paire de flottants.

[poukill]

Il faut arrêter de voir la recopie comme un démon tout droit sorti de l'enfer ; au contraire c'est bien souvent la meilleure solution.

Ca posera très rarement problème, encore plus pour une simple paire de flottants.

Oui, je comprend bien Laurent, je m'y mets!!! C'est d'ailleurs sur les conseils de Miles et autres que j'ai tout recodé dans cette optique!
Preuve de bonne volonté non?