Timer MFC erreur de LNK

Version imprimable

Bonjour à tous!

voila j'ai un problème au niveau d'un timer que j'ai créé de la manière suivante
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 #include<windows.h> #include <stdio.h> #include <stdlib.h> #include "Timer.h" // Timer dialog /*static*/void Timer::OnStartTimer() { UINT_PTR m_nTimer; m_nTimer = SetTimer(1, 10, 0); } /*static*/void Timer::OnStopTimer() { UINT_PTR m_nTimer; KillTimer(m_nTimer); } /*static*/ void Timer::OnTimer(UINT IdTimer) { int PedalState; PedalState=0; unsigned char Authmov[1]={140}; // get etat command unsigned char NoAuthmov[1]={141}; // get etat command CCom::ClosePortCom(); CCom::InitPortCom(); if(PedalState = 0) { WriteFile(hcom,Authmov, 1, &m, NULL); CCom::ReadBuffer(); if ((data[1]=1)&&(data[0]=140)) { OnStopTimer(); OnStartTimer(); } else { OnStopTimer(); OnStartTimer(); } } else { WriteFile(hcom,NoAuthmov, 1, &m, NULL); if ((data[1] = 1) && (data[0]=141)) { OnStopTimer(); OnStartTimer(); } else { OnStopTimer(); OnStartTimer(); } } }
J'ai essayé de suivre les indications du faq de farscape mais j'ai les erreurs suivantes:
error LNK 2019:unresolved external symbol "public:static unisgned int_cdecl Timer::SetTimer...
error LNK 2019:unresolved external symbol "public:static unisgned int_cdecl Timer::killTimer...

J'ai cherché dans les forums mais je ne comprends pas vraiment ...
Pouvez vous me donner un coup de main?

Merci d'avance.

19/05/2008, 18h15
farscape

salut,
normal la méthode SetTimer de la faq fait partie de la classe CWnd...
sinon il faut passer par l'api Win32 mais bon je ne te le conseil pas trop...

Merci pour ta réponse !

Il suffit donc je déclare les fonctions dans mon .h de la manière suivante?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 CWnd::SetTimer UINT SetTimer( UINT nIDEvent, // numéro du timer à partir de 1 UINT nElapse, // temps ecoulé entre 2 appels en ms. void (CALLBACK EXPORT* lpfnTimer)( HWND, UINT, UINT, DWORD) ); // fonction callback. CWnd::KillTimer BOOL KillTimer( int nIDEvent ); // numéro du Timer
cependant je suis débutante en C++ et ma fonction OnTimer est static il faut donc que SetTimer et KillTimer le soient aussi ... comment puis je faire?

20/05/2008, 09h20
Emyleet

Si je met static devant SetTimer et KillTimer dans le .h ca ne marche pas...:?

Aidez moi s'il vous plait !! je suis perdue ... :(
20/05/2008, 15h20
Emyleet

C'est bon j'ai réglé le problème de static. J'ai créé un objet CWnd timer et j'ai appelé ma fonction de la manière suivante : timer.SetTimer(1, 10, 0);
J'ai fait de même avec KillTimer.

Par contre maintenant j'ai l'erreur suivante à l'éxecution:
J'ai les lignes relatives à SetTimer et Killtimer de surligner dans le fichier afxwin2.h et le message suivant : .exe has triggered a breakpoint!

Que dois je faire?

salut,

as tu lu entierement la MSDN sur la fonction SetTimer() ???

il ne suffit pas de faire dériver une classe de CWnd pour utiliser SetTimer ...
ok le compilateur ne dit plus rien, mais c'est pas du tout fonctionnel :aie:

un Timer est soit rattaché à une fenetre, soit rattaché à un thread à condition qu'il possède une boucle à message...

dans ton cas, tu essayes de rattacher un timer à une fenetre qui n'existe pas ... donc normal que ça plante

utilises la fonction globale SetTimer() en indiquant une procédure et non la méthode SetTimer() de la classe CWnd
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 m_IdTimer=::SetTimer(NULL,0,1000,TimerProc); // création d'un timer qui bat à la seconde ... KillTimer(m_IdTimer); // pour arreter le timer ... VOID CALLBACK TimerProc(HWND /*hwnd*/,UINT /*uMsg*/,UINT_PTR /*idEvent*/,DWORD /*dwTime*/) { // ici la procédure du timer }
tu auras remarqué que j'ai mis le préfixe "m_" devant la variable m_IdTimer. C'est à dire que c'est une variable membre de la classe et qu'elle doit etre persistante pendant toute la durée de vie du timer, sinon tu pourras pas arreter le Timer ...

@+

22/05/2008, 09h13
Emyleet

Merci beaucoup pour ta réponse clair :)

cependant j'ai encore du mal faire quelquechose car j'ai l'erreur suivante:
errorC3867:'InstantResult::TimerProc' function call missing argument list; use '&InstantResult::TimerProc' to create a pointer to member.
Et si je fais ca... j'ai une autre erreur.

je ne comprends vraiment pas tout les exemples que j'étudie font la même chose. SetTimer prend bien comme 4ième argument un pointeur de type TIMERPROC vers la fonction callback.

Pourquoi ai je l'erreur précédente ?

Personne n'a d'idée ?

voila ce que j'ai fait :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
 
BOOL InstantResult::OnInitDialog()
{
 
m_IdTimer=::SetTimer(NULL,0,1000,TimerProc);  // création d'un timer qui bat à la seconde
 
return TRUE;
}
 
 
void InstantResult::OnBnClickedButton2()
{
 
	// TODO: Add your control notification handler code here
 
	KillTimer(m_IdTimer);  // pour arreter le timer
 
}
 
VOID CALLBACK TimerProc(HWND /*hwnd*/,UINT /*uMsg*/,UINT_PTR /*idEvent*/,DWORD /*dwTime*/)
{
  // ici la procédure du timer
 
	unsigned char Authmov[1]={140}; // authorization to move command
	unsigned char NoAuthmov[1]={141}; // not allowed to move command
 
 
	CCom::ClosePortCom();
	CCom::InitPortCom();
 
			WriteFile(hcom,Authmov, 1, &m, NULL);
			CCom::ReadBuffer();
}

et donc j'ai l'erreur 'TimerProc' function call missing argument use &InstantResult::TimerProc to create a pointer to member.

23/05/2008, 09h34
stephdim
tu es dans une boite de dialogue ? 8O
si c'est le cas, utilise un Timer lié à la fenetre de ta boite de dialogue, c'est plus simple.

sinon, la fonction TimerProc doit etre static si c'est une fonction déclarée dans une classe. a ce que je vois, dans ton code, il ne manque que le prototype de la fonction:
Code:

1 2 VOID CALLBACK TimerProc(HWND,UINT,UINT_PTR,DWORD);
a déclarer avant de faire référence à TimerProc.

@+
23/05/2008, 10h29
Emyleet

Rahhhh .... il me manque une librairie apparemment. Oui mais laquelle ? :)

J'ai déclaré TimerProc en static dans mon .h et je mis le prototype de cette fonction dans mon .cpp au début.

Au niveau librairie, j'ai pourtant tout ce qu'il faut je pensais :
<Time.h>
<windows.h>
23/05/2008, 10h34
stephdim

il ne te manque aucune librairie
les timer sont définis dans <windows.h> ...

tu travailles avec les MFC non ?
donc vire moi ces includes inutiles
tout est déjà dans le fichier StdAfx.h

tu es sous quelle version de VS ?

si c'est VS98 (VC6), il ne reconnaitra pas UINT_PTR ; remplaces le par UINT

@+
23/05/2008, 10h38
Emyleet

Oui je travaille avec les MFC.

Je suis sous Visual Studio 2005.

Bon alors comme tu mas dit j'ai fait ca dans mon .cpp:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
 
static VOID CALLBACK TimerProc(HWND /*hwnd*/,UINT /*uMsg*/,UINT_PTR /*idEvent*/,DWORD /*dwTime*/);
 
 
InstantResult::InstantResult(CWnd* pParent /*=NULL*/)
	: CDialog(InstantResult::IDD, pParent)
{
	//m_IdTimer=::SetTimer(NULL,0,1000,TimerProc);  // création d'un timer qui bat à la seconde
}
 
 
void InstantResult::OnBnClickedButton2()
{
 
	// TODO: Add your control notification handler code here
 
 
	KillTimer(m_IdTimer);  // pour arreter le timer
}
VOID CALLBACK TimerProc(HWND /*hwnd*/,UINT /*uMsg*/,UINT_PTR /*idEvent*/,DWORD /*dwTime*/)
{
  // ici la procédure du timer
 
	unsigned char Authmov[1]={140}; // authorization to move command
 
 
	CCom::ClosePortCom();
	CCom::InitPortCom();
	WriteFile(hcom,Authmov, 1, &m, NULL);
	CCom::ReadBuffer();
 
}

et dans mon .h:

Code:

1
2
3
4
5
6
 
public:
 
	UINT_PTR m_IdTimer;
 
	static VOID CALLBACK TimerProc(HWND /*hwnd*/,UINT /*uMsg*/,UINT_PTR /*idEvent*/,DWORD /*dwTime*/);

En effet ca ne change rien si je définis m_IdTimer en UINT.

ben sois tu définies la procédure dans la portée de ta classe, donc:

dans le .h
Code:

1 2 3 4 5 6 7 class InstantResult { ... static VOID CALLBACK TimerProc(HWND /*hwnd*/,UINT /*uMsg*/,UINT_PTR /*idEvent*/,DWORD /*dwTime*/); ... };
et donc dans le .cpp ça donne :
Code:

1 2 3 4 VOID CALLBACK InstantResult::TimerProc(HWND /*hwnd*/,UINT /*uMsg*/,UINT_PTR /*idEvent*/,DWORD /*dwTime*/) { }
et dans ce cas tu aurais oublié le InstantResult:: ...

sois tu définies la procédure en portée globale :

dans le .h :
Code:

1 2 3 4 5 6 7 VOID CALLBACK TimerProc(HWND /*hwnd*/,UINT /*uMsg*/,UINT_PTR /*idEvent*/,DWORD /*dwTime*/); class InstantResult { ... };
tu auras remarqué que la déclaration n'est pas à l'intérieur de la classe ...

et dans le .cpp
Code:

1 2 3 4 5 VOID CALLBACK TimerProc(HWND /*hwnd*/,UINT /*uMsg*/,UINT_PTR /*idEvent*/,DWORD /*dwTime*/) { ... }
et là pas de InstantResult:: devant

ça c'est les bases du C++

@+

23/05/2008, 11h08
Emyleet

youuuuuhouuuuuuuuuuu :yaisse2:

Ca marche :):) je reçois bien ce que je désire toutes les 10 ms!!!

Merci beaucoup ton aide m'est trés précieuse!
23/05/2008, 15h48
Emyleet
:'(

euhh juste encore un truc KillTimer(1); normalement ca arrête directement mon timer ?

la je l'appel dans la fonction suivante:
Code:

1 2 3 4 5 6 7 8 9 void InstantResult::OnBnClickedButton2()//return button { // TODO: Add your control notification handler code here toto = KillTimer(1); // pour arreter le timer }
(j'ai changé l'id du timer je l'ai mis à 1 dans settimer)
23/05/2008, 15h52
stephdim

si ton Timer n'est pas attaché à une fenetre, il faut conserver l'Id retourné par SetTimer()
c'est pas toi qui choisit l'Id mais le systeme ; c'est bien expliqué dans la MSDN

par contre, si tu fournis un HWND à SetTimer(), là c'est toi qui le choisit ...

@+
23/05/2008, 16h12
Emyleet
Oula il y a un truc louche.

Quand je fais afficher la valeur de m_IdTimer juste aprés SetTimer de cette manière:
Code:

1 2 3 4 m_IdTimer=::SetTimer(NULL,0,1000,TimerProc); // création d'un timer qui bat à la seconde str.Format(_T("%.1i"),m_IdTimer); AfxMessageBox(str);
j'ai m_IdTimer =26974

et quand je fais la même chose juste avant l'appel de KillTimer dans mon handle de bouton. J'ai:
m_IdTimer =26968

8O