Changed init cleanup to attach detach

Author: ArtifactForms

src/main/java/engine/components/AbstractComponent.java

@@ -37,5 +37,5 @@ public void setOwner(SceneNode owner) {
 	public SceneNode getOwner() {
 		return owner;
 	}
-	
+
 }

src/main/java/engine/components/CinematicBlackBarsRenderer.java

@@ -192,28 +192,12 @@ private void renderBottomBar(Graphics g) {
 		    (int) currentSize);
 	}
 
-	/**
-	 * Initialization logic for this renderer.
-	 * <p>
-	 * Currently unimplemented but provided as a placeholder for any setup
-	 * required when initializing this component in a larger system.
-	 * </p>
-	 */
 	@Override
-	public void initialize() {
-		// Placeholder for initialization logic.
+	public void onAttach() {
 	}
 
-	/**
-	 * Cleans up resources when the renderer is no longer needed.
-	 * <p>
-	 * Currently unimplemented but can be used to clean up graphics resources,
-	 * listeners, or other allocated resources in a larger system.
-	 * </p>
-	 */
 	@Override
-	public void cleanup() {
-		// Placeholder for cleanup logic.
+	public void onDetach() {
 	}
 
 }

src/main/java/engine/components/CircularAnimationComponent.java

@@ -99,28 +99,12 @@ public void update(float tpf) {
 		transform.setPosition(x, transform.getPosition().y, z);
 	}
 
-	/**
-	 * Handles any initialization logic required before the animation starts.
-	 * <p>
-	 * Currently, this is left blank but can be implemented if preconditions need
-	 * to be set before the animation begins.
-	 * </p>
-	 */
 	@Override
-	public void initialize() {
-		// Placeholder for potential initialization logic
+	public void onAttach() {
 	}
 
-	/**
-	 * Handles any cleanup logic when the animation component is no longer needed.
-	 * <p>
-	 * Currently, this is left blank but can be implemented if resources need to
-	 * be released or reset during cleanup.
-	 * </p>
-	 */
 	@Override
-	public void cleanup() {
-		// Placeholder for potential cleanup logic
+	public void onDetach() {
 	}
 
 }

src/main/java/engine/components/ControlWASD.java

@@ -73,15 +73,6 @@ public ControlWASD(Input input, float speed) {
 		this.speed = speed;
 	}
 
-	/**
-	 * Initializes the component. Currently no initialization logic is required,
-	 * but this is provided as a placeholder for future logic.
-	 */
-	@Override
-	public void initialize() {
-		// No additional initialization logic required at this stage.
-	}
-
 	/**
 	 * Updates the movement of the owning node based on keyboard input.
 	 * <p>
@@ -113,12 +104,16 @@ public void update(float tpf) {
 	 */
 	private Vector3f handleInput() {
 		Vector3f velocity = new Vector3f();
-		
+
 		// Check for movement inputs
-		if (input.isKeyPressed(Key.W)) velocity.addLocal(0, 0, -1);
-		if (input.isKeyPressed(Key.A)) velocity.addLocal(-1, 0, 0);
-		if (input.isKeyPressed(Key.S)) velocity.addLocal(0, 0, 1);
-		if (input.isKeyPressed(Key.D)) velocity.addLocal(1, 0, 0);
+		if (input.isKeyPressed(Key.W))
+			velocity.addLocal(0, 0, -1);
+		if (input.isKeyPressed(Key.A))
+			velocity.addLocal(-1, 0, 0);
+		if (input.isKeyPressed(Key.S))
+			velocity.addLocal(0, 0, 1);
+		if (input.isKeyPressed(Key.D))
+			velocity.addLocal(1, 0, 0);
 
 		// Normalize diagonal movement to prevent unintended speed boosts
 		if (velocity.length() > 0) {
@@ -127,16 +122,6 @@ private Vector3f handleInput() {
 		return velocity;
 	}
 
-	/**
-	 * Cleans up resources when the component is removed or the application is
-	 * shutting down. Currently, there is no cleanup logic necessary, but this is
-	 * a placeholder for extensibility.
-	 */
-	@Override
-	public void cleanup() {
-		// Cleanup logic placeholder
-	}
-
 	/**
 	 * Retrieves the current movement speed multiplier.
 	 * 
@@ -167,4 +152,12 @@ public void setSpeed(float speed) {
 		this.speed = speed;
 	}
 
+	@Override
+	public void onAttach() {
+	}
+
+	@Override
+	public void onDetach() {
+	}
+
 }

src/main/java/engine/components/Geometry.java

@@ -153,24 +153,12 @@ public void update(float tpf) {
 		// Placeholder for potential mesh state updates
 	}
 
-	/**
-	 * Called when the component is attached to a parent object in the scene. This
-	 * method is a hook for additional initialization or setup when the component
-	 * is added.
-	 */
 	@Override
 	public void onAttach() {
-		// Hook for additional setup on attachment
 	}
 
-	/**
-	 * Called when the component is detached from its parent object in the scene.
-	 * This method is a hook for cleanup or other actions when the component is
-	 * removed.
-	 */
 	@Override
 	public void onDetach() {
-		// Hook for cleanup on detachment
 	}
 
 }

src/main/java/engine/components/RotationComponent.java

@@ -112,13 +112,11 @@ public float getAngularSpeed() {
 	}
 
 	@Override
-	public void initialize() {
-		// Initialization logic if needed
+	public void onAttach() {
 	}
 
 	@Override
-	public void cleanup() {
-		// Cleanup logic if needed
+	public void onDetach() {
 	}
-	
+
 }