Timer MFC erreur de LNK

**Emyleet** · 19/05/2008, 17h16

Bonjour à tous!

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

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

**farscape** · 19/05/2008, 18h15

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

**Emyleet** · 19/05/2008, 21h06

Merci pour ta réponse !

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

Code :

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

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?

**Emyleet** · 20/05/2008, 09h20

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

Aidez moi s'il vous plait !! je suis perdue ...

**Emyleet** · 20/05/2008, 15h20

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?

**stephdim** · 20/05/2008, 17h35

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

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 :

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

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

@+

**Emyleet** · 22/05/2008, 09h13

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 ?

**Emyleet** · 23/05/2008, 09h02

Personne n'a d'idée ?

voila ce que j'ai fait :

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

**stephdim** · 23/05/2008, 09h34

tu es dans une boite de dialogue ?

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 :

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

1
2
 
VOID CALLBACK TimerProc(HWND,UINT,UINT_PTR,DWORD);

a déclarer avant de faire référence à TimerProc.

@+

**Emyleet** · 23/05/2008, 10h29

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>

**stephdim** · 23/05/2008, 10h34

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

@+

**Emyleet** · 23/05/2008, 10h38

Oui je travaille avec les MFC.

Je suis sous Visual Studio 2005.

**Emyleet** · 23/05/2008, 10h50

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

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
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 :

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

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.

**stephdim** · 23/05/2008, 10h59

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

dans le .h

Code :

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

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 :

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

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 :

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

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 :

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

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++

@+

**Emyleet** · 23/05/2008, 11h08

youuuuuhouuuuuuuuuuu

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!

**Emyleet** · 23/05/2008, 15h48

:'(

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

la je l'appel dans la fonction suivante:

Code :

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

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)

**stephdim** · 23/05/2008, 15h52

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

@+

**Emyleet** · 23/05/2008, 16h12

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 :

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

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

Timer MFC erreur de LNK

MFC

Discussions similaires

Partager

Partager