mqtt延时任务

agri-framework/src/main/java/com/agri/framework/interceptor/MqttMessageHandler.java

@@ -290,6 +290,8 @@ public class MqttMessageHandler implements SmartLifecycle {
         String funcType = null;
         Integer funcValue = null;
         boolean isAck = false;
+        // 新增：标记是否需要执行自动关任务（全局可用）
+        boolean needAutoOffTask = false;
 
         // 第二步：设备回执处理逻辑（完全移除Redis写入）
         if (payloadObj.containsKey("suc") && payloadObj.containsKey("prop")) {
@@ -322,7 +324,12 @@ public class MqttMessageHandler implements SmartLifecycle {
                     if (agriLimit != null) {
                         autoOffSeconds = LIMIT_MAP.getOrDefault(funcType, k -> 0).apply(agriLimit);
                     }
-                    scheduleAutoOff(deviceId, funcType, autoOffSeconds);
+                    // 新增：判断是否真的需要执行自动关任务（延迟秒数>0才是有效任务）
+                    needAutoOffTask = autoOffSeconds > 0;
+                    if (needAutoOffTask) {
+                        scheduleAutoOff(deviceId, funcType, autoOffSeconds);
+                        log.debug("【自动关任务】标记需要执行，deviceId={}, funcType={}, delay={}s", deviceId, funcType, autoOffSeconds);
+                    }
                 }
 
                 // ========== 关键：设备回执完全不写入Redis ==========
@@ -330,10 +337,9 @@ public class MqttMessageHandler implements SmartLifecycle {
             }
         }
 
-        // 第三步：仅处理非回执的设备状态包，且仅当是8个功能码结构时写入Redis
+        // 第三步：仅处理非回执的设备状态包，且仅当是8个功能码结构+需要执行自动关任务时写入Redis
+        // 有没有人订阅都得写，只要发送设备开的指令成功了就得写
         if (!isAck) {
-            boolean needWriteLatest = false;
-
             // 1) 先校验状态包是否包含8个固定功能码（核心：只有这种结构才写入）
             boolean isValidStatus = true;
             for (String validCode : VALID_FUNC_CODES) {
@@ -346,29 +352,21 @@ public class MqttMessageHandler implements SmartLifecycle {
                 log.debug("【设备状态包】结构不合法（非8个功能码），跳过Redis写入，deviceId={}；payload={}", deviceId,payload);
             }
 
-            // 2) 有人订阅时才写latest
+            // 2) 核心修改：仅当需要执行自动关任务时，才判断是否写入Redis
-            Set<String> subscribedClients = stringRedisTemplate.opsForSet().members("sub:" + deviceId);
+            // 条件1：当前已标记需要执行新的自动关任务
-            if (subscribedClients != null && !subscribedClients.isEmpty()) {
+            // 条件2：或已有未完成的自动关任务（避免任务执行中状态丢失）
-                needWriteLatest = true;
+            // 最终写入条件：有自动关任务（必要条件） + 有人订阅（可选条件，可按需删除）
-            }
+            // 3) 满足条件则写入完整的8功能码JSON到Redis
-
+            if (needAutoOffTask || hasAutoOffTask(deviceId)) {
-            // 3) 或者存在该设备的自动关任务时也写latest
-            if (!needWriteLatest && hasAutoOffTask(deviceId)) {
-                needWriteLatest = true;
-            }
-
-            // 4) 满足条件则写入完整的8功能码JSON到Redis
-            if (needWriteLatest) {
                 stringRedisTemplate.opsForValue().set(
                         "device:latest:" + deviceId,
                         payload,  // 完整的8功能码JSON
                         latestTtlSeconds,
                         TimeUnit.SECONDS
                 );
-                // log.debug("【设备状态包】写入Redis成功，deviceId={}", deviceId);
+                log.debug("【设备状态包】因需要执行自动关任务，写入Redis成功，deviceId={}", deviceId);
             }
         }
-
         // 非回执消息：正常转发给订阅前端
         // 查询Redis中订阅该设备的前端列表：sub:{deviceId}
         Set<String> subscribedClients = stringRedisTemplate.opsForSet().members("sub:" + deviceId);