problem with "connect" signal/slot

[rimie23]

Hi
i have class QOgrerenderwindow

void QOgreRenderWindow::mousePressEvent(QMouseEvent* evt)
{
    if (evt->buttons() & Qt::MiddleButton)
        mousePressStartPoint = Ogre::Vector2((float)evt->pos().x() / mVp->getActualWidth(),(float)evt->pos().y() / mVp->getActualHeight());
    else
        emit mousePress(evt);
}
void QOgreRenderWindow::wheelEvent(QWheelEvent * evt)
{
    Ogre::Real valueZoom;
 
    if(evt->delta() < 0)
        valueZoom = 1;
    else if(evt->delta() > 0)
        valueZoom = -1;
 
    mCamera->moveRelative(Ogre::Vector3(0, 0, valueZoom/* * dist*/));
 
    emit wheel(evt);
}

void QOgreRenderWindow::mousePressEvent(QMouseEvent* evt)
{
    if (evt->buttons() & Qt::MiddleButton)
        mousePressStartPoint = Ogre::Vector2((float)evt->pos().x() / mVp->getActualWidth(),(float)evt->pos().y() / mVp->getActualHeight());
    else
        emit mousePress(evt);
}
void QOgreRenderWindow::wheelEvent(QWheelEvent * evt)
{
    Ogre::Real valueZoom;

    if(evt->delta() < 0)
        valueZoom = 1;
    else if(evt->delta() > 0)
        valueZoom = -1;

    mCamera->moveRelative(Ogre::Vector3(0, 0, valueZoom/* * dist*/));

    emit wheel(evt);
}

To copy to clipboard, switch view to plain text mode 

and in Qogrewidget i have

connect(mQOgreRenderWindowList[i],SIGNAL(mousePress(QMouseEvent*)),this,SLOT(mousePress(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseRelease(QMouseEvent*)),this,SLOT(mouseRelease(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseMove(QMouseEvent*)),this,SLOT(mouseMove(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(wheel(QWheelEvent*)),this,SLOT(wheel(QWheelEvent*)));
void QOgreWidget::mousePress(QMouseEvent* evt)
{
    Q_UNUSED(evt);
    std::cout << sender()->objectName().toStdString() << " mousePress" << std::endl;
}
void QOgreWidget::wheel(QWheelEvent * evt)
{
    Q_UNUSED(evt);
    std::cout << sender()->objectName().toStdString() << " wheel" << std::endl;
}

connect(mQOgreRenderWindowList[i],SIGNAL(mousePress(QMouseEvent*)),this,SLOT(mousePress(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseRelease(QMouseEvent*)),this,SLOT(mouseRelease(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseMove(QMouseEvent*)),this,SLOT(mouseMove(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(wheel(QWheelEvent*)),this,SLOT(wheel(QWheelEvent*)));
void QOgreWidget::mousePress(QMouseEvent* evt)
{
    Q_UNUSED(evt);
    std::cout << sender()->objectName().toStdString() << " mousePress" << std::endl;
}
void QOgreWidget::wheel(QWheelEvent * evt)
{
    Q_UNUSED(evt);
    std::cout << sender()->objectName().toStdString() << " wheel" << std::endl;
}

To copy to clipboard, switch view to plain text mode 

the problem is when i debug it enter to wheel function only
even when i clic with mouse it dos not enter to mousepress
have you an idea why ?

[amleto]

show complete compilable example yadda yadda.

since it works for mouse wheel you have probably made a very simple mistake somewhere.

Have you checked console output to check that the connection is made successfully?
Is mousePress actually a slot?
Do you connect mousePress at the same place as wheel ?

All of these questions would be answered already if you would provide complete, compilable examples, instead of doing what you always do and post code fragments with no context.


This code

connect(mQOgreRenderWindowList[i],SIGNAL(mousePress(QMouseEvent*)),this,SLOT(mousePress(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseRelease(QMouseEvent*)),this,SLOT(mouseRelease(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseMove(QMouseEvent*)),this,SLOT(mouseMove(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(wheel(QWheelEvent*)),this,SLOT(wheel(QWheelEvent*)));

connect(mQOgreRenderWindowList[i],SIGNAL(mousePress(QMouseEvent*)),this,SLOT(mousePress(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseRelease(QMouseEvent*)),this,SLOT(mouseRelease(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseMove(QMouseEvent*)),this,SLOT(mouseMove(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(wheel(QWheelEvent*)),this,SLOT(wheel(QWheelEvent*)));

To copy to clipboard, switch view to plain text mode 

is meaningless - you don't even show what method it belongs to!

COMPLETE COMPILABLE EXAMPLES COMPLETE COMPILABLE EXAMPLES COMPLETE COMPILABLE EXAMPLES COMPLETE COMPILABLE EXAMPLES COMPLETE COMPILABLE EXAMPLES COMPLETE COMPILABLE EXAMPLES COMPLETE COMPILABLE EXAMPLES COMPLETE COMPILABLE EXAMPLES COMPLETE COMPILABLE EXAMPLES COMPLETE COMPILABLE EXAMPLES

[wysota]

I can only say that it is very unusual to try to pass event objects through signals to foreign slots.

[rimie23]

complete compilable example

yes i know that i must give you complete compilable exemple
and i know if i give you that you can help me
BUT
i tried to think an exemple like mine and withoute ogre i do not found an idea how i do it

Have you checked console output to check that the connection is made successfully?

yes , when i clic with mouse i do not see "mouse pressed" but if i execute run wheel i see " wheel"

Is mousePress actually a slot?

yes

Do you connect mousePress at the same place as wheel ?

yes

look that is my code in qogrerender and qogrewidget (it is complete code but with ogre )
QOgreRenderWindow.hQOgreWidget.hQOgreRenderWindow.cppQOgreWidget.cpp



and thanks to accept help me


EDIT

I can only say that it is very unusual to try to pass event objects through signals to foreign slots.

yes i know that is not normal
look at this post it will explain to you why i use 2 clases
http://www.qtcentre.org/threads/4915...ht=#post220802

i choose the secon advice of ChrisW67

You have 13 Ogre widgets sharing the same Ogre scene. One widget is the left half and the other 12 are in the grid. Put each Ogre widget inside the group boxes you already have. (Actually it could all be a single QGridLayout but you need to walk before running)

i subdevice ogrewidget on twoo class : qogrerenderwindow and qogrewidget

i create my object in "qogrewidget" BUT i render theme in "qogrerenderwindow" to render them i use camera for that i put mousepresseeven in "qogrerenderwindow" and because
the objects rendred are in oogrewidget class i need to make mousepress too

for that i make this melange

[StrikeByte]

Did you check if the event gets triggered? (qDebug :P)

---

Added after 7 minutes:

in the code box on the forum you use this code:

if (evt->buttons() & Qt::MiddleButton) //you filter out the middle mouse button (no signal will be emitted)
    mousePressStartPoint = Ogre::Vector2((float)evt->pos().x() / mVp->getActualWidth(),(float)evt->pos().y() / mVp->getActualHeight());
  else
    emit mousePress(evt);

if (evt->buttons() & Qt::MiddleButton) //you filter out the middle mouse button (no signal will be emitted)
    mousePressStartPoint = Ogre::Vector2((float)evt->pos().x() / mVp->getActualWidth(),(float)evt->pos().y() / mVp->getActualHeight());
  else
    emit mousePress(evt);

To copy to clipboard, switch view to plain text mode 

in the attachment you use this code:

if (evt->button() == Qt::LeftButton) //here you filter the left mouse button (no signal will be emitted)
      mousePressStartPoint = Ogre::Vector2((float)evt->pos().x() / mVp->getActualWidth(),(float)evt->pos().y() / mVp->getActualHeight());	
    else
      emit mousePress(evt);

if (evt->button() == Qt::LeftButton) //here you filter the left mouse button (no signal will be emitted)
      mousePressStartPoint = Ogre::Vector2((float)evt->pos().x() / mVp->getActualWidth(),(float)evt->pos().y() / mVp->getActualHeight());	
    else
      emit mousePress(evt);

To copy to clipboard, switch view to plain text mode 

what mouse button are you clicking and expecting an emit?

[wysota]

yes i know that is not normal
look at this post it will explain to you why i use 2 clases
http://www.qtcentre.org/threads/4915...ht=#post220802

i choose the secon advice of ChrisW67

i subdevice ogrewidget on twoo class : qogrerenderwindow and qogrewidget

i create my object in "qogrewidget" BUT i render theme in "qogrerenderwindow" to render them i use camera for that i put mousepresseeven in "qogrerenderwindow" and because
the objects rendred are in oogrewidget class i need to make mousepress too

for that i make this melange

I don't care that you use multiple widgets. This doesn't explain why you are doing weird things with the event system. Maybe if you tell us what you are trying to achieve, we'll find a better solution.

[rimie23]

i just change the code in attachement because the first code did not work i tried to change it

what mouse button are you clicking and expecting an emit?

what mouse button are you clicking and expecting an emit?

To copy to clipboard, switch view to plain text mode 

what you mean

if (evt->buttons() & Qt::MiddleButton) //you filter out the middle mouse button (no signal will be emitted)

if (evt->buttons() & Qt::MiddleButton) //you filter out the middle mouse button (no signal will be emitted)

To copy to clipboard, switch view to plain text mode 

why ? but why enter to weheel and not at the press

what you propose?


EDIT
when i run my program i see this
interfacee.jpg

my main objectif is when i clic on the big groupbox (big widget) i add an object(and see it in the others )

i created the widget (smalls and big ) in the class "Qogrewidget" and to render the scene in it i apply qogrerenderwidow like that(all that is in the attachement)

void QOgreWidget::setViewNum(int num)
{
 
 
   --------------------------------------------------------------
 
    if(num == 1)
     {
 
		  QWidget *central = new QGroupBox("My Scene", this);
     float wi=central->width();
	 float hi=central->height();
       QOgreRenderWindow * orw = new QOgreRenderWindow(QString("View2"),mSceneMgr,mRoot,nb_cam,/*central*/wi,hi); 
 
    QHBoxLayout* Layout = new QHBoxLayout();
    Layout->addWidget(orw);
 
    central->setLayout(Layout);
 
         mVerticalLayout->addWidget(central);
 
     }
---------------------------------
 for(int i = 0;i<mQOgreRenderWindowList.count();i++)
    {
        connect(mQOgreRenderWindowList[i],SIGNAL(keyPress(QKeyEvent*)),this,SLOT(keyPress(QKeyEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(keyRelease(QKeyEvent*)),this,SLOT(keyRelease(QKeyEvent*)));
       connect(mQOgreRenderWindowList[i],SIGNAL(mousePress(QMouseEvent*)),this,SLOT(mousePress(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseRelease(QMouseEvent*)),this,SLOT(mouseRelease(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseMove(QMouseEvent*)),this,SLOT(mouseMove(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(wheel(QWheelEvent*)),this,SLOT(wheel(QWheelEvent*)));
    }

void QOgreWidget::setViewNum(int num)
{
    

   --------------------------------------------------------------

    if(num == 1)
     {

		  QWidget *central = new QGroupBox("My Scene", this);
     float wi=central->width();
	 float hi=central->height();
       QOgreRenderWindow * orw = new QOgreRenderWindow(QString("View2"),mSceneMgr,mRoot,nb_cam,/*central*/wi,hi); 
		
    QHBoxLayout* Layout = new QHBoxLayout();
    Layout->addWidget(orw);
     
    central->setLayout(Layout);
     
         mVerticalLayout->addWidget(central);
		     
     }
---------------------------------
 for(int i = 0;i<mQOgreRenderWindowList.count();i++)
    {
        connect(mQOgreRenderWindowList[i],SIGNAL(keyPress(QKeyEvent*)),this,SLOT(keyPress(QKeyEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(keyRelease(QKeyEvent*)),this,SLOT(keyRelease(QKeyEvent*)));
       connect(mQOgreRenderWindowList[i],SIGNAL(mousePress(QMouseEvent*)),this,SLOT(mousePress(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseRelease(QMouseEvent*)),this,SLOT(mouseRelease(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(mouseMove(QMouseEvent*)),this,SLOT(mouseMove(QMouseEvent*)));
        connect(mQOgreRenderWindowList[i],SIGNAL(wheel(QWheelEvent*)),this,SLOT(wheel(QWheelEvent*)));
    }

To copy to clipboard, switch view to plain text mode 

it work with wheel when i make the curseur of mouse on widget i see that the camera change it position

Now i use
mousepresseevent in qogrerenderwindow : because in this class i create my camera and when i click with mouse i need camera
i use mousepress in qogrewidget : because i clic on this widget and my objects are created in this class

I did not found another idea to do what i want
Now , to add object by cliking with mouse i tried firstly to use simple "mousePressEvent(QKeyEvent* evt)"
it dos not enter to it

[StrikeByte]

I'll use this code as example

if (evt->button() == Qt::LeftButton) //here you filter the left mouse button (no signal will be emitted)
      mousePressStartPoint = Ogre::Vector2((float)evt->pos().x() / mVp->getActualWidth(),(float)evt->pos().y() / mVp->getActualHeight());
    else
      emit mousePress(evt);

if (evt->button() == Qt::LeftButton) //here you filter the left mouse button (no signal will be emitted)
      mousePressStartPoint = Ogre::Vector2((float)evt->pos().x() / mVp->getActualWidth(),(float)evt->pos().y() / mVp->getActualHeight());
    else
      emit mousePress(evt);

To copy to clipboard, switch view to plain text mode 

in this code if you click the left mouse button the signal mousePress won't be emitted.


and did you check if the mouseEvent (not the slot) is triggered when you click a mousebutton?

void QOgreRenderWindow::mousePressEvent(QMouseEvent* evt)
{
  qDebug() << "YEA event works!";
  if (evt->buttons() & Qt::MiddleButton)
    mousePressStartPoint = Ogre::Vector2((float)evt->pos().x() / mVp->getActualWidth(),(float)evt->pos().y() / mVp->getActualHeight());  
  else
    emit mousePress(evt);
}

void QOgreRenderWindow::mousePressEvent(QMouseEvent* evt)
{
  qDebug() << "YEA event works!";
  if (evt->buttons() & Qt::MiddleButton)
    mousePressStartPoint = Ogre::Vector2((float)evt->pos().x() / mVp->getActualWidth(),(float)evt->pos().y() / mVp->getActualHeight());  
  else
    emit mousePress(evt);
}

To copy to clipboard, switch view to plain text mode 

maybe an alternative to the signal/slot, is installing an eventFilter see http://doc.qt.io/qt-4.8/eventsandfilters

[rimie23]

@StrikeByte: i tried it it does not enter to it

[wysota]

So you mean you have 12 small widgets that should display the same data as the big widget? Are you doing that by adding same objects to those small widgets as you add to the big one? Isn't it simpler to make all of them share the same data instead of doing all the crazy stuff with events?

[rimie23]

So you mean you have 12 small widgets that should display the same data as the big widget?

Yes, But each widget display the same data with by the uses of defferent camera (widget1with camera1,widget2with camera2...............)

Isn't it simpler to make all of them share the same data instead of doing all the crazy stuff with events?

i tried to make that but it dos not work
the only idea that work is this

[StrikeByte]

If all the small widgets have the same object, but different views (camera's), why does the main window need to know if you clicked at one?
Is it to show the small widget in the big widget with the same view as the smaler one?


Is it possible to attach the entire project, so i can take a look at it after work?

[rimie23]

yes , i will be very hapyy if you can help me
But it is with ogre
( i will upload all the project)

[StrikeByte]

Thats not a problem i've tested ogre once
i can reinstall the lib so i can test it

[rimie23]

OK
i will upload it (all the projcet with files and dll)in website and i will put the link her

[wysota]

i tried to make that but it dos not work

So make it work

the only idea that work is this

Don't be offended but that's not the brightest approach. You are duplicating data and effectively spending much effort on synchronizing your scenes instead of focusing on the real task.

A trivial approach to solving your problem is having a single controller object, notifying all other objects upon changes in the scene. Then you can do your duplication based on high-level definition of things that happen around. So instead of transmitting "canvas x was clicked in position a,b" you can transmit "object A has been created at coordinates x,y,z". Then each "view" can query the "scene" for the object and create its local incarnation of it. Right now since all "views" can have different transformations, you'll spend lots and lots of time trying to recalculate coordinates from all other views based on their projections.

[rimie23]

Then you can do your duplication based on high-level definition of things that happen around

i don't understand what you mean her

So instead of transmitting "canvas x was clicked in position a,b" you can transmit "object A has been created at coordinates x,y,z". Then each "view" can query the "scene" for the object and create its local incarnation of it. Right now since all "views" can have different transformations, you'll spend lots and lots of time trying to recalculate coordinates from all other views based on their projections.

i do not understand very good the idea

and i do not know if i can do that in ogre or no

[wysota]

i don't understand what you mean her

I mean that you shouldn't care what was clicked where but rather what happened as a result of that click.

and i do not know if i can do that in ogre or no

OGRE has nothing to do with this. You'd only use OGRE inside the view, the scene would be composed of pure C++/Qt functionality. So yes, you can do that with OGRE. I'm not saying that's the best approach (having some kind of scene-view separation inside OGRE would be better) but it's definitely better than what you are doing now. If you still do not understand what I mean, read about the "model-view-controller" design pattern (just don't look at english wikipedia article on it, as the article is very poor).

[rimie23]

the scene would be composed of pure C++/Qt functionality.

no i use functionality of ogre to create scene (the scene is created in ogre ) and renderd using ogre camera ,functionality of ogre too
and in window of ogre the window of ogre we put than the window of ogre in widdget

EDIT
in the first time when i was just one class i can display many view as i want using thiss function

void OgreWidget::createViewports(void)
    {
    mWindow->addViewport(mCamera, 0, 0, 0, 0.5, 1); // large left half viewport
    int columns = 3;
    int rows = 4;
    for(int y = 0; y < rows; y++)
    for(int x = 0; x < columns; x++)
    {
    mWindow->addViewport(mCamera,
    x + y * columns + 1, // each viewport needs a unique zorder value
    0.5f + (0.5f / columns) * x, // left edge
    (1.0f / rows) * y, // top edge
    0.5f / columns, // tile width
    1.0f / rows); // tile height
    }
    }

void OgreWidget::createViewports(void)
    {
    mWindow->addViewport(mCamera, 0, 0, 0, 0.5, 1); // large left half viewport
    int columns = 3;
    int rows = 4;
    for(int y = 0; y < rows; y++)
    for(int x = 0; x < columns; x++)
    {
    mWindow->addViewport(mCamera,
    x + y * columns + 1, // each viewport needs a unique zorder value
    0.5f + (0.5f / columns) * x, // left edge
    (1.0f / rows) * y, // top edge
    0.5f / columns, // tile width
    1.0f / rows); // tile height
    }
    }

To copy to clipboard, switch view to plain text mode 

you can see that to render we use mwindow : it is ogre window , we render on it and put the result in widget
i change this idea because i can't put the sub-view in widgets indepedent on the the big widget(or big view)
when i write

setCentralWidget(ogrewidget)

setCentralWidget(ogrewidget)

To copy to clipboard, switch view to plain text mode 

it show to me all the view in one widget

that's why i use the new idea

and that's why i afraid that your idea do not work in ogre because we use the same window of ogre

[wysota]

no i use functionality of ogre to create scene (the scene is created in ogre ) and renderd using ogre camera ,functionality of ogre too

I don't think you understand.