Jenkins流水线Groovy项目Enum用法优化及现有代码正确性咨询

一、现有代码写法是否正确？ #

现有代码可以正常运行，但写法不规范，存在冗余问题：

• 枚举方法定义为static静态方法，却用枚举实例去调用，不符合静态方法的调用规范，可读性差
• 重复使用switch分支做判断，后续新增枚举类型需要同时修改两个方法的分支，漏改就会返回错误的默认值
• 静态方法本身需要额外传入枚举实例做判断，完全没有利用到枚举可以绑定实例属性的特性

二、如何避免在test方法中传递fileExtension参数？ #

你可以直接把需要的文件路径、扩展名作为属性绑定到每个枚举实例上，不需要写静态方法和switch判断，优化后不仅不需要额外传参，代码也更简洁易维护。

优化后的枚举代码： #

enum FileExtension {
    // 每个枚举实例初始化时直接绑定对应属性
    CSV("./master.csv", "csv"),
    JSON("./master.json", "json"),
    PROPERTIES("./master.properties", "properties")

    // 定义不可变的实例属性
    final String srcFile
    final String fileFormat

    // 枚举构造方法，实例化时自动赋值
    FileExtension(String srcFile, String fileFormat) {
        this.srcFile = srcFile
        this.fileFormat = fileFormat
    }
}

优化后的调用代码： #

public class Hello {
    public static void main(String[] ars) {
        FileExtension jsonType = FileExtension.JSON
        // 直接通过实例获取对应属性，不需要传参调用方法
        println(jsonType.srcFile) // 输出 ./master.json
        println(jsonType.fileFormat) // 输出 json
        println(test(FileExtension.CSV)) // 输出 ./master.csv
    }

    public static String test(FileExtension fileExtension){
        // 不需要向方法内再传额外参数，直接返回实例自带的属性即可
        return fileExtension.srcFile
    }
}

这种写法完全满足你避免硬编码、代码复用的需求，后续新增文件类型只需要在枚举定义里加一行对应关系即可，不需要修改任何逻辑代码，也不会出现漏改switch分支的问题。

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