La fonction callback dans c++

Version imprimable

J' ai une classe DialogImpl:
Code:

1 2 3 4 5 6 7 8 9 class DialogImpl { public: void my_callback(u_char *useless, const struct pcap_pkthdr* hdr, const u_char* packet); public slots: void sniffer(); // ............ les autres méthode }
Ma méthode sniffer() a cette fonction:
Code:

1 2 3 4 5 DialogImpl::sniffer() { //............ pcap_loop(descr,-1,my_callback,NULL); //................ }
Si je fais comme dessus, il va créer une erreur argument of type ‘void (DialogImpl:: )(u_char*, const pcap_pkthdr*, const u_char*)’ does not match ‘void ( * )(u_char*, const pcap_pkthdr*, const u_char*)’

Je sais qu'il faut mettre static à la fonction my_callback. Pourtant, je ne peux pas utiliser les fonctions de GUI dans my_callback. Est-ce qu' il y a une autre résolution?

Merci par avance pour votre aide!

Salut

Code:

pcap_loop(descr,-1,my_callback,NULL);

Je suppose que le dernier parametre (NULL) est le parametre de la fonction callback (c'est souvent le cas). Tu peux donc créer une fonction static :
Code:

1 2 3 4 5 6 7 8 9 static void MaBelleCallback(u_char *data, const struct pcap_pkthdr* hdr, const u_char* packet) { DialogImpl* pDlg = static_cast<data*>(data); assert(pDlg); pDlg->my_callback(hdr, packet); }
Il faut donc modifier ton code en conséquence :
Code:

1 2 3 4 5 DialogImpl::sniffer() { ... pcap_loop(descr,-1,my_callback,this); ... }
et
Code:

1 2 3 4 5 6 class DialogImpl { public: void my_callback(const struct pcap_pkthdr* hdr, const u_char* packet); ... }
En espérant que ça répond à ta question.
--
Jérémie

23/05/2009, 19h30
bboy0303

Merci bien, j'ai réussi à compiler le programme mais il y a une telle erreur dans la fonction pcap_loop que mon programme ne peut pas éxécuter normalement, je vais rechercher encore!

Citation:
Envoyé par jfouche
Il faut donc modifier ton code en conséquence :
Code:

1 2 3 4 5 DialogImpl::sniffer() { ... pcap_loop(descr,-1,my_callback,this); ... }
Je me suis trompé, c'est :
Code:

1 2 3 4 5 DialogImpl::sniffer() { ... pcap_loop(descr,-1,MaBelleCallback,this); ... }
--
Jérémie

Merci, je sais!
Il faut corriger aussi pour faire une forçage de type:

Code:

pcap_loop(descr,-1,MaBelleCallback,(u_char*)this);

Mais ton code a fait quelques erreurs en compilant:
Code:

1 2 3 4 5 6 static void MaBelleCallback(u_char *data, const struct pcap_pkthdr* hdr, const u_char* packet) { DialogImpl* pDlg = static_cast<data*>(data); assert(pDlg); pDlg->my_callback(hdr, packet); }
Citation:

src/dialogimpl.h:29: erreur: expected type-specifier before ‘data’
src/dialogimpl.h:29: erreur: expected `>' before ‘data’
src/dialogimpl.h:29: erreur: expected `(' before ‘data’
src/dialogimpl.h:29: erreur: expected primary-expression before ‘>’ token
src/dialogimpl.h:29: erreur: expected `)' before ‘;’ token

J' ai fais comme-ci, il peut être compiler mais le programme ne marche pas encore normalement:
Code:

1 2 3 4 5 static void MaBelleCallback(u_char *data, const struct pcap_pkthdr* hdr, const u_char* packet) { DialogImpl* pDlg = (DialogImpl*)data; pDlg->my_callback(hdr, packet); }
Tu connais une résolution?

24/05/2009, 11h03
jfouche

Utilises ton debuger pour voir ce qui passe mal.
Passes tu bien dans la MaBelleCallback ?
--
Jérémi

Selon moi, il ne passe pas MaBellCallback.
Quand je n'utilise pas la classe, ca marche bien. Mais avec la classe, ca se passe mal. Mon code avec la classe:
dialogimpl.h:

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
#ifndef DIALOGIMPL_H
#define DIALOGIMPL_H
//
#include <pcap.h>
#include <stdio.h>
#include <stdlib.h>
#include <string>
#include <errno.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <arpa/inet.h>
#include <netinet/if_ether.h>
#include <netinet/ip.h>
#include <netinet/tcp.h>
#include <netinet/udp.h>
#include <iostream>
#include <fstream>
using namespace std;
 
class DialogImpl
{
public:
	static void my_callback(u_char *useless, const struct pcap_pkthdr* hdr, const u_char* packet) {
		DialogImpl *This = (DialogImpl*)useless;
		This->got_packet(hdr, packet);
	}
 
	void got_packet(const struct pcap_pkthdr* hdr, const u_char* packet);
 
	void sniffer();
};
#endif

dialogimpl.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
#include "dialogimpl.h"
 
void DialogImpl::got_packet(const struct pcap_pkthdr* hdr, const u_char* packet) {
	cout << "Hello";
}
 
 
void DialogImpl::sniffer()
{
	char *dev;
	char errbuf[PCAP_ERRBUF_SIZE];
	pcap_t *descr;
 
	cout << "Start\n";
 
	dev = pcap_lookupdev(errbuf);
	if(dev == NULL) { 
		printf("%s\n",errbuf); 
		exit(1); 
	}
 
	descr = pcap_open_live(dev,BUFSIZ,0,-1,errbuf);
	if(descr == NULL) { 
		printf("pcap_open_live(): %s\n",errbuf); 
		exit(1); 
	}
 
	pcap_loop(descr,-1,my_callback,(u_char*)this);
	pcap_close(descr);
}

main.cpp:

Code:

1
2
3
4
5
6
7
8
#include "dialogimpl.h"
 
int main()
{
	DialogImpl win;
	win.sniffer();
	return 0;
}

Pourrais-tu m'aider?

Si je n'utilise pas la classe, ca marche bien:
foo.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
33
34
35
36
37
38
39
40
41
42
43
44
45
#include <pcap.h>
#include <stdio.h>
#include <stdlib.h>
#include <string>
#include <errno.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <arpa/inet.h>
#include <netinet/if_ether.h>
#include <netinet/ip.h>
#include <netinet/tcp.h>
#include <netinet/udp.h>
#include <iostream>
#include <fstream>
using namespace std;
 
void my_callback(u_char *useless,const struct pcap_pkthdr* hdr,const u_char* packet)
{	
	cout << "Hello\n";
}
 
int main()
{
	char *dev;
	char errbuf[PCAP_ERRBUF_SIZE];
	pcap_t *descr;
 
	cout << "Start\n";
 
	dev = pcap_lookupdev(errbuf);
	if(dev == NULL) { 
		printf("%s\n",errbuf); 
		exit(1); 
	}
 
	descr = pcap_open_live(dev,BUFSIZ,0,-1,errbuf);
	if(descr == NULL) { 
		printf("pcap_open_live(): %s\n",errbuf); 
		exit(1); 
	}
 
	pcap_loop(descr,-1,my_callback,NULL);
	pcap_close(descr);
   	return 0;
}

Compiler et éxécuter:
g++ foo.cpp -lpcap
sudo ./a.out

Je n'aurai pas mis my_callback en membre static de DialogImpl, mais plutôt une fonction static toute seule dans son coin.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#ifndef DIALOGIMPL_H
#define DIALOGIMPL_H
...
// As tu besoin d'inclure tous ces header dans ton .h ?
// ne défini pas de using namespace dans un .h, qui sais si l'utilisateur
// de ta classe n'en veux pas ?
// using namespace std; 
 
class DialogImpl
{
public:
	void got_packet(const struct pcap_pkthdr* hdr, const u_char* packet);
 
	void sniffer();
};
#endif

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
#include "dialogimpl.h"
 
// Ici tu peux, sans gener personne
using namespace std; 
 
static void my_callback(u_char *data, const struct pcap_pkthdr* hdr, const u_char* packet)
{
	cout << "my_callback(...)" << endl;
	DialogImpl *This = (DialogImpl*)data;
	This->got_packet(hdr, packet);
}
 
void DialogImpl::got_packet(const struct pcap_pkthdr* hdr, const u_char* packet)
{
	cout << "Hello";
}
 
void DialogImpl::sniffer()
{
	char *dev;
	char errbuf[PCAP_ERRBUF_SIZE];
	pcap_t *descr;
 
	cout << "Start" << endl;
 
	dev = pcap_lookupdev(errbuf);
	if(dev == NULL) {
		cerr << errbuf << endl; 
		exit(1); 
	}
 
	descr = pcap_open_live(dev,BUFSIZ,0,-1,errbuf);
	if(descr == NULL) { 
		cerr << "pcap_open_live(): " << errbuf << std::endl; 
		exit(1); 
	}
 
	pcap_loop(descr,-1,my_callback,(u_char*)this);
	pcap_close(descr);
}

--
Jérémie

24/05/2009, 14h23
bboy0303

Merci beaucoup de votre aide :D!
ça marche bien avec classe ou sans classe, je me suis trompé!
24/05/2009, 14h55
jfouche

Penses a signaler la conversation comme résolu (en cliquant sur le bouton résolu).
Merci
--
Jérémie