josh/ichigo
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

more comment

Browse source
This commit is contained in:
Josh Deprez 2021-08-31 13:41:26 +10:00
parent 30224c15bf
commit 80f00e8107
1 changed files with 17 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
engine/interface.go
View file

@ -129,15 +129,23 @@ type Scanner interface {
// Scener components are a scene (Scene or SceneRef). // Scener components are a scene (Scene or SceneRef).
type Scener interface { type Scener interface {
// Q: Why not make Scener a small interface with just Scene() ? // Q: Why not make Scene able to load itself?
// A: Everything in the engine would then need to type switch for Scener or SceneRef, i.e. // A: Having separate types makes it easier to reason about what is loading
// what. There is less ambiguity about what "save" means (the contents of
// the scene, or the path to the file?) Additionally, the gob decoder
// decodes over existing fields, which could lead to some fun bugs.
//
// Q: Why not make Scener a small interface, e.g. with just Scene() ?
// A: Everything in the engine would then need to type switch for Scener or
// SceneRef, i.e.
// switch x := i.(type) { // switch x := i.(type) {
// case Drawer: // case Drawer:
// i.Draw(screen, opts) // i.Draw(screen, opts)
// case Scener: // case Scener:
// i.Scene().Draw(screen, opts) // i.Scene().Draw(screen, opts)
// } // }
// It seems cleaner to let the engine assert only for the interface it needs at that moment. // It seems cleaner to let the engine assert only for the interface it
// needs at that moment.
Bounder Bounder
Disabler Disabler