Controller/service层方法粒度问题咨询

Question

以下哪种方式比较好？
条件：必须验证是否有权限执行，才能调用执行任务
方式1：

public class xxxController {
    @Autowried
    TaskService taskService;
    
    @RequestMapping("/executeTask")
    public void 执行任务() {
        taskService.验证是否有权限执行();
        taskService.执行任务();
    }

}

方式2：

public class xxxController {
    @Autowried
    TaskService taskService;
    
    @RequestMapping("/executeTask")
    public void 执行任务() {
        taskService.执行任务();
    }
}

public class TaskService {
        private void 验证是否有权限执行() {
        // code
    }
    public void执行任务() {
                验证是否有权限执行()
        // 执行任务code
    }
}

个人倾向于第2种。因为既然验证是必须的，那干嘛要分开，都写在一个service方法里方便
但是又怕一个方法职责过重
麻烦各位道友们指教

Answer 1

个人认为第一种可读性更好，各个方法所负责的功能更清楚😅