Add facilities to manage scenes #30

LumpBloom7 · 2019-03-11T15:18:14Z

This PR adds the facilities needed to manage Scenes properly.

Instead of placing GameObject, Registries, and Systems right in main() and "trying to" manage them, Scenes are object which contains everything needed for a particular scenario and blackboards are also easily created.

SceneManager handles the switching, update and rendering of a Scene.

Other changes:

Scenes render to it's own texture, before game renders that texture onto the rendering area.
Scenes can be rotated.
GameObjects can be assigned priorities, which makes them render on the top,bottom or in between other objects.
GameObjects can be assigned tags, which let's individual objects easy to access.
Sprites can also be rotated.

TODO:

Don't force a specific init function, instead the user should be able to provide their own.

WIP and not ready.

This prevents circular dependency when including them in Game.h

This prevents the program from continuously allocating memory, resulting in increased memory usage as time passes.

…/Scene

The class dtor would take care of it.

Also fix a build error in which an attempts to use `unload()` was made.

These functions will also set `m_sceneLoaded` to `true`

This change allows `Scene`s to be swapped out while retaining their state. (Something like suspend) Of course something would need to store the Scene or it'll just simply unload.

as it's stable and doesnэt use extra memory

Don't sort the `renderQueue` every frame. Instead, sort the whole GO list on every new `GameObject`. This should provide the same effect, while improving performance.

I'm not quite clear on how the registry sorting works, so I'm playing safe here.

This reverts commit 88c4f94.

This doesn't check whether the arguments provided are valid though

Protect eardrums and reduce scares XD.

`SDL_RenderClear()` caused the texture to be the set render color and it'll blend onto whatever that is underneath the scene. (Didn't happen with all colors, but still a problem.

LumpBloom7 · 2019-09-25T11:06:46Z

I'm considering dumping this PR in favor of the pure ECS model..
OOP for each entity is a pain in the ass to maintain, making sure they stay in scope so they don't destroy, implementing OOP alternatives to EnTT registry functions. Even if OOP is required, EnTT already provides a helper class for it. (https://github.com/skypjack/entt/wiki/Crash-Course:-entity-component-system#actor)

That is for GameObjects and Systems. Right now the alternative I'm thinking of is to use spawn functions to create entities, and convert system classes into functions instead.

LumpBloom7 and others added 30 commits October 5, 2018 20:33

Make System::update() non-pure virtual

c924a3f

Add Screen class

a2a93d4

Add some basic functions to Screen

ff979e1

WIP and not ready.

add real type compare

d051a2e

add automatic numbering for Screen Systems

b3ea51f

[Simplify] use SysPtr instead of Container

9626849

Fix potential runtime exception when entity is destroyed more than once

bd92973

Add Screen Testing to TestBench

07ef321

Made Screen protected variables private

8cff58f

Re-add dtor and improve update() function

3d8828e

Use forward declaration instead of includes in some files

b103f70

This prevents circular dependency when including them in Game.h

Remove comments

b9ffe87

Add functions in Game.h to make use of Screens

2e4bdca

Use the new functionality in TestBench

e2fa4f7

Make code consistent with the rest of the project

45ffcd4

Make DefaultSystem::update() a pure virtual function

cd8aa47

Make all build configs not generate xmldocs

f042b3c

Add SDL_RENDERER_TARGETTEXTURE to default renderer flags

9950953

[EXPERIMENTAL] Let Screens render to texture and let Game render it.

adad9ba

The unsafe Game ctor should also have the TARGETTEXTURE flag

7af06ea

Use SDL_RenderClear to clear Screen texture instead of recreating it

4ada583

This prevents the program from continuously allocating memory, resulting in increased memory usage as time passes.

Destroy Screen texture in dtor

d1667b9

Move Screen.h template definitions to prevent implicit inline

5c20f47

Use provided alias instead of just uint32_t in GameObject

f0b8292

The components don't depend on "stdIncludes.h"

78a3434

Implement ability to change render order of sprites

acafe28

Change grouping name from Priority to LayerGroup

0b956f2

Prevent implicit inline of Game::registerScreen()

fd98ec4

Merge branch 'master' into feature/Screens

9745ce5

Rename tmp2 to layerNo in RenderSystem

8eb9d16

LumpBloom7 and others added 29 commits January 15, 2019 21:47

Merge remote-tracking branch 'BloomCreativeWorks/master' into feature…

42071c8

…/Scene

Remove unload function

2531653

The class dtor would take care of it.

[Scene] Fix memory leak during destruction

87c9ed4

Fix some grammar

27a00ca

[Scene] Add functions to unregister all GameObjects/Systems

58774f6

Also fix a build error in which an attempts to use `unload()` was made.

[Scene] Add start and restart functions

4e06c1d

These functions will also set `m_sceneLoaded` to `true`

[SceneManager] Don't execute Scene::load if already loaded

a5824e8

This change allows `Scene`s to be swapped out while retaining their state. (Something like suspend) Of course something would need to store the Scene or it'll just simply unload.

[RenderQueue] Use std::list::sort instead of std::stable_sort

8b022e5

as it's stable and doesnэt use extra memory

Use size_t as entity type

88c4f94

Use optional::value_or instead of optional::value

9f8025b

Never use slow std::list, never

5861b67

Optimize LayerGroup sorting

c802ab4

Don't sort the `renderQueue` every frame. Instead, sort the whole GO list on every new `GameObject`. This should provide the same effect, while improving performance.

[TestChar] Always assign a LayerGroup component

cd8e310

I'm not quite clear on how the registry sorting works, so I'm playing safe here.

Bring back std::stable_sort:)

d10a4d8

Revert "Use size_t as entity type"

acd2a7b

This reverts commit 88c4f94.

Remove pure virtual init function

dbd1e13

Add check to ensure init function exists

dccd92e

This doesn't check whether the arguments provided are valid though

NoInit v3

2250ada

Merge remote-tracking branch 'xapdkop/test/no_init3' into test/No_Init

45eed87

Test lambda support. DON'T MERGE THIS!

e41456a

Update GameObject's names and types

d1d4b9d

Disable dat fckin' console spam

6d31d8a

Use more template power

87d186a

[SceneManager] Remove unused thisPtr

dd9c49f

[GameObject && Scene] Require init function to be specified

618690a

Fix CodeFactor issues

d8c5dc1

[MISC] Reduce volume for background music

9259185

Protect eardrums and reduce scares XD.

Merge branch 'master' into feature/Scene

8aa0999

[Scene] Scene textures should be transparent

787cecb

`SDL_RenderClear()` caused the texture to be the set render color and it'll blend onto whatever that is underneath the scene. (Didn't happen with all colors, but still a problem.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Add facilities to manage scenes #30

Add facilities to manage scenes #30

Uh oh!

LumpBloom7 commented Mar 11, 2019 •

edited

Loading

Uh oh!

LumpBloom7 commented Sep 25, 2019 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Add facilities to manage scenes #30

Are you sure you want to change the base?

Add facilities to manage scenes #30

Uh oh!

Conversation

LumpBloom7 commented Mar 11, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

LumpBloom7 commented Sep 25, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

LumpBloom7 commented Mar 11, 2019 •

edited

Loading

LumpBloom7 commented Sep 25, 2019 •

edited

Loading