mapToScene returns position with offset

[Peter_Dev]

@Christian-Ehrlicher Only change i made was in this function:

void MainWindow::mousePressEvent(QMouseEvent *event)
{
    QPoint origin = event->pos();
    QPointF res = ui->graphicsView->mapToScene(mapTo(ui->graphicsView,event->pos()));//here
    QPoint p = res.toPoint(); 
    qDebug() << p.rx();
    qDebug()<<p.ry();
    origin.setX(origin.rx()-50);
    origin.setY(origin.ry()-50);
    QSize pos(100,100);
    rubberBand->setGeometry(QRect(origin, pos));
    QPalette pal;
    pal.setBrush(QPalette::Highlight, QBrush(Qt::green));
    rubberBand->setPalette(pal);
    rubberBand->show();
}

[jsulm]

@Peter_Dev said in mapToScene returns position with offset:

PointF res = ui->graphicsView->mapToScene(mapTo(ui->graphicsView,event->pos()));//here

What is "mapTo" here?
Did you check all the pointers as already suggested?

[Peter_Dev]

@jsulm Checking now. mapTo maps click position relative to graphicsView, i guess. Am i wrong?

[jsulm]

@Peter_Dev Did you debug and/or check the pointers? This is actually the first thing to do if your app is crashing...

[Peter_Dev]

@jsulm it crushes at mapTo, when it tries to call mapToParent inside itself. I think problem is in passing ui->graphicsView as first argument in mapTo

[jsulm]

@Peter_Dev Again: did you check the pointers? Are those valid?

[Peter_Dev]

@jsulm Sorry, i'm new here. I don't quite understand what you mean by checking pointers, but as i understood you, yes, all pointers are valid. Before i put mapTo here, everything was working correctly. Sorry for wasting your time, but it's really important project for me, so i need your help very much.

[JonB]

@Peter_Dev
Your code shows

mapTo(ui->graphicsView,event->pos())

And you wrote

@jsulm it crushes at mapTo, when it tries to call mapToParent inside itself. I think problem is in passing ui->graphicsView as first argument in mapTo

If you're saying that crashes, you need to look at its code to determine why. If you're asking for our help, how can we help if you don't show mapTo()'s code?

[jsulm]

@JonB I think it's the mapTo from QMainWindow from QWidget :-)
This was my mistake also when reading this code :-)

[Peter_Dev]

@JonB

QPoint QWidget::mapTo(const QWidget * parent, const QPoint & pos) const
{
    QPoint p = pos;
    if (parent) {
        const QWidget * w = this;
        while (w != parent) {
            Q_ASSERT_X(w, "QWidget::mapTo(const QWidget *parent, const QPoint &pos)",
                       "parent must be in parent hierarchy");
            p = w->mapToParent(p); 
            w = w->parentWidget();
        }
    }
    return p;
}

QPoint QWidget::mapToParent(const QPoint &pos) const
{
    return pos + data->crect.topLeft(); //crushes here
}

[JonB]

@jsulm , @Peter_Dev
OK then. https://doc.qt.io/qt-5/qwidget.html#mapTo

QPoint QWidget::mapTo(const QWidget *parent, const QPoint &pos) const

Translates the widget coordinate pos to the coordinate system of parent. The parent must not be nullptr and must be a parent of the calling widget.

I think we can assume parent, ui->graphicsView, is not nullptr, else ui->graphicsView->mapToScene(...) would dump, and it wouldn't be to do with mapTo(). But you should check that ui->graphicsView pointer value.

If not that, I can't quite get my head around it, but ui->graphicsView cannot be parent of "calling widget" which is MainWindow, can it?

EDIT This post crossed with @Peter_Dev's latest.

 return pos + data->crect.topLeft(); //crushes here

Assuming you are seeing this in the debugger, what is the value of data and what does it point to?

[Peter_Dev]

@JonB written that "No such data". So, how to solve my problem? Should i use mapTo or something else?

[JonB]

@Peter_Dev
For the record, am I right that you (or at least the Qt mapTo() code you show) are not compiling for debug? I believe in debug it would have hit the

            Q_ASSERT_X(w, "QWidget::mapTo(const QWidget *parent, const QPoint &pos)",
                       "parent must be in parent hierarchy");

You'll have to ask the experts what's going on/you should do here, I think QMainWindow::mapTo(ui->graphicsView) is the wrong thing to be doing, but I don't claim to know what you should be doing....

[Peter_Dev]

Ok, i understand. I'll be trying to figure it out. Thanks to everyone who helped me here and sorry for wasting your time.