Discussion :

[wadcyr8_197]

j'ai un petit problème de passage d'argument dans une fonction que je ne comprends pas.

j'ai une classe Csock (pour créer des sockets) qui contient une surcharge du constructeur de la forme

Code :

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

Csock(string address, int port,network_type TCPorUDP);

jusque là rien de bien méchant !

j'ai fais une classe Telnet qui gère une connexion telnet (logique !) qui contient un objet Csock (connexion sur le port 23). Elle a cette forme :

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
25
class Telnet
{
    public:
 
    Telnet(); // contructeur utilisant comme argument par défaut
                //172.17.0.194:23 pour la connexion telnet
    Telnet(std::string addr, int port); // contructeur prenant en argument  
                                                 //l'adresse et le port pour la connexion
                                                //Telnet
    ~Telnet();
    std::string get_notify_addr();
    int get_notify_port();
    void set_notify_addr(std::string addr);
    void set_notify_port(int port);
 
    int sender(std::string commande);
    std::string reception();
 
    private:
 
...
    std::string notify_addr;
    int reader_port;
    Csock reader;
};

le problème c'est que quand j'essaie de compiler j'ai un problème à la construction de mon objet Telnet :

Code :

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

error: no match for call to `(Csock) (std::string&, int&, network_type)'

alors que mon contructeur ne semble pas avoir de problème :

Code :

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

1
2
3
4
5
6
Telnet::Telnet()
{
    notify_addr.assign( "172.17.0.194");
    notify_port = 23;
    reader(notify_addr, notify_port, TCP);
};

si quelqu'un a un idée, parce que là je suis à court, je comprends pas d'où peut venir mon problème.

pour info, j'utilise code::blocks sous windows

merci d'avance

[yorke]

Y'a un truc qui me parait bizard, tu cree ton reader en statique, et tu appelle un constructeur ensuite?

Declare:

CSock *reader;

et dans ton constructeur:

this->reader=new CSock(notify_addr, notify_port, TCP);

Edit:

Si tu veux le laisse en statique, il faut que tu creer une autre fonction dans ta classe CSock, equivalent à ton constructeur surchargé, genre: init(string address, int port,network_type TCPorUDP) et que tu l'appelle dans ton constructeur telnet.
Mais c'est plus pratique en dynamique

[befalimpertinent]

Tes inclusions de headers sont t'elles correctes ? Pas d'inclusions cycliques ou référence croisées?

[wadcyr8_197]

oui pour les header pas de problèmes.

j'ai isolé un peu le problème en essayant d'instancier directement un objet Csock dans une fonction main bateau (j'ai rien d'autre) et ça plante quand même.


Sinon, york, ça marche aussi en passant par un pointeur, mais va falloir que je remodifie tout mon prog derrière en conséquence.

Mais d'un autre côté, ce que je comprends pas, c'est que cette classe on l'a déjà utilisée ailleurs (pour la même chose), je n'ai pas eu de problèmes et la seule modification que j'ai faite depuis à été de changer le nom et de mettre en français les commentaire. c'est vraiment à se et

en tout cas merci !

[Alp]

Et pourquoi pas les listes d'initialisation :
http://c.developpez.com/faq/cpp/?pag...EUR_liste_init

Tu fais un constructeur avec des arguments par défaut, et tu utilises les listes d'initialisation pour créer ton reader.

[wadcyr8_197]

ça passe pas, mais tant pis, j'ai encore trop de bug et trop peu de temps pour continuer à m'arracher les cheuveux là dessus, je reste avec la méthode de Yorke qui marche bien.

merci quand même