如何重构调用同一服务方法但需不同常量的控制器方法？

你提出的传参判断方式是可行的，但可以从类型安全、扩展性、代码健壮性这几个维度做优化，具体如下：

1. 先修复现有实现的潜在问题 #

你当前代码里用param == "A"做字符串比较，在Java中这是比较对象引用而非字符串内容，容易出现逻辑错误，应该替换为"A".equals(param)（怕空指针的话可以用Objects.equals(param, "A")）。

2. 更优的重构方案 #

用枚举替代字符串参数 #

把"A"、"B"这类字符串常量换成枚举，既能避免拼写错误，又能获得编译期类型检查的保障：

// 定义枚举类
public enum ProjectType {
    A, B
}

控制器方法修改为：

public A() {
    return service.getTasks(ProjectType.A);
}

public B() {
    return service.getTasks(ProjectType.B);
}

用映射替代if-else判断 #

如果后续可能新增更多项目类型，不断叠加if-else会让代码臃肿难维护。可以用Map提前定义枚举类型和对应常量的映射关系，让逻辑更简洁：

// 在服务类中初始化映射（假设常量是List<Task>类型）
private static final Map<ProjectType, List<Task>> PROJECT_TASK_MAP = Map.of(
    ProjectType.A, constants.ProjectA,
    ProjectType.B, constants.ProjectB
);

private async List<Task> getTasks(ProjectType projectType) {
    // 原有的通用业务逻辑...
    return PROJECT_TASK_MAP.get(projectType);
}

3. 方案优势 #

• 类型安全：编译期就能发现参数错误，避免运行时因字符串拼写问题导致的bug
• 扩展性强：新增项目时，只需在枚举和映射中添加条目，无需修改getTasks的核心逻辑
• 代码简洁：摆脱冗长的if-else分支，逻辑更清晰直观

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