From 00a62de5ee9ff4e3dc4c91cb225dcf14d745ab00 Mon Sep 17 00:00:00 2001
From: zihluwang <real@zihluwang.me>
Date: Thu, 20 Aug 2026 15:43:03 +0800
Subject: [PATCH] refactor: retrieving value with Optional

---
 .../domain/converter/ModificationStatusConverter.java       | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/main/java/com/onixbyte/deltaforceguide/domain/converter/ModificationStatusConverter.java b/src/main/java/com/onixbyte/deltaforceguide/domain/converter/ModificationStatusConverter.java
index 5e5bab5..9da5368 100644
--- a/src/main/java/com/onixbyte/deltaforceguide/domain/converter/ModificationStatusConverter.java
+++ b/src/main/java/com/onixbyte/deltaforceguide/domain/converter/ModificationStatusConverter.java
@@ -4,6 +4,8 @@ import com.onixbyte.deltaforceguide.enumeration.ModificationStatus;
 import jakarta.persistence.AttributeConverter;
 import jakarta.persistence.Converter;
 
+import java.util.Optional;
+
 /**
  * JPA attribute converter that maps {@link ModificationStatus} enum to/from its integer database representation.
  *
@@ -14,7 +16,9 @@ public class ModificationStatusConverter implements AttributeConverter<Modificat
 
     @Override
     public Integer convertToDatabaseColumn(ModificationStatus attribute) {
-        return attribute == null ? null : attribute.getCode();
+        return Optional.ofNullable(attribute)
+                .map(ModificationStatus::getCode)
+                .orElse(null);
     }
 
     @Override