A little calculator problem

[bijan311]

ok i ran qmake in the folder of main.cpp then I pressed rebuild in Code::Blocks and it gave me the same error.

[wysota]

Did you implement the class destructor?

[squidge]

ok i ran qmake in the folder of main.cpp then I pressed rebuild in Code::Blocks and it gave me the same error.

Dump Code::Blocks, download Qt Creator. It will make your life easier. Once you have learned the basics, you can go back to Code::Blocks if you wish.

(For example, in Qt Creator there is no need for a console as you can do everything from inside the IDE)

[bijan311]

ok so I have installed QT Creator.

and what is a class destructor.

[wysota]

and what is a class destructor.

Ouch... we didn't learn C++, did we?

http://en.wikipedia.org/wiki/Destruc...puter_science)
http://www.parashift.com/c++-faq-lite/dtors.html

[bijan311]

I did learn c++ from
cprogramming.com
I just never had a good understanding of classes.

[wysota]

So you were learning C++ but didn't learn it. It's all about classes. It's like you were learning a language but never had a good understanding of words... I suggest you correct that mistake now.

[bijan311]

well actually, I got a better understanding of classes when reading this thread...

[bijan311]

OK, I got it to compile but when i press execute the LCDNumber thing doesn't change.
here is the cofr

#include <QApplication>
#include <QPushButton>
#include <QLabel>
#include <QSpinBox>
#include <QLCDNumber>
#include <QHBoxLayout>
#include <QVBoxLayout>
 
class calculator : public QWidget{
 
    public:
    calculator(QWidget *parent = 0) : QWidget(parent)
    {
    equal = new QPushButton("Execute");
    num1 = new QSpinBox;
    num2 = new QSpinBox;
    answer = new QLCDNumber;
 
    QHBoxLayout *layout = new QHBoxLayout;
    layout->addWidget(num1);
    layout->addWidget(num2);
    layout->addWidget(answer);
    QVBoxLayout *mainLayout = new QVBoxLayout(this);
    mainLayout->addLayout(layout);
    mainLayout->addWidget(equal);
 
     QObject::connect(equal, SIGNAL(clicked()), answer, SLOT(minus()));
    }
     public slots:
    void minus(){
        answer->display(num1->value()-num2->value());
    }
    private:
    QPushButton *equal;
    QSpinBox *num1;
    QSpinBox *num2;
    QLCDNumber *answer;
    };
 
int main(int argc, char *argv[])
{
    QApplication app(argc, argv);
    calculator w;
 
    w.show();
    return app.exec();
}

#include <QApplication>
#include <QPushButton>
#include <QLabel>
#include <QSpinBox>
#include <QLCDNumber>
#include <QHBoxLayout>
#include <QVBoxLayout>

class calculator : public QWidget{

    public:
    calculator(QWidget *parent = 0) : QWidget(parent)
    {
    equal = new QPushButton("Execute");
    num1 = new QSpinBox;
    num2 = new QSpinBox;
    answer = new QLCDNumber;

    QHBoxLayout *layout = new QHBoxLayout;
    layout->addWidget(num1);
    layout->addWidget(num2);
    layout->addWidget(answer);
    QVBoxLayout *mainLayout = new QVBoxLayout(this);
    mainLayout->addLayout(layout);
    mainLayout->addWidget(equal);

     QObject::connect(equal, SIGNAL(clicked()), answer, SLOT(minus()));
    }
     public slots:
    void minus(){
        answer->display(num1->value()-num2->value());
    }
    private:
    QPushButton *equal;
    QSpinBox *num1;
    QSpinBox *num2;
    QLCDNumber *answer;
    };

int main(int argc, char *argv[])
{
    QApplication app(argc, argv);
    calculator w;

    w.show();
    return app.exec();
}

To copy to clipboard, switch view to plain text mode 

[wysota]

OK, I got it to compile but when i press execute the LCDNumber thing doesn't change.

This doesn't surprise me as your code is not correct. Please open Qt Reference and read what arguments should you pass to connect(). Currently you miss some basic understanding of the signal/slot mechanism. Besides, you have already been given a working solution.

[bijan311]

I only made a few changes. I got rid of the Q_Object macro because with it the program wouldn't link to the .exe file. I tried to make a .moc file by using qmake in the directory of the program, the only files that are there are:

Calculator Gui.depend
Calculator Gui.layout
Calculator Gui.pro
Calculator Gui.pro.user
file
file.Debug
file.Release
Makefile
Makefile.Debug
Makefile.Release

thank you.

[squidge]

Watch the output window in Qt Creator, it'll tell you what your doing wrong. I can see an instant problem with your connect() call.

[wysota]

I only made a few changes. I got rid of the Q_Object macro because with it the program wouldn't link to the .exe file.

This macro wasn't there because it looks nice, placing it there had its purpose. I really really suggest you read a bit of docs before you start programming with Qt. At least go through the tutorial.

[sutee84]

QtCreator maybe can help you.

[wysota]

QtCreator maybe can help you.

No, Qt Creator won't help him understand what he is doing.

[sutee84]

You're right, but it's easier to connect signals and slots in QtCreator on the right way.

[squidge]

You're right, but it's easier to connect signals and slots in QtCreator on the right way.

Yes, if you know where the signals and slots need to be connected, but if you don't know that it could just become even more confusing when it doesn't let you do what you think is correct (like connecting a signal to a integer variable).

Reading the documentation or tutorial is a must before attempting anything, even a simple calculator program. If your a seasoned developer, the tutorial will probably take you about 10 minutes to go through.

[wysota]

You're right, but it's easier to connect signals and slots in QtCreator on the right way.

But you have to choose the right object as the source and destination of the signal. Without that Creator will not be helpful in any way other than not listing the signals and slot you'd like to connect which suggests you did something wrong. But then the compiler/Qt pair do exactly the same thing.