Header files and general questions.

JKSH

@Uberlinc said in Header files and general questions.:

Not sure if it is simply typos by me (possible, but I've gone over the code and can't find them!) or is Qt4 different sufficiently that even the simplest of code needs some tweaking to make it work?

The one major difference that will likely impact you is that widgets used to be in the Qt GUI module in Qt 4. Now, they are in their own Qt Widgets module in Qt 5.

Setting up the project is a bit different, but after that's done the code should still be mostly compatible.

how can I pick up where the language needs changing for parts where the book is outdated?

The changes are described at http://doc.qt.io/qt-5/portingguide.html. For the beginning parts of the book, you probably won't be affected by anything described here except for the Qt Widgets module change.

@Uberlinc said in Header files and general questions.:

I replaced, what was in the book as:

class QCheckBox;
class QLabel;
class QLineEdit;
class QPushButton;

with:

#include <QPushButton>
#include <QLabel>
#include <QLineEdit>
#include <QCheckBox>
#include <QBoxLayout>

Is this likely to be a mis-print in the original code in the book or has Qt5 changed from Qt4 such that this was correct for the previous version?

If you examine the book in depth, you will probably find that class QCheckBox; is written in the .h file while #include <QCheckBox> is written in the .cpp file.

As @aha_1980 mentioned, the former is called a forward declaration. See the following for more info on how to use them and why:

Uberlinc

I've read up a fair bit about forward declarations, and also re-read over the chapter from which I am working.
The chapter does indeed mention forward declarations, but I'm still fuzzy as to why the code, as they define it, did not work.

If I include the forward declarations of:
class QCheckBox;
class QLabel;
class QLineEdit;
class QPushButton;

... that's all well and good, but it doesn't answer where the code for these classes is defined.
If you don't include <QCheckbox> ..... <QPushbutton> then where are they kept?

The code worked once I commented out these forward declarations and simply included the include file for each of them, but I understand that this is a cumbersome way of doing it.

Where else is the full class definition of the above classes if not in its own QFunction include file?

If they are included in another, larger include file that the code has already specified, then why didn't the code work?
The only include files in the actual specified code are QDialog, QtGUI and QApplication.

Thanks for your help.

Regards,

Uberlinc.

aha_1980

@Uberlinc said in Header files and general questions.:

I've read up a fair bit about forward declarations, and also re-read over the chapter from which I am working.
The chapter does indeed mention forward declarations, but I'm still fuzzy as to why the code, as they define it, did not work.

If I include the forward declarations of:
class QCheckBox;
class QLabel;
class QLineEdit;
class QPushButton;

... that's all well and good, but it doesn't answer where the code for these classes is defined.
If you don't include <QCheckbox> ..... <QPushbutton> then where are they kept?

Well, they are kept in their header files, but at the point of forward declaration the compiler just knows that a class QCheckBox exists. Nothing more! you can't do more with the class at this point than declare a pointer to it. that is important to understand.

The code worked once I commented out these forward declarations and simply included the include file for each of them,

as said, to use the class, you need the include. but that's typically in the C++ file only, not in the header.

but I understand that this is a cumbersome way of doing it.

it is at least as legal to do so as to do fwd-declaration. you will see both in the wild.

Where else is the full class definition of the above classes if not in its own QFunction include file?

indeed.

If they are included in another, larger include file that the code has already specified, then why didn't the code work?
The only include files in the actual specified code are QDialog, QtGUI and QApplication.

????

Thanks for your help

you're welcome!

JKSH

@Uberlinc said in Header files and general questions.:

The chapter does indeed mention forward declarations, but I'm still fuzzy as to why the code, as they define it, did not work.

Which chapter? We could have a closer look.

Uberlinc

C++ GUI Programming with Qt 4, Second Edition
Blanchette and Summerfield
Chapter 2. Creating Dialogs.

VRonin

The code in that chapter is still 100% valid please note the:

The source code is spread across two files: finddialog.h and finddialog.cpp

I think you just put everything in the same file

Uberlinc

No, I definitely have, not 2, but 3 (three) files:
finddialog.h
finddialog.cpp
main.cpp

I believe that I have copied the code verbatim, but as it is interspersed between explanatory text, it is possible that I missed the declaration that qualifies these classes.

Can't find where, though!

Can anyone point out in the original text where the declaration is that qualifies the classes in question?
Is it one line like #include <QDoesItAll> or is it a block of several lines that I've missed?

Thanks,

Uberlinc.

mpergand

Hi,

You can download the source code from here:
https://github.com/GaoHongchen/CPPGUIProgrammingWithQt4

The project is in chap02>find folder
You need to proceed two modifications:

in the find.pro file, add at the top:

QT       += core gui
greaterThan(QT_MAJOR_VERSION, 4): QT += widgets

in the findDialog.cpp, at the top do the following:

//#include <QtGui>
#include <QtWidgets>

That's all

More info here:
https://wiki.qt.io/Transition_from_Qt_4.x_to_Qt5

JKSH

@mpergand said in Header files and general questions.:

in the findDialog.cpp, at the top do the following:
//#include <QtGui>
#include <QtWidgets>

Right.

#include <QtGui> is a quick way to include every single class in the Qt GUI module. Since QCheckBox et al. have been moved to the Qt Widgets module, you can replace this line with #include <QtWidgets> -- this will include every single class in the Qt Widgets module.

To see how this works,

Open the code in Qt Creator
Hover your mouse over the line #include <QtWidgets>
Press F2. You will see that you are actually #include-ing many many classes.

So to summarize:

Write class QCheckBox; in your .h file, to allow your class to contain a pointer to QCheckBox.
Write #include <QtWidgets> or #include <QCheckBox> in your .cpp file to allow your code to instantiate a QCheckBox.

Personally, I prefer to include individual classes when developing a proper project. However, including the whole module is quick an easy for rapid prototyping or for tutorials.

Uberlinc

Woo-hoo!
It works!
Thank you!

So, clearly there is a slight change in the code.
This explains why it didn't work beforehand.

Many thanks!