OLD-REPO: Fix method for initializing components

.editorconfig

@@ -2,3 +2,6 @@
 
 # IDE1006: Naming Styles
 dotnet_diagnostic.IDE1006.severity = none
+
+# CA1031: Do not catch general exception types
+dotnet_diagnostic.CA1031.severity = suggestion

Common/Behaviors/ModelStartupBehavior.cs → Common/Behaviors/CubeBehavior.cs

@@ -6,24 +6,19 @@
 
 namespace Common.Behaviors
 {
-    public class ModelStartupBehavior : BehaviorComponent
+    public class CubeBehavior : BehaviorComponent
     {
         #region Constructors
-        public ModelStartupBehavior() : base()
+        public CubeBehavior(string modelPath) : base()
         {
+            ModelPath = modelPath;
         }
         #endregion Constructors
 
         #region Members
 
         #region Public Members
-        public bool IsInitialized
-        {
-            get
-            {
-                return this._isInitialized;
-            }
-        }
+        public string ModelPath { get; protected set; }
         #endregion Public Members
 
         #region Protected Members
@@ -44,12 +39,15 @@ public override void Update(GameTime gameTime)
             {
                 _isInitialized = true;
                 var loadedTexture = KomodoGame.Content.Load<Texture2D>("player/idle/player_idle_01");
-                Parent.AddComponent(
-                    new ModelComponent("models/cube")
-                    {
-                        Texture = new KomodoTexture(loadedTexture)
-                    }
-                );
+                if (ModelPath != null)
+                {
+                    Parent.AddComponent(
+                        new ModelComponent(ModelPath)
+                        {
+                            Texture = new KomodoTexture(loadedTexture)
+                        }
+                    );
+                }
                 Parent.Scale = new KomodoVector3(20f, 20f, 20f);
             }
             Parent.Rotation += new KomodoVector3(0f, 0f, 0.001f);

Common/Behaviors/FPSCounterBehavior.cs

@@ -8,20 +8,15 @@ namespace Common.Behaviors
     public class FPSCounterBehavior : BehaviorComponent
     {
         #region Constructors
-        public FPSCounterBehavior(TextComponent textComponent) : base()
-        {
-            _counterText = textComponent;
-            _counterText.Position = KomodoVector3.Zero;
-        }
         #endregion Constructors
 
         #region Members
 
         #region Public Members
+        public TextComponent CounterText { get; protected set;  }
         #endregion Public Members
 
         #region Protected Members
-        public TextComponent _counterText { get; }
         #endregion Protected Members
 
         #region Private Members
@@ -32,9 +27,19 @@ public FPSCounterBehavior(TextComponent textComponent) : base()
         #region Member Methods
 
         #region Public Member Methods
+        public override void Initialize()
+        {
+            base.Initialize();
+
+            CounterText = new TextComponent("fonts/font", Color.Black, Game.DefaultSpriteShader, "")
+            {
+                Position = KomodoVector3.Zero
+            };
+            Parent.AddComponent(CounterText);
+        }
         public override void Update(GameTime gameTime)
         {
-            _counterText.Text = $"{Math.Round(Game.FramesPerSecond)} FPS";
+            CounterText.Text = $"{Math.Round(Game.FramesPerSecond)} FPS";
         }
         #endregion Public Member Methods
 

Common/Behaviors/FPSCounterStartupBehavior.cs → Common/Behaviors/PlayerBehavior.cs

@@ -1,24 +1,22 @@
+using Komodo.Core;
 using Komodo.Core.ECS.Components;
 using Microsoft.Xna.Framework;
 
 namespace Common.Behaviors
 {
-    public class FPSCounterStartupBehavior : BehaviorComponent
+    public class PlayerBehavior : BehaviorComponent
     {
         #region Constructors
-        public FPSCounterStartupBehavior() : base()
+        public PlayerBehavior(int playerIndex) : base()
         {
+            PlayerIndex = playerIndex;
         }
         #endregion Constructors
 
         #region Members
 
         #region Public Members
-        public bool IsInitialized
-        {
-            get;
-            private set;
-        }
+        public int PlayerIndex { get; }
         #endregion Public Members
 
         #region Protected Members
@@ -32,15 +30,25 @@ public bool IsInitialized
         #region Member Methods
 
         #region Public Member Methods
+        public override void Initialize()
+        {
+            base.Initialize();
+
+            Parent.AddComponent(new SpriteComponent("player/idle/player_idle_01", Game.DefaultSpriteShader));
+            if (PlayerIndex == 0)
+            {
+                Parent.AddComponent(new MoveBehavior(PlayerIndex));
+            }
+
+            Parent.AddComponent(
+                new TextComponent("fonts/font", Color.Black, Game.DefaultSpriteShader, $"Test {PlayerIndex}")
+                {
+                    Position = new KomodoVector3(0f, 20f, 0)
+                }
+            );
+        }
         public override void Update(GameTime gameTime)
         {
-            if (!IsInitialized)
-            {
-                IsInitialized = true;
-                var textComponent = new TextComponent("fonts/font", Color.Black, Game.DefaultSpriteShader, "");
-                Parent.AddComponent(textComponent);
-                Parent.AddComponent(new FPSCounterBehavior(textComponent));
-            }
         }
         #endregion Public Member Methods
 

DesktopGL/Startup.cs

@@ -23,12 +23,12 @@ static void Main()
                 {
                     Position = new KomodoVector3(0f, 0f, 100f),
                 };
-                player1Entity.AddComponent(new RootStartupBehavior(0));
+                player1Entity.AddComponent(new PlayerBehavior(0));
                 var player2Entity = new Entity(Game.ActiveScene)
                 {
                     Position = new KomodoVector3(0f, 0f, 100f),
                 };
-                player2Entity.AddComponent(new ModelStartupBehavior());
+                player2Entity.AddComponent(new CubeBehavior("models/cube"));
 
                 var camera = new CameraComponent()
                 {
@@ -45,7 +45,7 @@ static void Main()
                 var orthographicScene = new Scene(Game);
                 Game.ActiveScene.Children.Add(orthographicScene);
                 var counterEntity = new Entity(orthographicScene);
-                counterEntity.AddComponent(new FPSCounterStartupBehavior());
+                counterEntity.AddComponent(new FPSCounterBehavior());
 
                 var cameraEntity = new Entity(orthographicScene)
                 {
@@ -60,17 +60,7 @@ static void Main()
                 cameraEntity.AddComponent(camera);
                 camera.SetActive();
 
-                var startupEntities = new List<Entity>
-                {
-                    player1Entity,
-                    player2Entity,
-                    counterEntity,
-                    cameraEntity,
-                };
-
-                // Startup entities will be attached to the active scene once the monogame initialization has occurred.
-                // This allows the default shader to be created only after the graphics device has been initialized.
-                Game.Run(startupEntities);
+                Game.Run();
             }
         }
 

Komodo/Core/ECS/Components/Component.cs

@@ -8,7 +8,7 @@ namespace Komodo.Core.ECS.Components
     public abstract class Component : ISerializable<Component>
     {
         #region Constructors
-        public Component(bool isEnabled = true, Entity parent = null)
+        protected Component(bool isEnabled = true, Entity parent = null)
         {
             IsEnabled = isEnabled;
             Parent = parent;
@@ -19,6 +19,7 @@ public Component(bool isEnabled = true, Entity parent = null)
 
         #region Public Members
         public bool IsEnabled { get; set; }
+        public bool IsInitialized { get; private set; }
         [JsonIgnore]
         public Entity Parent { get; set; }
         public KomodoVector3 Position { get; set; }
@@ -78,6 +79,13 @@ public virtual void Draw(SpriteBatch spriteBatch)
         public virtual void Draw()
         {
         }
+        public virtual void Initialize()
+        {
+            if (!IsInitialized)
+            {
+                IsInitialized = true;
+            }
+        }
         public abstract SerializedObject Serialize();
         public virtual void Update(GameTime gameTime)
         {

Komodo/Core/ECS/Entities/Entity.cs

@@ -203,9 +203,11 @@ public bool RemoveComponent(Component componentToRemove)
 
         public SerializedObject Serialize()
         {
-            var serializedObject = new SerializedObject();
-            serializedObject.Type = this.GetType().ToString();
-
+            var serializedObject = new SerializedObject
+            {
+                Type = this.GetType().ToString()
+            };
+
             var components = new List<SerializedObject>();
             foreach (var component in Components)
             {