Trouble with creating custom class

[jstippey]

I have been trying to write a simple class file and can't figure out what I am doing wrong. It is one .h file, one .cpp file, and the main file. Any help would be greatly appreciated. It appears to not like my main file when compiling. I am getting errors on 11,13,16,17,19. In addition, I am getting errors on my QPushButton callout for "quit" and my QVBoxLayout in the main.cc

channelselected.h file

#ifndef CHANNELSELECTED_H
#define CHANNELSELECTED_H
 
#include <qvbox.h>
#include <qhbox.h>
#include <qlayout.h>
#include <qobject.h>
 
class QPushButton;
class QCheckBox;
 
class ChannelSelected : public QVBoxLayout
{
      Q_OBJECT
public:
 ChannelSelected ( QWidget *parent=0, const char *name=0 );
bool isChecked() const;
 
public slots:
void setChecked();
void setUnChecked();
 
signals:
void valueChanged();
 
private;
QPushButton *allpb;
QPushButton *clearpb;
QCheckBox *c0cb;
QCheckBox *c1cb;
QCheckBox *c2cb;
QCheckBox *c3cb;
QCheckBox *c4cb;
QCheckBox *c5cb;
QCheckBox *c6cb;
};
#endif // CHANNELSELECTED_H

#ifndef CHANNELSELECTED_H
#define CHANNELSELECTED_H

#include <qvbox.h>
#include <qhbox.h>
#include <qlayout.h>
#include <qobject.h>

class QPushButton;
class QCheckBox;

class ChannelSelected : public QVBoxLayout
{
      Q_OBJECT
public:
 ChannelSelected ( QWidget *parent=0, const char *name=0 );
bool isChecked() const;

public slots:
void setChecked();
void setUnChecked();

signals:
void valueChanged();

private;
QPushButton *allpb;
QPushButton *clearpb;
QCheckBox *c0cb;
QCheckBox *c1cb;
QCheckBox *c2cb;
QCheckBox *c3cb;
QCheckBox *c4cb;
QCheckBox *c5cb;
QCheckBox *c6cb;
};
#endif // CHANNELSELECTED_H

To copy to clipboard, switch view to plain text mode 

channelselected.cpp

#include "channelselected.h"
#include <qcheckbox.h>
#include <qpushbutton.h>
 
ChannelSelected::ChannelSelected ( QWidget *parent, const char *name)
      : QVBoxLayout (parent, name);
{
QCheckBox *c0cb = new QCheckBox ("Channel 0", this, "c0cb");
QCheckBox *c1cb = new QCheckBox ("Channel 1", this, "c1cb");
QCheckBox *c2cb = new QCheckBox ("Channel 2", this, "c2cb");
QCheckBox *c3cb = new QCheckBox ("Channel 3", this, "c3cb");
QCheckBox *c4cb = new QCheckBox ("Channel 4", this, "c4cb");
QCheckBox *c5cb = new QCheckBox ("Channel 5", this, "c5cb");
QCheckBox *c6cb = new QCheckBox ("Channel 6", this, "c6cb");
QPushButton *allpb = new QPushButton("All", this, "allpb");
QPushButton *clearpb = new QPushButton("Clear", this, "clearpb);
QVBoxLayout *box = new QVBoxLayout(this);
QHBox *hbox = new QHBox(box);
QHbox *hboxbutt= new QHBox(box);
hbox->addWidget(c0cb);
hbox->addWidget(c1cb); 
hbox->addWidget(c2cb); 
hbox->addWidget(c3cb); 
hbox->addWidget(c4cb); 
hbox->addWidget(c5cb); 
hbox->addWidget(c6cb);   
hboxbutt->addWidget(allpb);
hboxbutt->addWidget(clearpb);
box->addWidget(hbox);
box->addWidget(hboxbutt);
 
connect (allpb, SIGNAL(clicked()), c0cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c1cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c2cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c3cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c4cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c5cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c6cb, SLOT(setChecked()));
 
connect(clearpb, SIGNAL(clicked()), c0cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c1cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c2cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c3cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c4cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c5cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c6cb, SLOT(setChecked()));
}
 
bool ChannelSelected::isChecked() const
{
return;
}
 
void ChannelSelected::setChecked()
{
c0cb->setChecked(TRUE);
c1cb->setChecked(TRUE);
c2cb->setChecked(TRUE);
c3cb->setChecked(TRUE);
c4cb->setChecked(TRUE);
c5cb->setChecked(TRUE);
c6cb->setChecked(TRUE);
}
 
void ChannelSelected::setUnChecked()
{
c0cb->setChecked(FALSE);
c1cb->setChecked(FALSE);
c2cb->setChecked(FALSE);
c3cb->setChecked(FALSE);
c4cb->setChecked(FALSE);
c5cb->setChecked(FALSE);
c6cb->setChecked(FALSE);
}

#include "channelselected.h"
#include <qcheckbox.h>
#include <qpushbutton.h>

ChannelSelected::ChannelSelected ( QWidget *parent, const char *name)
      : QVBoxLayout (parent, name);
{
QCheckBox *c0cb = new QCheckBox ("Channel 0", this, "c0cb");
QCheckBox *c1cb = new QCheckBox ("Channel 1", this, "c1cb");
QCheckBox *c2cb = new QCheckBox ("Channel 2", this, "c2cb");
QCheckBox *c3cb = new QCheckBox ("Channel 3", this, "c3cb");
QCheckBox *c4cb = new QCheckBox ("Channel 4", this, "c4cb");
QCheckBox *c5cb = new QCheckBox ("Channel 5", this, "c5cb");
QCheckBox *c6cb = new QCheckBox ("Channel 6", this, "c6cb");
QPushButton *allpb = new QPushButton("All", this, "allpb");
QPushButton *clearpb = new QPushButton("Clear", this, "clearpb);
QVBoxLayout *box = new QVBoxLayout(this);
QHBox *hbox = new QHBox(box);
QHbox *hboxbutt= new QHBox(box);
hbox->addWidget(c0cb);
hbox->addWidget(c1cb); 
hbox->addWidget(c2cb); 
hbox->addWidget(c3cb); 
hbox->addWidget(c4cb); 
hbox->addWidget(c5cb); 
hbox->addWidget(c6cb);   
hboxbutt->addWidget(allpb);
hboxbutt->addWidget(clearpb);
box->addWidget(hbox);
box->addWidget(hboxbutt);

connect (allpb, SIGNAL(clicked()), c0cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c1cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c2cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c3cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c4cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c5cb, SLOT(setChecked()));
connect (allpb, SIGNAL(clicked()), c6cb, SLOT(setChecked()));

connect(clearpb, SIGNAL(clicked()), c0cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c1cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c2cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c3cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c4cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c5cb, SLOT(setChecked()));
connect(clearpb, SIGNAL(clicked()), c6cb, SLOT(setChecked()));
}

bool ChannelSelected::isChecked() const
{
return;
}

void ChannelSelected::setChecked()
{
c0cb->setChecked(TRUE);
c1cb->setChecked(TRUE);
c2cb->setChecked(TRUE);
c3cb->setChecked(TRUE);
c4cb->setChecked(TRUE);
c5cb->setChecked(TRUE);
c6cb->setChecked(TRUE);
}

void ChannelSelected::setUnChecked()
{
c0cb->setChecked(FALSE);
c1cb->setChecked(FALSE);
c2cb->setChecked(FALSE);
c3cb->setChecked(FALSE);
c4cb->setChecked(FALSE);
c5cb->setChecked(FALSE);
c6cb->setChecked(FALSE);
}

To copy to clipboard, switch view to plain text mode 

main.cpp file

#include <qapplication.h>
#include <qpushbutton.h>
#include <qfont.h>
#include <qvbox.h>
#include <qlayout.h>
#include <qobject.h>
 
#include "channelselected.h"
 
class MyWidget : public QVBoxLayout
{
public:
    MyWidget( QWidget *parent=0, const char *name=0);
};
 
MyWidget::MyWidget ( QWidget *parent, int, int, const char *name)
      :QVBoxLayout(parent, int, int, name)
{
QVBoxLayout *vbox = new QVBoxLayout ( this, 10, 10, "vbox");
 
QPushButton *quit = new QPushButton ("&Quit", vbox, "quit" );
quit->setFont ( QFont("Times", 18, QFont::Bold));
 
connect( quit, SIGNAL(clicked()), qApp, SLOT(quit())   );
ChannelSelected* channelPanel = new ChannelSelected (vbox, 10, 10, "channelPanel")
vbox->addWidget(channelPanel);
vbox->addWidget(quit);
}
 
int main ( int argc, char **argv)
{
QApplication a (argc, argv);
MyWidget w;
a.setMainWidget(&w);
w.show();
return a.exec();
}

#include <qapplication.h>
#include <qpushbutton.h>
#include <qfont.h>
#include <qvbox.h>
#include <qlayout.h>
#include <qobject.h>

#include "channelselected.h"

class MyWidget : public QVBoxLayout
{
public:
    MyWidget( QWidget *parent=0, const char *name=0);
};

MyWidget::MyWidget ( QWidget *parent, int, int, const char *name)
      :QVBoxLayout(parent, int, int, name)
{
QVBoxLayout *vbox = new QVBoxLayout ( this, 10, 10, "vbox");

QPushButton *quit = new QPushButton ("&Quit", vbox, "quit" );
quit->setFont ( QFont("Times", 18, QFont::Bold));

connect( quit, SIGNAL(clicked()), qApp, SLOT(quit())   );
ChannelSelected* channelPanel = new ChannelSelected (vbox, 10, 10, "channelPanel")
vbox->addWidget(channelPanel);
vbox->addWidget(quit);
}

int main ( int argc, char **argv)
{
QApplication a (argc, argv);
MyWidget w;
a.setMainWidget(&w);
w.show();
return a.exec();
}

To copy to clipboard, switch view to plain text mode 

[Zlatomir]

The first issue i see is that you are trying to create QWidgets but you are inheriting from a layout, this code:

class MyWidget : public QVBoxLayout

class MyWidget : public QVBoxLayout

To copy to clipboard, switch view to plain text mode 

should be something like:

class MyWidget : public QWidget

class MyWidget : public QWidget

To copy to clipboard, switch view to plain text mode 

Also don't forget to modify the parameters passing for constructor to QWidget (not whateverLayout)
Both your classes have the above issue.

And the second issue is that you create a QObject derived class in main.cpp, so my sugestion is to create for MyWidget class a MyWidget.h and MyWidget.cpp
Else you need to include main.moc in the end of the main.cpp.

[high_flyer]

I am getting errors on my QPushButton callout for "quit" and my QVBoxLayout in the main.cc

Also, you have a typo (the error should point to this line)

...
QCheckBox *c5cb = new QCheckBox ("Channel 5", this, "c5cb");
QCheckBox *c6cb = new QCheckBox ("Channel 6", this, "c6cb");
QPushButton *allpb = new QPushButton("All", this, "allpb");
QPushButton *clearpb = new QPushButton("Clear", this, "clearpb);//<<--Typo
QVBoxLayout *box = new QVBoxLayout(this);
QHBox *hbox = new QHBox(box);
QHbox *hboxbutt= new QHBox(box);
...

...
QCheckBox *c5cb = new QCheckBox ("Channel 5", this, "c5cb");
QCheckBox *c6cb = new QCheckBox ("Channel 6", this, "c6cb");
QPushButton *allpb = new QPushButton("All", this, "allpb");
QPushButton *clearpb = new QPushButton("Clear", this, "clearpb);//<<--Typo
QVBoxLayout *box = new QVBoxLayout(this);
QHBox *hbox = new QHBox(box);
QHbox *hboxbutt= new QHBox(box);
...

To copy to clipboard, switch view to plain text mode 

[jstippey]

Thank you guys for helping me with this issue. So I went back and changed the following lines

h file

class ChannelSelected : public QWidget


cpp file

ChannelSelected::ChannelSelected( QWidget *parent, const char *name)
:QWidget(parent, name)

checked the clear pb error, but I accidentally changed it when I copied it over.

Does this take care of issue 1 Zlatimir?

And I wanted to ask about issue number 2. Is this because I nested the quit button in main.cpp with a connect to the app. I just want to make sure that I understand exactly why. When I ran the code with the changes I made, it compiles and it shows that the signal is sent from the allpb/clearpb to the checkboxes, however is says that I have no such slot- QCheckBox :: setChecked and no such slot- QCheckBox::setUnChecked and the checkboxes don't get set/unset.

[Zlatomir]

There you want to call setCghecked() of your class (ChannelSelected:: ) i guess?
Then the code should be:

connect (allpb, SIGNAL(clicked()),    this,    SLOT(setChecked()));

connect (allpb, SIGNAL(clicked()),    this,    SLOT(setChecked()));

To copy to clipboard, switch view to plain text mode 

And you have some other issues in the other class, like this:

QPushButton *quit = new QPushButton ("&Quit", vbox, "quit" );

QPushButton *quit = new QPushButton ("&Quit", vbox, "quit" );

To copy to clipboard, switch view to plain text mode 

See the constructors of QPushButton

And you are trying to pass a layout as a parent (in the same QPushButton code above, vbox is a layout)
The parent must be a QWidget not a Layout, the layout can have a QWidget parent and re-parents all the widgets added to it to it's parent, but the layout can't be a parent for some widgets.
Solution is: 1) create a layout (if you set a parent you don't need step 3 ), 2) then use addWidget method to add widgets and addLayout for other nested layouts and 3) call setLayout to set a parent for all.

And i don't understand why you keep passing around NULL char pointer