这应该很容易,但是显然我缺乏基本知识。我有一项服务,可以在BugTracking系统中创建票证。创建插件非常简单。有一个界面
public interface BugTracker {
boolean createAndSendIssue(Issue issue);
boolean updateAndSendIssue(Issue issue);
boolean closeIssue(Issue issue);
}
和JIRA的API客户端:
@Component
public class JiraClient implements BugTracker {
@Override
public boolean createAndSendIssue(Issue issue) {
//Logic for using JIRA REST API
return false;
}
@Override
public boolean updateAndSendIssue(Issue issue) {
//Logic for using JIRA REST API
return false;
}
@Override
public boolean closeIssue(Issue issue) {
//Logic for using JIRA REST API
return false;
}
}
Bugzilla的API CLient:
@Component
public class BugzillaClient implements BugTracker {
@Override
public boolean createAndSendIssue(Issue issue) {
//Logic for using BUGZILLA REST API
return false;
}
@Override
public boolean updateAndSendIssue(Issue issue) {
//Logic for using BUGZILLA REST API
return false;
}
@Override
public boolean closeIssue(Issue issue) {
//Logic for using BUGZILLA REST API
return false;
}
}
问题Service正在处理中MVC。它看起来像:
@Service
public class BugTrackerService {
private final BugzillaClient bugzillaClient;
private final JiraClient jiraClient;
private static final String JIRA_TYPE = "Jira";
private static final String BUGZILLA_TYPE = "Bugzilla";
@Autowired
BugTrackerService(BugzillaClient bugzillaClient, JiraClient jiraClient) {
this.bugzillaClient = bugzillaClient;
this.jiraClient = jiraClient;
}
boolean processIssueTicketRequest(Issue issue){
if ( issue.getBugTrackerType().bugTrackingSystemType.name.equals(JIRA_TYPE)) {
return jiraClient.createAndSendIssue(issue);
} else if (issue.getBugTrackerType().bugTrackingSystemType.name.equals(BUGZILLA_TYPE)){
return bugzillaClient.createAndSendIssue(issue);
} else {
return false;
}
}
}
是否可以在BugTrackerService.processIssueTicketRequest没有自动连接所有客户端的情况下创建方法?目前,这不是什么大问题,但是在与越来越多的系统集成之后,该服务将变得复杂。
java spring 重构
将类型转换为,enum然后BugTracker决定是否可以处理。然后注入所有BugTracker实例的集合,遍历它们,如果它支持类型,则调用适当的方法。
public enum SystemType { JIRA, BUGZILLA }
然后添加一个方法,例如supports(Issue issue),你的BugTracker。
boolean supports(Issue issue)
;```
在实现中检查它是否可以处理,例如,JiraClient执行类似的操作。
```js
public boolean supports(Issue issue) {
return JIRA == issue.getBugTrackerType();
}
然后在BugTrackerService所有BugTracker实例的列表中,进行迭代并调用适当的实例。
@Service
public class BugTrackerService {
private final List<BugTracker> bugTrackers;
BugTrackerService(List<BugTracker> bugTrackers) {
this.bugTrackers=bugTrackers;
}
boolean processIssueTicketRequest(Issue issue){
for (BugTracker bugTracker : this.bugTrackers) {
if (bugTracker.supports(issue) ) {
bugTracker.createAndSendIssue(issue);
return true;
}
}
return false;
}
}
现在,您可以随心所欲地灵活工作,并且可以支持任意数量的支持。
版权声明:本文内容由阿里云实名注册用户自发贡献,版权归原作者所有,阿里云开发者社区不拥有其著作权,亦不承担相应法律责任。具体规则请查看《阿里云开发者社区用户服务协议》和《阿里云开发者社区知识产权保护指引》。如果您发现本社区中有涉嫌抄袭的内容,填写侵权投诉表单进行举报,一经查实,本社区将立刻删除涉嫌侵权内容。