Spring Boot中count(JobViewWrapper)方法重定义冲突的解决方案咨询

你碰到的这个问题其实是Java方法重载规则导致的——Java判断方法是否重复只看方法名+参数列表，哪怕两个方法的@PostMapping路径不一样，只要方法名和参数完全一致，编译器就会抛出method count(JobViewWrapper) is already defined的错误。

想要保留两个接口的功能，有几种简单可行的解决办法，最推荐的是重命名方法，另外也可以通过调整参数列表实现重载，下面具体说明：

方法一：重命名方法（最推荐） #

直接给两个方法起不同的、符合业务语义的名字，这样既解决编译问题，还能提升代码可读性，别人一看方法名就知道是做什么的。

修改后的控制器代码：

@PostMapping(value = "/count")
public long countDeletedJobs(@RequestBody(required = false) JobViewWrapper wrapper) {
    System.out.println("into controller count-deleted-jobs");
    if (wrapper == null) {
        wrapper = new JobViewWrapper();
    }
    System.out.println("Prepare to count deleted jobs service");
    return JobService.countLazyView();
}

@PostMapping(value = "/count-active")
public long countActiveDeletedJobs(@RequestBody(required = false) JobViewWrapper wrapper) {
    System.out.println("into controller count-active-deleted-jobs");
    if (wrapper == null) {
        wrapper = new JobViewWrapper();
    }
    System.out.println("Prepare to count active deleted jobs service");
    return JobService.countLazyViewIsActive();
}

这里我把方法名改成了countDeletedJobs和countActiveDeletedJobs，对应各自的业务逻辑，你也可以根据自己的习惯起更简洁的名字，比如count和countActive，只要两个名字不一样就行。

方法二：调整参数列表实现重载（不推荐） #

如果实在不想改方法名，可以给其中一个方法添加一个无实际业务意义的参数（比如一个可选的@RequestParam或者@HeaderParam），让两个方法的参数列表不一样，从而满足重载规则。

示例代码：

@PostMapping(value = "/count")
public long count(@RequestBody(required = false) JobViewWrapper wrapper) {
    System.out.println("into controller count");
    if (wrapper == null) {
        wrapper = new JobViewWrapper();
    }
    System.out.println("Prepare to count service");
    return JobService.countLazyView();
}

@PostMapping(value = "/count-active")
public long count(@RequestBody(required = false) JobViewWrapper wrapper, 
                  @RequestParam(required = false, defaultValue = "true") Boolean isActive) {
    System.out.println("into controller count-active");
    if (wrapper == null) {
        wrapper = new JobViewWrapper();
    }
    System.out.println("Prepare to count active service");
    return JobService.countLazyViewIsActive();
}

不过这种方式不推荐，因为添加的参数没有实际业务作用，只是为了绕过编译错误，会让代码变得不直观，增加维护成本。

总结 #

优先选择方法一重命名，这是最简洁、最符合代码规范的解决方案，完全能保留你两个接口的所有功能，还能让代码更易读。

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