Reduce complexity

Author: bouwew

plugwise_usb/nodes/helpers/pulses.py

@@ -507,6 +507,19 @@ def _update_log_direction(
         if self._logs is None:
             return
 
+        self._check_prev_production(address, slot, timestamp)
+        self._check_next_production(address, slot, timestamp)
+        if self._prev_log_exists and self._next_log_exists:
+            # _log_production is True when 2 out of 3 consecutive slots have
+            # the same timestamp, otherwise it is False
+            self._log_production = (
+                next_timestamp == timestamp and prev_timestamp != timestamp
+            ) or (next_timestamp == prev_timestamp and next_timestamp != timestamp)
+
+    def _check_prev_production(
+        self, address: int, slot: int, timestamp: datetime
+    ) -> None:
+        """Check the previous slot for production pulses."""
         prev_address, prev_slot = calc_log_address(address, slot, -1)
         if self._log_exists(prev_address, prev_slot):
             prev_timestamp = self._logs[prev_address][prev_slot].timestamp
@@ -528,6 +541,10 @@ def _update_log_direction(
         elif self._prev_log_exists:
             self._prev_log_exists = False
 
+    def _check_next_production(
+        self, address: int, slot: int, timestamp: datetime
+    ) -> None:
+    """Check the next slot for production pulses."""
         next_address, next_slot = calc_log_address(address, slot, 1)
         if self._log_exists(next_address, next_slot):
             next_timestamp = self._logs[next_address][next_slot].timestamp
@@ -549,13 +566,6 @@ def _update_log_direction(
         elif self._next_log_exists:
             self._next_log_exists = False
 
-        if self._prev_log_exists and self._next_log_exists:
-            # _log_production is True when 2 out of 3 consecutive slots have
-            # the same timestamp, otherwise it is False
-            self._log_production = (
-                next_timestamp == timestamp and prev_timestamp != timestamp
-            ) or (next_timestamp == prev_timestamp and next_timestamp != timestamp)
-
     def _update_log_interval(self) -> None:
         """Update the detected log interval based on the most recent two logs."""
         if self._logs is None or self._log_production is None: