From cb54c16584cf3be746a1a536c1e37cb3022a2f1b Mon Sep 17 00:00:00 2001 From: Christian Ehrlicher Date: Fri, 6 Sep 2019 22:38:45 +0200 Subject: Cleanup QtWidgets (widgets) examples Cleanup QtWidgets widgets examples: - use member-init (clang-tidy) - fix includes/don't include QtWidgets globally - include own header first - use nullptr (clang-tidy) - avoid c-style casts - use QVector instead QList Change-Id: Ib56bb507eb2ef885f1ddc664050d3c7af92adb70 Reviewed-by: Friedemann Kleint --- examples/widgets/doc/src/calculator.qdoc | 3 ++ examples/widgets/doc/src/tooltips.qdoc | 2 +- .../widgets/widgets/analogclock/analogclock.cpp | 4 ++- examples/widgets/widgets/calculator/button.cpp | 2 -- examples/widgets/widgets/calculator/calculator.cpp | 22 ++++++------- examples/widgets/widgets/calendarwidget/main.cpp | 2 +- examples/widgets/widgets/calendarwidget/window.cpp | 19 +++++++---- .../widgets/charactermap/characterwidget.cpp | 37 ++++++++++++---------- .../widgets/widgets/charactermap/characterwidget.h | 6 ++-- .../widgets/widgets/charactermap/mainwindow.cpp | 22 ++++++++++--- examples/widgets/widgets/codeeditor/codeeditor.cpp | 11 ++++--- examples/widgets/widgets/codeeditor/codeeditor.h | 13 ++++---- examples/widgets/widgets/codeeditor/main.cpp | 2 +- .../widgets/widgets/digitalclock/digitalclock.cpp | 3 +- examples/widgets/widgets/groupbox/window.cpp | 9 ++++-- examples/widgets/widgets/icons/iconpreviewarea.cpp | 3 +- examples/widgets/widgets/icons/iconsizespinbox.cpp | 2 +- examples/widgets/widgets/icons/imagedelegate.cpp | 2 +- examples/widgets/widgets/icons/mainwindow.cpp | 23 +++++++++++--- .../widgets/widgets/imageviewer/imageviewer.cpp | 23 ++++++++++++-- examples/widgets/widgets/imageviewer/imageviewer.h | 2 +- examples/widgets/widgets/lineedits/window.cpp | 13 ++++++-- examples/widgets/widgets/movie/movieplayer.cpp | 10 +++++- examples/widgets/widgets/scribble/mainwindow.cpp | 26 ++++++++------- examples/widgets/widgets/scribble/scribblearea.cpp | 11 +++---- examples/widgets/widgets/scribble/scribblearea.h | 8 ++--- .../widgets/widgets/shapedclock/shapedclock.cpp | 9 ++++-- examples/widgets/widgets/sliders/slidersgroup.cpp | 5 ++- examples/widgets/widgets/sliders/window.cpp | 9 ++++-- examples/widgets/widgets/spinboxes/window.cpp | 10 ++++-- .../widgets/widgets/styles/norwegianwoodstyle.cpp | 5 ++- examples/widgets/widgets/styles/widgetgallery.cpp | 19 ++++++++++- examples/widgets/widgets/stylesheet/main.cpp | 2 +- examples/widgets/widgets/stylesheet/mainwindow.cpp | 4 +-- examples/widgets/widgets/stylesheet/mainwindow.h | 2 +- .../widgets/stylesheet/stylesheeteditor.cpp | 4 ++- examples/widgets/widgets/tablet/main.cpp | 2 -- examples/widgets/widgets/tablet/mainwindow.cpp | 11 +++++-- examples/widgets/widgets/tablet/mainwindow.h | 2 +- .../widgets/widgets/tablet/tabletapplication.cpp | 2 -- examples/widgets/widgets/tablet/tabletcanvas.cpp | 29 ++++++++--------- examples/widgets/widgets/tablet/tabletcanvas.h | 25 +++++++-------- examples/widgets/widgets/tetrix/tetrixboard.cpp | 10 +++--- examples/widgets/widgets/tetrix/tetrixpiece.cpp | 2 +- examples/widgets/widgets/tetrix/tetrixwindow.cpp | 15 +++++---- examples/widgets/widgets/tooltips/main.cpp | 2 +- examples/widgets/widgets/tooltips/sortingbox.cpp | 14 +++++--- examples/widgets/widgets/tooltips/sortingbox.h | 2 +- .../widgets/widgets/validators/validatorwidget.cpp | 2 +- examples/widgets/widgets/wiggly/wigglywidget.cpp | 9 +++--- .../widgets/windowflags/controllerwindow.cpp | 30 ++++++++++-------- .../widgets/widgets/windowflags/previewwindow.cpp | 21 ++++++------ 52 files changed, 330 insertions(+), 197 deletions(-) diff --git a/examples/widgets/doc/src/calculator.qdoc b/examples/widgets/doc/src/calculator.qdoc index e8f8030207..7d34a86c19 100644 --- a/examples/widgets/doc/src/calculator.qdoc +++ b/examples/widgets/doc/src/calculator.qdoc @@ -142,6 +142,9 @@ pendingAdditiveOperator and \c pendingMultiplicativeOperator variables don't need to be initialized explicitly, because the QString constructor initializes them to empty strings. + It is also possible to initialize those variable directly in the + header. This is called \c member-initializaton and avoids a long + initialization list. \snippet widgets/calculator/calculator.cpp 1 \snippet widgets/calculator/calculator.cpp 2 diff --git a/examples/widgets/doc/src/tooltips.qdoc b/examples/widgets/doc/src/tooltips.qdoc index a278215503..35e3b1e29f 100644 --- a/examples/widgets/doc/src/tooltips.qdoc +++ b/examples/widgets/doc/src/tooltips.qdoc @@ -95,7 +95,7 @@ \snippet widgets/tooltips/sortingbox.h 2 - We keep all the shape items in a QList, and we keep three + We keep all the shape items in a QVector, and we keep three QPainterPath objects holding the shapes of a circle, a square and a triangle. We also need to have a pointer to an item when it is moving, and we need to know its previous position. diff --git a/examples/widgets/widgets/analogclock/analogclock.cpp b/examples/widgets/widgets/analogclock/analogclock.cpp index c7b3f66cca..06e298659d 100644 --- a/examples/widgets/widgets/analogclock/analogclock.cpp +++ b/examples/widgets/widgets/analogclock/analogclock.cpp @@ -50,7 +50,9 @@ #include "analogclock.h" -#include +#include +#include +#include //! [0] //! [1] AnalogClock::AnalogClock(QWidget *parent) diff --git a/examples/widgets/widgets/calculator/button.cpp b/examples/widgets/widgets/calculator/button.cpp index a1ce0bf428..cc370a563c 100644 --- a/examples/widgets/widgets/calculator/button.cpp +++ b/examples/widgets/widgets/calculator/button.cpp @@ -50,8 +50,6 @@ #include "button.h" -#include - //! [0] Button::Button(const QString &text, QWidget *parent) : QToolButton(parent) diff --git a/examples/widgets/widgets/calculator/calculator.cpp b/examples/widgets/widgets/calculator/calculator.cpp index dd908cf40a..2c3669b7a8 100644 --- a/examples/widgets/widgets/calculator/calculator.cpp +++ b/examples/widgets/widgets/calculator/calculator.cpp @@ -48,21 +48,18 @@ ** ****************************************************************************/ -#include "button.h" #include "calculator.h" +#include "button.h" -#include - -#include +#include +#include +#include //! [0] Calculator::Calculator(QWidget *parent) - : QWidget(parent) + : QWidget(parent), sumInMemory(0.0), sumSoFar(0.0) + , factorSoFar(0.0), waitingForOperand(true) { - sumInMemory = 0.0; - sumSoFar = 0.0; - factorSoFar = 0.0; - waitingForOperand = true; //! [0] //! [1] @@ -78,9 +75,8 @@ Calculator::Calculator(QWidget *parent) //! [2] //! [4] - for (int i = 0; i < NumDigitButtons; ++i) { + for (int i = 0; i < NumDigitButtons; ++i) digitButtons[i] = createButton(QString::number(i), SLOT(digitClicked())); - } Button *pointButton = createButton(tr("."), SLOT(pointClicked())); Button *changeSignButton = createButton(tr("\302\261"), SLOT(changeSignClicked())); @@ -194,6 +190,8 @@ void Calculator::additiveOperatorClicked() //! [10] //! [11] { Button *clickedButton = qobject_cast