Обработчик кнопок для калькулятора

[voodoo]

Доброе время суток!

Решил для освоения Qt написать калькулятор.
Как при нажатии кнопки мыши по клавише, определить какая именно клавиша была нажата?

Код:

void Dialog::mousePressEvent(QMouseEvent *e)
{
    QPushButton *button_clicked = qobject_cast<QPushButton *>(sender());
    ...
    ...
    ...
}

Хочу просто сделать обработчик для клавиш 0-9, чтобы код был покомпактнее.

Заранее спасибо.

[LisandreL]

Это же событие, а не слот. Вместо сендера используйте this, каста не надо.
Самый банальный вариант - взять цифру из текста на кнопке.

[voodoo]

Это же событие, а не слот. Вместо сендера используйте this, каста не надо.

Я немного не понял как это должно выглядеть в коде. Не могли бы Вы показать?

[BuRn]

можно так
где-то в коде на батон

Код:

QPushButton *button = new QPushButton;
button->setText("-");
button->setObjectName(name);
connect(button,SIGNAL(clicked()),this,SLOT(RemoveItem()));

и далее получить обжект нейм в слоте

Код:

QObject *element = (QObject *)sender();
QString name=element->objectName();

[BuRn]

Код:

void Dialog::mousePressEvent(QMouseEvent *e)
{
    QPushButton *button_clicked = qobject_cast<QPushButton *>(sender());
    ...
    ...
    ...
}

И если я прав так не делаются обработчики на клавиши, ты тут обрабатываешь нажатие клавиш мыши, а нужно обрабатывать нажатие кнопки (QPushbutton ) соответственно и обработчик на него

[voodoo]

BuRn, как тогда сделать обработчик на клавиши?

[BuRn]

Код:

connect(button,SIGNAL(clicked()),this,SLOT(RemoveItem()));

вот
можно через гуи тоже самое сделать, на кнопку повесить обработчик ...
в классе объявляешь слот, который будет отвечать за нажатие кнопки
public slots:
void MySlot();
далее в конструкторе можно явно прописать
у тебя будет что-то типа
connect(ui->pushButton,SIGNAL(clicked()),this,SLOT(MySlot()));

[voodoo]

Да ну это понятно )) Так сделать - самый простой вариант, но будет куча лишнего кода.

[BuRn]

ну засунь в него тогда  mousePressEvent(QMouseEvent *e), то , что я тебе писал кинуть в обработчик

[andrew.k]

Да ну это понятно )) Так сделать - самый простой вариант, но будет куча лишнего кода.

Одна строка это куча? Это в твоем варианте будет куча лишнего кода.

[voodoo]

Одна строка на 10 кнопок:) итого:)

[andrew.k]

Ты коннекты сигналов за код считаешь?

[LisandreL]

Я немного не понял как это должно выглядеть в коде. Не могли бы Вы показать?

Код

C++ (Qt)
void NumberButton::mousePressEvent(QMouseEvent* e)
{
    QPushButton::mousePressEvent(e);
    bool error;
    uint number = text().toUInt(&error);
    if ( error || number > 9 )
    {
          qt_assert_x("error", "wrong number", __FILE__, __LINE__);
    }
    else
    {
          // обработчик нажатой цифровой кнопки на калькуляторе
          numberPressed( number );
    }
}

Отмечу, что события обрабатываем у кнопки, а не у диалога, как у вас.

Но вообще через сигналы кошерней. Для клавиш 0-9 можно использовать QSignalMapper.

[voodoo]

Попытался сделать через QSignalMapper. Но не знаю к какому событию диалога привязывать "биндовку" клавиш. Попытался так:

Код:

void Dialog::finished()
{
    QSignalMapper *signalMapper;

    for(int i=0; i<10; i++)
    {
        QString button_name = "num_" + QString::number(i);
        connect(button_name,SIGNAL(clicked()),signalMapper,SLOT(map()));
        signalMapper->setMapping(button_name,QString::number(i));
        connect(signalMapper, SIGNAL(mapped(const QString &)),
                this, SLOT(clicked(const QString &)));

    }
}

Но не работает =/ ошибок куча

[kambala]

а память кто выделять будет?

да и смысл замапить числа, а не строки