From 51a551a7f8eae7654f973d45cd2fa2952774673d Mon Sep 17 00:00:00 2001
From: linwei <872216696@qq.com>
Date: Fri, 07 Aug 2026 10:56:55 +0800
Subject: [PATCH] perf(task): 优化巡查任务审核性能并添加详细日志记录
---
drone-service/drone-fw/src/main/java/org/sxkj/fw/area/mapper/FwAreaDivideMapper.xml | 311 ++++++++++++++++++++++++---------------------------
1 files changed, 149 insertions(+), 162 deletions(-)
diff --git a/drone-service/drone-fw/src/main/java/org/sxkj/fw/area/mapper/FwAreaDivideMapper.xml b/drone-service/drone-fw/src/main/java/org/sxkj/fw/area/mapper/FwAreaDivideMapper.xml
index 979404b..47c700f 100644
--- a/drone-service/drone-fw/src/main/java/org/sxkj/fw/area/mapper/FwAreaDivideMapper.xml
+++ b/drone-service/drone-fw/src/main/java/org/sxkj/fw/area/mapper/FwAreaDivideMapper.xml
@@ -79,7 +79,7 @@
<if test="param2.areaTypeKeyList != null and param2.areaTypeKeyList.size > 0">
and (
<foreach collection="param2.areaTypeKeyList" item="key" separator=" or ">
- ARRAY_POSITION(string_to_array(ad.area_type_keys, ','), #{key}) IS NOT NULL
+ ARRAY_POSITION(string_to_array(ad.area_type_keys, ','), #{key}::text) IS NOT NULL
</foreach>
)
</if>
@@ -169,7 +169,7 @@
<if test="areaTypeKeyList != null and areaTypeKeyList.size > 0">
and (
<foreach collection="areaTypeKeyList" item="key" separator=" or ">
- ARRAY_POSITION(string_to_array(ad.area_type_keys, ','), #{key}) IS NOT NULL
+ ARRAY_POSITION(string_to_array(ad.area_type_keys, ','), #{key}::text) IS NOT NULL
</foreach>
)
</if>
@@ -241,115 +241,99 @@
<select id="selectFwAreaDivideStatisticsPage" resultMap="fwAreaDivideStatisticsResultMap">
select
- stats.id,
- stats.area_name,
- stats.area_type_keys,
- stats.scene_name,
- stats.scene_type,
- stats.device_count,
- stats.alarm_count,
- stats.counter_count,
- stats.counter_success_count,
- case
- when stats.counter_count = 0 then '/'
- when stats.counter_success_count * 100.0 / stats.counter_count >= 80 then '好'
- when stats.counter_success_count * 100.0 / stats.counter_count >= 60 then '一般'
- else '差'
- end as control_effect
+ stats.id,
+ stats.area_name,
+ stats.area_type_keys,
+ stats.scene_name,
+ stats.scene_type,
+ stats.device_count,
+ stats.alarm_count,
+ stats.counter_count,
+ stats.counter_success_count,
+ case
+ when stats.counter_count = 0 then '/'
+ when stats.counter_success_count * 100.0 / stats.counter_count >= 80 then '好'
+ when stats.counter_success_count * 100.0 / stats.counter_count >= 60 then '一般'
+ else '差'
+ end as control_effect
from (
- select
- ad.id,
- ad.area_name,
- ad.area_type_keys,
- (
- select string_agg(distinct ds.scene_name, ',')
- from ja_fw_defense_scene ds
- join ja_fw_defense_scene_manage dsm on dsm.defense_scene_id::VARCHAR = ds.id::VARCHAR and dsm.is_deleted = 0
- where ds.is_deleted = 0
- AND ad.id::VARCHAR = ANY(string_to_array(ds.area_divide_ids, ','))) AS scene_name,
- (
- select string_agg(distinct ds.scene_type, ',')
- from ja_fw_defense_scene ds
- join ja_fw_defense_scene_manage dsm on dsm.defense_scene_id::VARCHAR = ds.id::VARCHAR and dsm.is_deleted = 0
- where ds.is_deleted = 0
- AND ad.id::VARCHAR = ANY(string_to_array(ds.area_divide_ids, ','))) AS scene_name,
- case
- when ad.device_ids is null or trim(ad.device_ids) = '' then 0
- else length(replace(ad.device_ids, ' ', '')) - length(replace(replace(ad.device_ids, ' ', ''), ',', '')) + 1
- end as device_count,
- case
- when ad.device_ids is null or trim(ad.device_ids) = '' then 0
- else (
- select count(1)
- from ja_fw_drone_alarm_record ar
- join ja_fw_device d on d.id::VARCHAR = ar.device_id::VARCHAR and d.is_deleted = 0
- where ar.is_deleted = 0
- and ad.id::VARCHAR = ANY(string_to_array(ad.device_ids, ','))
- )
- end as alarm_count,
- case
- when ad.device_ids is null or trim(ad.device_ids) = '' then 0
- else (
- select count(1)
- from ja_fw_effect_eval ee
- join ja_fw_device d on d.id = ee.device_id::VARCHAR and d.is_deleted = 0
- where ee.is_deleted = 0
- and ad.id::VARCHAR = ANY(string_to_array(ad.device_ids, ','))
- )
- end as counter_count,
- case
- when ad.device_ids is null or trim(ad.device_ids) = '' then 0
- else (
- select count(1)
- from ja_fw_effect_eval ee
- join ja_fw_device d on d.id = ee.device_id::VARCHAR and d.is_deleted = 0
- where ee.is_deleted = 0
- and ee.counter_effect = '1'
- and ad.id::VARCHAR = ANY(string_to_array(ad.device_ids, ','))
- )
- end as counter_success_count
- from ja_fw_area_divide ad
- <where>
- ad.is_deleted = 0
+ select
+ ad.id,
+ ad.area_name,
+ ad.area_type_keys,
+ (SELECT string_agg(DISTINCT ds.scene_name, ',')
+ FROM ja_fw_defense_scene ds
+ JOIN ja_fw_defense_scene_manage dsm ON dsm.defense_scene_id::text = ds.id::text AND dsm.is_deleted = 0
+ WHERE ds.is_deleted = 0
+ AND ad.id::text = ANY(string_to_array(ds.area_divide_ids, ','))) AS scene_name,
+
+ (SELECT string_agg(DISTINCT ds.scene_type, ',')
+ FROM ja_fw_defense_scene ds
+ JOIN ja_fw_defense_scene_manage dsm ON dsm.defense_scene_id::text = ds.id::text AND dsm.is_deleted = 0
+ WHERE ds.is_deleted = 0
+ AND ad.id::text = ANY(string_to_array(ds.area_divide_ids, ','))) AS scene_type,
+
+ CASE WHEN ad.device_ids IS NULL OR trim(ad.device_ids) = '' THEN 0
+ ELSE length(replace(ad.device_ids, ' ', '')) - length(replace(replace(ad.device_ids, ' ', ''), ',', '')) + 1 END AS device_count,
+
+ CASE WHEN ad.device_ids IS NULL OR trim(ad.device_ids) = '' THEN 0
+ ELSE (SELECT count(1) FROM ja_fw_drone_alarm_record ar
+ JOIN ja_fw_device d ON d.id = ar.device_id AND d.is_deleted = 0
+ WHERE ar.is_deleted = 0
+ AND d.id = ANY(string_to_array(replace(ad.device_ids, ' ', ''), ',')::bigint[])) END AS alarm_count,
+
+ CASE WHEN ad.device_ids IS NULL OR trim(ad.device_ids) = '' THEN 0
+ ELSE (SELECT count(1) FROM ja_fw_effect_eval ee
+ JOIN ja_fw_device d ON d.id = ee.device_id AND d.is_deleted = 0
+ WHERE ee.is_deleted = 0
+ AND d.id = ANY(string_to_array(replace(ad.device_ids, ' ', ''), ',')::bigint[])) END AS counter_count,
+
+ CASE WHEN ad.device_ids IS NULL OR trim(ad.device_ids) = '' THEN 0
+ ELSE (SELECT count(1) FROM ja_fw_effect_eval ee
+ JOIN ja_fw_device d ON d.id = ee.device_id AND d.is_deleted = 0
+ WHERE ee.is_deleted = 0 AND ee.counter_effect = '1'
+ AND d.id = ANY(string_to_array(replace(ad.device_ids, ' ', ''), ',')::bigint[])) END AS counter_success_count
+ from ja_fw_area_divide ad
+ <where>
+ ad.is_deleted = 0
+ and exists (
+ SELECT 1 FROM ja_fw_defense_scene ds
+ JOIN ja_fw_defense_scene_manage dsm ON dsm.defense_scene_id::text = ds.id::text AND dsm.is_deleted = 0
+ WHERE ds.is_deleted = 0
+ AND ad.id::text = ANY(string_to_array(ds.area_divide_ids, ','))
+ )
+ <if test="param2.id != null and param2.id != ''">
+ and ad.id = #{param2.id}::bigint
+ </if>
+ <if test="param2.areaName != null and param2.areaName != ''">
+ and ad.area_name like concat('%', #{param2.areaName}, '%')
+ </if>
+ <if test="param2.sceneType != null and param2.sceneType != ''">
and exists (
- select 1
- from ja_fw_defense_scene ds
- join ja_fw_defense_scene_manage dsm on dsm.defense_scene_id::VARCHAR = ds.id::VARCHAR and dsm.is_deleted = 0
- where ds.is_deleted = 0
- and ad.id::VARCHAR = ANY(string_to_array(ad.device_ids, ','))
+ select 1
+ from ja_fw_defense_scene ds
+ where ds.is_deleted = 0
+ and ad.id::text = ANY(string_to_array(ds.area_divide_ids, ','))
+ and ds.scene_type = #{param2.sceneType}
)
- <if test="param2.id != null and param2.id != ''">
- and ad.id = #{param2.id}
- </if>
- <if test="param2.areaName != null and param2.areaName != ''">
- and ad.area_name like concat('%', #{param2.areaName}, '%')
- </if>
- <if test="param2.sceneType != null and param2.sceneType != ''">
- and exists (
- select 1
- from ja_fw_defense_scene ds
- where ds.is_deleted = 0
- and ad.id::VARCHAR = ANY(string_to_array(ad.device_ids, ','))
- and ds.scene_type = #{param2.sceneType}
- )
- </if>
- <if test="param2.areaCode != null and param2.areaCode != ''">
- and ad.area_code = #{param2.areaCode}
- </if>
- <if test="param2.areaTypeKeyList != null and param2.areaTypeKeyList.size > 0">
- and (
- <foreach collection="param2.areaTypeKeyList" item="key" separator=" or ">
- ARRAY_POSITION(string_to_array(ad.area_type_keys, ','), #{key}) IS NOT NULL
- </foreach>
- )
- </if>
- <if test="param2.deptList != null and param2.deptList.size > 0">
- and ad.create_dept in
- <foreach collection="param2.deptList" item="deptId" open="(" separator="," close=")">
- #{deptId}
- </foreach>
- </if>
- </where>
+ </if>
+ <if test="param2.areaCode != null and param2.areaCode != ''">
+ and ad.area_code = #{param2.areaCode}
+ </if>
+ <if test="param2.areaTypeKeyList != null and param2.areaTypeKeyList.size > 0">
+ and (
+ <foreach collection="param2.areaTypeKeyList" item="key" separator=" or ">
+ array_position(string_to_array(ad.area_type_keys, ','), #{key}::text) is not null
+ </foreach>
+ )
+ </if>
+ <if test="param2.deptList != null and param2.deptList.size > 0">
+ and ad.create_dept in
+ <foreach collection="param2.deptList" item="deptId" open="(" separator="," close=")">
+ #{deptId}::bigint
+ </foreach>
+ </if>
+ </where>
) stats
</select>
@@ -467,73 +451,76 @@
from ja_fw_area_divide ad
left join ja_fw_police_station ps on ps.id::VARCHAR = ad.police_station_id::VARCHAR and ps.is_deleted = 0
inner join (
- SELECT
- d.id
- FROM
- ja_fw_device d
- WHERE
- d.is_deleted = 0
- AND d.is_enabled = 1
- AND d.final_outbound_area IS NOT NULL
- <if test="regionCode != null and regionCode != ''">
- and ( d.final_outbound_area_code like concat(#{regionCode},'%')
- <if test="currentDeptId != null and currentDeptId != '' ">
- or d.id in (
- SELECT device_id FROM ja_fw_device_per_share
- WHERE loan_to_dept_id = #{currentDeptId}
- AND is_deleted = 0
- )
- </if>
- )
- </if>
- GROUP BY
- d.id
- ) dd on dd.id = ANY(string_to_array(replace(ad.device_ids, ' ', ''), ',')::bigint[])
+ SELECT
+ d.id
+ FROM
+ ja_fw_device d
+ WHERE
+ d.is_deleted = 0
+ AND d.is_enabled = 1
+ AND d.final_outbound_area IS NOT NULL
+ <if test="regionCode != null and regionCode != ''">
+ and ( d.final_outbound_area_code like concat(#{regionCode},'%')
+ <if test="currentDeptId != null and currentDeptId != '' ">
+ or d.id in (
+ SELECT device_id FROM ja_fw_device_per_share
+ /* 修改点1:参数强制转为 bigint 匹配数据库字段 */
+ WHERE loan_to_dept_id = #{currentDeptId}::bigint
+ AND is_deleted = 0
+ )
+ </if>
+ )
+ </if>
+ GROUP BY
+ d.id
+ /* 修改点2:dd.id 是 bigint,转 VARCHAR 匹配文本数组,去掉不稳定的 ::bigint[] */
+ ) dd on dd.id::VARCHAR = ANY(string_to_array(replace(ad.device_ids, ' ', ''), ','))
<where>
ad.is_deleted = 0
<if test="isSetSceneManage != null and isSetSceneManage == 1">
and exists (
select 1
- from ja_fw_defense_scene ds
- join ja_fw_defense_scene_manage dsm on dsm.defense_scene_id::VARCHAR = ds.id::VARCHAR and dsm.is_deleted = 0
- where ds.is_deleted = 0
- and ds.area_divide_ids is not null
- and ds.area_divide_ids != ''
- and ad.id = ANY(string_to_array(replace(ds.area_divide_ids, ' ', ''), ',')::bigint[])
+ from ja_fw_defense_scene ds_sub
+ join ja_fw_defense_scene_manage dsm_sub on dsm_sub.defense_scene_id::VARCHAR = ds_sub.id::VARCHAR and dsm_sub.is_deleted = 0
+ where ds_sub.is_deleted = 0
+ and ds_sub.area_divide_ids is not null
+ and ds_sub.area_divide_ids != ''
+ /* 修改点3:ad.id 转 VARCHAR 匹配 ANY */
+ and ad.id::VARCHAR = ANY(string_to_array(replace(ds_sub.area_divide_ids, ' ', ''), ','))
)
</if>
<if test="isSetSceneManage != null and isSetSceneManage == 1 and flyTime != null">
and exists (
select 1
- from ja_fw_defense_scene ds
- join ja_fw_defense_scene_manage dsm on dsm.defense_scene_id::VARCHAR = ds.id::VARCHAR and dsm.is_deleted = 0
- where ds.is_deleted = 0
- and ds.area_divide_ids is not null
- and ds.area_divide_ids != ''
- and ad.id = ANY(string_to_array(replace(ds.area_divide_ids, ' ', ''), ',')::bigint[])
- and (dsm.effective_date_start is null or dsm.effective_date_start <= #{flyTime})
- and (dsm.effective_date_end is null or dsm.effective_date_end >= #{flyTime})
+ from ja_fw_defense_scene ds_sub2
+ join ja_fw_defense_scene_manage dsm_sub2 on dsm_sub2.defense_scene_id::VARCHAR = ds_sub2.id::VARCHAR and dsm_sub2.is_deleted = 0
+ where ds_sub2.is_deleted = 0
+ and ds_sub2.area_divide_ids is not null
+ and ds_sub2.area_divide_ids != ''
+ and ad.id::VARCHAR = ANY(string_to_array(replace(ds_sub2.area_divide_ids, ' ', ''), ','))
+ and (dsm_sub2.effective_date_start is null or dsm_sub2.effective_date_start <= #{flyTime})
+ and (dsm_sub2.effective_date_end is null or dsm_sub2.effective_date_end >= #{flyTime})
)
</if>
<if test="filterSelected != null and filterSelected == 1">
and (
not exists (
select 1
- from ja_fw_defense_scene ds
- where ds.is_deleted = 0
- and ds.area_divide_ids is not null
- and ds.area_divide_ids != ''
- and ad.id = ANY(string_to_array(replace(ds.area_divide_ids, ' ', ''), ',')::bigint[])
+ from ja_fw_defense_scene ds_sub3
+ where ds_sub3.is_deleted = 0
+ and ds_sub3.area_divide_ids is not null
+ and ds_sub3.area_divide_ids != ''
+ and ad.id::VARCHAR = ANY(string_to_array(replace(ds_sub3.area_divide_ids, ' ', ''), ','))
)
<if test="sceneId != null">
or exists (
select 1
- from ja_fw_defense_scene ds
- where ds.is_deleted = 0
- and ds.id = #{sceneId}
- and ds.area_divide_ids is not null
- and ds.area_divide_ids != ''
- and ad.id = ANY(string_to_array(replace(ds.area_divide_ids, ' ', ''), ',')::bigint[])
+ from ja_fw_defense_scene ds_sub4
+ where ds_sub4.is_deleted = 0
+ and ds_sub4.id = #{sceneId}::bigint
+ and ds_sub4.area_divide_ids is not null
+ and ds_sub4.area_divide_ids != ''
+ and ad.id::VARCHAR = ANY(string_to_array(replace(ds_sub4.area_divide_ids, ' ', ''), ','))
)
</if>
)
@@ -541,18 +528,18 @@
<if test="sceneId != null and (filterSelected == null or filterSelected != 1)">
and exists (
select 1
- from ja_fw_defense_scene ds
- where ds.is_deleted = 0
- and ds.id = #{sceneId}
- and ds.area_divide_ids is not null
- and ds.area_divide_ids != ''
- and ad.id = ANY(string_to_array(replace(ds.area_divide_ids, ' ', ''), ',')::bigint[])
+ from ja_fw_defense_scene ds_sub5
+ where ds_sub5.is_deleted = 0
+ and ds_sub5.id = #{sceneId}::bigint
+ and ds_sub5.area_divide_ids is not null
+ and ds_sub5.area_divide_ids != ''
+ and ad.id::VARCHAR = ANY(string_to_array(replace(ds_sub5.area_divide_ids, ' ', ''), ','))
)
</if>
<if test="areaTypeKeyList != null and areaTypeKeyList.size > 0">
and (
<foreach collection="areaTypeKeyList" item="key" separator=" or ">
- ARRAY_POSITION(string_to_array(ad.area_type_keys, ','), #{key}) IS NOT NULL
+ ARRAY_POSITION(string_to_array(ad.area_type_keys, ','), #{key}::VARCHAR) IS NOT NULL
</foreach>
)
</if>
--
Gitblit v1.9.3