Class, inline et AnsiString

Bonjour,

J'ai créé une Class:

Code:

---

1 2 3 4 5 6 7 8 9 10

class Client { public: int Handle; AnsiString Name; AnsiString Password; char Tel[10]; void Create(int handle, AnsiString Name, AnsiString password, char tel[10]); };

---

et un inline

Code:

---

1 2 3 4 5 6 7 8

inline void Client::Create(int handle, AnsiString name, AnsiString password, char tel[10]) { Handle = handle; Name = name; Password = password; strcpy(Tel, tel); }

---

mais j'ai un message d'avertissement inline...

Egalement déclaré un vecteur

Code:

---

1 2 3

vector <Client> vecClient; vector <Client>::iterator iterClient;

---

et dans ma class de la form

Code:

---

1 2 3

vecClient vClient; iterClient itClient;

---

et j'ai un message d'erreur comme quoi Create n'existe pas quand j'utilise le vector pour créer un client

Code:

---

1 2	vClient.push_back(Create(2, "Toto", "****", "0320534704"));

---

pouvez-vous m'aiguiller?

Bonsoir,
Juste une proposition:
....
Ne faut il pas déclarer une instance de la class 'Client' dans la TForm ?
C'est a dire mettre en .H:

Code:

---

1 2	Client AClient;

---

et après appeler sa méthode avec :

Code:

---

1 2	AClient.Create(...);

---

@ + ;)

Bonjour

A la place de ta fonction Create, mets un constructeur pour ta classe Client, le constructeur porte le mon de la classe :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

// dans ta déclaration de classe Client(int handle, AnsiString Name, AnsiString password, char tel[10]);   // dans le fichier .cpp Client::Client(int handle, AnsiString name, AnsiString password, char tel[10]) { Handle = handle; Name = name; Password = password; strcpy(Tel, tel); }

---

Code:

---

1 2	vector <Client> vecClient;

---

Là tu crée un vecteur d'objets de type Client et ce vecteur s'appelle vecClient.
Si tu veux remplir le vecteur, tu y mets des objets Client.

Code:

---

1 2	Client Zidane = new Client(2, "Toto", "****", "0320534704"); vecClient.push_back(*Zidane);

---

Quelque chose comme ça doit marcher.
Laisse tomber les inline, tu les utiliseras plus tard.

Un apprentissage des notions de base de C++ au moyen d'un bon bouquin te faciliterais beaucoup les choses (C++ facile, N° 889, editions Marabout, pas cher)

Citation:

---

mais j'ai un message d'avertissement inline...

---

Les fonctions inline doivent être entièrement définies dans des en-têtes (pas dans des .cpp). La raison est simple : un appel à une fonction inline étant remplacé par le code de la fonction, si celui-ci n'est pas disponible le compilo ne pourra pas faire son boulot.

Citation:

---

Client Zidane = new Client(2, "Toto", "****", "0320534704");
vecClient.push_back(*Zidane);

---

Rien de mieux pour chopper des fuites de mémoire (qui va détruire l'instance que tu as allouée dynamiquement ?)

Code:

---

vecClient.push_back(Client(2, "Toto", "****", "0320534704"));

---

Citation:

---

Les fonctions inline doivent être entièrement définies dans des en-têtes (pas dans des .cpp). La raison est simple : un appel à une fonction inline étant remplacé par le code de la fonction, si celui-ci n'est pas disponible le compilo ne pourra pas faire son boulot.

---

Dans mon exemple c'est ce que j'ai fait, la fonction inline est déclarée juste en dessous de ma class Client dans le même fichier.h

Citation:

---

Rien de mieux pour chopper des fuites de mémoire (qui va détruire l'instance que tu as allouée dynamiquement ?)

Code:

---

1 2	vecClient.push_back(Create(2, "Toto", "****", "0320534704"));

---

---

Donc la solution c'est quoi?
Je me base sur le livre de G Leblanc.
Sauf que cette syntaxe me cause une erreur disant que Create n'est pas un membre de la class Client!

Citation:

---

Dans mon exemple c'est ce que j'ai fait, la fonction inline est déclarée juste en dessous de ma class Client dans le même fichier.h

---

Ah, autant pour moi...
Commençons dont par le début : quel est le message d'avertissement que te donne le compilo ?

Citation:

---

Donc la solution c'est quoi?
Je me base sur le livre de G Leblanc.
Sauf que cette syntaxe me cause une erreur disant que Create n'est pas un membre de la class Client!

---

Code:

---

vecClient.push_back(Create(2, "Toto", "****", "0320534704"));

---

Je n'ai pas écrit ça... Relis mon post.

Désolé, par moyen de me connecter :(
Voici le message de mon compilateur:

Citation:

---

[C++ Avertissement] fichier.h(45): W8026 Les fonctions prendre les arguments classe-par-valeur ne sont pas développées inline

---

On dirait que ceci est dû aux types AnsiString dans ma class

Ca veut dire que comme tu passes tes paramètres par valeur, et que le constructeur par copie n'est pas trivial, le compilo ne peut pas inliner la fonction puisque cela signifierait éliminer l'appel au constructeur par copie.

Passe simplement tes paramètres AinsiString par référence constante :

Code:

---

Create(int handle, const AnsiString& name, const AnsiString& password, char tel[10])

---

PS : ils se sont pas foulés pour la traduction des messages d'erreur... c'est vraiment caca

Du coup j'obtiens ce message d'erreur!

Citation:

---

[C++ Avertissement] Fichier.h(97): W8027 Les fonctions contenant missing return statements ne sont pas développées inline

---

L'avertissement surligne la fin de la déclaration inline!

J'ai une autre demande,
Où et comment initialiser une class Toto?
ex:

Code:

---

1 2 3 4 5 6 7 8 9

class Toto { public: AnsiString Name; int age; Toto(); ~Toto(); };

---

Les constructeurs et destructeurs sont-ils obligatoires dans une appli bcb?
Où initialiser ma class Toto et comment dans mon *cpp?

Code:

---

1 2 3

... Toto toto("tata", 23); // me donne une erreur de correspondance

---

Citation:

---

[C++ Avertissement] Fichier.h(97): W8027 Les fonctions contenant missing return statements ne sont pas développées inline

---

On peut voir ta fonction ?

Citation:

---

Où et comment initialiser une class Toto?

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

class Toto { public :   Toto(const AnsiString& Name_, int Age_);   ... };   Toto::Toto(const AnsiString& Name_, int Age_) : Name(Name_), Age(Age_) {   }

---

Franchement c'est pas encore clair pour moi les class, il y a trop de manières différents de les aborder!
Voici mon code:

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

class CLIENT { public: int Handle; AnsiString UserName; AnsiString Password; int Nk; int Nb; char Key[32];   Create(int H, const AStr &N, const AStr &P, int nk, int nb, char key[32]); CLIENT() {} // Constructeur par défaut };   inline CLIENT::Create(int H, const AStr &N, const AStr &P, int nk, int nb, char key[32]) { Handle = H; UserName = N; Password = P; Nk = nk; Nb = nb; strcpy(Key, key); }

---

J'obtiens donc une erreur,
Comment initialiser CLIENT, sachant que j'ai déclaré un vector de n CLIENT

Code:

---

1 2 3

typedef vector<CLIENT> VCLIENT; typedef vector<CLIENT>::iterator itVCLIENT;

---

Comme on te l'a déjà dit, il faut remplacer ta fonction Create par un constructeur. Un constructeur a toujours le même nom que la classe (voir mon exemple sur Toto, ou encore des bouquins / tutoriels si nécessaire).

mon constructeur c'est

Code:

---

CLIENT() {}

---

et il se trouve dans la déclaration de ma

Code:

---

1 2	class CLIENT{... }

---

Create c'est une fonction quelconque également déclarée dans cette class CLIENT.

Comment construire CLIENT vu que j'ai déclaré un vector <CLIENT>?

Merci

Citation:

---

Envoyé par Laurent Gomila

Comme on te l'a déjà dit, il faut remplacer ta fonction Create par un constructeur

---

Code:

---

1 2 3 4 5 6 7 8 9

inline CLIENT::CLIENT(int H, const AStr &N, const AStr &P, int nk, int nb, char key[32]) { Handle = H; UserName = N; Password = P; Nk = nk; Nb = nb; strcpy(Key, key); }

---

Si tu utilise le vector de la STL, les objets que tu mets dans ton vecteur (ici ta classe Client) doivent obligatoirement avoir :
* un constructeur par copie public
* un constructeur par défaut public
* un destructeur public
* un opérateur d'affectation public

Sinon la conservation des données dans ton vector quand tu travailles dessus n'est pas garantie.
Le constructeur et le destructeur peuvent être implicites. Ces 2 méthodes n'ont aucun type.

Par contre toutes les autres, comme la fonction Create auquel tu sembles tenir doivent avoir un type : void, int, etc... Par défaut c'est int.
Donc inline Toto(...) équivaut à inline int Toto(...) et doit renvoyer un int.

Citation:

---

* un constructeur par défaut public

---

Pas spécialement. Si on ne pouvait stocker dans un conteneur que des classes possédant obligatoirement un constructeur par défaut, ce serait bien embêtant.