Discussion :

[Ganondorf]

Bonjour à tous,

Je me permet de venir vers vous car j'ai un problème lors de l'appel de la fonction delete sur un objet stocké dans une QList.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void TcpServer::OnSocketStateChanged(TcpSocket *socket, int state)
{
    switch (state)
    {
    case TcpSocket::UnconnectedState:
        qDebug() << "Deconnexion du socket" << socket->socketDescriptor();
        socketList.removeOne(socket);
        delete socket; // aie !!!
        break;
    }
}

Voici le header de tcpsocket si besoin.

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
#ifndef TCPSOCKET_H
#define TCPSOCKET_H
 
#include <QtNetwork/QTcpSocket>
 
class TcpSocket
    : public QTcpSocket
{
    Q_OBJECT
public:
    TcpSocket() = delete;
    TcpSocket(const TcpSocket &) = delete; // desactive le constructeur de copie
    TcpSocket& operator=(const TcpSocket &) = delete; // desactive l'opérateur d'affectation
 
    TcpSocket(qintptr handle, QObject* parent = nullptr);
    ~TcpSocket();
 
signals:
    void SigReadyRead(TcpSocket* socket);
    void SigStateChanged(TcpSocket* socket, int state);
};
 
#endif // TCPSOCKET_H

La liste d'objet de type pointeur TcpSocket

Code :

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

QList<TcpSocket*> socketList;

Lors du delete, l'application plante, je tiens à préciser que TcpSocket est une classe qui hérite de QTcpSocket et TcpServer hérite de QTcpServer.
D'après ce que j'ai pu comprendre dans la doc QList::removeOne ne fait pas appel au deleteur de l'objet, si j'affiche l'adresse du pointeur 'socket' il semble toujours actif.
Si l'un d'entre vous a une idée je suis preneur !

Bonne fin de journée à tous.

Anthony

[dourouc05]

si j'affiche l'adresse du pointeur 'socket' il semble toujours actif.

delete ne met pas l'adresse à zéro, il appelle le destructeur de l'objet et s'arrête là. Peux-tu toujours appeler des méthodes de l'objet sans plantage ? (Même si on nage là en plein dans les comportements non définis…)

Pourquoi as-tu besoin de pointeurs dans ta liste ? Ne peux-tu pas avoir un QList<TcpSocket> socketList, en récupérant des références avec [] ? (Dans l'idée qu'un code sans pointeur est un code avec moins de chances de planter…)

Vu ton erreur, je regarderais de plus près le destructeur de TcpSocket. Quand tu utilises un débogueur, où ton code plante-t-il ?

Pourquoi utiliser removeOne ? Vu ton architecture, je vois le besoin, mais ça serait plus propre et largement plus efficace d'utiliser un index avec erase.

[Ganondorf]

Salut,

Alors après pas mal de test pour voir d’où venait l'erreur je n'ai toujours pas trouvé !
J'ai donc changé mon fusil d'épaule et je passe maintenant par une liste de 'shared_ptr', je n'ai plus qu'a supprimer un élément dans la liste pour automatiquement appeler le deleteur.
En ce qui concerne l'utilisation des pointeurs c'est une sale manie que j'ai, j'ai commencé le C et maintenant le C++ un peu comme un gland sans forcement comprendre exactement tout ce que je fais !
Merci pour ton aide.

Anthony