refactor(common): 基于 Hutool 简化 LocalDateTime 日期解析，判空提前返回，补全边界用例

dadiyang
1 parent 9a41ee23
Showing 2 changed files with 47 additions and 50 deletions
yudao-framework/yudao-common/src/main/java/cn/iocoder/yudao/framework/common/util/json/databind/TimestampLocalDateTimeDeserializer.java
yudao-framework/yudao-common/src/test/java/cn/iocoder/yudao/framework/common/util/json/databind/TimestampLocalDateTimeDeserializerTest.java
 package cn.iocoder.yudao.framework.common.util.json.databind;
+import cn.hutool.core.date.DateUtil;
+import cn.hutool.core.date.LocalDateTimeUtil;
+import cn.hutool.core.util.StrUtil;
 import com.fasterxml.jackson.core.JsonParser;
-import com.fasterxml.jackson.core.JsonToken;
 import com.fasterxml.jackson.databind.DeserializationContext;
 import com.fasterxml.jackson.databind.JsonDeserializer;
 import java.io.IOException;
 import java.time.Instant;
-import java.time.LocalDate;
 import java.time.LocalDateTime;
-import java.time.OffsetDateTime;
-import java.time.ZoneId;
 import java.time.format.DateTimeFormatter;
-import java.time.format.DateTimeParseException;
 /**
- * 基于时间戳的 LocalDateTime 反序列化器
- *
- * 兼容范围（覆盖前端/接口实际在用的全部格式，避免把"原先能静默入库"的格式变成报错）：
- * 1. epoch 毫秒：JSON 数字 / 数字字符串（原契约，保持不变）
- * 2. 常见日期时间字符串：yyyy-MM-dd HH:mm:ss（含毫秒）
- * 3. ISO-8601：yyyy-MM-ddTHH:mm:ss（本地 / 带时区，后者覆盖前端 Date 对象序列化）
- * 4. 纯日期：yyyy-MM-dd（当日零点）
- * 无法解析时抛清晰错误（含原始值），不再静默落 1970-01-01（消除静默数据损坏）。
+ * LocalDateTime 反序列化器，支持毫秒时间戳、常见日期时间字符串和 ISO-8601 日期。
  *
  * @author 老五
  */
@@ -30,51 +21,22 @@ public class TimestampLocalDateTimeDeserializer extends JsonDeserializer&lt;LocalDa
     public static final TimestampLocalDateTimeDeserializer INSTANCE = new TimestampLocalDateTimeDeserializer();
-    private static final DateTimeFormatter DT = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss");
-    private static final DateTimeFormatter DT_MILLIS = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss.SSS");
-
     @Override
     public LocalDateTime deserialize(JsonParser p, DeserializationContext ctxt) throws IOException {
-        // 1. JSON 数字 = epoch 毫秒（原契约，保持）
-        if (p.currentToken() == JsonToken.VALUE_NUMBER_INT) {
-            return LocalDateTime.ofInstant(Instant.ofEpochMilli(p.getLongValue()), ZoneId.systemDefault());
-        }
         String text = p.getText();
-        if (text == null || text.trim().isEmpty()) {
+        if (StrUtil.isBlank(text)) {
             return null;
         }
         text = text.trim();
-        // 1. 数字字符串 = epoch 毫秒
         if (text.matches("-?\\d+")) {
-            return LocalDateTime.ofInstant(Instant.ofEpochMilli(Long.parseLong(text)), ZoneId.systemDefault());
-        }
-        // 2. yyyy-MM-dd HH:mm:ss / .SSS
-        try {
-            return LocalDateTime.parse(text, DT);
-        } catch (DateTimeParseException ignored) {
-        }
-        try {
-            return LocalDateTime.parse(text, DT_MILLIS);
-        } catch (DateTimeParseException ignored) {
-        }
-        // 3. ISO-8601 本地（2026-09-11T00:00:00 / .SSS）
-        try {
-            return LocalDateTime.parse(text);
-        } catch (DateTimeParseException ignored) {
-        }
-        // 3. ISO-8601 带时区（前端 Date 对象序列化：2026-09-11T00:00:00.000Z / +08:00）
-        try {
-            return OffsetDateTime.parse(text).atZoneSameInstant(ZoneId.systemDefault()).toLocalDateTime();
-        } catch (DateTimeParseException ignored) {
+            return LocalDateTimeUtil.of(Long.parseLong(text));
         }
-        // 4. 纯日期（2026-09-11）→ 当日零点
-        try {
-            return LocalDate.parse(text).atStartOfDay();
-        } catch (DateTimeParseException ignored) {
+        // Hutool 5.x 的自动解析会忽略部分负时区偏移，并截断纳秒。
+        if (text.contains("T")) {
+            return LocalDateTimeUtil.of(DateTimeFormatter.ISO_DATE_TIME.parseBest(
+                    text, Instant::from, LocalDateTime::from));
         }
-        // 无法解析：抛清晰错误（不再静默落 1970）
-        throw new IllegalArgumentException("无法解析时间值：" + text
-                + "（支持 epoch 毫秒 / yyyy-MM-dd HH:mm:ss / ISO-8601 / yyyy-MM-dd）");
+        return LocalDateTimeUtil.of(DateUtil.parse(text).toInstant());
     }
 }
@@ -63,6 +63,12 @@ public class TimestampLocalDateTimeDeserializerTest {
     }
     @Test
+    public void testIsoLocalWithNanos() throws Exception {
+        assertEquals(LocalDateTime.of(2026, 9, 11, 8, 0, 0, 123456789),
+                parse("\"2026-09-11T08:00:00.123456789\""));
+    }
+
+    @Test
     public void testIsoWithOffset() throws Exception {
         // 前端 Date 对象序列化：带 Z（UTC）。2026-09-11T00:00:00Z 换算到系统时区
         LocalDateTime expected = java.time.OffsetDateTime.parse("2026-09-11T00:00:00Z")
@@ -92,16 +98,45 @@ public class TimestampLocalDateTimeDeserializerTest {
     }
     @Test
+    public void testIsoWithNegativeOffsetAndMillis() throws Exception {
+        LocalDateTime expected = java.time.OffsetDateTime.parse("2026-09-11T23:30:00.123-05:30")
+                .atZoneSameInstant(java.time.ZoneId.systemDefault()).toLocalDateTime();
+        assertEquals(expected, parse("\"2026-09-11T23:30:00.123-05:30\""));
+    }
+
+    @Test
+    public void testZeroAndNegativeEpochMilli() throws Exception {
+        for (long millis : new long[]{0L, -1L}) {
+            LocalDateTime expected = LocalDateTime.ofInstant(java.time.Instant.ofEpochMilli(millis),
+                    java.time.ZoneId.systemDefault());
+            assertEquals(expected, parse(Long.toString(millis)));
+            assertEquals(expected, parse("\"" + millis + "\""));
+        }
+    }
+
+    @Test
+    public void testSurroundingWhitespace() throws Exception {
+        assertEquals(LocalDateTime.of(2026, 9, 11, 8, 0, 0), parse("\" 2026-09-11 08:00:00 \""));
+    }
+
+    @Test
     public void testNullAndEmpty() throws Exception {
         assertNull(parse("null"));
         assertNull(parse("\"\""));
+        assertNull(parse("\"   \""));
+    }
+
+    @Test
+    public void testHutoolDateFormats() throws Exception {
+        assertEquals(LocalDateTime.of(2026, 9, 11, 0, 0), parse("\"2026/09/11\""));
+        assertEquals(LocalDateTime.of(2026, 9, 11, 8, 0), parse("\"2026/09/11 08:00:00\""));
     }
     @Test
     public void testInvalidThrows() {
         // 非法输入抛清晰错误，不再静默落 1970
         assertThrows(Exception.class, () -> parse("\"not-a-date\""));
-        assertThrows(Exception.class, () -> parse("\"2026/09/11\""));
+
     }
 }