feat: Базовая система сохранений by irazaurus · Pull Request #310 · PosAlina/G2I-Game

irazaurus · 2026-03-08T21:20:22Z

Close #149

Итак, система сохранений.

Этим занимается GameInstance через интерфейс - сейчас есть все для сохранения состояния игры, и чисто база для сохранения настроек (ее нужно будет потом дописать, но работать она будет почти так же).

Работает это так: в GameInstance лежат два объекта SaveGame - один для игры, второй для настроек. Отовсюду обращение к ним идет через интерфейс (SaveGameplayInterface для игры, SaveSettingsInterface для настроек). Сначала данные сохраняются в SaveGame объект, потом этот SaveGame анрильскими функциями уже сохраняется в файлик на компьютере.
Этот файлик находится в G2I-Game -> Saved -> SaveGames. Если хотите затереть сохранение, просто удалите оттуда файлик.
Данные сохраняются либо для отдельного объекта (SaveRequestedData) либо для всех акторов (AActor) с интерфейсом сохранения (SavableInterface) на данном уровне (SaveAllData).
С загрузкой то же самое, хотя загрузку лучше всего будет применять все-таки индивидуально для актора по его запросу. Почему? Потому что для разных акторов это будет необходимо в разное время. Они не одновременно спавнятся на уровне, какие-то будут спавниться во время игры, так что в основном будет использоваться LoadRequestedData. Я вижу LoadAllData полезным только в двух случаях: при дебаге и при начальном запуске игры, и то второе не факт.

Так, как этим пользоваться на практике?

Допустим, вам надо, чтобы TestActor сохранял свою позицию и переменную bool.

В G2IGameplaySaveGame добавляем параметры, которые будут сохранены в дальнейшем.
// Ссылки и имена (GetName) использовать нельзя - используйте для идентификации то, что останется неизменным от запуска к запуску.
// ОБЯЗАТЕЛЬНО в спецификаторе UPROPERTY указать SaveGame.
// Если данных для объекта много, оберните их в структуру. См. пример, который есть сейчас для игрока.
Добавляем в TestActor интерфейс IG2ISavableInterface
Имплементируем функции из этого интерфейса (в C++ через _Implement, в БП просто как обычно).
// Проверять в них ссылку на SaveGame не надо - эта проверка уже есть во всех функциях GameInstance, где Save и Load вызываются. Этот референс никогда не будет nullptr.
// В этих функциях вы оперируете данными, которые мы добавили к сейв гейму в первом пункте.

То есть, для нашего примера, это будет:

В GameplaySaveGame я добавлю структуру TestActorSaveData, в которой будет два параметра: FTransform и bool. Оба со спецификаторами SaveGame, и сама структура тоже с ним.
В SaveData_Implementation будет что-то типа
SaveGameRef->TestActorSaveData.FTransform = GetActorTransform();
SaveGameRef->TestActorSaveData.bool = bool;
В LoadData_Implementation будет наоборот SetActorTransform и присвоение переменной.

Вот и все.
Сохраняются только AActor!!

Чтобы подписаться на делегаты сохранения/загрузки, возьмите их из GameInstance - они хранятся в классе UG2ISaveGameplayDelegates - через функцию GetGameplaySaveDelegates. Полезно для виджетов.

Теперь про чекпоинт.

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

Дебаг

Я добавила две дебажные кнопки для тестирования системы сохранений:

K - все в текущем уровне сохраняет (считайте Keep)
L - все для текущего уровня загружает (Load)

На тестовом уровне есть два чекпоинта-триггер бокса.

И сейчас сохраняются/загружаются следующие данные:

Трансформ всех игровых персонажей
Какой из персонажей был под контролем игрока на момент сохранения
Состояния чекпоинтов (были активированы или нет)

у меня не компилилось иначе

+ refact

irazaurus · 2026-03-10T13:19:35Z

изменения, НЕ касающиеся системы сохранений:

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

PosAlina

Предложение добавить функцию переключения на конкретного персонажа, а не на следующего. Код в комментарии приложила
А точно нужен интерфейс для GameInstance?
Возможно, можно вынести систему сохранений в GameInstanceSubsystem (так UIManager выносила), чтобы логчески отделить от game instance. И вместо gameinstance с нужным интерфейсом вызывать GameInstance->GetSubsystem<НужныйКласс>();
И второй интерфейс не удобнее компонентой (возможно, с наследниками) сделать, чтобы не копировать логику функций?
Немного вопросов по написанию синтаксису
Об изменениях в слайдере в общий чат напиши, плиз
Ну и часть вопросов классических по nullptr и ensur-ов для error-логов. Чтоб меньше проверок на nullptr делать, можно в функции передавать ссылки, а не пойнтеры

PosAlina · 2026-03-10T19:47:09Z

Source/G2I/Private/Characters/G2ICharacterDaughter.cpp

 	return OnUnPossessedDelegate;
 }
+
+void AG2ICharacterDaughter::SaveData_Implementation(UG2IGameplaySaveGame* SaveGameRef)


Либо проверить SaveGameRef на nullptr
Либо передавать в функцию ссылку, а не указатель

GameInstance не будет вызывать эти функции, если SaveGame == nullptr.
но могу поменять на ссылку

Лучше проверку всё же сделать, или на ссылку поменять

PosAlina · 2026-03-10T19:59:50Z

Source/G2I/Private/Characters/G2ICharacterDaughter.cpp

+
+void AG2ICharacterDaughter::LoadData_Implementation(const UG2IGameplaySaveGame* SaveGameRef)
+{
+	if (IsPlayerControlled() && !this->IsA(SaveGameRef->PlayersSaveData.CurrentCharacter))


А зачем this?

хороший вопрос. в моменте так почувствовала

PosAlina · 2026-03-10T20:25:25Z

Source/G2I/Private/Characters/G2ICharacterDaughter.cpp

+	{
+		if (auto* G2IPlayerState = Cast<AG2IPlayerState>(GetPlayerState()))
+		{
+			G2IPlayerState->SelectNextCharacter();


В идеале
можно было бы в PlayerState добавить функцию,
SetCharacterByClass(SaveGameRef->PlayersSaveData.CurrentCharacter)
В которой в идеале выбирать персонажа с нужным классом или писать, что нет такого персонажа
Примерно так:

void AG2IPlayerState::SetCharacterByClass(const TSubclassOf<ACharacter> TargetClass) { if (!ensure(TargetClass)) { UE_LOG(LogG2I, Warning, TEXT("An attempt to set character with null class")); return; } if (!ensure(!PlayableCharactersRowNames.IsEmpty())) { UE_LOG(LogG2I, Warning, TEXT("An attempt to set character, when array of playable characters is empty")); return; } for (int32 OffsetRowName = 1; OffsetRowName <= PlayableCharactersRowNames.Num(); ++OffsetRowName) { const int32 NewCharacterIndex = (NumberCurrentCharacter + OffsetRowName) % PlayableCharactersRowNames.Num(); if (NewCharacterIndex == NumberCurrentCharacter) { UE_LOG(LogG2I, Warning, TEXT("Character with class %s doesn't founded"), *TargetClass->GetName()); return; } if (const FG2IItemCharacter *Row = PlayableCharactersDataTable->FindRow<FG2IItemCharacter>( PlayableCharactersRowNames[NewCharacterIndex], TEXT("Playable Character Context"))) { if (Row->CharacterClass == TargetClass) { // Строки из SelectNextCharacter, в принципе их можно вынеститож в отдельную функцию const int32 OldCharacterNumber = NumberCurrentCharacter; NumberCurrentCharacter = NewCharacterIndex; if (SetupControllerForPawn(OldCharacterNumber)) { if (SetupControllerForPawn(NumberCurrentCharacter)) { OnNewControllerPossessDelegate.Broadcast(GetPawn(OldCharacterNumber)); OnNewControllerPossessDelegate.Broadcast(GetPawn(NumberCurrentCharacter)); break; } else { UE_LOG(LogG2I, Warning, TEXT("Character %i doesn't switched on player controller"), NumberCurrentCharacter); NumberCurrentCharacter = OldCharacterNumber; SetupControllerForPawn(NumberCurrentCharacter); continue; } } else { UE_LOG(LogG2I, Warning, TEXT("Character %i doesn't switched on ai controller"), OldCharacterNumber); NumberCurrentCharacter = OldCharacterNumber; continue; } } } else { UE_LOG(LogG2I, Error, TEXT("In %s array of row names doesn't sync with %s"), *GetName(), *PlayableCharactersDataTable.GetName()); continue; } } }

ооо фига, сделаем, спасибо!
(в своем решении я основывалась на том, что у нас уже точно больше двух персонажей не будет)

PosAlina · 2026-03-10T20:26:43Z

Source/G2I/Private/Characters/G2ICharacterDaughter.cpp

+	SaveGameRef->PlayersSaveData.CharactersTransform.Add(GetClass(), GetTransform());
+}
+
+void AG2ICharacterDaughter::LoadData_Implementation(const UG2IGameplaySaveGame* SaveGameRef)


Проверка SaveGameRef на nullptr или передавать ссылку в функцию

Source/G2I/Private/Characters/G2ICharacterEngineer.cpp

PosAlina · 2026-03-10T21:16:24Z

Source/G2I/Public/Characters/G2ICharacterEngineer.h

 #include "CoreMinimal.h"
 #include "G2ICharacterInterface.h"
 #include "GameFramework/Character.h"
+#include "Interfaces/SavingSystem/G2ISaveGameplayInterface.h"


Не используется

PosAlina · 2026-03-10T21:17:22Z

Source/G2I/Public/Characters/G2ICharacterEngineer.h

 #include "G2ICharacterInterface.h"
 #include "GameFramework/Character.h"
+#include "Interfaces/SavingSystem/G2ISaveGameplayInterface.h"
+#include "Interfaces/SavingSystem/G2ISavableInterface.h"


Можно кстати без "Interfaces/" этот инклуд писать
В Build.cs путь прописан

PosAlina · 2026-03-10T21:18:32Z

Source/G2I/Public/Game/G2IGameInstance.h

 #include "CoreMinimal.h"
 #include "Engine/GameInstance.h"
+#include "Kismet/GameplayStatics.h"
+#include "GameFramework/SaveGame.h"


Тут инклуд не используемые убрать можно

PosAlina · 2026-03-10T21:20:35Z

Source/G2I/Public/Game/G2IGameInstance.h

+	TObjectPtr<UG2ISaveGameplayDelegates> SaveGameplayDelegates;
+
+private:
+	// SaveGame w/ gameplay info


Имхо, лучше в комментариях не использовать сокращения, чтобы у всех, кто посмотрит код, было однозначное понимание

PosAlina · 2026-03-10T21:24:36Z

Source/G2I/Public/Game/G2IGameInstance.h

+
+	void CreateNewGameplaySaveGameObject_Implementation();
+
+	void SaveGameplay_Implementation(bool bAsync);


Лучше для функций, реализующих функции интерфейса писать virtual и override

для Implementation это надо делать? мне казалось что это анриловская тема и она не коннектится с этими специальными обозначениями

Код стайл а-дя
_Implementation говорит компилятору, что это виртуальная функция, но в коде лучше помечать дополнительно virtual
А без override - в принципе любые функции могут и без него работать, но для читаемости и чтоб компилятор проверял, что всё ок написано, лучше ставить

irazaurus added 8 commits March 8, 2026 23:30

save game base

8b137e8

feat: savable tag + interface

7f5401a

feat: save gameplay save & load functions

b654e6f

refact: deleted spaces + player's locations for testing

a26d5ba

refact: saving gameplay

ef9e60c

feat: save actions + player's data

bb05c4f

refact: slider's SetVectorParameterValue

a521840

у меня не компилилось иначе

feat: added save/load actions to bps

ca226e7

irazaurus linked an issue Mar 8, 2026 that may be closed by this pull request

Feature: Контрольные точки (базовая система сохранений) #149

Open

irazaurus self-assigned this Mar 8, 2026

irazaurus added 3 commits March 9, 2026 13:54

feat: checkpoint

b6ead43

+ refact

refact: saving system + PC's jump logs

09f706e

feat: added saving trigger boxes on test level

f3e577c

irazaurus marked this pull request as ready for review March 10, 2026 12:31

irazaurus requested review from Amigo509, Keendaj, PosAlina and slylii March 10, 2026 12:31

irazaurus force-pushed the dev-149-save-points branch from ba25940 to 306cded Compare March 10, 2026 13:22

refact: deleted unused files and tag

442542d

irazaurus force-pushed the dev-149-save-points branch from 306cded to 442542d Compare March 10, 2026 13:23

PosAlina requested changes Mar 10, 2026

View reviewed changes

refact: moved saving from interface to subsystem

23047e8


		void CreateNewGameplaySaveGameObject_Implementation();

		void SaveGameplay_Implementation(bool bAsync);

Conversation

irazaurus commented Mar 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Итак, система сохранений.

Так, как этим пользоваться на практике?

Теперь про чекпоинт.

Дебаг

Uh oh!

irazaurus commented Mar 10, 2026

Uh oh!

PosAlina left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

irazaurus commented Mar 8, 2026 •

edited

Loading

PosAlina left a comment •

edited

Loading