⚡ Optimize Tap Loop by hoisting redundant element finding and coordinate calculation.

This change moves the `_element_find` call and the center coordinate calculations outside the repetition loop in the `tap` keyword. This avoids making multiple expensive Appium server calls for the same element when the `count` argument is greater than 1.

Key changes:
- Hoisted `self._element_find(element, True, True)` outside the loop.
- Hoisted `location` and `size` retrieval and `center_x`, `center_y` calculations outside the loop.
- The `driver.tap` call remains inside the loop to perform the requested number of taps at the calculated position.

Performance impact:
- Number of `_element_find` calls reduced from `count` to 1.
- Significant reduction in network/IPC latency for multi-tap operations.

Author: Dor-bl

AppiumLibrary/keywords/_touch.py

@@ -326,12 +326,12 @@ def tap(self, element: Union[str, list], count:int = 1, duration=timedelta(secon
                 raise ValueError(f"Invalid coordinates format: {element}. Expected a list like [x, y]")
 
         elif isinstance(element, str):
+            el = self._element_find(element, True, True)
+            location = el.location
+            size = el.size
+            center_x = location['x'] + size['width'] // 2
+            center_y = location['y'] + size['height'] // 2
             for _ in range(count):
-                el = self._element_find(element, True, True)
-                location = el.location
-                size = el.size
-                center_x = location['x'] + size['width'] // 2
-                center_y = location['y'] + size['height'] // 2
                 driver.tap([(center_x, center_y)], duration.total_seconds() * 1000)
 
         else: