Nauka Qt z pewnej książki ( pytanie o radę i problemy w kodzie )

Question

Hej, zacząłem uczyć się Qt z tej książki: http://www.arxen.com/descargas/Books/Game%20Programming%20Using%20Qt%205,%20Begginer%27s%20Gu%20-%20Witold%20Wysota,%20Lorenz%20Haas.pdf

Utknąłem na stronie 38. Oto mój kod:

*.h

#ifndef TICTACTOEWIDGET_H
#define TICTACTOEWIDGET_H

#include <QWidget>
#include <QPushButton>
#include <QGridLayout>
class QPushButton;

class TicTacToeWidget : public QWidget {
    Q_OBJECT
public:
    TicTacToeWidget(QWidget *parent = nullptr);
    ~TicTacToeWidget();
private:
    QGridLayout *gridLayout;
    QList<QPushButton*> board;
    void setupBoard();
    void initNewGame();
};

#endif // TICTACTOEWIDGET_H

*.cpp

#include "tictactoewidget.h"
#include "ui_tictactoewidget.h"
#include <QPushButton>
#include <QGridLayout>
#include <QSignalMapper>

TicTacToeWidget::TicTacToeWidget(QWidget *parent)
    : QWidget(parent) {
    setupBoard();
    setLayout(gridLayout);
}

TicTacToeWidget::~TicTacToeWidget() {}

void TicTacToeWidget::setupBoard() {
    gridLayout = new QGridLayout(this);
    QSignalMapper* mapper = new QSignalMapper(this);
    for(int row = 0; row < 3; ++row) {
        for(int column = 0; column < 3; ++column) {
            QPushButton *button = new QPushButton;
            button->setSizePolicy(QSizePolicy::Minimum,QSizePolicy::Minimum);
            button->setText(" ");
            gridLayout->addWidget(button, row, column);
            board.append(button);
            mapper->setMapping(button, board.count()-1);
            connect(button, SINGAL(clicked())), mapper, SLOT(map()));
        }
    }
}

void TicTacToeWidget::initNewGame() {
    for(int i=0; i<9; ++i) board.at(i)->setText(" ");
}

Problemy mam takie:

26: błąd: use of undeclared identifier 'clicked'

17: ostrzeżenie: 'QSignalMapper' is deprecated

Jak zobaczyłem że w książce w przykładach pokazane jest używanie nieaktualnych już klas to zacząłem się zastanawiać czy warto się z niej jeszcze uczyć, miał ktoś może jakiś kontakt z nią? Czy powinienem ją kontynuować? Druga sprawa dotyczy oczywiście funkcji clicked()...

Z góry dziękuje za pomoc w rozwiązaniu problemów i rady :)

Answer 1

Witam,

tak jak kolega powiedział prawdopodobnie błąd tkwi w literówce. Staraj się używać nowszej składni w przypadku pisania connect. W tym przypadku:

connect(button, &QPushButton::clicked, mapper, &QSignalMapper::map);

Tutaj więcej informacji : https://doc.qt.io/qt-5/signalsandslots.html. Co do QSignalMapper to ta klasa dalej jest aktualna. Książkę, którą teraz przerabiasz posiada 2 wydania, mimo wszystko nawet pierwsze wydanie jest dosyć nowe. Sam ją przerobiłem i wydaje mi się przyzwoita. Jeśli chcesz tworzyć gry z użyciem Qt polecam Ci zapoznać się z frameworkiem Qt czyli felgo: https://felgo.com/, który bazuje głownie na module QtQuick. Przykład użycia tego frameworka do stworzenie popularnej gry Flappy Bird https://felgo.com/doc/howto-flappybird-game/

Comments

Co do QSignalMapper to ta klasa dalej jest aktualna

Autorowi pytania pewnie chodziło o:

https://doc.qt.io/qt-5/qsignalmapper.html

This class is obsolete. It is provided to keep old source code working. We strongly advise against using it in new code.

---

Ok, masz racje nie zauważyłem tego.

---

Dziękuje, ogólnie tworzenie gier nie jest moim głównym celem, po prostu spodobała mi się na książka bo opisuje Qt od podstaw w praktyce na przykładach różnych projektów. A wydaje mi się że pisanie gier bardzo dobrze pozwala uczyć się języka programowania, algorytmiki, logicznego myślenia itd...

Mam tylko pytanie dotyczące klasy QSignalMapper, czy powinienem ją ostatecznie czymś zastąpić czy na razie nie muszę sobie zawracać tym głowy?

---

@Jakub 0

nie zawracaj sobie tym głowy. Według źródła (2 edycji tej książki) można to tak zostawić.Według : https://github.com/PacktPublishing/Game-Programming-Using-Qt-5-Beginners-Guide-Second-Edition/blob/master/Chapter03/Qt%20GUI%20Programming/tictactoe/tictactoe%20ver7/tictactoewidget.cpp

...
TicTacToeWidget::TicTacToeWidget(QWidget *parent) : QWidget(parent)
{
    m_currentPlayer = Player::Invalid;
    QGridLayout *gridLayout = new QGridLayout(this);
    QSignalMapper *mapper = new QSignalMapper(this);
    for(int row = 0; row < 3; ++row) {
        for(int column = 0; column < 3; ++column) {
            QPushButton *button = new QPushButton(" ");
            button->setSizePolicy(QSizePolicy::Preferred,
                                  QSizePolicy::Preferred);
            gridLayout->addWidget(button, row, column);
            m_board.append(button);
            mapper->setMapping(button, m_board.count() - 1);
            connect(button, SIGNAL(clicked()), mapper, SLOT(map()));
        }
    }
    connect(mapper, SIGNAL(mapped(int)),
            this,   SLOT(handleButtonClick(int)));
}
...

Co do gier to możesz wejść na mojego GitHuba i zobaczyć przykładowe repozytoria gdzie wzorowałem się na tej książce. Może coś się przyda:

• https://github.com/Przemekkkth/Draughts-Cpp-Qt
• https://github.com/Przemekkkth/games
• https://github.com/Przemekkkth/Qt-C-SimpleGame [aktualnie robię]

Założyłem nawet post na tym forum odnośnie powyższych repo: https://forum.pasja-informatyki.pl/344914/proste-gry-qt.

---

Hej, wiem że teraz robię bardzo źle ale mam kolejny drobny problem i nie widzę sensu koniecznie zakładać dla niego pytania. Więc wyjątkowo zadam jeszcze jedno pytanie pod tym tematem ( już więcej razy nie będę ). Mam nadzieje że mi to wybaczysz ;)

Kwestia dotyczy strony 55, a dokładnie tej funkcji:
 

void ConfigurationDialog::updateOKButtonState() {
    bool pl1NameEmpty = ui->player1Name->text().isEmpty();
    bool pl2NameEmpty = ui->player2Name->text().isEmpty();
    QPushButton* okButton = ui->buttonBox->button(QDialogButtonBox::Ok);
    okButton->setDisabled(pl1NameEmpty || pl2NameEmpty); 
}

Mam takie błędy:

configurationdialog.cpp:21: błąd: member access into incomplete type 'QPushButton'

qdialog.h:51: forward declaration of 'QPushButton'

Raczej nie jest to literówka bo mogę skopiować to z książki i nadal jest tak samo... będę bardzo wdzięczny za pomoc i pozdrawiam :)

 

---

Mógłbym prosić o cały kod?

---

Problem rozwiązany, nie dodałem odpowiedniego pliku nagłówkowego...

Answer 2

SINGAL

SIGNAL?

Comments

Dziękuje, faktycznie :)