[SOLVED] Calling a function from another class

[p3c0]

Hi,

The glavni class object is destroyed as soon as the function exits and hence nothing is displayed.
You should rather create the object on heap.

[Demmy]

@t3685

Yes, the label does not change. I previously put in some other functions from the same class to call and they do not execute. If I put in a variable, it does not change. It is as if the function does not trigger at all.

@p3c0

Can you suggest how it should look? Would be much appriciated.

[p3c0]

Just use the new keyword.
So,
@
glavni *g = new glavni;
g->show();
g->elipsaOFF();
@

[Demmy]

[quote author="p3c0" date="1403359381"]Just use the new keyword.
So,
@
glavni *g = new glavni;
g->show();
g->elipsaOFF();
@[/quote]

ok, this works but it creates a completely new window. I would like to change the existing one.

[p3c0]

bq. change the existing one.

Do you mean you want to add this widget ?

[Demmy]

[quote author="p3c0" date="1403361027"]bq. change the existing one.

Do you mean you want to add this widget ?[/quote]

Yes - add to the existing widget, glavni.cpp is a widget with a graphicsview and some other stuff in it and nasascena.cpp is basically qgraphicsscene which is imported in on the view in glavni.cpp. I want to be able to change things on the existing widget of glavni.cpp when I click inside the scene, changing the label name (using elipsaOFF()) was just a way for me to check on how it all works.

I have to figure it out somehow :)

For a future reference and why I'm asking for this:
I would like to get coordinates on a mouse click on the scene (already done below in the nasascena.cpp). They already are an extern float, but I would like to trigger a function inside glavni.cpp which would for example draw a line with coordinates provided.
I thought calling a function would be way simpler to do though.

[p3c0]

So basically you can use "setScene":http://qt-project.org/doc/qt-4.8/qgraphicsview.html#setScene function to set the QGraphicsScene to a QGraphicsView.
So in general for eg.
@
QGraphicsScene *m_scene = new QGraphicsScene(this);
ui->graphicsView->setScene(m_scene);
@

Adjust the QGraphicView and QGraphicsScene Objects as per your need.

[Demmy]

[quote author="p3c0" date="1403362183"]So basically you can use "setScene":http://qt-project.org/doc/qt-4.8/qgraphicsview.html#setScene function to set the QGraphicsScene to a QGraphicsView.
So in general for eg.
@
QGraphicsScene *m_scene = new QGraphicsScene(this);
ui->graphicsView->setScene(m_scene);
@

Adjust the QGraphicView and QGraphicsScene Objects as per your need.[/quote]
it is already done:
glavni.cpp
@
#include "glavni.h"
#include "ui_glavni.h"
#include "nasascena.h"
#include "nasaelipsa.h"

#include<QGraphicsScene>
glavni::glavni(QWidget *parent) :
QMainWindow(parent),
ui(new Ui::glavni)
{
ui->setupUi(this);

scene= new nasaScena();
ui->graphicsView->setScene(scene);
elipsa[0] = new nasaElipsa(); //irrelevant in this case
scene->addItem(elipsa[0]); //irrelevant in this case

}

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

void glavni::elipsaOFF()
{
//Pritisnuto=true;
//elipsaToggle();
ui->label->setText("YOU PRESSED IT");
//elipsaON();
}
@
at this point it is very important to me to somehow trigger this function though, scene and view work fine.

Pic:

http://i58.tinypic.com/33e6duh.jpg

but it is all irrelevant at this point, all I want to do is to execute elipsaOFF() in glavni.cpp from nasascena.cpp without creating a completely new window.

[p3c0]

Ok. So better would be to use the "Signals and Slots":http://qt-project.org/doc/qt-4.8/signalsandslots.html mechanism.
Emit a custom signal say for eg. onSceneClicked() from mousePressEvent() function of nasaScena class.
And connect this signal to elipsaOFF() (you need to declare this as a slot)

[Demmy]

[quote author="p3c0" date="1403363202"]Ok. So better would be to use the "Signals and Slots":http://qt-project.org/doc/qt-4.8/signalsandslots.html mechanism.
Emit a custom signal say for eg. onSceneClicked() from mousePressEvent() function of nasaScena class.
And connect this signal to elipsaOFF() (you need to declare this as a slot)[/quote]

Interesting approach, will try. I thought something like calling a function would be the most basic way to perform it.

[p3c0]

Signals and Slots is the most recommended and an important aspect of Qt programming.

[Demmy]

[quote author="p3c0" date="1403363782"]Signals and Slots is the most recommended and an important aspect of Qt programming.[/quote]

Ok, I tried this (correct plz if I misunderstood)
@#include "nasascena.h"
#include "glavni.h"

#include <QGraphicsSceneMouseEvent>

nasaScena::nasaScena()
{
connect(mousePressEvent, SIGNAL(event), glavni, SLOT(glavni::elipsaOFF()));
}

void nasaScena::mousePressEvent(QGraphicsSceneMouseEvent *event)
{
//glavni g;
//g.elipsaOFF();
update();
QGraphicsScene::mousePressEvent(event);
}@

This does not work; returns "expected primary-expression before ',' token...

points at "glavni," at the end of it in connect.

[p3c0]

That won't work since event is not a signal and the Connect's syntax is wrong.

First you must declare a new signal in nasaScena.h

nasaScena.h
@
signals:
void onSceneClicked();
@

Emit this signal in mousePressEvent
@
void nasaScena::mousePressEvent(QGraphicsSceneMouseEvent *event)
{
emit onSceneClicked();
}
@

Then in Main class glavni declare elipsaOFF() as a slot.

glavni.h
@
protected slots:
void elipsaOFF();
@

and then in glavni.cpp connect the signal onSceneClicked() to the newly created slot. So

glavni.cpp
@
ui->setupUi(this);
scene= new nasaScena();
connect(scene,SIGNAL(onSceneClicked()),this,SLOT(elipsaOFF()));
@

Thus the signal should trigger the slot.

[Demmy]

[quote author="p3c0" date="1403365994"]That won't work since event is not a signal and the Connect's syntax is wrong.

First you must declare a new signal in nasaScena.h

nasaScena.h
@
signals:
void onSceneClicked();
@

Emit this signal in mousePressEvent
@
void nasaScena::mousePressEvent(QGraphicsSceneMouseEvent *event)
{
emit onSceneClicked();
}
@

Then in Main class glavni declare elipsaOFF() as a slot.

glavni.h
@
protected slots:
void elipsaOFF();
@

and then in glavni.cpp connect the signal onSceneClicked() to the newly created slot. So

glavni.cpp
@
ui->setupUi(this);
scene= new nasaScena();
connect(scene,SIGNAL(onSceneClicked()),this,SLOT(elipsaOFF()));
@

Thus the signal should trigger the slot.[/quote]

Did all that, now it says: "undefined reference to 'nasaScena::onSceneClicked()'
this is on the @emit onSceneClicked();@ line. Googled for it, no luck. Why this won't work is beyond me. I declared the signal the way you said:

in nasascena.h
@signals:
void onSceneClicked();@

and it gives me shit when I try to emit it from nasascena.cpp

[Demmy]

Ok, apparently I had to add Q_OBJECT macro in nasascena.h and run qmake for it to work. Although I'd still like to know how to run a function from the outside I declare this problem solved. Many thanks to p3c0 for bearing with me.

[t3685]

Hi Demmy,

What you seem to confused about is the following:

• you have to call functions on instances of a class, in your case glavni

• what you are doing is making a glavni instance g, and calling your function on that object

• you need to which object you want to call your function on

• best to get reading on C++ or object oriented programming in general

[Demmy]

[quote author="t3685" date="1403428669"]Hi Demmy,

What you seem to confused about is the following:

• you have to call functions on instances of a class, in your case glavni

• what you are doing is making a glavni instance g, and calling your function on that object

• you need to which object you want to call your function on

• best to get reading on C++ or object oriented programming in general[/quote]
  I thought it was something like that after p3c0 gave me an example with new glavni; but sadly I still don't understand how to call on existing object or how is it in this case even defined (can't call on "this" from other cpp file :D ). I guess I could use some serious reading.

[t3685]

Yes, reading up a bit would do you good :). It's not difficult but you need to know some basic things.