How to use methods of an object from a different class?

J.Hilk

mmh,
did you declare send as a slot in your mainwindow cpp?

private slots:
//public slots:
     void send(QString s);

Edit:

just noticed, your Send-Function wants a char and you send a QString that will most likly not work.

change one or the other. I would suggest chaning the Mainwindow::Send to accept QString, makes it easier.

jsulm

@bask185 said in How to use methods of an object from a different class?:

Keyboard *keyboard = new Keyboard();
connect(keyboard, &Keyboard::KeyPressed, this &MainWindow::send);

Aren't you missing a comma?

Keyboard *keyboard = new Keyboard();
   connect(keyboard, &Keyboard::KeyPressed, this, &MainWindow::send);

jsulm

@J.Hilk said in How to use methods of an object from a different class?:

//Something like this
Keyboard kBoard;

Please do NOT do this in the constructor: kBoard would be a local variable and would disappear when the constructor finishes!

J.Hilk

@jsulm @bask185
Oh, sorry about that.
ApparentlyI missed that when I wrote my answer.

I tried to cover all bases, just in case, but that's an error my from my part.

bask185

@jsulm said in How to use methods of an object from a different class?:

Aren't you missing a comma?

That is what I thought as well, but no. Adding comma will give errors

@jsulm said in How to use methods of an object from a different class?:

Please do NOT do this in the constructor: kBoard would be a local variable and would disappear when the constructor finishes!

at first I used Keyboard *keyboard = new Keyboard(); with those strange pointer things. Now I declare keyboard with the rest of my variables and I initialize it in the constructor.

@J.Hilk said in How to use methods of an object from a different class?:

ust noticed, your Send-Function wants a char and you send a QString that will most likly not work

I also noticed that and changed it already.

There are a few more things I found out. I was using 'keyboard' at first, but that turns red so it might interfere. I swapped out 'keyboard' for Kboard. But my way of decalaring it in the variable section and initializing it in the constructor suddenly upset the compiler so I went back to the constructor and now I have:

Keyboard *Kboard = new Keyboard();
   connect(Kboard, &Keyboard::KeyPressed, this &MainWindow::send);

also in keyboard.h I changed private by public because I was getting an error message there at first.

But I am still having this error in the connect function:

C:\Users\sknippels\Documents\GUI\mainwindow.cpp:51: error: invalid use of member (did you forget the '&' ?)
    connect(Kboard, &Keyboard::KeyPressed, this &MainWindow::send);
                                                             ^

how my code looks like atm:

Keyboard *Kboard = new Keyboard();  // mainwindow.cpp constructor
   connect(Kboard, &Keyboard::KeyPressed, this &MainWindow::send);

void MainWindow::send(QString c)  // send function in mainwindow.ccp
{
  serial->write("c"); // still does not send the value of c, just "c"
  qDebug() << c;
}

void Keyboard::on_a_clicked() // push button function in keyboard.cpp
{
   QString c;
   c = ui->a->text();
   emit KeyPressed(c);
   }

public slots: // keyboard.h
    void KeyPressed(QString c); // also tried KeyPressed(QString); and KeyPressed(); no effect
}

private slots: //mainwindow.h
    void serialReceived(); // one of many other functions
    void send(QString c);

I think we have all the pieces of this strange C++ puzzle now

jsulm

@bask185 You're still missing the comma, please add it as it is needed. And if you then still get an error then please post it here: just saying there is an error does not help.

J.Hilk

@bask185

additionally to what @jsulm said,


public slots: // keyboard.h
    void KeyPressed(QString c); // also tried KeyPressed(QString); and KeyPressed(); no effect

ought to be a signal

signals: // keyboard.h
    void KeyPressed(QString c); // also tried KeyPressed(QString); and KeyPressed(); no effect

not a slot

bask185

@jsulm said in How to use methods of an object from a different class?:

@bask185 You're still missing the comma, please add it as it is needed. And if you then still get an error then please post it here: just saying there is an error does not help.

Keyboard *Kboard = new Keyboard();
   //connect(Kboard, &Keyboard::KeyPressed, this &MainWindow::send);
   connect(Kboard, &Keyboard::KeyPressed, this, &MainWindow::send);

V

C:\Users\sknippels\Documents\GUI\mainwindow.cpp:52: error: undefined reference to `Keyboard::KeyPressed(QString)'

C:\Users\sknippels\Documents\GUI\keyboard.cpp:76: error: undefined reference to `Keyboard::KeyPressed(QString)'

C:\Users\sknippels\Documents\build-GUI-Desktop_Qt_5_8_0_MinGW_32bit-Debug\debug\moc_keyboard.cpp:251: error: undefined reference to `Keyboard::KeyPressed(QString)'

collect2.exe:-1: error: error: ld returned 1 exit status

jsulm

@bask185 Please read what @J-Hilk wrote

bask185

@jsulm said in How to use methods of an object from a different class?:

@bask185 Please read what @J-Hilk wrote

I did and changed it and I can finally atleast build it. only pressing the 'a' on the keyboard does not do anything.

I know that void Keyboard::on_a_clicked() gets executed using Qdebug I can also tell c has the correct value.

But in mainwindow void MainWindow::send(QString c) does not get executed. I added qDebug() << "hello world"; but nothing happens :(

jsulm

@bask185 Can you show the whole code where

Keyboard *Kboard = new Keyboard();
   //connect(Kboard, &Keyboard::KeyPressed, this &MainWindow::send);
   connect(Kboard, &Keyboard::KeyPressed, this, &MainWindow::send);

is embedded?

bask185

@jsulm said in How to use methods of an object from a different class?:

@bask185 Can you show the whole code where
Keyboard *Kboard = new Keyboard();
   //connect(Kboard, &Keyboard::KeyPressed, this &MainWindow::send);
   connect(Kboard, &Keyboard::KeyPressed, this, &MainWindow::send);
is embedded?
I don't know what you mean??

I moved Keyboard *Kboard; to the global variables of mainwindow but it seems that does not make a difference. Button 'a' of the keyboard still does not trigger the send function in the main window

jsulm

@bask185 Can you please post your code? Else I don't know what is wrong.

J.Hilk

@bask185

what @jsulm meant, is, can you post the whole content between the brackets {} where

Keyboard *Kboard = new Keyboard();
   //connect(Kboard, &Keyboard::KeyPressed, this &MainWindow::send);
   connect(Kboard, &Keyboard::KeyPressed, this, &MainWindow::send);

is copied out of.

bask185

The one with the comma remains ;)

#include "mainwindow.h" // mainwindow.cpp
#include "ui_mainwindow.h"
#include <QPixmap>
#include <QDebug>
#include <QPalette>
#include <QtSerialPort>
#include "keyboard.h"
#include "numpad.h"
QSerialPort *serial;
void readCommand();

char lettres[5][40][16];     // initializes all to be used variables;
int index1 = 0;
int index2 = 0;
int colom[5];
int row[5];
int colorIndex[5];
char c;
Keyboard *Kboard;

MainWindow::MainWindow(QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::MainWindow)
{
    ui->setupUi(this);
    serial = new QSerialPort();
    serial->setPortName("COM4");
    serial->open(QIODevice::ReadWrite);
    serial->setBaudRate(QSerialPort::Baud115200);
    serial->setDataBits(QSerialPort::Data8);
    serial->setParity(QSerialPort::OddParity);
    serial->setStopBits(QSerialPort::OneStop);
    //serial.setFlowControl(QSerialPort::NoFlowControl);
    //serial.open(QIODevice::ReadWrite);

    QPixmap logo(":/resources/img/logo.jpg");
    int w = ui->label->width();
    int h = ui->label->height();
    ui->label->setPixmap(logo.scaled(w,h,Qt::KeepAspectRatio));

    for(int i=0;i<5;i++) {
        for (int j=0;j<40;j++) {
            for (int k=0;k<16;k++){
                lettres[i][j][k] =' ';   // initializes the letters array
            }
        }
    }
   connect(serial, &QSerialPort::readyRead, this, &MainWindow::serialReceived);

   Kboard = new Keyboard();
   connect(Kboard, &Keyboard::KeyPressed, this, &MainWindow::send);
}

MainWindow::~MainWindow()
{
   delete ui;
}

void MainWindow::serialReceived()
{
    QByteArray b = serial->readAll();
    qDebug() << b;
}

void MainWindow::send(QString c)
{
  serial->write("c");
  qDebug() << "hello world " << c;
}

#include "keyboard.h" //keyboard.cpp
#include "ui_keyboard.h"
#include <QtSerialPort>
#include "mainwindow.h"


bool caps = true, shift = true;

Keyboard::Keyboard(QWidget *parent) :
    QDialog(parent),
    ui(new Ui::Keyboard)

{
    ui->setupUi(this);
}

Keyboard::~Keyboard()
{
    delete ui;
}

void Keyboard::on_a_clicked()
{
   QString c;
   c = ui->a->text();
   emit KeyPressed(c);
   qDebug() << c;

   if(caps == true && shift == false){ // disables the effect of the shift key when any lettre has been pressed.
       shift = false;
       on_shift1_clicked();
   }
}

#ifndef MAINWINDOW_H //main window.h
#define MAINWINDOW_H

#include <QMainWindow>
#include "keyboard.h"
#include "numpad.h"


namespace Ui {
class MainWindow;
}

class MainWindow : public QMainWindow
{
    Q_OBJECT

public:
    explicit MainWindow(QWidget *parent = 0);
    ~MainWindow();

private slots:
    void send(QString c);

private:
    Ui::MainWindow *ui;
    Keyboard *keyboard;
    Numpad *numpad;
};

#endif // MAINWINDOW_H

#ifndef KEYBOARD_H
#define KEYBOARD_H

#include <QDialog>

namespace Ui {  //keyboard.h
class Keyboard;
}

class Keyboard : public QDialog
{
    Q_OBJECT

public:
    explicit Keyboard(QWidget *parent = 0);
    ~Keyboard();
signals:
    void KeyPressed(QString);

private slots:
    void on_a_clicked();
    

private:
    Ui::Keyboard *ui;
};

#endif // KEYBOARD_H

jsulm

@bask185 One note: why do you use global variables? This is bad habit!
For example Kboard should be a member variable of your MainWindow class.

bask185

idk to get it to work I suppose, but I should declare it in the constructor above the connect function??

In processing I sometimes use global variables because I can acces and modify them in every class I write. In Qt I know very little of what the firetruck I am doing. I do know that in Qt it is a lot more work to get an exact same application than in processing..

J.Hilk

@bask185 said in How to use methods of an object from a different class?:

Keyboard *keyboard;

I think I found the mixup,

You have Keyboard as a priavte member of your mainwindow.h, as it should be, and as a global Variable too.

I think the instance of your keyboard, that you show and have input of, is not the one you connected your Signal from.

Change that.

jsulm

@bask185 said in How to use methods of an object from a different class?:

void MainWindow::send(QString c)
{
serial->write("c");
qDebug() << "hello world " << c;
}

Can you try to comment out serial->write("c"); and try again?
Also: I guess you want to send the string/character entered by user over serial bus, right?
In that case you should change

serial->write("c");

to

serial->write(c.toLatin1()); // If you only use ASCII characters

Eeli K

@bask185 I didn't know you know so little about C++ and Qt. It's true that C++ is in many ways a low-level language. Qt takes it a bit higher, but certainly it's still means more work for certain things than many other languages. But on the other hand C++ is on the most popular and used general purpose languages in the world, and unlike with processing, you can call yourself a real programmer if you master C++ and Qt. Every language has its place and has pros and cons. If you want to just get one program done in your life C++ isn't the right language for you, otherwise it pays back what you put into learning it.