Discussion :

[CyberSlan]

Bonjour,

Je débute en C++, et je n'arrive pas à faire appel à une fonction définie dans une classe.

Je suis sous Visual studio 2005, avec un projet MFC, et j'ai créé 2 fichiers pour ma classe :

Classes.h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
#ifndef CCLASSES_H
#define CCLASSES_H
 
class CClasses
{
	DECLARE_DYNAMIC(CClasses)
public:
	void retourAccueil();
};
 
#endif

Classes.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
#include "stdafx.h"
#include "CryptageCle.h"
#include "AccueilDlg.h"
#include "Classes.h"
 
void CClasses::retourAccueil()
{
	AccueilDlg *m_AccueilDlgPtr;
	m_AccueilDlgPtr =NULL;
	BOOL ret=false;
	m_AccueilDlgPtr = new AccueilDlg();
	// créé un dialogue à pertir du modele en ressource
	ret = m_AccueilDlgPtr->Create(IDD_ACCUEIL);
	if(!ret) //Create failed.
	MessageBox(0,_T("Erreur de création de la fenêtre principale"),NULL,0);
	m_AccueilDlgPtr->ShowWindow(SW_SHOW);
}

Dans mon programme, j'ai inclu mon fichier classes.h

Quand je fais :

Code :

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

CClasses::retourAccueil();

J'obtiens l'erreur de compilation suivante :

1>c:\documents and settings\sylvain\bureau\pair\projet\cryptagecle\cryptagecle\ajoutsupprserrure.cpp(56) : error C2352: 'CClasses::retourAccueil' : appel non conforme d'une fonction membre non static

J'ai regardé dans la FAQ, et j'ai l'impression que mon code est correct, je ne vois vraiment pas où est l'erreur.

Merci d'avance pour votre aide

[bolhrak]

Hello,

La méthode doit être appelée sur une instance de classe et non la classe elle même :

Code :

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

1
2
3
4
 
CClasses a;
//...
a.retourAccueil;

[CyberSlan]

Bonjour,

Merci pour ta réponse rapide.

Je croyais que les deux solutions étaient possible, j'avais vu des appels avec la syntaxe que j'ai mis.
Je viens de me rendre compte que c'était des méthodes virtuelles, ça doit venir de là.

Cependant, en utilisant une instance de la classe, comme tu me l'as montré, j'ai l'erreur suivante :

1>AjoutSupprSerrure.obj : error LNK2001: symbole externe non résolu "public: virtual struct CRuntimeClass * __thiscall CClasses::GetRuntimeClass(void)const " (?GetRuntimeClass@CClasses@@UBEPAUCRuntimeClass@@XZ)
1>C:\Documents and Settings\Sylvain\Bureau\PAIR\Projet\CryptageCle\Debug\CryptageCle.exe : fatal error LNK1120: 1 externes non résolus

Je pense que ça dois venir de ma classe qui est mal définie, et non de l'appel, cette fois ci, mais je ne vois pas comment corriger ça.

[CyberSlan]

J'ai trouvé le problème.

J'ai supprimé la ligne

Code :

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

DECLARE_DYNAMIC(CClasses)

et j'arrive à exécuter mon programme.

Merci encore pour ton aide.

[Médinoc]

DECLARE_DYNAMIC te donnait une erreur parce qu'il manquait le IMPLEMENT_DYNAMIC.

Ensuite, tu n'en as généralement pas besoin en effet.

[CyberSlan]

DECLARE_DYNAMIC te donnait une erreur parce qu'il manquait le IMPLEMENT_DYNAMIC.

Ensuite, tu n'en as généralement pas besoin en effet.

Merci pour la précision.

Par contre, j'ai rajouté une deuxième méthode à cette classe, et j'ai à nouveau une erreur.

Voici mon code, la deuxième méthode permettrait de convertir des TCHAR* en char*.

Classes.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
#include "stdafx.h"
#include "CryptageCle.h"
#include "AccueilDlg.h"
#include "Classes.h"
 
void CClasses::retourAccueil()
{
	AccueilDlg *m_AccueilDlgPtr;
	m_AccueilDlgPtr =NULL;
	BOOL ret=false;
	m_AccueilDlgPtr = new AccueilDlg();
	// créé un dialogue à pertir du modele en ressource
	ret = m_AccueilDlgPtr->Create(IDD_ACCUEIL);
	if(!ret) //Create failed.
	MessageBox(0,_T("Erreur de création de la fenêtre principale"),NULL,0);
	m_AccueilDlgPtr->ShowWindow(SW_SHOW);
}
 
void wtoc(char* Dest, TCHAR* Source)
{
USES_CONVERSION;
 
strcpy (Dest, T2A (Source));
}

Classes.h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#ifndef CCLASSES_H
#define CCLASSES_H
 
class CClasses
{
 
public:
	void retourAccueil();
public:
	void wtoc(char* Dest, TCHAR* Source);
};
 
#endif

J'ai l'erreur suivante :

1>AjoutAcces.obj : error LNK2019: symbole externe non résolu "public: void __thiscall CClasses::wtoc(char *,wchar_t *)" (?wtoc@CClasses@@QAEXPADPA_W@Z) référencé dans la fonction "public: void __thiscall CAjoutAcces::OnBnClickedButtonSupprimerAcces(void)" (?OnBnClickedButtonSupprimerAcces@CAjoutAcces@@QAEXXZ)

Désolé de rouvrir ce topic après l'avoir mis en résolu

Merci d'avance

[Médinoc]

Une erreur commune. T'imagines pas la fréquence à laquelle ça m'arrive.

Code C++ :

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

1
2
3
4
5
6
void CClasses::wtoc(char* Dest, TCHAR* Source)
{
	USES_CONVERSION;
 
	strcpy (Dest, T2A (Source));
}

[Médinoc]

PS: La fonction devrait s'appeler ttoc() et non wtoc().
Ou mieux, faire deux (voire quatre) fonctions ctoc() surchargées.

[CyberSlan]

Merci pour la rapidité.

Ca m'énerve de bloquer sur des trucs comme ça, quand la solution est si simple et qu'en plus je l'ai déjà juste au dessus

J'ai changé le nom de la fonction, en ttoc, c'est vrai que c'est plus logique.
Par contre, je vois pas ce que tu veux dire par des fonctions ctoc() surchargées.

Je vais donc laisser comme ça pour le moment (sauf si tu as le temps de m'expliquer rapidement, mais je me débrouillerais sans, c'est juste pour apprendre, car mon programme fonctionne maintenant).

[Médinoc]

Je pensais à un truc comme ça:

Code C++ :

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
void CClasses::ctoc(char* Dest, char const * Source)
{
	strcpy (Dest, Source);
}
 
void CClasses::ctoc(char* Dest, wchar_t const * Source)
{
	USES_CONVERSION;
 
	strcpy (Dest, W2A(Source));
}
 
void CClasses::ctoc(wchar_t * Dest, char const * Source)
{
	USES_CONVERSION;
 
	wcscpy (Dest, A2W(Source));
}
 
void CClasses::ctoc(wchar_t * Dest, wchar_t const * Source)
{
	wcscpy (Dest, Source);
}

[CyberSlan]

OK, merci.

Je ne savais pas qu'il était possible de faire un truc de ce genre.