QPainter, paintEvent как правильно рисовать?

[n4ela]

Задача стоит. отмечать на QWidget точки при клики мыши.
Код:
cpp:

Код

C++ (Qt)
Widget::Widget(QWidget *parent)
    : QWidget(parent), ui(new Ui::Widget)
{
    ui->setupUi(this);
    x = 10;
    y = 10;
}
void Widget::paintEvent( QPaintEvent *event )
{
    QPainter painter( this );
    painter.setPen( QPen( Qt::black, 3 ) );
    painter.drawPoint( QPoint( x, y ) );
}
Widget::~Widget()
{
    delete ui;
}
 
void Widget::mousePressEvent( QMouseEvent *mouseEvent )
{
    if ( mouseEvent->button() == Qt::LeftButton ) 
    {
        x = mouseEvent->x();
        y = mouseEvent->y();
        update();
    }
}
 

h:

Код

C++ (Qt)
namespace Ui
{
    class Widget;
}
 
class Widget : public QWidget
{
    Q_OBJECT
 
public:
    Widget(QWidget *parent = 0);
    ~Widget();
 
protected:
    void paintEvent( QPaintEvent *event );
    virtual void mousePressEvent( QMouseEvent *mouseEvent );
 
private:
    Ui::Widget *ui;
    int x;
    int y;
};
 

Проблема в том что при клики, предыдущая точка затирается. В гугли нашел что при таком случаи надо рисовать на QImage, сейчас буду читать доку, но есть еще пару вопросов.
И еще если я переопределяю метод как:

Код

C++ (Qt)
void paintEvent( QPaintEvent *event, int x, int y );
 

а в mouseEvent просто ставлю update() то на виджете не рисуется почему?
И последний вопрос, если использовать код который сейчас, то при запуске программы первая точка рисуется сам, по координатам заданным в конструкторе, как сделать так что бы первая точка рисовалась только по клику и по координатам этого клика?
Заранее спасибо.

[StogovOleg]

есть вариант - использовать массив или вектор (что удобнее) для запоминания координат точек и в paintEvent в цикле рисовать каждую точку

[StogovOleg]

к вопросу о первой точке - предыдущий ответ решит эту проблему
к вопросу о "переопределении" функции - добавляя параметры, создаешь новую функцию (перегружаешь), которая, не смотря на схожее название paintEvent является все-таки другой функцией, которую метод update() не вызывает, поэтому и не рисует

[n4ela]

Спасибо большое, все понял.
Добавлено:
А кстати то что компилятор кидает warning параметр ‘event’ не используется, нормально? Имею виду может его где то надо было использовать или отключить подобного рода варнинги?

[Vexator]

передай event дальше QWidgetу )

[Danila_Bagrofff]

передай event дальше QWidgetу )

И тогда передав событие стандартному обработчику, должен заработать и update, так как событие отловит теперь уже стандартный обработчик.

Передача дальше:
QWidget::event(event);

[Igors]

Имею виду может его где то надо было использовать или отключить подобного рода варнинги?

Использовать или нет - по обстановке (не ошибка). Отключать такой warning не очень хорошо, может в др. случае быть полезным. Проще добавить в функцию пустое использование, например

(void) event;

[pastor]

Проще добавить в функцию пустое использование, например
(void) event;

Для этого есть макрос Q_UNUSED(...);

Q_UNUSED ( name )
Indicates to the compiler that the parameter with the specified name is not used in the body of a function. This can be used to suppress compiler warnings while allowing functions to be defined with meaningful parameter names in their signatures.

[IGHOR]

Вроде так корректнее передавать ивент дальше: event->accept(); а заглушить так event->ignore();

[Alex Custov]

Вроде так корректнее передавать ивент дальше: event->accept(); а заглушить так event->ignore();

QWidget сам ничего не рисует, передавать paint event дальше не нужно.

[Alex Custov]

Спасибо большое, все понял.
Добавлено:
А кстати то что компилятор кидает warning параметр ‘event’ не используется, нормально? Имею виду может его где то надо было использовать или отключить подобного рода варнинги?

сотри имя переменной "event", или используй Q_UNUSED

[Vexator]

а как это повлияет на работу?

[SABROG]

а как это повлияет на работу?

Никак. Если переменная объявленная как параметр метода не используется, то msvc'шный компилятор перестанет ругаться. MinGW в этом плане мозг программиста не насилует.