[WIN32] SendMessage qui bloque. **new** code minimal qui buggue

SendMessage qui bloque. **new** code minimal qui buggue [WIN32] - C - Programmation

Marsh Posté le 09-10-2003 à 09:50:14    

salut,
 
 
j'ai un probleme que je n arrive pas a resoudre. J'ai une fonction qui me permet de logguer dans un ListCtrl ce que je veux. Ma fonction qui insere dans le ListCtrl est la suivante :
 

Code :
  1. void LogToWindow(DWORD dwLevel) {
  3. if (logWindow != NULL) {
  4.  if (plogLast) {
  5.   if (plogLast->dwLevel & logLevelFilter) {
  7.    char szBuffer[64];
  8.    LVITEM lvi;
  10.                                                           sprintf(szBuffer,"%d",plogLast->dwID);
  12.    lvi.mask = LVIF_TEXT;
  13.    lvi.iItem = 0;
  14.    lvi.iSubItem = 0;
  15.    lvi.pszText = szBuffer;
  16.    lvi.cchTextMax = 64;
  18.    SendMessage(logWindow,LVM_INSERTITEM,0,(LPARAM)&lvi);
  22.   }
  23.  }
  24. }
  25. }


 
 
cela marche tres bien a 99% du temps. Mais sur un cas bien precis la fonction SendMessage bloque indefiniment.
 
 
Ce cas est le suivant :
 
J'ai un bouton de type ON/OFF , j appuie une fois dessus ca declenche une action, une fois a nouveau ca la stoppe.  
     - ON : je cree un thread, je mets une variable V a TRUE et celui ci m'envoie a intervalle regulier (200 ms)par le biais d une boucle sur V un caractere sur le port serie, puis un 2eme (20  mn apres le premier). Chaque ecriture sur le port serie est logguée, et ca apparait tres bien dans la fenetre de log.
     - OFF : Je mets ma variable V a FALSE ce qui a pour but d arreter la boucle d envoi des caractere, puis envoie un caractere  different sur le port (caractere de stop). Sachant que toutes les ecritures sont logguées , je loggue egalement celle ci, le log fichier, memoire fonctionne bien mais des que je loggue dans la fenetre de log mon SendMessage d insertion de ligne bloque. C est aussi simple que ca.
 
 
Pendant le ON, je peux envoyer autant de caractere que necessaire, ca ne bloque pas. Mais des que je stoppe ,ca bloque.
 
 
Quelqu un aurait il une idee ?
 
 
merci :)


Message édité par xilebo le 16-10-2003 à 13:42:15
Reply

Marsh Posté le 09-10-2003 à 09:50:14   

Reply

Marsh Posté le 09-10-2003 à 09:55:52    

Essaye de faire un PostMessage à la place

Reply

Marsh Posté le 09-10-2003 à 10:05:27    

SendMessage est une méthode bloquante (ne sort que quand le message a été traité)
Tu peux effectivement utiliser PostMessage qui n'est pas bloquant.

Reply

Marsh Posté le 09-10-2003 à 10:10:12    

Kyle_Katarn a écrit :

Essaye de faire un PostMessage à la place


+1
SendMessage est synchrone, tant que le message que tu envoies n'est pas traité, ton programme est bloqué. Pour peu que tu aies bcp de messages en attente, ça le fera pas.
 
PostMessage est asynchrone, il n'attend pas que le message soit traité.
 
edit: [:benou_grilled]


Message édité par Harkonnen le 09-10-2003 à 10:10:49

---------------
J'ai un string dans l'array (Paris Hilton)
Reply

Marsh Posté le 09-10-2003 à 10:18:49    

De plus, faut faire gaffe avec PostMessage:
Tu passes un pointeur vers ta structure lvi, quand ton message sera traité, cette structure n'existera peut etre plus...

Reply

Marsh Posté le 09-10-2003 à 10:25:29    

oui mais si je fais un postmessage, le message ne sera alors pas traité ? ou il sera mis a la queue ? Et ca peut bloquer meme si seulement 3 ou 4 message (enfin moins de 20 c sur) sont envoyés.

Reply

Marsh Posté le 09-10-2003 à 11:01:32    

Bon ca ne fonctionne pas du tout avec PostMessage et ca plante meme (surement a cause du fait que ma structure LVITEM n existe plus en memoire, mais je n ai pas le choix)
 
 
De toute facon, si le SendMessage bloque a l infini c est qu il y a un probleme mais je ne vois pas lequel.  
 
Peut etre du a l'appel SendMessage dans un thread ? en tout cas j ai vérifié la valeur du Handle de fenetre, il est correct. Je ne comprends vraiment pas ce qu il se passe. Et vu que je n ai pas le code source de SendMessage (merci  ms) je ne peux pas voir quel est le probleme et pourquoi ca boucle.
 

Reply

Marsh Posté le 14-10-2003 à 16:06:33    

bon j ai tenté de faire un code minimal qui fait la meme chose et ca plante de la meme facon.
 
C est tres simple: j'ai une boite de dialogue avec un bouton (check box) et un ListCtrl (MFC)
 
le button :  
lorsqu on appuie sur le bouton , il met une variable a 1 et cree un thread .
lorsqu on réappuie sur le bouton, il met cette meme variable a 0 et attend que le thread stoppe.
 
 
le thread :
 
il boucle sur cette variable, tant qu elle est a un, il insere une ligne dans le listctrl PUIS s endort 500 ms. Lorsqu il sort de la boucle il insere a nouveau une ligne (que je differencie par le texte inséré) et la il bloque sur cette insertion.
 
 
 
 
Comment ca se fait ? y a t il un mecanisme que j ai pas compris ?
 
 
 
je vous mets egalement le code de cette application minimale.
 
 

Code :
  1. // TESTDlg.cpp : implementation file
  2. //
  4. #include "stdafx.h"
  5. #include "TEST.h"
  6. #include "TESTDlg.h"
  8. #ifdef _DEBUG
  9. #define new DEBUG_NEW
  10. #undef THIS_FILE
  11. static char THIS_FILE[] = __FILE__;
  12. #endif
  14. /////////////////////////////////////////////////////////////////////////////
  15. // CAboutDlg dialog used for App About
  17. class CAboutDlg : public CDialog
  18. {
  19. public:
  20. CAboutDlg();
  22. // Dialog Data
  23. //{{AFX_DATA(CAboutDlg)
  24. enum { IDD = IDD_ABOUTBOX };
  25. //}}AFX_DATA
  27. // ClassWizard generated virtual function overrides
  28. //{{AFX_VIRTUAL(CAboutDlg)
  29. protected:
  30. virtual void DoDataExchange(CDataExchange* pDX);    // DDX/DDV support
  31. //}}AFX_VIRTUAL
  33. // Implementation
  34. protected:
  35. //{{AFX_MSG(CAboutDlg)
  36. //}}AFX_MSG
  37. DECLARE_MESSAGE_MAP()
  38. };
  40. CAboutDlg::CAboutDlg() : CDialog(CAboutDlg::IDD)
  41. {
  42. //{{AFX_DATA_INIT(CAboutDlg)
  43. //}}AFX_DATA_INIT
  44. }
  46. void CAboutDlg::DoDataExchange(CDataExchange* pDX)
  47. {
  48. CDialog::DoDataExchange(pDX);
  49. //{{AFX_DATA_MAP(CAboutDlg)
  50. //}}AFX_DATA_MAP
  51. }
  53. BEGIN_MESSAGE_MAP(CAboutDlg, CDialog)
  54. //{{AFX_MSG_MAP(CAboutDlg)
  55.  // No message handlers
  56. //}}AFX_MSG_MAP
  57. END_MESSAGE_MAP()
  59. /////////////////////////////////////////////////////////////////////////////
  60. // CTESTDlg dialog
  62. CTESTDlg::CTESTDlg(CWnd* pParent /*=NULL*/)
  63. : CDialog(CTESTDlg::IDD, pParent)
  64. {
  65. //{{AFX_DATA_INIT(CTESTDlg)
  66.  m_dwOnSend = 0;
  67. //}}AFX_DATA_INIT
  68. // Note that LoadIcon does not require a subsequent DestroyIcon in Win32
  69. m_hIcon = AfxGetApp()->LoadIcon(IDR_MAINFRAME);
  70. }
  72. void CTESTDlg::DoDataExchange(CDataExchange* pDX)
  73. {
  74. CDialog::DoDataExchange(pDX);
  75. //{{AFX_DATA_MAP(CTESTDlg)
  76.  // NOTE: the ClassWizard will add DDX and DDV calls here
  77. //}}AFX_DATA_MAP
  78. }
  80. BEGIN_MESSAGE_MAP(CTESTDlg, CDialog)
  81. //{{AFX_MSG_MAP(CTESTDlg)
  82. ON_WM_SYSCOMMAND()
  83. ON_WM_PAINT()
  84. ON_WM_QUERYDRAGICON()
  85. ON_BN_CLICKED(IDC_ON, OnOn)
  87. //}}AFX_MSG_MAP
  88. END_MESSAGE_MAP()
  90. /////////////////////////////////////////////////////////////////////////////
  91. // CTESTDlg message handlers
  93. BOOL CTESTDlg::OnInitDialog()
  94. {
  95. CDialog::OnInitDialog();
  97. // Add "About..." menu item to system menu.
  99. // IDM_ABOUTBOX must be in the system command range.
  100. ASSERT((IDM_ABOUTBOX & 0xFFF0) == IDM_ABOUTBOX);
  101. ASSERT(IDM_ABOUTBOX < 0xF000);
  103. CMenu* pSysMenu = GetSystemMenu(FALSE);
  104. if (pSysMenu != NULL)
  105. {
  106.  CString strAboutMenu;
  107.  strAboutMenu.LoadString(IDS_ABOUTBOX);
  108.  if (!strAboutMenu.IsEmpty())
  109.  {
  110.   pSysMenu->AppendMenu(MF_SEPARATOR);
  111.   pSysMenu->AppendMenu(MF_STRING, IDM_ABOUTBOX, strAboutMenu);
  112.  }
  113. }
  115. // Set the icon for this dialog.  The framework does this automatically
  116. //  when the application's main window is not a dialog
  117. SetIcon(m_hIcon, TRUE);   // Set big icon
  118. SetIcon(m_hIcon, FALSE);  // Set small icon
  120. CListCtrl *pCtrl ;
  122. pCtrl = (CListCtrl *)GetDlgItem(IDC_LIST);
  124. pCtrl->InsertColumn(0,"message",LVCFMT_LEFT,200);
  126. return TRUE;  // return TRUE  unless you set the focus to a control
  127. }
  129. void CTESTDlg::OnSysCommand(UINT nID, LPARAM lParam)
  130. {
  131. if ((nID & 0xFFF0) == IDM_ABOUTBOX)
  132. {
  133.  CAboutDlg dlgAbout;
  134.  dlgAbout.DoModal();
  135. }
  136. else
  137. {
  138.  CDialog::OnSysCommand(nID, lParam);
  139. }
  140. }
  142. // If you add a minimize button to your dialog, you will need the code below
  143. //  to draw the icon.  For MFC applications using the document/view model,
  144. //  this is automatically done for you by the framework.
  146. void CTESTDlg::OnPaint()
  147. {
  148. if (IsIconic())
  149. {
  150.  CPaintDC dc(this); // device context for painting
  152.  SendMessage(WM_ICONERASEBKGND, (WPARAM) dc.GetSafeHdc(), 0);
  154.  // Center icon in client rectangle
  155.  int cxIcon = GetSystemMetrics(SM_CXICON);
  156.  int cyIcon = GetSystemMetrics(SM_CYICON);
  157.  CRect rect;
  158.  GetClientRect(&rect);
  159.  int x = (rect.Width() - cxIcon + 1) / 2;
  160.  int y = (rect.Height() - cyIcon + 1) / 2;
  162.  // Draw the icon
  163.  dc.DrawIcon(x, y, m_hIcon);
  164. }
  165. else
  166. {
  167.  CDialog::OnPaint();
  168. }
  169. }
  171. // The system calls this to obtain the cursor to display while the user drags
  172. //  the minimized window.
  173. HCURSOR CTESTDlg::OnQueryDragIcon()
  174. {
  175. return (HCURSOR) m_hIcon;
  176. }
  178. void CTESTDlg::OnOn()
  179. {
  180. if ( ((CButton *)GetDlgItem(IDC_ON))->GetCheck() ) { // stopper le thread
  182.   m_dwOnSend = 1; // redemarrer le thread
  183.   hSendThread = CreateThread(NULL, 0, SendThreadFunc,this, 0, &SendThreadID);
  187. }
  188. else {
  189.   m_dwOnSend = 0;
  190.   WaitForSingleObject(hSendThread,INFINITE);
  191. }
  193. }
  198. DWORD WINAPI SendThreadFunc(LPVOID pParam) {
  199. CTESTDlg *pDlg = (CTESTDlg *)pParam;
  200. static int i = 0;
  201. CString tmp;
  203. CListCtrl *pCtrl = (CListCtrl *)pDlg->GetDlgItem(IDC_LIST);
  205. while (pDlg->m_dwOnSend) {
  207.  tmp.Format("send : %d",i++);
  209.  pCtrl->InsertItem(0,tmp,0);
  210.  Sleep(500);
  211. }
  213. tmp.Format(" stop : %d",i++);
  214. pCtrl->InsertItem(0,tmp,0);
  217. return 0;
  218. }


 

Reply

Marsh Posté le 14-10-2003 à 17:33:53    

j'ai pas tout lu (trop de mfc), et je vois rien dans le thread qui peut bloquer indéfiniment, mais essaye d'éviter le "WaitForSingleObject(hSendThread,INFINITE);" et remplace le INFINITE par disons 1 ou 2 sec. ensuite tu testes la valeur de retour : par ex si ca te renvoie WAIT_TIMEOUT (timeout écoulé) bah t'as plus qu'a killer le thread (TerminateThread), mais franchement ca serait bizarre que ca soit le thread qui bloque comme ca...

Reply

Marsh Posté le 14-10-2003 à 17:41:04    

bah mfc j ai pas trop le choix mais ca change rien , effectivement les 2 fonctions interessantes sont OnOn() et SendThreadFunc() (le reste on s en fout). ET si je vire le WaitSingleObject ca passe (normal vu que j attends pas). Effectivement c est mon thread qui bloque parceque tout simplement , l appel a InsertItem (qui est en fait un sendmessage) bloque. donc thread bloqué -> processus principal egalement bloqué.
 
 
Mon réel problème est : pourquoi mon InsertItem(stop) bloque a ce moment la alors que si je le mets en commentaire, AUCUN des autres InsertItem ne bloque.
 
 
 
 
Je vous mets au défi : essayer de faire une appli minimale (meme sans MFC) qui insere des lignes dans un ListCtrl tous les X millisecondes par l'intermediaire d'un thread, et bien ca bloquera .

Reply

Marsh Posté le 14-10-2003 à 17:41:04   

Reply

Marsh Posté le 14-10-2003 à 17:58:15    

moui pourquoi pas... pas grand chose a faire la.
 
sinon pour resoudre ton pb, tu fais un beau TerminateThread et puis voila...

Reply

Marsh Posté le 14-10-2003 à 18:13:15    

bon... a priori tu aurais raison, ca bloque sur la derniere insertion, apres la boucle while du thread.
pour infos : proj en win32, en C, et insertion dans une listbox.
 
vais regarder ca de plus près... y a pas de raison pour que ca marche pas.

Reply

Marsh Posté le 14-10-2003 à 18:23:45    

je viens de comprendre pourquoi :  (attention c'est complexe a expliquer)
 
apres la mise a 0 de la variable qui permet de dire au thread de se finir, tu fais un WaitForSingleObject. Donc la boucle while du thread se termine, et l'appel a SendMessage s'effectue. Or le WaitForSingleObject (qui est un peu l'équivalent d'un Sleep) se trouve dans la DlgProc, et bloque donc tous les messages en queue, ce qui bloque le SendMessage de la listbox, et donc tu te retrouves dans une position ou le SendMessage du thread attend que le msg soit traité, et ta DlgProc attend que le thread se termine => on appelle ca un deadlock je crois : les 2 threads attendent un évenement de la part de l'autre thread, et sont donc bloqués...
 
remplacer le dernier SendMessage par un PostMessage résoud la situation.
 
edit:
en fait remplacer le Send par un Post c'est pas forcèment la bonne solution, pasque bloquer la DlgProc c'est jamais bon. tu devrais donc eviter le WaitForSingleObject, et trouver un autre moyen, selon tes besoins.
En mettant juste la variable a 0 ca suffirait pas ?


Message édité par Konar le 14-10-2003 à 18:33:03
Reply

Marsh Posté le 14-10-2003 à 18:39:26    

Deux suggestions :

  • Pourquoi ne pas utiliser un évenement HEVENT plutôt qu'une variable pour indiquer à la thread de se terminer.

Ton code dans ta thread serait du genre :

Code :
  1. while( WaitForSingleObject( pDlg->m_hStopThread, 500 ) == WAIT_TIMEOUT )
  2. {
  3. /*... mise à jour de la liste ... */
  4. }


 

  • De même, je crois qu'il est déconseillé de mettre à jour des contrôles avec des PostMessage (dans la documentation, Microsoft parle systématiquement de SendMessage). Donc ce que tu peut faire, c'est faire un message privé que ta thread va envoyer via PostMessage à ta boîte de dialogue et dans le traîtement de ce message ta boîte de dialogue ajoute un élement à ta liste.


---------------
each day I don't die is cheating
Reply

Marsh Posté le 14-10-2003 à 18:48:15    

Ca change pas le pb de la DlgProc bloquée.
ca revient au même de virer le SendMessage (ou le InsertItem dans son cas) situé après le while du thread, pour le mettre apres le WaitForSingleObject de l'évenement du clic de bouton. La DlgProc sera bloquée au max 500 ms (quoique c'est pas critique non plus...)

Reply

Marsh Posté le 14-10-2003 à 18:55:03    

merci beaucoup , je n avais pas pensé que le WaitForSingleObject bloquait le traitement des messages. Je vais donc essayer de trouver une solution a ce probleme.
 
eh beh , c est pas facile a détecter ce genre de probleme.
 
Encore merci de votre aide , ca m'aide beaucoup :)... je vous dirai demain si j ai reussi a trouver une autre solution.
 
edit : konar, ton explication etait tres claire ;)


Message édité par xilebo le 14-10-2003 à 18:57:23
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed