fix(useGamepad): improve data updating logic (#3775)

vueuse · Feb 20, 2024 · 9b8ed55 · 9b8ed55
1 parent 8c73515
commit 9b8ed55
Showing 1 changed file with 6 additions and 15 deletions.
diff --git a/packages/core/useGamepad/index.ts b/packages/core/useGamepad/index.ts
@@ -92,14 +92,9 @@ export function useGamepad(options: UseGamepadOptions = {}) {
   const updateGamepadState = () => {
     const _gamepads = navigator?.getGamepads() || []
 
-    for (let i = 0; i < _gamepads.length; ++i) {
-      const gamepad = _gamepads[i]
-      if (gamepad) {
-        const index = gamepads.value.findIndex(({ index }) => index === gamepad.index)
-
-        if (index > -1)
-          gamepads.value[index] = stateFromGamepad(gamepad)
-      }
+    for (const gamepad of _gamepads) {
+      if (gamepad && gamepads.value[gamepad.index])
+        gamepads.value[gamepad.index] = stateFromGamepad(gamepad)
     }
   }
 
@@ -125,13 +120,9 @@ export function useGamepad(options: UseGamepadOptions = {}) {
   tryOnMounted(() => {
     const _gamepads = navigator?.getGamepads() || []
 
-    if (_gamepads) {
-      for (let i = 0; i < _gamepads.length; ++i) {
-        const gamepad = _gamepads[i]
-
-        if (gamepad)
-          onGamepadConnected(gamepad)
-      }
+    for (const gamepad of _gamepads) {
+      if (gamepad && gamepads.value[gamepad.index])
+        onGamepadConnected(gamepad)
     }
   })