c++ - 在列表中使用 QWidget 指针时，它们无法显示

Question

我正在我的 PC 上创建一个日历程序，但是我在创建一个 Widget 来显示 Day-details 时遇到了一个非常奇怪的问题。

它应该显示的是顶部的选定日期，带有上一个和下一个按钮，在该日期下是一个以 1 小时为间隔的带有 QPushButton 子类的列表，但由于某种原因，我只能获得选定日期 + 上一个按钮 + 当我尝试运行它时的下一个按钮。

小部件指针放在列表中似乎是个问题，但对我来说这是个问题似乎很奇怪。但是当我测试不使用列表并硬编码它工作的所有内容时，为什么我将这些指针放在列表中会出现问题？

GUICalendarDay::GUICalendarDay(QDate date)
{
    m_date = date;
    //creating buttons
    m_prevDay = new QPushButton(QString("<"));
    m_nextDay = new QPushButton(QString(">"));

    connect(m_nextDay, SIGNAL(clicked()), this, SLOT(nextDay()));
    connect(m_prevDay, SIGNAL(clicked()), this, SLOT(prevDay()));

    //creating the cells
    for(int i = 0; i++; (i<TOTALHOURS))
    {
        m_shedule[i] = new GUICalendarCell(QTime(i, 0, 0), QTime(i, 59, 59));//creating 1-hour intervals
        connect(m_shedule[i], SIGNAL(clickedRange(QTime,QTime)), this, SLOT(selectedRange(QTime,QTime)));
    }

    //creating the labels
    m_dateLabel = new QLabel(m_date.toString(QString("dd/MM/yyyy")));

    QLabel *hourLabel[TOTALHOURS];

    for(int i = 0; i++; (i<TOTALHOURS))
    {
        hourLabel[i] = new QLabel(QTime(i,0,0).toString(QString("hh:mm")));
    }


    //placing everything in a layout
    QGridLayout *mainLayout = new QGridLayout;
    mainLayout->addWidget(m_prevDay, 0, 1);
    mainLayout->addWidget(m_dateLabel, 0, 2);
    mainLayout->addWidget(m_nextDay, 0, 3);
    for(int i = 0; i++; (i<TOTALHOURS))
    {
        mainLayout->addWidget(hourLabel[i], i+1, 0);
        hourLabel[i]->show();
        mainLayout->addWidget(m_shedule[i], i+1, 1, 3, 1);
        m_shedule[i]->show();
    }
    setLayout(mainLayout);

}

这不是什么大问题，但我想保持我的代码干净，我也想知道为什么这不起作用。

Answer 1

您在上一个 for 中用循环测试切换了循环增量环形。它应该显示为 for (int i = 0; i < TOTALHOURS; ++) .

您的代码还有其他几个样式问题。

1. 使用容器类，而不是原始数组。

2. 您可以将所有成员放入类中，而不是显式地将它们存储在堆中。

3. 您可以使用 PIMPL 使 header 不包含实现细节。

下面的示例需要 C++11 和 Qt 5。它可以通过使用 Q_PRIVATE_SLOT 适应 Qt 4 和 C++9x。 .

// Interface (header file)

#include <QWidget>

class QDate;
class GUICalendarDayPrivate;
class GUICalendarDay : public QWidget {
   Q_OBJECT
   Q_DECLARE_PRIVATE(GUICalendarDay)
   QScopedPointer<GUICalendarDayPrivate> const d_ptr;
public:
   explicit GUICalendarDay(const QDate & date, QWidget * parent = 0);
   ~GUICalendarDay();
};

// Implementation (.cpp file)

#include <QGridLayout>
#include <QPushButton>
#include <QLabel>
#include <QDate>

// dummy to make the example compile
class GUICalendarCell : public QWidget {
   Q_OBJECT
public:
   explicit GUICalendarCell(const QTime &, const QTime &, QWidget * parent = 0) : QWidget(parent) {}
   Q_SIGNAL void clickedRange(const QTime &, const QTime &);
};

class GUICalendarDayPrivate {
public:
   QDate date;
   QGridLayout layout;
   QPushButton prevDay, nextDay;
   QLabel dateLabel;
   QList<GUICalendarCell*> schedule;

   QList<QLabel*> hourLabels;
   explicit GUICalendarDayPrivate(const QDate &, QWidget * parent);
   void onNextDay() {}
   void onPrevDay() {}
   void selectedRange(const QTime &, const QTime &) {}
};

GUICalendarDay::GUICalendarDay(const QDate &date, QWidget *parent) : QWidget(parent),
   d_ptr(new GUICalendarDayPrivate(date, this))
{}

GUICalendarDay::~GUICalendarDay() {}

GUICalendarDayPrivate::GUICalendarDayPrivate(const QDate & date_, QWidget * parent) :
   date(date_),
   layout(parent),
   prevDay("<"), nextDay(">"), dateLabel(date.toString(QString("dd/MM/yyyy")))
{
   const int TOTALHOURS = 24;
   QObject::connect(&nextDay, &QAbstractButton::clicked, [this]{ onNextDay(); });
   QObject::connect(&prevDay, &QAbstractButton::clicked, [this]{ onPrevDay(); });

   layout.addWidget(&prevDay, 0, 1);
   layout.addWidget(&dateLabel, 0, 2);
   layout.addWidget(&nextDay, 0, 3);

   for(int i = 0, row = layout.rowCount(); i<TOTALHOURS; ++i, ++row) {
      auto from = QTime(i, 0, 0), to = QTime(i, 59, 59);
      auto cell = new GUICalendarCell(from, to);
      auto hourLabel = new QLabel(from.toString(QString("hh:mm")));

      QObject::connect(cell, &GUICalendarCell::clickedRange,
                       [this](const QTime & from, const QTime & to){
         selectedRange(from, to);
      });

      schedule << cell;
      hourLabels << hourLabel;
      layout.addWidget(hourLabel, row, 0);
      layout.addWidget(cell, row, 1, 3, 1);
   }
}

// main.cpp
#include <QApplication>

int main(int argc, char *argv[])
{
   QApplication a(argc, argv);
   GUICalendarDay day(QDate::currentDate());
   day.show();
   return a.exec();
}

// Only needed if all of the example is in a single file
#include "main.moc"