Discussion :

[JeanNoel53]

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 :

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

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

[Davidbrcz]

Pourquoi utiliser des pointeurs de listes ?

[JulienDuSud]

Ca va pas mieux en initialisant explicitement ?

Code :

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

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

(au passage: je rejoins Davidbrcz pour te demander : pourquoi des objets dynamiques ?)

[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).

[3DArchi]

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 :

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

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 ?