Discussion :

[James_]

Bonjour,

J'ai créé une Class:

Code :

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

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 :

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

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 :

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

1
2
3
 
    vector <Client> vecClient;
    vector <Client>::iterator iterClient;

et dans ma class de la form

Code :

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

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 :

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

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

pouvez-vous m'aiguiller?

[Sunchaser]

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 :

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

1
2
 
Client AClient;

et après appeler sa méthode avec :

Code :

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

1
2
 
AClient.Create(...);

@ +

[Orangina]

Bonjour

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

Code :

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

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 :

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

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 :

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

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)

[Laurent Gomila]

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.

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 :

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

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

[James_]

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

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

Code :

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

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!

[Laurent Gomila]

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 ?

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 :

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

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

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

[James_]

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

[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

[Laurent Gomila]

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 :

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

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

[James_]

Du coup j'obtiens ce message d'erreur!

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

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

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 :

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

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

[Laurent Gomila]

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

On peut voir ta fonction ?

Où et comment initialiser une class Toto?

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
class Toto
{
public :
 
    Toto(const AnsiString& Name_, int Age_);
 
    ...
};
 
Toto::Toto(const AnsiString& Name_, int Age_) :
Name(Name_),
Age(Age_)
{
 
}

[James_]

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 :

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

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

1
2
3
 
typedef vector<CLIENT> VCLIENT;
typedef vector<CLIENT>::iterator itVCLIENT;

[Laurent Gomila]

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

[James_]

mon constructeur c'est

Code :

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

CLIENT() {}

et il se trouve dans la déclaration de ma

Code :

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

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

[Laurent Gomila]

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

Code :

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

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);
    }

[Orangina]

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.

[Laurent Gomila]

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