diff --git a/src/main/java/com/genersoft/iot/vmp/service/impl/AlarmServiceImpl.java b/src/main/java/com/genersoft/iot/vmp/service/impl/AlarmServiceImpl.java
index 40b6b177c..80f1dad9f 100644
--- a/src/main/java/com/genersoft/iot/vmp/service/impl/AlarmServiceImpl.java
+++ b/src/main/java/com/genersoft/iot/vmp/service/impl/AlarmServiceImpl.java
@@ -31,7 +31,6 @@ import org.springframework.stereotype.Service;
 import java.io.File;
 import java.util.ArrayList;
 import java.util.List;
-import java.util.UUID;
 import java.util.concurrent.ConcurrentLinkedQueue;
 
 @Slf4j
diff --git a/src/main/java/com/genersoft/iot/vmp/storager/dao/AlarmMapper.java b/src/main/java/com/genersoft/iot/vmp/storager/dao/AlarmMapper.java
index 23e183bf0..5efc4152c 100644
--- a/src/main/java/com/genersoft/iot/vmp/storager/dao/AlarmMapper.java
+++ b/src/main/java/com/genersoft/iot/vmp/storager/dao/AlarmMapper.java
@@ -43,11 +43,11 @@ public interface AlarmMapper {
     @Insert("<script>" +
             "INSERT INTO wvp_alarm (channel_id, description, snap_path, record_path, longitude, latitude, alarm_type, alarm_time)" +
             " VALUES " +
-            "<foreach collection='handlerCatchDataList' item='item' open='(' separator=',' close=')'>" +
-            "#{item.channelId}, #{item.description}, #{item.snapPath}, #{item.recordPath}, #{item.longitude}, #{item.latitude}, #{item.alarmType}, #{item.alarmTime}" +
+            "<foreach collection='handlerCatchDataList' item='item' separator=','>" +
+            "(#{item.channelId}, #{item.description}, #{item.snapPath}, #{item.recordPath}, #{item.longitude}, #{item.latitude}, #{item.alarmType}, #{item.alarmTime})" +
             "</foreach>" +
             "</script>")
-    void insertAlarms(List<Alarm> handlerCatchDataList);
+    void insertAlarms(@Param("handlerCatchDataList") List<Alarm> handlerCatchDataList);
 
     @Select("SELECT snap_path FROM wvp_alarm WHERE id = #{id}")
     String getSnapPathById(@Param("id") Long id);