Discussion :

[leame]

bonjour,

dans mon projet, j'ouvre une CDialog, qui est composé d'une ListBox.
dans mon application, je rempli une liste de CString. et c'est cette liste que je vais mettre dans la listBox.
mon code:

dans ma class application:

Code :

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

1
2
3
4
5
6
7
 
 
[...]
CDiagFichierPlusRecent diag;
diag.setList(listeFichierPlusRecent);
diag.DoModal();
[...]

dan ma classe CDiag

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
 
void CDiagFichierPlusRecent::setList(list<CString> listeFichier)
{
	//CString str;
	list<CString>::iterator it;
	for(it=listeFichier.begin(); it!=listeFichier.end();it++)
	{
		//char * texte= (*it).GetBuffer((*it).GetLength()+1);
		//str = (*it);
	            //str.Format("%s.",(*it));
                         // str.Format(_T("blabla"));
                         // str= _T("blabla");
		//m_listBox.AddString(str);
		//m_listBox.AddString(texte);
		m_listBox.AddString((*it));
	}
 
}

mon probleme.. le addString ne marche pas. vous avez en commentaire les différents essais que j'ai effectué.
mais à chaque ajout j'ai un message d'erreur qui apparait et ma listBox reste vide...
je pense que c'est un probleme de conversion LPCSTR -->CString..
j'ai déjà eu plusieurs fois le problème de conversion mais jusque la j'ai pu recréer mes propres méthodes... sauf que la je seche...

[farscape]

il serait judicieux de passer listfichier par référence et non par copie.
tu rentres dans la boucle d'itération ?
tu pourrais rajouter un TRACE des différentes valeurs avant le addstring pour vérifier le contenu.

[leame]

il serait judicieux de passer listfichier par référence et non par copie.
tu rentres dans la boucle d'itération ?
tu pourrais rajouter un TRACE des différentes valeurs avant le addstring pour vérifier le contenu.

ma liste de fichier est rempli et oui je rentre dans la boucle, lorsque j'affecte mon (*it) a str par exemple, str prend bien la valeur de (*it). c'est seulement au moment du AddString(..) qu'une boite de dialogue apparait. le titre de la boite de dialogue c'est " Microsoft Visual C++ Debug Library. " apres dans la boite de dialogue il y a marqué: "Debut Assertion failed" etc.

mais c'est pas une erreur qui arrete mon application.. il y a trois bouton "Ignorer", "recommencer", "abandonner". si j'appuie sur Ignorer, mon programme continue.. il entre a nouveau dans la boucle, avec l'element suivant de ma liste.. il le reaffecte bien a la variable... et a nouveau, au AddString RE message d'erreur.. et comme ca jusqu'à ce que le parcours de ma liste soit fini. ensuite ma boite de dialogue apparait mais avec rien dans la listBox

[TheGzD]

t'es sûr que ta liste est vraiment créée ?

[leame]

t'es sûr que ta liste est vraiment créée ?

que ma liste est créée? tu veux dire la "listeFichier" que j'ai en parametre?
si c'est celle là oui, je suis sure, j'ai mis des point d'arret j'ai regardé, et ma liste contient bien des CString.

j'en suis sure aussi parce que je passe dans la boucle.. hors si ma liste etait vide elle n'y entrerai pas..

[farscape]

tu peux poster la ligne de code qui est affichée pour l'assertion d'erreur ?

[leame]

tu peux poster la ligne de code qui est affichée pour l'assertion d'erreur ?

je comprend pas ce que je dois poster....
je mets en piece l'erreur qui s'affiche...

[farscape]

tu presses le bouton recommencer et tu verras le code en question qui provoque l'erreur.
dans mon tuto le dernier et celui de vc6 il y a un chapitre sur le debugage qui explique tout cela..

[leame]

Code :

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

1
2
3
4
 
AFXWIN_INLINE int CListBox::AddString(LPCTSTR lpszItem)
	{ ASSERT(::IsWindow(m_hWnd)); return (int)::SendMessage(m_hWnd, LB_ADDSTRING, 0, (LPARAM)lpszItem); }
_AFXWIN_INLINE int CListBox::DeleteString(UINT nIndex)

le point d'arret se situe au niveau du "ASSERT". c'est dans la classe afxwin2.h

[farscape]

ben voila le problème est résolu non ?
m_hwnd est null ce qui veut dire que ta lisbox n'est pas initialisée graphiquement .
voir faq :http://cpp.developpez.com/faq/vc/ind...nge#UpdateData

[leame]

ben voila le problème est résolu non ?
m_hwnd est null ce qui veut dire que ta lisbox n'est pas initialisée graphiquement .
voir faq :http://cpp.developpez.com/faq/vc/ind...nge#UpdateData

c'est ce que ca veut dire...
vais voir ton lien

[farscape]

Toute variable contrôle déclarée dans une fenêtre dialogue (CDialog ou CFormView etc ..) qui ne fera pas partie de la fonction DoDataExchange ne sera pas « subclassée » son handle de fenêtre sera égal à NULL et son utilisation provoquera une assertion.

[TheGzD]

que ma liste est créée? tu veux dire la "listeFichier" que j'ai en parametre?
si c'est celle là oui, je suis sure, j'ai mis des point d'arret j'ai regardé, et ma liste contient bien des CString.

j'en suis sure aussi parce que je passe dans la boucle.. hors si ma liste etait vide elle n'y entrerai pas..

je parlais de ta list box

[leame]

Toute variable contrôle déclarée dans une fenêtre dialogue (CDialog ou CFormView etc ..) qui ne fera pas partie de la fonction DoDataExchange ne sera pas « subclassée » son handle de fenêtre sera égal à NULL et son utilisation provoquera une assertion.

bah oui mais elle fait partie de DodataExchange

Code :

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

1
2
3
4
5
6
 
void CDiagFichierPlusRecent::DoDataExchange(CDataExchange* pDX)
{
	CDialog::DoDataExchange(pDX);
	DDX_Control(pDX, IDC_LIST1, m_listBox); // ici!!
}

.....................

j'ai essayé de mettre le UpdateData(False);
ca me lance un message:

une opération non prise en charge a été tentée

..

[TheGzD]

Seulement tu fais ceci :

Code :

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

1
2
3
CDiagFichierPlusRecent diag;
diag.setList(listeFichierPlusRecent); //ici tu utilises des ressources graphiques
diag.DoModal(); // ici tu instancies les ressources graphiques : tu vois le soucis ?

passe le setList dans le OnInitDialog de CDiagFichierPlusRecent et je pense que ton pb disparaitra.

[leame]

Seulement tu fais ceci :

Code :

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

1
2
3
CDiagFichierPlusRecent diag;
diag.setList(listeFichierPlusRecent); //ici tu utilises des ressources graphiques
diag.DoModal(); // ici tu instancies les ressources graphiques : tu vois le soucis ?

passe le setList dans le OnInitDialog de CDiagFichierPlusRecent et je pense que ton pb disparaitra.

en effet en faisant cela ça marche... mais pourquoi de ma façon ca ne marche pas??
parce que là.. ça m'oblige à faire de ma liste une variable membre de la classe application pour pouvoir la récupérer alors que j'en ai besoin que pour une simple vérification...

.................;

Merci farscape!!
Merci TheGzD!!

[farscape]

mince je n'avais pas fait attention a la première portion de code du premier post.

pour répondre à ta question:
http://cpp.developpez.com/faq/vc/ind...ToWorkWithCtrl

ceci dit je maintiens ma remarque sur le passage par référence à la place du passage par valeur.
pourquoi provoquer un copie de la list surtout si elle est importante ...
ça devrait même etre par defaut une réference constante.

[leame]

ceci dit je maintiens ma remarque sur le passage par référence à la place du passage par valeur.
pourquoi provoquer un copie de la list surtout si elle est importante ...
ça devrait même etre par defaut une réference constante.

sans raison précise.. je passe par référence quand ma méthode modifie cette valeur. sinon..... je mets pas par référence... pourquoi c'est pas bien de mettre juste en paramètre sans passage par référence?

[TheGzD]

tu perds du temps de calcul et tu utilises plus de mémoire

[farscape]

il me semble avoir répondu:

pourquoi provoquer une copie de la list surtout si elle est importante ...

ta liste va être recrée par itération sur celle d'origine avec le constructeur de copie.
en gros pourquoi faire perdre du temps et de la mémoire à ton programme ?