What is the mistake in my code?

Question

The sample code mentioned below is not compiling. Why?

#include "QprogressBar.h"

#include <QtGui>
#include <QApplication>
#include<qprogressbar.h>
#include <qobject.h>

lass myTimer: public QTimer
{

public:
    myTimer(QWidget *parent=0):QTimer(parent)
{}

public slots:
void recivetime();
};
void myTimer::recivetime()
    {

    }

class Progressbar: public QProgressDialog
    {
public:
    Progressbar(QWidget *parent=0):QProgressDialog(parent)
            {

            }


    };

int main(int argc, char *argv[])
{
    QApplication a(argc, argv);


 QObject::connect(QTimer,SIGNAL(timeout()),QTimer,SLOT(recivetime()));


 return a.exec();

}

It is giving me a problem when it tries to connect. I think that maybe it is fine to write the connect code in the main function.

Can you at least tell us the error? Also, you have `lass` instead of `class` — GManNickG, Feb 18 '10 at 14:56
This family of question was discussed at http://meta.stackexchange.com/questions/40164/should-we-close-fix-my-program-questions . Although the discussion there was at least for semantic problems, not syntactic/typing ones. — Pascal Cuoq, Feb 22 '10 at 14:22

score 4 · Answer 1 · answered Feb 18 '10 at 14:57

4

Where is your QTimer? I think that's the problem. I haven't done Qt for a while, but as far as I remember, the first and third arguments of connect are pointers to objects, and you don't have a QTimer pointer.

answered Feb 18 '10 at 14:57

Skilldrick

69,215
34
177
229

score 3 · Accepted Answer · edited Feb 22 '10 at 14:12

To sum-up the previous comments and answers:

the compiler tells you at least what it does not understand if not straight-up what's wrong with your code => if you don't understand what the compiler says post the error message with your question so that it helps those who speak "compilese"
"connect" will connect a object's signal with another object's slot -> pass objects to connect, not classes
the connected objects must exist for the intended duration of your connection you are connecting now at best automatic instances of QTimer which will be out of scope by the time the connect call ends.

The correct way of doing this:

int main(int argc, char *argv[])
    {
        QApplication a(argc, argv);

        myTimer myTimerObject(a);

        QObject::connect(&myTimerObject, SIGNAL(timeout()), &myTimerObject, SLOT(recivetime()));

        return a.exec();

    }

As a side note this has nothing to do with Symbian, nor is it specific to Qt 4.x. Also Qt is not QT just as QT is not Qt ;)

Matthieu · Answer 3 · 2010-02-18T23:30:49.580

Skilldrick is right !
See the qt doc on signals and slots. The connect method needs a pointer or reference of the sender and receiver object !
But in your code :

QObject::connect(QTimer,SIGNAL(timeout()),QTimer,SLOT(recivetime()));

QTimer is a class name and not a object of this class ! I mean, you need to create an object. For example :

QTimer* pTimer = new QTimer(a);   // QTimer object   
myTimer* pReciever = new myTimer(a);  // Your custom QTimer object with progress bar 
QObject::connect(pTimer,SIGNAL(timeout()), pReciever,SLOT(recivetime()));   
...

Hope it helps !

score 0 · Answer 4 · answered Feb 18 '10 at 15:00

0

Not sure, but try:

QObject::connect(myTimer,SIGNAL(timeout()),this,SLOT(recivetime()));

Oops, thought myTimer was an instance of QTimer rather than a subclass. Make an instance of QTimer and give that as the first parameter. And this as the third.

answered Feb 18 '10 at 15:00

Kurt

4,477
2
26
34

There is no `this` in main, lest I'm misunderstanding. – GManNickG Feb 18 '10 at 15:08

What is the mistake in my code?

4 Answers4