Question

我是这个网站的新手，这是我的第一个问题。

我必须做一个网站，我使用的是java和Struts2，但我是Struts2的新手。

在我的网站中，我必须向Facebook提出请求并通过OAuth进行身份验证。

我正在执行操作页面的执行方法中的所有过程（对受保护资源进行身份验证和请求），这个过程非常复杂，并且在Facebook和我的网络之间有很多重定向。

前几天我读到这个“不要用多种方法创建动作：”执行“并且你要执行的操作（例如”createUser“）应该足够”（来自http://freeopenidea.blogspot.com.es/2010/04/struts2-best-practices.html）。

大部分代码可以在另一个时刻从我网站的另一部分调用，因为我第一次连接时会执行此过程，但我可以执行此操作（或类似的操作）来刷新联系人列表。

1 - 我应该为我需要的方法创建一个单独的类（不是动作），并从“execute”方法中调用它们吗？

2 - 我应该将代码保留在操作页面中，而不是“执行”之外的方法吗？每当我需要完成一些任务时，请调用此页面。

我不知道在哪里放置代码（我知道，我必须存储de accessToken。我只是粘贴代码来显示复杂性，但不要查看更正）。

public String execute() throws Exception{
   if (code!=null){
      Verifier verifier = new Verifier(code);
      //get the accessToken to do requests
      Token accessToken = service.getAccessToken(EMPTY_TOKEN, verifier);
      OAuthRequest requestList = new OAuthRequest(Verb.GET, PROTECTED_RESOURCE_URL);
      service.signRequest(accessToken, requestList);
      Response responseList = requestList.send();
      if (responseList.getCode() == 200){
        //I get the Friends List
        JsonParse parser = new JsonParse();
        JSONObject json = (JSONObject) JSONSerializer.toJSON(responseList.getBody());
        JSONArray datos = json.getJSONArray("data");
        for (Object o : datos) 
        {//for every friend of the list I do this
            JSONObject elem = (JSONObject) o;
            String id = elem.getString("id");                
            OAuthRequest requestFriend = new OAuthRequest(Verb.GET,"https://graph.facebook.com/"+id);
            service.signRequest(accessToken, requestFriend);
            Response responseFriend = requestFriend.send();
            if (responseFriend.getCode() == 200){
                JsonParse parserAux = new JsonParse();
                PerfilContacto pcBean = parserAux.parseFacebookElement(responseFriend.getBody());
                pcDAO.insertarContacto(pcBean); 
            }
        }
      }
      return SUCCESS; 
   }      
   else return ERROR;    
 }

Answer 1

IMO在动作方法中的代码太多了。操作应该处理Web和业务层之间的层，而不是其他。这种级别的耦合，特别是对于硬编码的类实例化，使得单独测试动作的逻辑非常困难。

将所有代码基本上移到服务中。该操作仅涉及服务交互。该框架验证了code的存在。该服务在Struts 2之外进行测试。该操作使用模拟服务进行测试。

完全未经测试，但我怀疑我的代码看起来更接近下面的内容。它将一种复杂性换成另一种，但带来了多种好处。每种方法都紧密集中且易于阅读。服务调用是隔离的，这允许我们测试不同的服务失败模式。它是系统行为和功能的精细表示。

<强>动作

public String execute() throws Exception {
    fbService.updateFriends(code);
    return SUCCESS;
}

FB服务

public void updateFriends(String code) {
    Token accessToken = getAccessToken(code);
    Response response = getFriends(accessToken);
    if (response.getCode() == 200) {
        processFriends(accessToken, response);
    }
}

private void processFriends(Token accessToken, Response response) {
    JSONObject json = (JSONObject) JSONSerializer.toJSON(response.getBody());
    JSONArray datos = json.getJSONArray("data");
    for (Object o : datos)  {
        JSONObject friend = (JSONObject) o;
        processFriend(friend);
    }
}

private void processFriend(Token accessToken, JSONObject friend) {
    Response response = getFriendGraph(accessToken, friend.getString("id"));
    if (response.getCode() == 200){
        PerfilContacto pcBean = new JsonParse().parseFacebookElement(response.getBody());
        pcDAO.insertarContacto(pcBean); 
    }
}

//~ Service interaction

private Response getFriends(Token accessToken) {
    return sendSignedGetRequest(PROTECTED_RESOURCE_URL, accessToken);
}

private Response getFriendGraph(Token accessToken, String id) {
    return sendSignedGetRequest("https://graph.facebook.com/" + id, accessToken);
}

private Token getAccessToken(String code) {
    return service.getAccessToken(EMPTY_TOKEN, new Verifier(code));
}

private Response sendSignedGetRequest(String url, Token accessToken) {
    OAuthRequest request = new OAuthRequest(Verb.GET, url);
    service.signRequest(accessToken, request);
    return request.send();
}

如果我们考虑指标，我们会得到以下结论。

原件：

Average Function NCSS:      24.00
Average Function CCN:        6.00
Program NCSS:               25.00

返工：

Average Function NCSS:       3.63
Average Function CCN:        1.38
Program NCSS:               31.00

最终，它意味着：

重要LOC的数量没有增加太多。
每个功能都是＆lt; 4行（易于阅读）。
每个函数的圈复杂度是＆lt; 2（易于测试和推理）。
程序更像是在做什么，可以更快地理解。

权衡是我们有更多的方法（在任何合理的IDE或文本编辑器中都不应该是巨大的问题），是一种复杂性。但是：

我们可以在我们选择的深度停止阅读。
我们可以更大的粒度进行测试。
我们可以确定我们可能错过的成功/失败模式。
我们可以更轻松地交换和/或扩展功能。

Struts2 - 多种方法的动作？

1 个答案: