Сбой на keyPressEvent

Question

Сбой на keyPressEvent

0

Я создаю программу, которая рисует цвета, а другой класс проверяет их и возвращает true, если они совпадают. Весь код которого находится в диалоговом окне. colQ - класс, который просто хранит значения RGB и другие вещи; мы будем использовать их для получения RGB.

colDialog::colDialog(QWidget *parent) // etc..
{
  // initialization code

    colQ.setColor(255, 0, 0); // red
    r_ans = g_ans = b_ans = nullptr;
}

Затем у меня есть событие рисования, которое делает рисунок. r1 и br1 являются указателями на их соответствующие классы.

void Atn_QDialog::paintEvent(QPaintEvent *e)
{
    QPainter painter(this);

    r1 = new QRect(100, 100, 175, 400);
    br1 = new QBrush(QColor(colQ.getR, colQ.getG, colQ.getB));
    painter.fillRect(*r1, *br1);
    painter.drawRect(*r1);

    // other color code
}

На данный момент у меня будет прямоугольник, нарисованный красным цветом. Теперь я хочу, чтобы пользователь нажал клавишу (например, 1), а затем colQ проверит, соответствует ли цвет, соответствующий цвету на прямоугольнике, на который нажата клавиша.

void colDialog::keyPressEvent(QKeyEvent *Ev)
{
    if (Ev->key() == Qt::Key_1)
    {
         br1->color().getRgb(r_ans, g_ans, b_ans);
         colQ.answer(*r_ans, *g_ans, *b_ans);

         if (colQ.isCorrect())
              ui->resultlabel->setText("correct!");
         else ui->resultlabel->setText("wrong!"); 
    }

    // other key presses here
    QDialog::keyPressEvent(Ev);
}

Ошибка возникает, когда я нажимаю клавишу "1", где код до сих пор реализован. Прежде, нажатие клавиши отвечает нормально, но это не происходит, когда этот код запущен. Fyi, colQ не имеет кода Qt Q, и опечатать его код по-прежнему вызывает сбой моей программы. Что я делаю здесь, я хочу получить цвет, хранящийся в br типа QBrush *. Я передал указатели в getRgb (..), поскольку это было то, о чем просили. Код срабатывает эффективно на keyPressEvent (Ev); такие вещи, как ui-> resultlabel- ~> setText ("..."), не приводят к сбою программы при нажатии клавиши. Это происходит только с вышеназванными вызовами функций.

Poriferous 18 нояб. 2014, в 01:18

Источник

Теги:

c++

qt

1 ответ

Ещё вопросы

Iuliu · Accepted Answer · 2014-11-17T22-32-00.000Z

Ваша программа вылетает здесь:

br1->color().getRgb(r_ans, g_ans, b_ans);

r_ans, g_ans и b_ans должны быть инициализированы, и память должна быть выделена для этих указателей. QColor::getRgb не выделяет вам память, он просто меняет значения и из-за этого он выходит из строя с ошибкой сегментации/доступом.

Внутри конструктора выделить память для этих 3 указателей:

colDialog::colDialog(QWidget *parent) // etc..
{
  // initialization code

    colQ.setColor(255, 0, 0); // red
    r_ans = new int;
    g_ans = new int;
    b_ans = new int;
}

Если у вас есть nullptr и вы пытаетесь разыменовать его: int *p = nullptr; int x = *p; int *p = nullptr; int x = *p; программа выйдет из строя. Это то, что происходит внутри QColor::getRgb: три указателя разыменовываются, чтобы изменить значения, на которые они указывают, и поэтому ваше приложение выходит из строя.

Также ужасно, что вы делаете это:

r1 = new QRect(100, 100, 175, 400);
br1 = new QBrush(QColor(colQ.getR, colQ.getG, colQ.getB));

внутри paintEvent. Даже если вы освобождаете память где-то в функции paintEvent это плохая идея. Объявите r1 в стеке и инициализируйте его в конструкторе, здесь вам не нужны указатели. Также создайте br1 в стеке. Нет необходимости в указателях и распределении динамической памяти.

Я инициализировал r_ans, g_ans и b_ans в nullptr в конструкторе. Разве этого недостаточно для работы кода?
@ Vormeph Нет, вам нужно выделить память для них, смотрите мои правки.
Сначала это не сработало, но позже я прекратил встроенную инициализацию указателей и перенес ответы на другую функцию. Спасибо за вашу помощь!
r_ans , g_ans , b_ans было бы лучше, если бы они были не указателями, а автоматическими переменными. Вы можете передать их как ссылку на метод QColor::getColor : br1.color().getRgb(&r_ans, &g_ans, &b_ans); Помните, не используйте указатели, если у вас нет веских причин для этого.