From 73f8d5c558ebc0820ede241e64a876ff1c5ccefb Mon Sep 17 00:00:00 2001
From: WBoerenkamps <wrj.boerenkamps@student.avans.nl>
Date: Thu, 12 Dec 2024 07:57:24 +0100
Subject: seperat ints converted to float

---
 src/crepe/system/InputSystem.cpp | 77 ++++++++++++++++++----------------------
 1 file changed, 34 insertions(+), 43 deletions(-)

(limited to 'src/crepe/system/InputSystem.cpp')

diff --git a/src/crepe/system/InputSystem.cpp b/src/crepe/system/InputSystem.cpp
index f5d8536..e76b1ec 100644
--- a/src/crepe/system/InputSystem.cpp
+++ b/src/crepe/system/InputSystem.cpp
@@ -2,7 +2,6 @@
 #include "../api/Button.h"
 #include "../manager/ComponentManager.h"
 #include "../manager/EventManager.h"
-
 #include "InputSystem.h"
 
 using namespace crepe;
@@ -41,15 +40,15 @@ void InputSystem::update() {
 			|| event.event_type == SDLContext::EventType::MOUSEMOVE
 			|| event.event_type == SDLContext::EventType::MOUSEWHEEL) {
 
-			int adjusted_mouse_x = event.mouse_data.mouse_position.x + camera_origin.x;
-			int adjusted_mouse_y = event.mouse_data.mouse_position.y + camera_origin.y;
-
+			ivec2 adjusted_mouse;
+			adjusted_mouse.x = event.mouse_data.mouse_position.x + camera_origin.x;
+			adjusted_mouse.y = event.mouse_data.mouse_position.y + camera_origin.y;
 			// Check if the mouse is within the viewport
 			bool mouse_in_viewport
-				= !(adjusted_mouse_x < camera_origin.x
-					|| adjusted_mouse_x > camera_origin.x + current_cam.viewport_size.x
-					|| adjusted_mouse_y < camera_origin.y
-					|| adjusted_mouse_y > camera_origin.y + current_cam.viewport_size.y);
+				= !(adjusted_mouse.x < camera_origin.x
+					|| adjusted_mouse.x > camera_origin.x + current_cam.viewport_size.x
+					|| adjusted_mouse.y < camera_origin.y
+					|| adjusted_mouse.y > camera_origin.y + current_cam.viewport_size.y);
 
 			if (!mouse_in_viewport) continue;
 
@@ -57,45 +56,44 @@ void InputSystem::update() {
 			switch (event.event_type) {
 				case SDLContext::EventType::MOUSEDOWN:
 					event_mgr.queue_event<MousePressEvent>({
-						.mouse_pos = {adjusted_mouse_x, adjusted_mouse_y},
+						.mouse_pos = {adjusted_mouse.x, adjusted_mouse.y},
 						.button = event.mouse_data.mouse_button,
 					});
-					this->last_mouse_down_position = {adjusted_mouse_x, adjusted_mouse_y};
+					this->last_mouse_down_position = {adjusted_mouse.x, adjusted_mouse.y};
 					this->last_mouse_button = event.mouse_data.mouse_button;
 					break;
 
 				case SDLContext::EventType::MOUSEUP: {
 					event_mgr.queue_event<MouseReleaseEvent>({
-						.mouse_pos = {adjusted_mouse_x, adjusted_mouse_y},
+						.mouse_pos = {adjusted_mouse.x, adjusted_mouse.y},
 						.button = event.mouse_data.mouse_button,
 					});
-					int delta_x = adjusted_mouse_x - this->last_mouse_down_position.x;
-					int delta_y = adjusted_mouse_y - this->last_mouse_down_position.y;
+					int delta_x = adjusted_mouse.x - this->last_mouse_down_position.x;
+					int delta_y = adjusted_mouse.y - this->last_mouse_down_position.y;
 
 					if (this->last_mouse_button == event.mouse_data.mouse_button
 						&& std::abs(delta_x) <= click_tolerance
 						&& std::abs(delta_y) <= click_tolerance) {
 						event_mgr.queue_event<MouseClickEvent>({
-							.mouse_pos = {adjusted_mouse_x, adjusted_mouse_y},
+							.mouse_pos = {adjusted_mouse.x, adjusted_mouse.y},
 							.button = event.mouse_data.mouse_button,
 						});
-						this->handle_click(event.mouse_data.mouse_button, adjusted_mouse_x,
-										   adjusted_mouse_y);
+						this->handle_click(event.mouse_data.mouse_button, adjusted_mouse);
 					}
 					break;
 				}
 
 				case SDLContext::EventType::MOUSEMOVE:
 					event_mgr.queue_event<MouseMoveEvent>({
-						.mouse_pos = {adjusted_mouse_x, adjusted_mouse_y},
+						.mouse_pos = {adjusted_mouse.x, adjusted_mouse.y},
 						.mouse_delta = event.mouse_data.rel_mouse_move,
 					});
-					this->handle_move(event, adjusted_mouse_x, adjusted_mouse_y);
+					this->handle_move(event, adjusted_mouse);
 					break;
 
 				case SDLContext::EventType::MOUSEWHEEL:
 					event_mgr.queue_event<MouseScrollEvent>({
-						.mouse_pos = {adjusted_mouse_x, adjusted_mouse_y},
+						.mouse_pos = {adjusted_mouse.x, adjusted_mouse.y},
 						.scroll_direction = event.mouse_data.scroll_direction,
 						.scroll_delta = event.mouse_data.scroll_delta,
 					});
@@ -148,54 +146,57 @@ void InputSystem::update() {
 }
 
 void InputSystem::handle_move(const SDLContext::EventData & event_data,
-							  const int adjusted_mouse_x, const int adjusted_mouse_y) {
+							  const ivec2& mouse_pos) {
 	ComponentManager & mgr = this->mediator.component_manager;
 
 	RefVector<Button> buttons = mgr.get_components_by_type<Button>();
 
 	for (Button & button : buttons) {
+		if(!button.active) continue;
 		RefVector<Transform> transform_vec
 			= mgr.get_components_by_id<Transform>(button.game_object_id);
 		Transform & transform(transform_vec.front().get());
 
 		bool was_hovering = button.hover;
-		if (button.active
-			&& this->is_mouse_inside_button(adjusted_mouse_x, adjusted_mouse_y, button,
+		if (this->is_mouse_inside_button(mouse_pos, button,
 											transform)) {
 			button.hover = true;
-			if (!was_hovering && button.on_mouse_enter) {
+			if(!button.on_mouse_enter) continue;
+			if (!was_hovering) {
 				button.on_mouse_enter();
 			}
 		} else {
 			button.hover = false;
 			// Trigger the on_exit callback if the hover state just changed to false
-			if (was_hovering && button.on_mouse_exit) {
+			if(!button.on_mouse_exit) continue;
+			if (was_hovering) {
 				button.on_mouse_exit();
 			}
 		}
 	}
 }
 
-void InputSystem::handle_click(const MouseButton & mouse_button, const int adjusted_mouse_x,
-							   const int adjusted_mouse_y) {
+void InputSystem::handle_click(const MouseButton & mouse_button, const ivec2& mouse_pos) {
 	ComponentManager & mgr = this->mediator.component_manager;
 
 	RefVector<Button> buttons = mgr.get_components_by_type<Button>();
 
 	for (Button & button : buttons) {
+		if(!button.active) continue;
+		if (!button.on_click) continue;
 		RefVector<Transform> transform_vec
 			= mgr.get_components_by_id<Transform>(button.game_object_id);
 		Transform & transform = transform_vec.front().get();
-
-		if (button.active
-			&& this->is_mouse_inside_button(adjusted_mouse_x, adjusted_mouse_y, button,
+		
+		if (this->is_mouse_inside_button(mouse_pos, button,
 											transform)) {
-			this->handle_button_press(button);
+			 
+			button.on_click();
 		}
 	}
 }
 
-bool InputSystem::is_mouse_inside_button(const int mouse_x, const int mouse_y,
+bool InputSystem::is_mouse_inside_button(const ivec2& mouse_pos,
 										 const Button & button, const Transform & transform) {
 	int actual_x = transform.position.x + button.offset.x;
 	int actual_y = transform.position.y + button.offset.y;
@@ -204,17 +205,7 @@ bool InputSystem::is_mouse_inside_button(const int mouse_x, const int mouse_y,
 	int half_height = button.dimensions.y / 2;
 
 	// Check if the mouse is within the button's boundaries
-	return mouse_x >= actual_x - half_width && mouse_x <= actual_x + half_width
-		   && mouse_y >= actual_y - half_height && mouse_y <= actual_y + half_height;
+	return mouse_pos.x >= actual_x - half_width && mouse_pos.x <= actual_x + half_width
+		   && mouse_pos.y >= actual_y - half_height && mouse_pos.y <= actual_y + half_height;
 }
 
-void InputSystem::handle_button_press(Button & button) {
-	if (button.is_toggle) {
-		if (!button.is_pressed && button.on_click) {
-			button.on_click();
-		}
-		button.is_pressed = !button.is_pressed;
-	} else if (button.on_click) {
-		button.on_click();
-	}
-}
-- 
cgit v1.2.3