Erreur de compilation avec CString

Version imprimable

Bonjour,

Je débute en C++ et j'ai une question, lors de la compilation du code ci-dessous (avec Dev-C++) j'ai pas mal d'erreurs dû à " [Linker error] undefined reference to `CString::~CString()' " ou à " [Linker error] undefined reference to `CString::CString(char const*)' ".... etc....

Toujours lié à CString !!! J'ai bien mon fichier cstring.h et j'en fais l'inclusion dans mon programme mais il dois me manquer qqch !!!

Pourriez-vous m'aider ?

PS: Cstring.h, je l'ai trouvé sur le net :?

Merci d'avance

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
76
77
78
79
80
81
82
83
84
85
86
 
#include <cstdlib>
#include <iostream>
#include <stdio.h>
#include <cstring.h>
using namespace std;
 
class CSecurity
{
 
  public:  
 
   CString CSecurity::Crypt(LPCSTR p_lpszTerm)
   {
    	CString strTerm = p_lpszTerm;
    	//strTerm.MakeUpper();
    	LPSTR lpczCursor = (LPSTR) (LPCSTR) strTerm;
    	CString strCrytedTerm;
    	CString strByte;
 
    	int nLength = strTerm.GetLength();
    	int nCounter = 3;
    	int nOffset;
    	int nRotation;
    	unsigned char cCrypted;
    	while (*lpczCursor)
    	{	
    		cCrypted = *lpczCursor++;
 
    		nOffset = m_pOffsetTable[nCounter++ % m_nOffsetTableSize];
    		nRotation = m_pOffsetTable[nCounter++ % m_nOffsetTableSize];
    		//cCrypted = RotLeft(cCrypted,nRotation);
     		cCrypted += nOffset;
 
    		//strByte.Format("%.2X",cCrypted);
    		strCrytedTerm += strByte;
    	}
    	strCrytedTerm = Scramble(strCrytedTerm);
    	return strCrytedTerm;
    }
    //static int DefaultOffsetTable[] = {5,9,2,6,5,3,5,9,1,4,1,5,9,2,6,5,3,5,9,1};
 
    CSecurity::CSecurity()
    {
		 int DefaultOffsetTable[] = {5,9,2,6,5,3,5,9,1,4,1,5,9,2,6,5,3,5,9,1};
         SetOffsetTable(DefaultOffsetTable,20);
    }
 
    private :
 
    CString CSecurity::Scramble(LPCSTR p_lpszTerm)
    {
    		 CString strTerm = p_lpszTerm;
    		 int nMidPoint = strTerm.GetLength()/2;
    		 CString strLeft = strTerm.Left(nMidPoint);
    		 CString strRight = strTerm.Mid(nMidPoint);
 
    		 CString strScrambled;
    		 LPSTR lpszLeftCursor = (LPSTR) (LPCSTR) strLeft;
    		 LPSTR lpszRightCursor = (LPSTR) (LPCSTR) strRight;
    		 while ((*lpszLeftCursor) && (*lpszRightCursor))
    		 {
    		 		 if (*lpszRightCursor)
    		 		 		 strScrambled += *lpszRightCursor++;
    		 		 if (*lpszLeftCursor)
    		 		 		 strScrambled += *lpszLeftCursor++;
    		 }
    		 return strScrambled;
    }
 
    inline void CSecurity::SetOffsetTable(int* p_pOffsetTable, int p_nTableSize)
    {
    		 m_pOffsetTable = p_pOffsetTable;
    		 m_nOffsetTableSize = p_nTableSize;
    }
 
    int* m_pOffsetTable;
    int  m_nOffsetTableSize; 
};
 
int main()
{
    CSecurity Test;
    Test.CSecurity::Crypt("test");
    return(0);
}

13/08/2008, 08h18
zabibof

Pourquoi tu n'utilises tout simplement pas std::string? (il y a une raison particulière pour que tu utilises CString?)

Sinon pour ton problème, peut-être que tu as simplement téléchargé l'en-tête, pas l'implémentation ;)
13/08/2008, 08h22
bianconeri

Hello,

Merci de ta réponse si rapide, pourquoi CString, car dans le code avec lequel je me suis inspirer il y avait déjà CString et j'ai continuer dans cette voie !

Citation:

peut-être que tu as simplement téléchargé l'en-tête, pas l'implémentation

Il y a bien des chances, qu'est-ce que l'imprimentation ? :oops:
13/08/2008, 08h44
3DArchi

Bonjour,
Et bien, en plus du .h, tu dois avoir à minima un .lib pour le lien et s'il s'agit d'un lien dynamique, à l'exécution tu auras besoin d'une DLL. CString, ca ressemble aux MFC. Je ne suis pas sur que tu puisses l'avoir comme ça.
13/08/2008, 08h44
zabibof
Citation:

Envoyé par bianconeri

Il y a bien des chances, qu'est-ce que l'implémentation ?

En gros, t'as tout simplement téléchargé "CString.h", pas "CString.cpp" (ou la .lib? ou dll?).
Dans l'en-tête, tu as la définition de ta classe, ça te dis ce que la classe peut faire et dans le .cpp, tu as l'implémentation, ça te dis comment la classe fait ceci ou cela (je l'explique peut-être mal mais en gros c'est ça :aie:), parfois l'implémentation est dans un fichier en-tête mais en générale, c'est dans le .cpp (sauf pour les templates et les fonctions inlines)

Exemple:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 #include <iostream> class A { public: void foo(); }; // implémentation de la méthode foo de la classe A void A::foo() { std::cout << "foo\n"; }
13/08/2008, 13h26
Spout

CString est la classe de texte des MFC, et n'est en aucun cas définie dans cstring.h!
cstring.h contient la définition des fonctions de manipulation de texte héritées du C (d'où le cstring.h), alors que la classe CString est définie dans afxstr.h.
N'essaye pas d'inclure ce fichier dans ton projet, ce n'est pas aussi facile :D. Ca ne fonctionnera qu'avec Visual (en version complète), avec un projet MFC. J'en conclus donc que le code dont tu t'es inspiré avait été développé sous Visual.

NB: je ne vois dans ton code aucun appel à une des fonctions de cstring.h. Il semble être inutile ici...
13/08/2008, 13h39
bianconeri

Merci de l'information, en cherchant un peu (beaucoup) sur le net il me semble être tomber sur la même analyse que toi spoutspout !!!

Autant pour moi, est-il possible de faire la même chose avec du "String" standard ?

Merci de votre réponse
13/08/2008, 13h47
Spout

Citation:

Envoyé par bianconeri

Autant pour moi, est-il possible de faire la même chose avec du "String" standard ?

Si tu parles tu std::string, oui. Tu peux faire autant qu'avec un CString et plus encore.
Voir doc STL.

J'ai commencé à refaire le code en "String" malheureusement j'ai déjà des erreurs d'entré sans commencer à coder...

Pouvez-vous me mettre sur la bonne voie ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 #include <cstdlib> #include <string.h> #include <iostream> using std::string; class CSecurity { public : string CSecurity::Crypt(LPCSTR p_lpszTerm) { String strTerm = p_lpszTerm; return strTerm; } private : };
Voici les erreurs

11 C:\Dev-Cpp\Projects\GED\main.cpp expected `;' before '(' token
17 C:\Dev-Cpp\Projects\GED\main.cpp expected `;' before "private"

Voilà ton code corrigé avec des remarques:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
#include <cstdlib>
#include <string>
#include <iostream>

// soit tu mets
using namespace std;
// soit tu mets des std:: devant tous tes appels à la STL, mais en aucun cas ça: using std::string; 

class CSecurity
{
      public :
      
      // CSecurity:: inutile car tu es déjà dans la définition de la classe, et donc dans son scope
      string Crypt(LPCSTR p_lpszTerm)
      {
             // string et non String
             string strTerm = p_lpszTerm; 
    	     return strTerm;
      };  // ';' obligatoire car c'est une définition, pas une implémentation
      
      private :
};

13/08/2008, 14h48
buggen25

Bonjour;

et si tu met

using namespace std;

au lieu de

using std::string;

Merci beaucoup !

Pourtant pour le ; à la fin j'ai pris des exemple sur le net !!!
Pour le S de String, on va dire que c'est une mauvaise habitude de VB !

Mais j'ai toujours un erreur vraiment bizarre,
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 #include <cstdlib> #include <string> #include <iostream> using namespace std; class CSecurity { string Crypt(LPCSTR p_lpszTerm) { string strTerm = p_lpszTerm; return strTerm; }; };
9 C:\Dev-Cpp\Projects\GED\main.cpp expected `;' before '(' token

13/08/2008, 15h05
Spout

Est-ce bien le code de main.cpp que tu nous montres? D'une part parceque ça ressemble plus à un *.h plutôt qu'à un *.cpp, et d'autre part car le code que tu as posté fonctionne très bien chez moi (VS2005).
13/08/2008, 15h14
Noxen

Il me semble que spoutspout s'est trompé pour le ; après la fonction, puisqu'il y a déclaration et définition (entre les {} ).

Sinon, c'est quoi une LPCSTR ? Est-ce-que ça peut se convertir en chaîne C style ou en std::string ? Si c'est un typedef pour un char* ou assimilé ça ira, mais sinon je ne pense pas qu'il y ait de constructeur de std::string qui permette de faire :

Code:

string strTerm = p_lpszTerm;

C'est à quelle ligne qu'il veut un ';' ? Je n'arrive pas à identifier la 'ligne 9'.

Oui c'est bien main.cpp !

Et avec ceci cela fonctionne
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 #include <cstdlib> #include <string> #include <iostream> using namespace std; class CSecurity{ string Crypt(char * p_lpszTerm) { string strTerm = p_lpszTerm; return strTerm; }; //<-- Fonctionne avec ou sans le point virgule };
Je suppose que LPCSTR est un "truc" de CString et que c'est cela qui faisait planter ma classe ! Je pense que je vais bien morfler pour passer de CString en String !

Dans tous les cas, merci de votre aide les gars !!!

13/08/2008, 15h24
Spout

Citation:

Envoyé par Noxen

Il me semble que spoutspout s'est trompé pour le ; après la fonction, puisqu'il y a déclaration et définition (entre les {} ).

Je persiste ;).
Ca reste une définition (dans laquelle tu places l'implémentation).

Citation:

Envoyé par Noxen

Sinon, c'est quoi une LPCSTR ? Est-ce-que ça peut se convertir en chaîne C style ou en std::string ?

Oui. LPCSTR est un Long Pointer Const sur un STRing. Autrement dit, un const char*. L'opérateur = du string de la STL accepte ce type d'argument.

Citation:

Envoyé par Noxen

mais sinon je ne pense pas qu'il y ait de constructeur de std::string qui permette de faire :

Code:

string strTerm = p_lpszTerm;

Effectivement, cette ligne est fausse. Lui préférer:

Code:

string strTerm(p_lpszTerm);
13/08/2008, 15h30
Spout

J'ai oublié de préciser que LPCSTR est bien un typedef spécifique de Windows :aie:.

Citation:

Envoyé par bianconeri

Fonctionne avec ou sans le point virgule

C'est vrai qu'en réfléchissant bien je n'ai jamais essayé sans. Mais dans la logique je le laisserais pour les raisons que j'ai avancées plus tôt.
13/08/2008, 15h58
Noxen

Citation:

Envoyé par spoutspout

Ca reste une définition (dans laquelle tu places l'implémentation).

Définition ou déclaration ? De toute façon, ici, il y a déclaration et définition, c'est d'ailleurs une fonction inline.

Bon, effectivement, avec ou sans le ';', j'arrive à compiler (du moins sous visual, il faudrait tester avec gcc vu la discipline de M$ à respecter les normes qu'ils n'ont pas imposées eux-même). J'avoue que ça me laisse quand même perplexe. Si tu regardes tous les exemples de codes donnés sur le site, ils sont sans ';' dans le cas d'une fonction définie dans la classe. :koi::koi:

Code:

string strTerm = p_lpszTerm;

et

Code:

string strTerm(p_lpszTerm);

devraient être équivalent, ce sont deux appels au constructeur. Ceci-dit, après test, même l'opérateur d'affectation est compatible avec LPCSTR.
13/08/2008, 16h22
zabibof

spoutspout a raison, LPCSTR n'est qu'un typedef, ça ne devrait pas poser de problème (du moment qu'on include windows.h)

En tout cas, passer de CString à std::string ne devrait pas poser de problème puisque tous deux peuvent travailler sur un const char* et LPCSTR est un const char*
13/08/2008, 16h30
Spout

Citation:

Envoyé par Noxen

Définition ou déclaration ? De toute façon, ici, il y a déclaration et définition, c'est d'ailleurs une fonction inline.

Au temps pour moi, je voulais bien dire déclaration.

Citation:

Envoyé par Noxen

Code:

string strTerm = p_lpszTerm;

et

Code:

string strTerm(p_lpszTerm);

devraient être équivalent, ce sont deux appels au constructeur. Ceci-dit, après test, même l'opérateur d'affectation est compatible avec LPCSTR.

Attention, dans le premier cas, tu appelles le constructeur puis l'opérateur d'affectation, tandis que dans le deuxième cas, tu n'appelles que le constructeur avec un paramètre.
J'ai parlé un peu vite en disant que c'était faux :aie: : ce n'est juste pas la même chose.
13/08/2008, 16h36
Spout

Citation:

Envoyé par zabibof

LPCSTR n'est qu'un typedef, ça ne devrait pas poser de problème (du moment qu'on include windows.h)

WinNT.h suffit, mais attention aux includes sauvages.

Citation:

Envoyé par zabibof

En tout cas, passer de CString à std::string ne devrait pas poser de problème puisque tous deux peuvent travailler sur un const char* et LPCSTR est un const char*

Dans la théorie oui, sauf qu'il faut se retaper tous les appels aux méthodes de CString et en adapter le contexte à un appel à une fonction équivalente de la STL. Attention donc au rechercher/remplacer!
Mais dans le fond, je suis d'accord avec toi, même si je préfère travailler explicitement sur un const char* plutôt qu'un LPCSTR.
13/08/2008, 16h52
stillman

Citation:

Envoyé par spoutspout

Citation:

Envoyé par Noxen

Code:

string strTerm = p_lpszTerm;

et

Code:

string strTerm(p_lpszTerm);

devraient être équivalent, ce sont deux appels au constructeur. Ceci-dit, après test, même l'opérateur d'affectation est compatible avec LPCSTR.

Attention, dans le premier cas, tu appelles le constructeur puis l'opérateur d'affectation, tandis que dans le deuxième cas, tu n'appelles que le constructeur avec un paramètre.
J'ai parlé un peu vite en disant que c'était faux :aie: : ce n'est juste pas la même chose.

Non, Noxen a raison, il s'agit de deux appels au constructeur. La première ligne est une initialisation, pas une affectation.
13/08/2008, 18h58
Spout

Citation:

Envoyé par stillman

Non, Noxen a raison, il s'agit de deux appels au constructeur. La première ligne est une initialisation, pas une affectation.

Au temps pour moi. Après vérification avec VS2005, on ne passe bien que dans le constructeur, et non pas dans l'opérateur =.
Pourtant je l'aurai parié :? !
14/08/2008, 07h30
bianconeri
Bonjour,

Merci pour toutes vos informations, mais j'ai une petite question qui n'a plus trop a voir avec le sujet ! (Modo -> Dois-je ouvrir une nouvelle discution)

Quel est l'équivalent de la ligne ci-dessous
Code:

1 2 //LPSTR lpczCursor = (LPSTR) (LPCSTR) strTerm;
en cherchant un peu je pense que cela devrait ressemblé à cela ?
Code:

1 2 char * lpczCursor = (char *) strTerm.c_str();
Est-ce bien cela ?
14/08/2008, 09h39
Spout
C'est exactement ça, mais c'est une erreur car tu "contournes" le fait que la chaîne renvoyée ne peut être modifiée. En effet, un LPCSTR est un pointeur constant, c'est à dire que l'objet pointé (chaîne de caractère) ne peut être modifié.

Dans ton cas ici, la chaîne de caractère renvoyée par c_str() est en plus une partie d'un objet plus gros, std::string. Cela veut dire que si tu utilises le pointeur non constant lpczCursor pour modifier cette chaîne, tu vas accéder et modifier une partie de l'objet std::string sans passer par les méthodes disponibles. Tu risques alors de "foutre le souk" dans le conteneur :D. C'est pour cette raison que c_str() renvoie un const char*, et non pas un char*.

Si tu veux tout de même modifier la chaîne renvoyée, tu as deux solution:
- Agir directement sur le std::string en utilisant ses méthodes
- Copier la chaîne renvoyée dans un tableau de char via sprintf() par exemple, et modifier cette copie à ta guise