Connect QLineEdit with my class

[jsulm]

@navyseabear said in Connect QLineEdit with my class:

connect(GameWindow.ui->input_edit(),SIGNAL(returnPressed()),this,SLOT(checkResult()));

This is wrong, remove () after input_edit

[navyseabear]

@navyseabear said in Connect QLineEdit with my class:

/home/tce/CTux/mytextitem.cpp:18: error: expected primary-expression before ‘.’ token

still the same error:

/home/tce/CTux/mytextitem.cpp:18: error: expected primary-expression before ‘.’ token

What do I do wrong??

Tobias

[mpergand]

try:
connect(ui->input_edit,SIGNAL(returnPressed()),this,SLOT(checkResult()));

[navyseabear]

now I have

error: ‘ui’ was not declared in this scope

I have both files

#include "gamewindow.h"
#include "ui_gamewindow.h"

added. So I should work. Or do I make something wrong in my thinking?

Tobias

[VRonin]

Are you inside gamewindow.cpp?

[navyseabear]

No, inside my class (mytextitem).

I thought I should put it there, because I create several instances of my class during runtime.

Tobias

[SGaist]

Hi and welcome to devnet,

You can't connect a class. You need an object instance for that.

[navyseabear]

Hmm, and how do I do that?
I'm really new in C++ and Qt. Normaly I use Pascal and VB.

Tobias

[jsulm]

@navyseabear said in Connect QLineEdit with my class:

Hmm, and how do I do that?

You should already have an instance of GameWindow somewhere in your app. How else would you use it?
Also it is bad praxis to access private data of a class (like ui) from other classes. You should do this connection from GameWindow not from mytextitem.

[navyseabear]

OK, so far I understood that.
My problem is just that I have created the class, because I create there individual objects, which then walk independently on my QGraphicView.
Using the QLineEdit, each individual object should now check whether the input for the object is correct and then destroy itself or execute a corresponding routine.
Probably I have not quite understood this with the SIGNAL / SLOTS, but how else should I distribute the input of the QLineEdit to all objects so that they then compare the value? I do not know how many objects are created at runtime.

Tobias

[SGaist]

Do you mean that you have many QGraphicsItem objects in your scene that you want to do a check on based on the content of the QLineEdit ?

[navyseabear]

Yes, and I do not know how to connect them to the QLineEdit.

Tobias

[mrjj]

@navyseabear
Hi
Where you create the new mytextitem, there you would hook it up.
Pr mytextitem.
Hopefully its same class as where the UI lives.

[navyseabear]

Here is my code:

gamewindow.h

#ifndef GAMEWINDOW_H
#define GAMEWINDOW_H
#include <QGraphicsScene>
#include <QWidget>
#include "mytextitem.h"

namespace Ui {
class GameWindow;
}

class GameWindow : public QWidget
{
    Q_OBJECT

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

private slots:
    void on_pushButton_2_clicked();

private:
    Ui::GameWindow *ui;
    QGraphicsScene *scene;
    MyTextItem *textitem;
    int timerID;

protected:
    void timerEvent(QTimerEvent *event);
    void on_lineEdit_returnPressed();

};

#endif 
// GAMEWINDOW_H

gamewindow.cpp

#include "gamewindow.h"
#include "ui_gamewindow.h"
#include "mytextitem.h"
#include <QGraphicsTextItem>
#include <QGraphicsScene>

GameWindow::GameWindow(QWidget *parent) :
    QWidget(parent),
    ui(new Ui::GameWindow)private slots:

    void on_pushButton_2_clicked();
{
    ui->setupUi(this);
    this->setWindowState(Qt::WindowMaximized);
    scene = new QGraphicsScene(this);
}

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

void GameWindow::on_pushButton_2_clicked()
{
    int Xmax;
    int Ymax;
    float Scale;
    Xmax = ui->graphicsView->width()-10;
    Ymax = ui->graphicsView->height()-10;
    Scale = (float)Xmax / (float)800;
     if (Scale < 1) {
         Scale = 1;
     }
    Xmax = Xmax / Scale;
    Ymax = Ymax / Scale;
    scene->setSceneRect(0,0,Xmax,Ymax);
    QTransform transform;
    transform.scale(Scale,Scale);
    ui->graphicsView->setTransform(transform);
    ui->graphicsView->update();
    ui->graphicsView->setScene(scene);
    textitem = new MyTextItem("TEST",80);
    textitem->setX_Max(Xmax);
    textitem->setY_Max(Ymax);
    textitem->setSpeed(100);
    scene->addItem(textitem);
}

void GameWindow::on_lineEdit_returnPressed()
{
    // some code
}

mytextitem.h

// mytextitem.h

#ifndef MYTEXTITEM_H
#define MYTEXTITEM_H

#include <QPainter>
#include <QGraphicsTextItem>
#include <QObject>

// class for customization
class MyTextItem : public QGraphicsTextItem
{
    Q_OBJECT

public:
    MyTextItem(QString,int);
    void StartTimer();
    void StopTimer();
    void setX_Max(int);
    void setY_Max(int);
    void setSpeed(int);
    int getSpeed();

private:
    int timerID;
    int Y_Pos;
    int Y_Max;
    int X_Max;
    int Speed;

public slots:
    void checkResult();

protected:
    void timerEvent(QTimerEvent *event);
};

#endif 
// MYTEXTITEM_H

mytextitem.cpp

// mytextitem.cpp

#include "mytextitem.h"
#include "gamewindow.h"
#include "ui_gamewindow.h"

MyTextItem::MyTextItem(QString txt, int spd)
{
    this->setPlainText(txt);
    this->setDefaultTextColor(Qt::red);
    this->setX(100);
    this->setY(0);
    setSpeed(spd);
    Y_Pos = 0;
    StartTimer();

   connect(ui->input_edit,SIGNAL(returnPressed()),this,SLOT(checkResult()));
}

void MyTextItem::StartTimer()
{
    timerID = startTimer(Speed);
}

void MyTextItem::StopTimer()
{
    killTimer(timerID);
}

void MyTextItem::setX_Max(int x)
{
    X_Max = x;
}

void MyTextItem::setY_Max(int y)
{
    Y_Max = y;
}

void MyTextItem::setSpeed(int spd)
{
    Speed = spd;
}

int MyTextItem::getSpeed()
{
    return Speed;
}

void MyTextItem::checkResult()
{
   // some code
}

void MyTextItem::timerEvent(QTimerEvent *event)
{
    this->setY(Y_Pos);
    Y_Pos = Y_Pos + 1;
    if (Y_Pos > Y_Max){
        StopTimer();
    }
}

Any idea how I can do it the correct way?

Tobias

[mrjj]

Hi
in void GameWindow::on_pushButton_2_clicked()
after
textitem = new MyTextItem("TEST",80);
you can connect

qDebug() << "connect works: " << connect(ui->input_edit,SIGNAL(returnPressed()),textitem ,SLOT(checkResult()));

(must say connect works:true)

and remove connect from
MyTextItem::MyTextItem(QString txt, int spd)

[navyseabear]

Thanks!!!! :-)
Now it works!

Thank you very very much!!!!!

Tobias

[mrjj]

@navyseabear
Super :)
When using the SIGNAL()/ SLOT() syntax, its good to always check
it returns true. ( the qDebug() part)
There is actually a new syntax that can fail compile time if something wrong.
https://wiki.qt.io/New_Signal_Slot_Syntax