Как наиболее просто и правильно организовать для свойств класса поведение, аналогичное «транзакции» (или ctrl-z)?

Question

[Kalombyr]

Добрый день!
Есть некий класс, с Q_PROPERTY(....)
Из UI задаются им значения.

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

Пока что я при подтверждении транзакции я заношу в массив значения всех свойств,
при отмене, соответственно восстанавливаю оттуда.

Что не устраивает - нужно каждое свойство прописывать руками для каждого класса, в котором нужно такое поведение, что несколько утомляет, да и уже пару раз столкнулся с ошибкой копи-паста.

В идеале решение проблемы было бы унаследоваться от какого-либо класса, где были бы реализованы функции подтверждения и отмены транзакции, что бы все свойства запоминались или восстанавливались.

Подскажите, пожалуйста, куда копать и как бы реализовать?

Answer 1

[al_sh]

насколько я понял Вы используете Qt. В таком случае не надо велосипедить doc.qt.io/qt-5/qtwidgets-tools-undoframework-examp...

Answer 2

[Максим Мосейчук]

Кладите в массив не все члены объекта, а сам объект. И тут не массив нужен, тут идеально стек подходит.

Вот я вам накидал примерчик https://ideone.com/tOXA0S

Пример

#include <iostream>
#include <stack>
#include <memory>

template<typename T>
class Undo : public T {
private:
    std::shared_ptr<std::stack<Undo>> history_ptr = std::make_shared<std::stack<Undo>>();

public:
    template <typename... Args>
    explicit Undo(const Args&... args): T(args ...) {}

    void save() {
        history_ptr->push(*this);
    }

    void undo() {
        if (!history_ptr->empty()) {
            *this = history_ptr->top();
            history_ptr->pop();
        }
    }
};

class Foo {
public:
    int a = 0;
    std::string s;

    Foo(const int _a, std::string _s): a(_a), s(std::move(_s)) {}
};

std::ostream &operator<<(std::ostream &out, const Foo &foo) {
    out << "Foo(" << foo.a << ", \"" << foo.s << "\")";
    return out;
}

int main() {
    Undo<Foo> foo(1, "Hello World!");

    foo.save();
    std::cout << "Save\t" << foo << std::endl;

    foo.a = 6661313;
    foo.save();
    std::cout << "Save\t" << foo << std::endl;

    foo.s = "Lakad Matataaag!";
    foo.save();
    std::cout << "Save\t" << foo << std::endl;

    foo.a = 9000;
    foo.s = "Normalin, Normalin!";
    foo.save();
    std::cout << "Save\t" << foo << std::endl;

    foo.a = 12345678;
    foo.s = "Da next lebel play!";
    std::cout << "Current\t" << foo << std::endl;

    foo.undo();
    std::cout << "Undo\t" << foo << std::endl;

    foo.undo();
    std::cout << "Undo\t" << foo << std::endl;

    foo.undo();
    std::cout << "Undo\t" << foo << std::endl;

    foo.undo();
    std::cout << "Undo\t" << foo << std::endl;

    return 0;
}

Но вам нужно следить за реализацией конструктора копирования и оператора присваивания, что бы они корректно себя вели.

Comments

[Kalombyr]

Эм... Моя немного не понимать.
Ведь транзакция должна был в пределах одного экземпляра класса.

[Максим Мосейчук]

Kalombyr, есть у вас объект, который вы модифируете. Копию оригинала кладете в стек, затем проводите любые модификации.

Стек можно хранить внутри шаблонного класса по shared_ptr. В этом классе сделать методы для сохранения и возврата состояния.

От этого класса и наследуйтесь.

[Максим Мосейчук]

Kalombyr, обновил ответ

[al_sh]

fshp, а, если в классе динамически выделяется метров 100 на куче - хранить на стеке копию класса весом > 100 метров? указатель хранить не получится, поскольку при изменении свойств исходного экземпляра изменятся свойства и экзепмляра в стэке, поскольку это один и тот же экземпляр по сути. Ваш метод годен для легких классов без свойст на куче.

[Максим Мосейчук]

al_sh, а если при каждой транзакции менять все 100 мегабайт данных?

На счёт хранения указателей я предупредил о необходимости корректной реализации копирования и присваивания.