BoomingTech / Piccolo

Piccolo (formerly Pilot) – mini game engine for games104
MIT License
5.79k stars 1.8k forks source link

fix access modifiers, change repeatd protected to private. #373

Closed Adrian145 closed 1 year ago

Adrian145 commented 1 year ago

class PiccoloEngine 内第二个protected看起来应该是private,下方的变量都是类内部使用。

hyv1001 commented 1 year ago

用protect会因为未来可能会有子类继承,这样子类可以访问这些内部成员变量

Adrian145 commented 1 year ago

用protect会因为未来可能会有子类继承,这样子类可以访问这些内部成员变量

33行处已经有protected标识,此处的第二个protected是起分隔函数和成员变量,作为视觉分隔的作用吗?

另外想请教一下 PiccoloEngine 子类的场景预期,当前PiccoloEngine没虚析构函数,但是使用上是申请了就不会释放, 子类是预计依然保持不释放吗? 如果做运行时引擎切换,释放,似乎就需要给PiccoloEngine添加一下虚析构函数。