J'instancier une méthode

Version imprimable

1 pièce(s) jointe(s)

Bonjour,

j'ai une methode ''SENDFRAME" qu est sur un fichier ANCREwifi.cpp.

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
/**
 * \brief Fonction permettant d'envoyer un message à une ancre
 *
 * \param num : numero de l'ancre destinataire (0 pour toutes)
 * \param sendFrameBuffer : buffer d'envoi du message
 * \param length : longueur du message
 *
 */
void anchor::sendFrame( int num, QByteArray sendFrameBuffer, unsigned int length)
{
    unsigned int start, stop;
    if( num == 0)
    {
        start = 1;
        stop = NbAnchor;
    }
    else
    {
        start = stop = num;
    }
    for(unsigned int i=start; i<=stop; i++)
    {
        if( (Connected[i] == CONNECTED) ||  (Connected[i] == CONNECTING) )
        {
            socket.at(i)->write(sendFrameBuffer, length);
        }
    }
}

je veux utiliser (instancier) cette methode dans un autre fichier "mainwindow.cpp"

ici

Code:

1
2
3
4
5
6
7
8
9
10
11
//...............................................................START.......................................................//
 
void MainWindow::on_start_clicked()
{
    unsigned int lenght;
 
                //QByteArray frame ;
                Frame[1]  = 0x80;
 
                anchor::sendFrame(1,Frame,lenght);
}

quand je compile ca me sort ca ==

Code:

'cannot call member function void anchor::sendFrame( int , QByteArray sendFrameBuffer, unsigned int ) without objets

de quels objets il parle ??

je vous mets aussi le AncreWifi.h

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
class anchor : public QObject
{
    Q_OBJECT
public:
    explicit anchor(QObject *parent = 0);
 
signals:
    void setLed(unsigned char nbLed);
    void ackMode(unsigned int);
    void ackStart(unsigned int);
    void ackStop(unsigned int);
    void ackTabTag(unsigned int);
 
public slots:
 
    void sendFrame(int num, QByteArray sendFrameBuffer, unsigned int length);

04/08/2017, 11h12
Bktero

Une méthode ne s'instancie pas, elle s'appelle puisqu'une une méthode est une fonction. En revanche, une classe s'instancie. L'instanciation d'une classe résulte en la création d'un objet, qu'on appelle instance. Pour appeler une méthode, il faut un objet (une instance) sur lequel la méthode s'appuie. Et c'est exactement ton message d'erreur : tu n'as pas d'objet de type anchor avec lequel faire my_anchor.sendFrame().

Il est possible d'appeler une méthode sans objet à la condition que cette méthode soit statique. Voir http://en.cppreference.com/w/cpp/language/static

Une autre solution est de ne pas utiliser de classe anchor. Tu peux définir des fonctions en dehors de toute classe en C++ (contrairement à Java par exemple).

Tes questions ressemblent à des questions de débutant en C++. Avant d'aller plus loin, il serait bien que tu relises ton tutoriel sur les classes en C++. ;)
04/08/2017, 11h22
koupiss

ca fait longtemps que j'ai pas programmer en c++, c pour ca j'ai perdu qlq bases.

oui justement ma methode est bien static je sais qu il ya une astuce pour la faire appeler sans instance mais j ai oublié comment faire ...

merci pour ton conseil
04/08/2017, 11h41
Bktero

Citation:

oui justement ma methode est bien static

Qu'est ce qui te fait dire ça ?
04/08/2017, 12h02
koupiss

"this" n'existe pas
04/08/2017, 12h05
bacelar
Citation:

"this" n'existe pas

Ça c'est une conséquence du fait qu'une méthode est statique mais qu'est-ce qui te fait dire que la méthode "sendFrame" est statique dans :
Code:

1 2 3 4 5 6 7 class anchor : public QObject { ... void sendFrame(int num, QByteArray sendFrameBuffer, unsigned int length); ... }
04/08/2017, 12h12
koupiss

Ah d'accord, oui vous avez raison ce n 'est pas une statique, du coup il me faut absolument une instance (objet) .mais je pense que votre solution

Citation:

Une autre solution est de ne pas utiliser de classe anchor. Tu peux définir des fonctions en dehors de toute classe en C++

est plus adéquate avec mon programme parce que je vais utilisé SENDFRAME ailleurs aussi. donc si vous pouvez m éclaircir SVP comment faire cette Méthode je serai très reconnaissant :)

Je n'ai pas de projet Qt prêt sous la main, donc j'ai mimé ton organisation :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 #include <iostream> using namespace std; namespace anchor { void sendFrame(int num, int start, int stop) { cout << num << start << stop << endl; } } void clicked_start() { anchor::sendFrame(42, 0, 666); }
J'utilise un namespace anchor qui est optionnel, mais qui permet d'avoir des fonctions avec des noms simples sans risquer un conflit avec des noms d'autres bibliothèques ou d'autres parties du programme. Ca permet de plus une "thématisation" des fonctions.

Note au passage que tu as une écriture très similaire avec une méthode statique :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 class anchor { public: static void sendFrame(int num, int start, int stop) { cout << num << start << stop << endl; } }; void clicked_start() { anchor::sendFrame(42, 0, 666); }
Il est souvent dit que static n'est pas orienté objet car......tu n'as pas d'objet pour appeler la méthode ! Quitte à faire du structuré au milieu de POO, je trouve la solution des fonctions libres (mais fournies dans un namespace) plus "logique".

04/08/2017, 14h13
koupiss

quand j'ai fait ca dans le mainwindow.cpp, ca me donne une noubelle erreur

Citation:

''namespace anchor{}'redeclared as différent kind of symbol

?? et si j'utilise un autre namespace cela ne va pas faire reference a la methode souhaité -_-