Groovy定义双数组引发Jenkins流水线方法过大错误排查

我需要从Jenkins作业向Groovy函数传递两个参数imagelist1和imagelist2，并将它们分别读取为数组，代码实现如下：

def abc = {
  container('xyz') {

    String[] arrspt = imagelist1.split(' ')
    for(int i=0; i < arrspt.length; i++) {
      env.image_name = arrspt[i] 
      sh 'echo "${image_name}" >> images_to_scan.txt' 
    }
    
    String[] arrStr = imagelist2.split(' ')
    for(int i=0; i < arrStr.length; i++) {
       env.image_name2 =  arrStr[i]
       sh 'echo "${image_name2}" >> images_to_scan.txt'
    }
  }
}

单独传递一个参数到单个数组时可正常运行，但传递两个参数时出现异常，错误信息如下：

[Pipeline] Start of Pipeline
[Pipeline] End of Pipeline
org.jenkinsci.plugins.workflow.cps.CpsCompilationErrorsException:
startup failed:
General error during class generation: Method too large:
Pipeline.___cps___19086323
()Lcom/cloudbees/groovy/cps/impl/CpsFunction;
groovyjarjarasm.asm.MethodTooLargeException: Method too large:
Pipeline.___cps___19086323
()Lcom/cloudbees/groovy/cps/impl/CpsFunction;

请问我在Groovy中声明两个数组的方式是否存在错误？

你的数组声明方式没有错误，String[] arrspt = imagelist1.split(' ')和String[] arrStr = imagelist2.split(' ')的写法是合法的Groovy代码。触发错误的根源是Jenkins的CPS编译器特性：

Jenkins Pipeline的CPS会把整个脚本编译成单个方法，当imagelist1或imagelist2包含大量元素时，循环里的每一次sh调用都会被转换成CPS节点，导致生成的方法字节码超过JVM单个方法的最大限制（65535字节），从而抛出MethodTooLargeException。

可以通过以下优化解决问题：

1. 提取重复逻辑为独立函数：把写入文件的逻辑抽成单独函数，减少主方法的代码量
2. 批量写入文件：避免循环中多次调用sh，一次性把所有镜像名写入文件，大幅减少CPS转换的节点数量
3. 避免使用env传递临时值：直接在sh中使用变量，减少不必要的环境变量操作

优化后的代码示例：

// 提取独立函数处理镜像列表写入
def writeImagesToFile(String imageList) {
    def images = imageList.split(' ')
    // 将所有镜像名拼接为换行分隔的字符串，一次性写入
    def imageContent = images.join('\n')
    sh """
        echo '${imageContent}' >> images_to_scan.txt
    """.trim()
}

def abc = {
    container('xyz') {
        writeImagesToFile(imagelist1)
        writeImagesToFile(imagelist2)
    }
}

如果镜像名中包含特殊字符（如引号、空格），可以改用更安全的方式传递变量：

def writeImagesToFile(String imageList) {
    def images = imageList.split(' ')
    // 使用Groovy的字符串插值和sh的参数传递避免转义问题
    sh script: "echo '${images.join('\n')}' >> images_to_scan.txt", returnStdout: false
}

这种优化方式会大幅减少生成的字节码数量，避免触发方法过大的错误，同时代码更简洁易维护。

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