Activiti中检查流程变量是否为空时出现编译错误求助

问题描述 #

我在维护流程任务累计总数时，需要检查一个名为totaltasks的Integer类型本地流程变量是否为空，于是写了一段Groovy脚本做空值检查和累加逻辑，但控制台抛出了编译错误。

我的脚本代码：

//If process variable has never been initialised
if( execution.getVariable("totaltasks") === null && execution.getVariable("totaltasks") === 'undefined' ) {
    //set default value to 1 and initialise it
    int totalTasks = 1;
    execution.setVariable("totaltasks", totalTasks);
} else {
    //otherwise, just increment the current value
    int totalTasks = (int)execution.getVariable("totaltasks");
    totalTasks += 1;
    execution.setVariable("totaltasks", totalTasks);
}

控制台抛出的错误：

org.springframework.web.util.NestedServletException: Request processing failed; nested exception is org.activiti.engine.ActivitiException: problem evaluating script: org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed: General error during class generation: null. Script6.groovy

问题分析与修复 #

你的脚本有几个关键问题导致了编译和运行错误，咱们一步步拆解：

1. 逻辑错误的空值判断 #

• 你用了&&（逻辑与）判断变量同时为null和'undefined'，这根本不可能发生——一个变量不可能同时满足这两个条件。
• Groovy里没有'undefined'这个概念，未初始化的流程变量通过execution.getVariable()获取时返回的就是null，完全不需要判断字符串'undefined'。
• Groovy中判断值相等用==就足够，===是用来判断对象引用完全相同的，这里完全没必要用它。

2. 不安全的类型强制转换 #

即使你的空值判断逻辑正确，直接用(int)强制转换null也会抛出空指针异常，这是潜在的运行时风险。

3. 冗余的Groovy语法 #

Groovy支持类型推断，不需要显式声明int类型，而且有更简洁的语法处理空值和累加逻辑。

修正后的脚本（清晰版） #

// 获取流程变量，未初始化则返回null
def totalTasks = execution.getVariable("totaltasks")

// 空值处理：null则初始化为1，否则累加1
totalTasks = totalTasks == null ? 1 : totalTasks + 1

// 更新流程变量，确保类型为Integer
execution.setVariable("totaltasks", totalTasks as Integer)

更简洁的Groovy风格写法 #

利用Groovy的Elvis运算符?:（空值快捷处理），一行搞定初始化和累加：

// 空值时用0兜底，加1后刚好初始化为1；已有值则直接累加
def totalTasks = (execution.getVariable("totaltasks") ?: 0) + 1
execution.setVariable("totaltasks", totalTasks as Integer)

补充说明 #

• 始终记住：execution.getVariable()在变量未定义时返回null，不是字符串'undefined'，别被其他语言的习惯带偏了。
• as Integer用来确保变量类型和你定义的totaltasks（Integer类型）匹配，避免类型不兼容的问题。
• Groovy的语法特性可以帮你简化很多冗余代码，写脚本时尽量利用这些特性，减少出错概率。

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