Qt: обновить растровую сетку со значениями двумерного массива

Вы вручную ввели и назвали 200 виджетов ярлыков? Пусть никто не называет вас ленивым. :)

Согласно вашему обновлению, теперь вы знаете, как использовать QLabel::setPixmap(). Как вы думаете, вам нужно получить указатель QLabel из имени, что будет комбинацией двух вещей:

• QWidget::findChild, чтобы получить QWidget* из QString

• qobject_cast, чтобы получить QLabel* из QWidget

Если вы пойдете по этому пути, вы получите что-то вроде:

QWidget* cellWidget = ui->findChild(TR_position);
QLabel* cellLabel = qobject_cast<QLabel*>(cellWidget);
cellLabel->setPixmap(QPixmap(":/images/missspace.png"));

Но ОСТЕРЕГАЙТЕСЬ! В этом подходе много неправильного.

• Он хрупкий. Что, если виджета с таким именем не существует (загадочный сбой)? Или, что еще хуже, что, если существует несколько виджетов с таким именем, а этот код продолжает блаженно игнорировать это странное условие, которое, вероятно, является ошибкой?

• Это плохое ООП: хотя есть несколько достойных случаев использования динамического приведения (или "понижающего приведения"), обычно это указывает на недостаток в дизайне. Вы знаете, что все QLabels являются QWidgets, но не все QWidgets являются QLabels... так что вызов qobject_cast может вернуть NULL. Это просто еще одна точка отказа. Иногда вы не можете избежать этого, но на самом деле нет причин, по которым ваша программа должна быть структурирована таким образом.

• Это ужасно медленно: поиск виджета по его имени, по сути, является наивным рекурсивным поиском. Если вы выделили отдельный фрейм виджета для каждой сетки и ищете только его, Qt придется выполнить 100 сравнений строк, чтобы найти последний элемент (то есть 50 в среднем случае). Представьте себе очистку сетки с помощью цикла... теперь вы говорите о 100*50 сравнениях строк!

Всего этого можно избежать. Точно так же, как можно использовать циклы для установки изображений в элементах управления по имени, циклы можно использовать в первую очередь для создания виджетов. В основном вы оставляете область для игрового поля пустой в инструменте дизайна, а затем динамически создаете элементы управления с помощью кода... прикрепляете их к макету с помощью кода... и сохраняете указатели на них в 2D-массиве. (В этот момент вы не сможете получить к ним доступ по имени метки, вы будете индексировать их так же, как вы индексируете свою доску.)

Вы можете создать свой собственный класс, производный от QLabel (например, класс GameCell), который будет содержать информацию о вашей ячейке платы и связанных с ней методах. Тогда вам не понадобится массив виджетов меток параллельно массиву, представляющему вашу доску. У вас будет просто один массив объектов, который позаботится об обоих аспектах реализации.

ОБНОВЛЕНИЕ: поскольку вы просили в комментариях уточнить подробности, вот класс GameCell:

class GameCell : public QLabel
{
    Q_OBJECT    

public:
    enum State { Undiscovered, Hit, Miss };

    GameCell (QWidget *parent = 0) : QLabel (parent),
        currentState (Undiscovered)
    {
        syncBitmap();
    }

    State getState() const { return currentState; }

    void setState(State newState) {
        if (currentState != newState) {
            currentState = newState;
            syncBitmap();
        }
    }

private:
    void syncBitmap() { // you'd use setPixmap instead of setText
        switch (currentState) {
        case Undiscovered: setText("U"); break;
        case Hit: setText("H"); break;
        case Miss: setText("M"); break;
        }
    }

    State currentState;
};

Это выполняет двойную функцию, ведя себя как QWidget, а также сохраняя часть внутреннего состояния. Затем виджет GameMap может использовать QGridLayout этих ячеек GameCell:

class GameMap : public QWidget {
    Q_OBJECT

public:
    static const int Rows = 10;
    static const int Columns = 10;

    GameMap (QWidget* parent = 0) :
        QWidget (parent)
    {
        layout = new QGridLayout (this);
        for (int column = 0; column < Columns; column++) {
            for (int row = 0; row < Rows; row++) {
                GameCell* cell = new GameCell (this);
                cells[column][row] = cell;
                layout->addWidget(cell, row, column);
            }
        }
    }

private:
    GameCell* cells[Columns][Rows];
    QGridLayout* layout;
};

Если вы хотите, вы можете просто оставить пробелы в своем макете в дизайнере, который вы хотите заполнить виджетом GameMap. Или вы можете нажать и сделать все это программно. Для простоты я просто поставлю две доски рядом друг с другом с вертикальным разделителем на поверхности диалога:

class Game : public QDialog
{
    Q_OBJECT

public:
    Game (QWidget *parent = 0)
        : QDialog(parent)
    {
        targetMap = new GameMap (this);
        fleetMap = new GameMap (this);

        verticalSeparator = new QFrame (this);
        verticalSeparator->setFrameShape(QFrame::VLine);
        verticalSeparator->setFrameShadow(QFrame::Sunken);

        layout = new QHBoxLayout (this);
        layout->addWidget(targetMap);
        layout->addWidget(verticalSeparator);
        layout->addWidget(fleetMap);
        setLayout(layout);

        setWindowTitle(tr("Battleship"));
    }

private:
    GameMap* targetMap;
    QFrame* verticalSeparator;
    GameMap* fleetMap;
    QHBoxLayout* layout;
};

Я не собираюсь писать здесь целую игру или приукрашивать ее. Это только суть, показывающая, как получить 200 меток программным способом:

С моим кодом получение GameCell из координаты (x, y) не требует в среднем 50 сравнений строк. Из-за формализованного и предсказуемого характера двумерных массивов индексация в cells[x][y] требует только одной операции умножения и одной операции сложения. Нет понижения, и вы можете просто написать:

cells[x][y].setState(GameCell::Miss);

ДОПОЛНЕНИЕ: создание QWidget для каждой ячейки сетки не всегда является правильным решением. Кто-то может подумать, что это «тяжеловес». Если бы ваша игра разыгрывалась на большом виртуальном пространстве плиток, она могла бы быть слишком медленной. Возможно, вам будет полезно изучить QGraphicsGridLayout, который может быть более подходящий подход в долгосрочной перспективе:

http://doc.qt.io/qt-5/qtwidgets-graphicsview-basicgraphicslayouts-example.html

Однако использование QWidgets не будет большой проблемой с сеткой 10x10, поэтому, если вы хотите просто придерживаться этого, вы можете. Если вы собираетесь делать это таким образом, то, по крайней мере, вам не следует размещать их все вручную!