push après un new

Version imprimable

push après un new

Bonjour

Je continue a nettoyer le code de mon application et je butte sur un problème nouveau. Je dois pousser dans une liste trois données et mon programme plante à la deuxième: voici le code:

Code:

1
2
3
4
5
6
7
8
9
10
11
 
  std::list<std::wstring>* wsListEndSentPos;    //list of POS label which point out end of sentence
  std::list<std::wstring>::iterator itSentPos;
  std::list<std::wstring>* wsListEndSentPosDefault; // list of POS label with default value
  m_bOptionProblem = false;
  int pos;
  wsListEndSentPos = new std::list<std::wstring>;
  wsListEndSentPosDefault = new std::list<std::wstring>;
  wsListEndSentPosDefault->push_back(L".");         // Default values of end of sentence
  wsListEndSentPosDefault->push_back(L"?");         // Default values of end of sentence
  wsListEndSentPosDefault->push_back(L"!");         // Default values of end of sentence

qui me donnera le conseil pour tourner cette difficulté, ce sera :ccool:

21/09/2009, 18h28
Davidbrcz

Pourquoi utiliser des pointeurs de listes ?
21/09/2009, 19h33
JulienDuSud

Ca va pas mieux en initialisant explicitement ?

Code:

wsListEndSentPosDefault = new std::list<std::wstring>();

(au passage: je rejoins Davidbrcz pour te demander : pourquoi des objets dynamiques ?)
22/09/2009, 04h51
koala01

Salut,

Je rejoins mes deux compères, surtout en ce qui concerne wsListEndSentPos...

Si l'on s'en réfère au commentaire, ce n'est "rien d'autre" qu'un itérateur pointant sur... la fin de la liste.

Tu devrais donc utiliser l'itérateur ou l'itérateur constant et l'initialiser sur... la fin de la liste (malist.end(), ou, pour suivre l'exemple, vraisemblablement wsListEndSentPosDefault.end(), du moins aussi vite que possible une fois qu'elle existe ;))

Et, de manière générale, les conteneurs de la STL sont, justement, créés pour éviter de manipuler des pointeurs et la gestion dynamique de la mémoire chaque fois que ce n'est pas strictement indispensable.

Il n'y a donc aucune raison à déclarer wsListEndSentPosDefault comme étant un pointeur, à moins qu'il ne soit en réalité question d'avoir une référence vers une liste qui existe par ailleurs.

Mais, dans ce cas, il faut veiller à ce que, lorsque la liste référencée est effectivement détruite (principalement parce que l'objet dans lequel elle aparait effectivement est détruit) le pointeur soit remis à NULL (en n'oubliant pas de tester ce fait avant d'essayer de l'utiliser).

Salut,
Je ne vais pas en rajouter une couche sur l'allocation dynamique de la classe liste même si je n'en pense pas moins. Juste pour dire que ton code marche :
Code:

1 2 3 4 wsListEndSentPosDefault = new std::list<std::wstring>; wsListEndSentPosDefault->push_back(L"."); // Default values of end of sentence wsListEndSentPosDefault->push_back(L"?"); // Default values of end of sentence wsListEndSentPosDefault->push_back(L"!"); // Default values of end of sentence
Ton problème doit être ailleurs. As-tu la fonction complète ?