C++

probleme de pointeur

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : probleme de pointeur

dreamkiller

Salut a tous bon voila j'ai un probleme de pointeur apparament il est mal initialisé et me plante dans les pates des que je m'en sert

Code :

INT ThreadHorloge(LPVOID pParam)
{
CHologe* pObject = (CHologe*)pParam;
CTime horloge /*= CTime::GetCurrentTime()*/;
int nHeure;
int nMin;
int nSec;
while(pObject->m_bFin!=true)
{
horloge = CTime::GetCurrentTime();
nHeure=horloge.GetHour();
nMin=horloge.GetMinute();
nSec=horloge.GetSecond();
pObject->m_strHeure.Format("%d",nHeure);
pObject->m_strMin.Format("%d",nMin);
pObject->m_strSec.Format("%d",nSec);
pObject->m_pDialog->Invalidate(); //ici
Sleep(1000);
}
return 1;
}

declaration du pointeur

Code :

CPCControleDlg *m_pDialog;

initialisation

Code :

m_pDialog=NULL;

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

Publicité

SomeBugsInMe

life suxx

Quand tu arrives dans ta fonction m_pDialog est-il toujours NULL ?
Si c'est le cas, tu ne peux pas faire m_pDialog->Invalidate()

tu peux faire :

Code :

if (p_Object->m_pDialog) pObject->m_pDialog->Invalidate();

dreamkiller

ouai il est tj nul

ouai mais t as solution ne sert a rien ici il ne fait jamais de invalidate

Message édité par dreamkiller le 07-04-2005 à 14:44:30

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

IrmatDen

salut,
Il me semble que c'est le même topic que là, non ?
http://forum.hardware.fr/forum2.ph [...] subcat=386

dreamkiller

ouai mais on mas pas rtepondu

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

IrmatDen

tiens je t'ai trouvé ça : http://www.mvps.org/vcfaq/mfc/11.htm et le lien qu'il y a en bas de la page est aussi à voir. Pour mieux comprendre, utilise la msdn...

dreamkiller

ok j'essaie

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

dreamkiller

ouai mais pour le hwnd j'ai besoin d'un pointeur sur la fenetre d'affichage donc retour case depart

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

IrmatDen

au lieu de faire de pParam un pointeur vers ta classe CHorloge, tu en fais un pointeur vers un hWnd...

dreamkiller

pourquoi passé par pParam??? moiu j'ai tapé ca

Code :

::PostMessage(pObject->m_pDialog->hwnd, MY_WM_INVALIDATE, (WPARAM)0, (LPARAM)0);

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

Publicité

IrmatDen

aussi... du moment que ça marche c'est l'essentiel... Mais le fait de passer un pointeur vers un HWND t'empêche d'appeler une quelconque fonction membre ayant une incidence sur le thread de l'UI.

bjone

Insert booze to continue

pourquoi un thread ?

tu crées un Timer qui va t'envoyer un message qui va te faire rafraichir ton bouzin.

Message édité par bjone le 07-04-2005 à 17:12:13

dreamkiller

IrmatDen a écrit :

aussi... du moment que ça marche c'est l'essentiel... Mais le fait de passer un pointeur vers un HWND t'empêche d'appeler une quelconque fonction membre ayant une incidence sur le thread de l'UI.

ouai mais le prob c'est que justement c'est mon pointeur qui plante!?!

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

dreamkiller

dans ce cas la je fais mon horloge aussi dans un timer c'est pas une solution

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

IrmatDen

dreamkiller a écrit :

ouai mais le prob c'est que justement c'est mon pointeur qui plante!?!

D'après ton code, non c'est pas ton pointeur qui plante mais l'utilisation que tu en fais !!
Et le coup du timer, pourquoi c'est pas une solution ?

bjone

Insert booze to continue

si c'est LA solution.

tu as un évenement qui t'arrives périodiquement quand tu dois actualiser quelque chose.

ce que tu fais est ultra-classique y'a pas besoin de thread avec un sleep dedans pour actualiser une horloge dans une interface.

dreamkiller

je persiste a croire que c'est mon pointeur

pObject->m_pDialog->mClock.GetHeure(); cette ligne plante a l'execution (mClock donne membre de la classe qui contient mon thread)

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

bjone

Insert booze to continue

si ton pointeur est NULL oui y'a déjà ça.

mais après, au moment de l'appel à invalidate() es-tu que tout ce qui se passe soit thread-safe ?

bjone

Insert booze to continue

sinon ton "CPCControleDlg *m_pDialog", on est d'accord qu'il est bien dans ta classe CHologe, et que tu n'en as pas d'autres ailleurs.

poste le code de la création du thread, voir comment tu initialises le CHorloge passé en paramètre au thread.

dreamkiller

tennez pas compte des commentaires c'est des essai

.cpp

Code :

CHologe::CHologe(/*CPCControleDlg *pDialog*/ /*CString a*/)
{
m_pTHreadHorloge=AfxBeginThread(ThreadHorloge,this,THREAD_PRIORITY_LOWEST);
m_pTHreadHorloge->m_bAutoDelete=false;
// m_pDialog=pDialog;
// CString test=a;
m_pDialog=NULL;
m_bFin=false;
}
CHologe::~CHologe()
{
m_bFin=true;
}
UINT ThreadHorloge(LPVOID pParam)
{
CHologe* pObject = (CHologe*)pParam;
CTime horloge /*= CTime::GetCurrentTime()*/;
int nHeure;
int nMin;
int nSec;
while(pObject->m_bFin!=true)
{
horloge = CTime::GetCurrentTime();
nHeure=horloge.GetHour();
nMin=horloge.GetMinute();
nSec=horloge.GetSecond();
pObject->m_strHeure.Format("%d",nHeure);
pObject->m_strMin.Format("%d",nMin);
pObject->m_strSec.Format("%d",nSec);
// pObject->m_pDialog->Invalidate();
// if (pObject->m_pDialog) pObject->m_pDialog->Invalidate();
//::InvalidateRect(hwnd, NULL, TRUE);
//::PostMessage(pObject->hwnd, MY_WM_INVALIDATE, (WPARAM)0, (LPARAM)0);
pObject->m_pDialog->mClock.GetHeure();
Sleep(1000);
}
return 1;
}
CString CHologe::GetHeure()
{
return m_strHeure;
}
CString CHologe::GetMin()
{
return m_strMin;
}
CString CHologe::GetSec()
{
return m_strSec;
}

Code :

#include "PC ControleDlg.h"
#if !defined(AFX_HOLOGE_H__E8178F81_42E7_442A_9C7F_A0A51303DEB2__INCLUDED_)
#define AFX_HOLOGE_H__E8178F81_42E7_442A_9C7F_A0A51303DEB2__INCLUDED_
#if _MSC_VER > 1000
#pragma once
#endif // _MSC_VER > 1000
#define MY_WM_INVALIDATE (WM_APP + 1)
class CPCControleDlg;
class CHologe
{
public:
CString GetSec();
CString GetMin();
CString GetHeure();
CString m_strMin;
CString m_strHeure;
CString m_strSec;
CHologe(/*CPCControleDlg *pDialog*/);
CPCControleDlg *m_pDialog;
virtual ~CHologe();
friend UINT ThreadHorloge(LPVOID pParam);
private:
bool m_bFin;
CWinThread *m_pTHreadHorloge;
HANDLE m_hThreadHorloge;
HWND hwnd;
};
#endif // !defined(AFX_HOLOGE_H__E8178F81_42E7_442A_9C7F_A0A51303DEB2__INCLUDED_)

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

bjone

Insert booze to continue

alors déjà:

Code :

* CHologe::CHologe(/*CPCControleDlg *pDialog*/ /*CString a*/)
* {
*
* m_pTHreadHorloge=AfxBeginThread(ThreadHorloge,this,THREAD_PRIORITY_LOWEST);
* m_pTHreadHorloge->m_bAutoDelete=false;
* // m_pDialog=pDialog;
* // CString test=a;
* m_pDialog=NULL;
* m_bFin=false;
* }

quand AfxBeginThread() crée le thread, si il donne tout de suite un timeslice, le thread va commencer à s'éxécuter avec des données CHorloge non initialisés, et le pObject->m_pDialog->mClock.GetHeure() se fera avec un "m_pDialog" à pseudo-rien, ni à NULL ni à autre chose de désiré.

ie faire plustôt:

Code :

CHologe::CHologe(/*CPCControleDlg *pDialog*/ /*CString a*/)
* {
*
* // m_pDialog=pDialog;
* // CString test=a;
* m_pDialog=NULL;
* m_bFin=false;
* m_pTHreadHorloge=AfxBeginThread(ThreadHorloge,this,THREAD_PRIORITY_LOWEST);
* m_pTHreadHorloge->m_bAutoDelete=false;
* }

surtout le m_pDialog qui doit être initialisé à autre chose que NULL, sinon kaboum....

Message édité par bjone le 07-04-2005 à 17:51:33

dreamkiller

oui mais l'initialisé a quoi

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

bjone

Insert booze to continue

dans le doute tu peux aussi créer ton thread avec un:

AfxBeginThread(ThreadHorloge,this,THREAD_PRIORITY_LOWEST,0,CREATE_SUSPENDED);

et faire un ResumeThread() pour lancer le thread un fois que tu as terminé tes initialisations.

comme spécifié à: http://msdn.microsoft.com/library/ [...] nct_18.asp

bjone

Insert booze to continue

dreamkiller a écrit :

oui mais l'initialisé a quoi

c'est pas moi qui vais savoir

bjone

Insert booze to continue

mais comme je te dis, l'approche du thread est une mauvaise approche pour ce que tu veux faire.

tu crées un Timer, tu mappes le message du timer sur ta fonction qui va bien et puis voilà...

dreamkiller

et comment on fiat un timer (et oui j'suis tributaire de mes cours d'info et ca c'est pas dans mes cours)

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

Lam's

Profil: bas.

dreamkiller a écrit :

et comment on fiat un timer

Bah tu vas là: http://www.google.com/microsoft
Et tu tapes "timer". Faut pas prendre le 1er choix (qui est en .NET), mais le 2ème. C'est quand même pas compliqué, nan ? (comme d'hab, y a même un exemple dedans, et la doc est très lisible)

Ou sinon, un bête google sur "msdn timer" donne ça en première page:
http://www.codeproject.com/system/timers_intro.asp

dreamkiller a écrit :

(et oui j'suis tributaire de mes cours d'info et ca c'est pas dans mes cours)

Pas du tout. Tu crois qu'on connais tous les APIs par coeur, ou qu'on garde jalousement nos polys de cours 10 ans après l'obtention du diplôme, même pour coder des trucs abscons ?

Bah nan. Pour être efficace dans la vie, faut savoir chercher (c'est aussi un truc à apprendre: comment trouver la bonne info de la bonne manière). En l'occurence, tu es gâté, la programmation sous Windows est hyper bien documentée sur la MSDN, et on trouve un milliard d'exemples sur le net. Donc, Google est ton ami.

dreamkiller

[ouai j'ai realisé que j'avais dit une connerie dsl merci quand memem

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

dreamkiller

ouai ca marche :bounce:

merci beaucoup la solution timer en faite n'etait pas une mauvaise idee

---------------
debian sib sur asus a6va peut etre prochainement un freebsd

bjone

Insert booze to continue

:jap:

Publicité

FORUM HardWare.fr

Programmation

C++

probleme de pointeur

Sujets relatifs
Problème avec pointeur dans une structure	Problème de typage de pointeur de pointeur
[MFC/VC++] Problème de pointeur	problème de pointeur sur fonction
[C] problème de pointeur	probleme pointeur void alors qu'un cast de char* marche :\
probleme avec pointeur sur objet	Problème de pointeur... (oui, je sais.)
[C] Liste chaînée / pointeur : problème !	probleme pour comparer le contenu d'un pointeur et un valeur!!
Plus de sujets relatifs à : probleme de pointeur

Page générée en 0.087 secondes