ColdFusion数组计数器性能优化及空元素删除报错问题

一、正确缩减数组的方法 #

你之前正向循环删除元素时，删除操作会导致数组长度实时缩短、后续元素索引前移，循环到原数组长度时就会访问不存在的元素，引发索引越界报错。正确的做法是反向遍历数组，从最后一个元素往前处理：

<cfloop index="i" from="#arrayLen(myarray)#" downTo="1">
    <cfif myarray[i][2] eq 0> 
        <cfset arrayDeleteAt(myarray, i)>
    </cfif> 
</cfloop>

反向遍历时，删除当前元素不会影响未遍历的前面元素索引，能顺利剔除统计值为0的元素。

二、更高效的计数器实现方案 #

原二维数组初始化8000个元素的方式冗余度极高，结合你仅约400个元素有统计值的场景，推荐用**结构体（Struct）**替代二维数组，同时优化数据库查询减少交互次数：

1. 用结构体替代数组，仅存储有数据的项 #

结构体无需预初始化所有可能的adigit，只在遇到对应adigit时新增或累加计数，内存占用和后续排序开销都会大幅降低：

<!--- 初始化空结构体作为计数器 --->
<cfset countStruct = {}>

<!--- 优化数据库查询：用JOIN替代循环内嵌套查询，减少数据库访问次数 --->
<cfquery datasource="xxx" name="countQuery">
    SELECT m2.adigit
    FROM mydatabase m1
    JOIN mydatabase m2 ON m1.pattern1 = m2.pattern2 
        AND m2.sequencenum < m1.sequencenum
    WHERE m1.id IN (SELECT id FROM mydatabase ORDER BY sequencenum DESC LIMIT 10)
</cfquery>

<!--- 累加计数 --->
<cfoutput query="countQuery">
    <cfif structKeyExists(countStruct, adigit)>
        <cfset countStruct[adigit] = countStruct[adigit] + 1>
    <cfelse>
        <cfset countStruct[adigit] = 1>
    </cfif>
</cfoutput>

2. 将结构体转为数组并高效排序 #

如果需要排序展示，把结构体键值对转为数组后，用ColdFusion内置的arraySort()函数排序，性能远高于自定义双重循环：

<!--- 将结构体转为二维数组 --->
<cfset resultArray = []>
<cfloop collection="#countStruct#" item="adigit">
    <cfset arrayAppend(resultArray, [adigit, countStruct[adigit]])>
</cfloop>

<!--- 按统计值降序排序 --->
<cfset arraySort(resultArray, function(a, b) {
    return b[2] - a[2];
})>

3. 核心优化点 #

• 数据库层面用JOIN替代循环内嵌套查询，减少数据库连接开销，大幅提升查询效率；
• 抛弃手动双重循环排序，内置arraySort()是优化过的排序算法，性能更优；
• 结构体仅存储有实际数据的项，无需预初始化大量空元素，节省内存和初始化时间。

内容的提问来源于stack exchange，提问作者Jack