Discussion :

[LinuxUser]

Bonjour,

J'ai un soucis avec l'allocation d'un vector.

En fait, je n'arrive pas à comprendre pourquoi les exemples suivants ne sont pas équivalents:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
//Exemple 1:
 
std::vector<double> v1;
v1.reserve(5); // edit: v1.resize(5);
v1.push_back(1);
v1.push_back(2);
v1.push_back(3);
v1.push_back(4);
v1.push_back(5);
 
std::vector<double> v2 = std::vector<double>(v1.size(), 0.0);
v2[v2.size()-1] = 3.14; // !!! seg fault

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
//Exemple 2:
 
std::vector<double> v1;
v1.reserve(5); // edit: v1.resize(5);
v1.push_back(1);
v1.push_back(2);
v1.push_back(3);
v1.push_back(4);
v1.push_back(5);
 
std::vector<double> v2;
v2.resize(v1.size, 0.0);
v2[v2.size()-1] = 3.14; // OK

Sachant que dans l'Exemple 1 on aura une seg fault (out of range) si on veut accéder au dernier élément du vector (à savoir v2[4]) et pas dans l'Exemple 2.
Pourtant, je vérifie les caractéristiques du vector v2 qui indiquent: size =5 et capacity = 5.

Je ne comprends absolument pas pourquoi.

[RPGamer]

Salut,

moi je vois pas de problème dans ton premier code et il s'exécute parfaitement chez moi. Quel compilateur est-ce que tu utilise et avec quelles options activées ?
En revanche dans ton 2ème code il manque des parenthèses, il ne compile pas.

[LinuxUser]

Salut,

Il s'agit de Visual C++ (VS 2010).

Pour être plus précis je mix du C# et du C++ en passant par un wrapper C++/CLI.

Le vector "v1" est alloué dans le wrapper CLI puis passé par référence au code C++ qui l'utilise pour initialiser le vector "v2".

En faisant:

Code :

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

std::vector<double> v2 = std::vector<double>(v1.size(), 0.0);

Si ensuite je fais:

Code :

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

1
2
3
4
for(i=0; i < v2.size(); i++)
{
  v2[i] = .... ;
}

Cela va marcher jusque v2.[v2.size()-2] et planter pour v2[v2.size()-1];
Incompréhensible

[Bousk]

Juste comme ça : avec un resize(5) suivi de 5 push_back, ton vector contient 10 éléments et non 5.
Quand ça crash, qu'indique le debugger ? Quelle est la véritable size de v1 ? Et celle de v2 ? As-tu essayé avec une ranged-for loop ?

[LinuxUser]

Pardon, je voulais écrire

Code :

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

v1.reserve(5); // au lieu de v1.resize(5);

Le debug de VS 2010 crash avec un "out of range" alors que juste avant de crasher j'affiche "size" et "capacity" dans le debugguer et elles valent toutes les deux 5 (idem pour v1 et v2).

[Bousk]

Mué, je reste sceptique. Ton problème doit forcément venir d'ailleurs. Comme comment fais-tu ton "transfert de C# vers C++" ? Regarde plutôt du côté de ton wrapper dont on ne sait rien pour le réel problème.
Parce qu'en C++ seulement, le code que tu montres n'as absolument aucune chance de planter, sauf à faire n'importe quoi avec plusieurs threads qui modifient v2. Ou d'avoir bidouiller la stl n'importe comment.

[LinuxUser]

Du côté wrapper, cela ressemble à:

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
	std::vector<double> Wrapper::ListToVector(List<double>^ list)
	{
		std::vector<double> result;
 
		if (list == nullptr) 
		{
			throw gcnew ArgumentNullException(L"list");
		}
		else
		{
			result.reserve(list->Count);
			for each (double elem in list)
			{
				result.push_back(elem);
			}
		}
 
		return result;
	}
/*----------------------------------------------------------------------------*/
void Wrapper::callNativeCode(List<double>^ list)
{
  Native object = Native();
  std::vector<double> v1 = ListToVector(list);
 
  object.calculation(v1);
 
 
}

Du côté code C++ de calculs:

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
void Native::calculation(const std::vector<double>& v1)
{
  unsigned int i = 0;
  // plante dans la boucle for (out of range)
  //std::vector<double> v2= std::vector<double>(v1.size(), 0.0);
  // ne plante pas
  std::vector<double> v2;
  v2.resize(v1.size(), 0.0);
  // plante à la dernière itération
  for(i=0; i < v2.size(); i++)
  {
     v2[i] = 3.14;
  }
 
}

[Médinoc]

C'est bizarre, je ne vois aucune erreur dans ton code C++/CLI...

[bacelar]

Y a un truc que je ne comprends pas, c'est comment avoir une liste .NET de "double" natif ?
Vous êtes sûr qu'on a le même type de double en entré et en sortie ???

[Médinoc]

Oui, à ma connaissance System::Double et le double natif sont traités comme exactement le même type par le Common Type System.